Hi Geert,
Thanks for the feedback.
> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: Friday, January 26, 2024 1:53 PM
> Subject: Re: [PATCH v2 5/5] arm64: dts: renesas: r9a07g043u11-smarc-cru-
> csi-ov5645: Reduce I2C frequency
>
> Hi Biju,
>
> On Fri, Jan 26, 2024 at 2:31 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> wrote:
> > Reduce i2c freq from 400->100 kHz on RZ/G2UL SMARC EVK as the captured
> > image is not proper with the sensor configuration over I2C.
> >
> > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
>
> Thanks for your patch!
>
> > ---
> > a/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc-cru-csi-ov5645.dtso
> > +++ b/arch/arm64/boot/dts/renesas/r9a07g043u11-smarc-cru-csi-ov5645.dt
> > +++ so
> > @@ -19,3 +19,7 @@ &ov5645 {
> > enable-gpios = <&pinctrl RZG2L_GPIO(4, 4) GPIO_ACTIVE_HIGH>;
> > reset-gpios = <&pinctrl RZG2L_GPIO(0, 1) GPIO_ACTIVE_LOW>; };
> > +
> > +&i2c0 {
> > + clock-frequency = <100000>;
> > +};
>
> Is this a limitation of one of the I2C devices on the bus, or a PCB design
> issue?
Currently versa3 clock generator connected to the same bus and it works ok with
400kHz clock. Maybe it is stressed not that much compared to OV5645 sensor configuration.
At the moment with 400kHz I2C bus clock, Camera capture is not working properly
on RZ/G2UL, but with same bus frequency the same works fine on RZ/{G2L,G2LC,V2L}.
There may be some hardware differences which is causing this issue.
>
> Doesn't this need a Fixes tag?
I can create a new patch updating bus frequency as 100kHz and add fixes tag.
After this I will drop this patch as it no longer needed.
Please let me know.
Cheers,
Biju
