[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aR72ZPGIJD9JWOCr@tom-desktop>
Date: Thu, 20 Nov 2025 12:07:23 +0100
From: Tommaso Merciai <tommaso.merciai.xr@...renesas.com>
To: Wolfram Sang <wsa+renesas@...g-engineering.com>
Cc: tomm.merciai@...il.com, linux-renesas-soc@...r.kernel.org,
biju.das.jz@...renesas.com,
Geert Uytterhoeven <geert+renesas@...der.be>,
Magnus Damm <magnus.damm@...il.com>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] arm64: dts: renesas: r9a09g047e57-smarc: Add overlay
for P3T1085UK-ARD
Hi Wolfram,
Thanks for your review!
On Wed, Nov 19, 2025 at 07:12:56AM +0100, Wolfram Sang wrote:
>
> > Your suggestion is to keep only Patch 1/2 dropping alias line right?
>
> Yes.
>
> We could think about enabling I3C unconditionally by adding this to
> rzg3e-smarc-som.dtsi:
Fine to me. Thank you.
>
> + i2c-scl-hz = <400000>;
> + i3c-scl-hz = <12500000>;
> + status = "okay";
>
> So, I3C will just work when you connect devices to it. The I3C frequency
> might be depending a bit on the I3C target board and how it is wired.
> Maybe use 10Mhz as a safe value and add a comment?
>
Since our current testing setup works fine with 12.5 MHz, I would go with that.
If another target board is used, it can override this value in its board overlay.
I will send v2 addressing your comments.
Kind Regards,
Tommaso
Powered by blists - more mailing lists