Re: [PATCH 2/2] media: i2c: Add Sony IMX908 image sensor driver

[Lachlan.Michael@xxxxxxxx]

Dear Dave,

>In case you'd missed it, I'm trying to convert Jai's IMX678 driver
>into a generic Starvis 2 driver[1].

Thanks for looping me in.

>IMX662 that I'm interested in is streaming with a total of 7 changes
>to defines or const structs in the current imx678 driver.
>IMX675 (5MPix Starvis2) frames with a similar number of changes but
>currently isn't giving me images (test patterns work).
>IMX585 looks like it may fit the same pattern, but I'm waiting on a
>module for that one.
>
>I'm therefore looking to parameterise those diffs but otherwise keep
>it all common. This avoids the hassle of having to convert potentially
>4 (or more) drivers to the Common Raw Sensor Model once it is
>finalised.

We will be following your work and also plan to convert IMX908 to the 
Common Raw Sensor Model at the appropriate time.

>IMX908 here looks to very much follow the same pattern, so unless
>there are additional diffs between Starvis 2 and Starvis 3 it would
>potentially fit into the same framework.

Currently the IMX908 driver is SDR only. We plan to add Clear HDR3 [1] support
in a future patch. Clear HDR3 does the image combining internally in the HW
and outputs one combined HDR image, reducing artifacts for moving objects.

>I'm hoping to get an RFC out this week as I'm then on holiday for a
>week. It'll be interesting to see if IMX908 will fit with the same
>pattern.
>
>Lachlan: Are there any particular people we should contact for details
>of IMX908, or just go through our normal contact?

For details of HW, please go through your normal contacts.
If anything to do with the driver, please contact me.

Best Regards,

Lachlan

[1] https://www.sony-semicon.com/en/products/is/security/security/IMX908.html