[v1] media: i2c: imx214: Add support for more clock frequencies

[PATCH 0/4] media: i2c: imx214: Add support for more clock frequencies

Posted by André Apitzsch via B4 Relay 8 months, 1 week ago

The imx214 driver currently supports only a 24 MHz 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 additional clock frequencies.

This series supersedes
https://lore.kernel.org/linux-media/20250308-imx214_clk_freq-v1-0-467a4c083c35@apitzsch.eu/

Signed-off-by: André Apitzsch <git@apitzsch.eu>
---
André Apitzsch (4):
      media: i2c: imx214: Reorder imx214_parse_fwnode call
      media: i2c: imx214: Prepare for variable clock frequency
      media: i2c: imx214: Make use of CCS PLL calculator
      media: i2c: imx214: Read clock frequency from device tree

 drivers/media/i2c/Kconfig  |   1 +
 drivers/media/i2c/imx214.c | 262 +++++++++++++++++++++++++++++++++++----------
 2 files changed, 205 insertions(+), 58 deletions(-)
---
base-commit: 5b37f7bfff3b1582c34be8fb23968b226db71ebd
change-id: 20250406-imx214_ccs_pll-e4aed0e9e532

Best regards,
-- 
André Apitzsch <git@apitzsch.eu>

Re: [PATCH 0/4] media: i2c: imx214: Add support for more clock frequencies

Posted by Ricardo Ribalda Delgado 7 months, 4 weeks ago

On Wed, Apr 16, 2025 at 5:22 AM André Apitzsch via B4 Relay
<devnull+git.apitzsch.eu@kernel.org> wrote:
>
> The imx214 driver currently supports only a 24 MHz 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 additional clock frequencies.
>
> This series supersedes
> https://lore.kernel.org/linux-media/20250308-imx214_clk_freq-v1-0-467a4c083c35@apitzsch.eu/
>
> Signed-off-by: André Apitzsch <git@apitzsch.eu>
Acked-by: Ricardo Ribalda <ribalda@chromium.org>
> ---
> André Apitzsch (4):
>       media: i2c: imx214: Reorder imx214_parse_fwnode call
>       media: i2c: imx214: Prepare for variable clock frequency
>       media: i2c: imx214: Make use of CCS PLL calculator
>       media: i2c: imx214: Read clock frequency from device tree
>
>  drivers/media/i2c/Kconfig  |   1 +
>  drivers/media/i2c/imx214.c | 262 +++++++++++++++++++++++++++++++++++----------
>  2 files changed, 205 insertions(+), 58 deletions(-)
> ---
> base-commit: 5b37f7bfff3b1582c34be8fb23968b226db71ebd
> change-id: 20250406-imx214_ccs_pll-e4aed0e9e532
>
> Best regards,
> --
> André Apitzsch <git@apitzsch.eu>
>
>

Re: [PATCH 0/4] media: i2c: imx214: Add support for more clock frequencies

Posted by André Apitzsch 7 months, 3 weeks ago

Hi Sakari,

Am Donnerstag, dem 24.04.2025 um 08:28 +0800 schrieb Ricardo Ribalda
Delgado:
> On Wed, Apr 16, 2025 at 5:22 AM André Apitzsch via B4 Relay
> <devnull+git.apitzsch.eu@kernel.org> wrote:
> > 
> > The imx214 driver currently supports only a 24 MHz 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 additional clock frequencies.
> > 
> > This series supersedes
> > https://lore.kernel.org/linux-media/20250308-imx214_clk_freq-v1-0-467a4c083c35@apitzsch.eu/
> > 
> > Signed-off-by: André Apitzsch <git@apitzsch.eu>
> Acked-by: Ricardo Ribalda <ribalda@chromium.org>
> > ---
> > André Apitzsch (4):
> >       media: i2c: imx214: Reorder imx214_parse_fwnode call
> >       media: i2c: imx214: Prepare for variable clock frequency
> >       media: i2c: imx214: Make use of CCS PLL calculator
> >       media: i2c: imx214: Read clock frequency from device tree
> > 
> >  drivers/media/i2c/Kconfig  |   1 +
> >  drivers/media/i2c/imx214.c | 262
> > +++++++++++++++++++++++++++++++++++----------
> >  2 files changed, 205 insertions(+), 58 deletions(-)
> > ---
> > base-commit: 5b37f7bfff3b1582c34be8fb23968b226db71ebd
> > change-id: 20250406-imx214_ccs_pll-e4aed0e9e532
> > 
> > Best regards,
> > --
> > André Apitzsch <git@apitzsch.eu>
> > 
> > 

Any chance, this can be applied for the next merge window?

Best regards,
André