Re: [PATCH v2 0/2] media: imx415: Add more clock configurations

[Alexander Stein <alexander.stein@ew.tq-group.com>]

Hi,

Am Dienstag, 12. Dezember 2023, 08:26:35 CET schrieb Alexander Stein:
> Hi,
> 
> this small series converts the driver to using CCI helpers.
> On top of that more clock setups are added to the driver.
> These are all taken from the datasheet. I suspect they all can be calculated
> dynamically later on.
> Tested on TQMa8MPxL + MBa8MPxL + ISP + Vision Components IMX415 camera.
> 
> While working on the driver I notived that imx415 sets mbus code to
> MEDIA_BUS_FMT_SGBRG10_1X10, while imx290 uses MEDIA_BUS_FMT_SRGGB10_1X10.
> But the datasheets show the same color coding pattern. But both settings
> seem to be correct, e.g. the resulting image has correct colors.
> 
> I had to remove the identity check locally. I suspect that Vision Components
> interjects reading some registers.
> 
> Changes in v2:
> * Removed explicit selection of REGMAP_I2C
> * Dropped patch 3 for now. Mode selection shall be done by dynamic VMAX/HMAX
> calculations later on

I forgot to mention: This series depends on [1] which is yet to be applied.

Best regards
Alexander

[1] https://lore.kernel.org/all/20231102095048.3222110-2-alexander.stein@ew.tq-group.com/

> Best regards,
> Alexander
> 
> Alexander Stein (2):
>   media: i2c: imx415: Convert to new CCI register access helpers
>   media: i2c: imx415: Add more clock configurations
> 
>  drivers/media/i2c/Kconfig  |   1 +
>  drivers/media/i2c/imx415.c | 651 +++++++++++++++++++++++--------------
>  2 files changed, 416 insertions(+), 236 deletions(-)


-- 
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/