Skip to content

Isp driver fixes #3695

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 30, 2020
Merged

Isp driver fixes #3695

merged 3 commits into from
Jun 30, 2020

Conversation

naushir
Copy link
Contributor

@naushir naushir commented Jun 29, 2020

Fixed based on upstream review feedback comments.

@naushir
Copy link
Contributor Author

naushir commented Jun 29, 2020

@6by9, can you review please?

s->r.height = node->q_data.height;
} else {
return -EINVAL;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

switch case?

s->r.top = crop.rect.y;
s->r.width = crop.rect.width;
s->r.height = crop.rect.height;
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And again, switch / case

naushir added 3 commits June 30, 2020 10:34
Add V4L2_SEL_TGT_CROP_DEFAULT and V4L2_SEL_TGT_CROP_BOUND targets.
Disable the appropriate ioctls for the meta capture nodes - this now
passes v4l2-compliance tests.

Signed-off-by: Naushir Patuck <[email protected]>
Register the video node at the end of the probe, swapping order with
registering the controls.

Signed-off-by: Naushir Patuck <[email protected]>
Rename pad_[] to padding[].
struct bcm2835_isp_rational.den is now unsigned.

Signed-off-by: Naushir Patuck <[email protected]>
@6by9
Copy link
Contributor

6by9 commented Jun 30, 2020

LGTM

@pelwell pelwell merged commit 9d6cffd into raspberrypi:rpi-5.4.y Jun 30, 2020
@naushir naushir deleted the isp_fixes branch June 30, 2020 16:49
popcornmix added a commit to raspberrypi/firmware that referenced this pull request Jul 2, 2020
kernel: drm/vc4: Allow interlaced HDMI modes from FKMS
See: raspberrypi/linux#3698

kernel: Isp driver fixes
See: raspberrypi/linux#3695

kernel: Add Bluetooth overlays
See: raspberrypi/linux#3682

firmware: arm_loader: Allow interlaced HDMI modes from FKMS
See: raspberrypi/linux#3698

firmware: arm_loader: Limit rather than reject boosts with disable_auto_turbo

firmware: i2c: Clearing the TA bit may time out
See: #1422

firmware: arm_loader: Add an accelerated memmove
firmware: arm_loader: memmove kernel to preferred text_offset
See: #1421

firmware: filesystem: Fix GPT regression on USB after SD fix
See: #1420

firmware: bootloader: Some tweaks for LED, UART, USB timeouts
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request Jul 2, 2020
kernel: drm/vc4: Allow interlaced HDMI modes from FKMS
See: raspberrypi/linux#3698

kernel: Isp driver fixes
See: raspberrypi/linux#3695

kernel: Add Bluetooth overlays
See: raspberrypi/linux#3682

firmware: arm_loader: Allow interlaced HDMI modes from FKMS
See: raspberrypi/linux#3698

firmware: arm_loader: Limit rather than reject boosts with disable_auto_turbo

firmware: i2c: Clearing the TA bit may time out
See: raspberrypi/firmware#1422

firmware: arm_loader: Add an accelerated memmove
firmware: arm_loader: memmove kernel to preferred text_offset
See: raspberrypi/firmware#1421

firmware: filesystem: Fix GPT regression on USB after SD fix
See: raspberrypi/firmware#1420

firmware: bootloader: Some tweaks for LED, UART, USB timeouts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants