[PATCH] arm64: dts: imx8mm: Drop CSI1 PHY reference clock configuration

Adam Ford aford173 at gmail.com
Wed Jul 26 05:24:23 PDT 2023


On Wed, Jul 26, 2023 at 2:46 AM Marco Felsch <m.felsch at pengutronix.de> wrote:
>
> Hi Fabio,
>
> On 23-07-25, Fabio Estevam wrote:
> > From: Fabio Estevam <festevam at denx.de>
> >
> > The CSI1 PHY reference clock is limited to 125 MHz according to:
> > i.MX 8M Mini Applications Processor Reference Manual, Rev. 3, 11/2020
> > Table 5-1. Clock Root Table (continued) / page 307
> > Slice Index n = 123 .
> >
> > Currently the IMX8MM_CLK_CSI1_PHY_REF clock is configured to be
> > fed directly from 1 GHz PLL2 , which overclocks them. Instead, drop
> > the configuration altogether, which defaults the clock to 24 MHz REF
> > clock input, which for the PHY reference clock is just fine.
> >
> > Based on a patch from Marek Vasut for the imx8mn.
>
> The 8MP does it right?

The 8MP appears to leave the PHY ref clock to its default and only
sets IMX8MP_CLK_MEDIA_CAM1_PIX rate to 500MHz.

>
> > Fixes: e523b7c54c05 ("arm64: dts: imx8mm: Add CSI nodes")
> > Signed-off-by: Fabio Estevam <festevam at denx.de>
>
> Reviewed-by: Marco Felsch <m.felsch at pengutronix.de>

Reviewed-by: Adam Ford <aford173 at gmail.com>



More information about the linux-arm-kernel mailing list