Skip to content

Colorspace fixes #4643

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 7 commits into from
Oct 22, 2021
Merged

Conversation

davidplowman
Copy link
Contributor

@6by9 @naushir
This is a set of small changes to fix colour space issues noticed during libcamera development. Most of them involve sensors that were incorrectly reporting sRGB rather than RAW colour spaces. Additionally I've also made our ISP advertise that it wants full, rather than limited, range RAW input.

Nothing was actually failing before, it was a case of the debug warnings showing that colour spaces were wrong, even though they were being over-written later. I've worked through each of the affected camera modules double-checking that the changes don't break anything!

@6by9
Copy link
Contributor

6by9 commented Oct 21, 2021

The content looks fine, but none of the commit messages contain the name of the driver that they're changing.
Normally they'd be
media: i2c: imx219: Blah
or similar.

Tested on Raspberry Pi running libcamera.

Signed-off-by: David Plowman <[email protected]>
Tested on Raspberry Pi running libcamera.

Signed-off-by: David Plowman <[email protected]>
Tested on Raspberry Pi running libcamera.

Signed-off-by: David Plowman <[email protected]>
Tested on Raspberry Pi running libcamera.

Signed-off-by: David Plowman <[email protected]>
Tested on Raspberry Pi running libcamera.

Signed-off-by: David Plowman <[email protected]>
Tested on Raspberry Pi running libcamera.

Signed-off-by: David Plowman <[email protected]>
… space

RAW color spaces are more usually reported as having full range
quantization.

Tested using libcamera.

Signed-off-by: David Plowman <[email protected]>
@davidplowman
Copy link
Contributor Author

Ah yes indeed, that should be better now!

@pelwell pelwell merged commit b2b29b4 into raspberrypi:rpi-5.10.y Oct 22, 2021
popcornmix added a commit to raspberrypi/firmware that referenced this pull request Oct 25, 2021
kernel: ARM: dts: vc4-kms-v3d: Always disable firmware HDMI
See: raspberrypi/linux#4652

kernel: media: bcm2835-codec: Limit video callbacks
See: raspberrypi/linux#4646

kernel: staging/bcm2835-camera: Add support for H264_MIN_QP, H264_MAX_QP, MPEG_VIDEO_FORCE_KEY_FRAME
See: raspberrypi/linux#4647

kernel: bcm2835-isp: Colorspace fixes
See: raspberrypi/linux#4643

kernel: Unicam Media Controller support
See: raspberrypi/linux#4641

kernel: config: Enable FSFS_FS_SECURITY
See: raspberrypi/linux#4637

kernel: gpio: bcm-virt: Fix the get() method
See: raspberrypi/linux#4638

firmware: userland: Reduce debug_sym error messages
See: https://forums.raspberrypi.com/viewtopic.php?f=98&t=322238

firmware: arm_dt: Increase maximum line length to 98
See: raspberrypi/linux#4638
popcornmix added a commit to raspberrypi/rpi-firmware that referenced this pull request Oct 25, 2021
kernel: ARM: dts: vc4-kms-v3d: Always disable firmware HDMI
See: raspberrypi/linux#4652

kernel: media: bcm2835-codec: Limit video callbacks
See: raspberrypi/linux#4646

kernel: staging/bcm2835-camera: Add support for H264_MIN_QP, H264_MAX_QP, MPEG_VIDEO_FORCE_KEY_FRAME
See: raspberrypi/linux#4647

kernel: bcm2835-isp: Colorspace fixes
See: raspberrypi/linux#4643

kernel: Unicam Media Controller support
See: raspberrypi/linux#4641

kernel: config: Enable FSFS_FS_SECURITY
See: raspberrypi/linux#4637

kernel: gpio: bcm-virt: Fix the get() method
See: raspberrypi/linux#4638

firmware: userland: Reduce debug_sym error messages
See: https://forums.raspberrypi.com/viewtopic.php?f=98&t=322238

firmware: arm_dt: Increase maximum line length to 98
See: raspberrypi/linux#4638
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