Hi Sakari, Am Montag, dem 10.03.2025 um 11:11 +0000 schrieb Sakari Ailus: > Hi André, > > On Sat, Mar 08, 2025 at 10:47:54PM +0100, André Apitzsch via B4 Relay > wrote: > > The imx214 driver currently supports only a 24MHz external clock. > > But > > there are devices, like Qualcomm-MSM8916-based phones, which cannot > > provide this frequency. To make the sensor usable by those devices, > > add > > support for 23.88MHz clock. > > > > Signed-off-by: André Apitzsch <git@xxxxxxxxxxx> > > --- > > André Apitzsch (4): > > media: i2c: imx214: Calculate link bit rate from clock > > frequency > > media: i2c: imx214: Prepare for variable clock frequency > > media: i2c: imx214: Read clock frequency from device tree > > media: i2c: imx214: Add support for 23.88MHz clock > > > > drivers/media/i2c/imx214.c | 188 > > +++++++++++++++++++++++++++++++++++---------- > > 1 file changed, 146 insertions(+), 42 deletions(-) > > Thanks for the patches. > > Do you think the driver could use the CCS PLL calculator? The PLL > appears to be compliant. The AR0234 driver will do the same. (The > sensor might just work with the CCS driver, too, but that's another > discussion.) > Using the CCS PLL calculator seems quite complicated compared to switching to the CCS driver. That's why I looked at the later first. But for it to work, quirks already need to be applied in ccs_power_on(), to disable writing to COMPRESSION_MODE, and in ccs_identify_module(), to change the MODULE_MANUFACTURER_ID register. I'll check if CCS PLL calculator could be used. Best regards, André
