Skip to content

IMX585 fixups to Starvis2 driver - #7668

Merged
6by9 merged 2 commits into
raspberrypi:rpi-6.18.yfrom
6by9:rpi-6.18.y-starvis2
Oct 1, 2026
Merged

6by9 merged 2 commits into
raspberrypi:rpi-6.18.yfrom
6by9:rpi-6.18.y-starvis2

Conversation

@6by9

@6by9 6by9 commented Sep 30, 2026

Copy link
Copy Markdown
Contributor

Fixups now that I've got an IMX585 module.

@pelwell pelwell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Change the Fixes: tag to use a 12-digit hash, then merge at will.

6by9 added 2 commits October 1, 2026 11:37
IMX585 requires bit 4 of the WINMODE register (0x3018) to be
set, whilst the bottom nibble selects between all-pixel and
window mode.

Switch to using cci_update_bits so that only the bottom nibble
is updated, thereby working on all supported Starvis2 sensors.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
The common registers and minimum HMAX values merged for IMX585 were
incorrect. Validate against the information from Sony, and confirmed
working.

Fixes: bcb59d4 ("media: sony-starvis2: Add support for IMX585")

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
@6by9
6by9 force-pushed the rpi-6.18.y-starvis2 branch from 2474600 to f67232a Compare October 1, 2026 10:38
@6by9

6by9 commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

Well that's weird. I have a --pretty=fixes option defined that was meant to give the relevant Linux kernel coding style for Fixes:, but it's giving me 13 chars on the hash.

...

Ah, https://git-scm.com/docs/pretty-formats#Documentation/pretty-formats.txt-abbrevnumber

abbrev=<number>
    Instead of using the default number of hexadecimal digits (which will vary according to the number of objects in the repository with a default of 7) of the abbreviated object name, use <number> digits, or as many digits as needed to form a unique object name.

I guess I've got more objects in my repo now.

Updated. That hash is only on our rpi-6.18.y branch, so it wasn't too critical anyway. It'll get squashed when I submit upstream.

@6by9
6by9 merged commit 7044e8a into raspberrypi:rpi-6.18.y Oct 1, 2026
12 checks passed
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.

2 participants