[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <f6c8abba-75f6-4a68-846a-780d37bb94fe@ti.com>
Date: Mon, 8 Sep 2025 18:25:51 -0500
From: Judith Mendez <jm@...com>
To: Andrew Davis <afd@...com>, Nishanth Menon <nm@...com>
CC: Moteen Shah <m-shah@...com>, Vignesh Raghavendra <vigneshr@...com>,
Tero
Kristo <kristo@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof
Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
<linux-arm-kernel@...ts.infradead.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 1/2] arm64: dts: ti: k3-am62p/j722s: Remove HS400
support from common
Hi,
On 9/8/25 3:44 PM, Andrew Davis wrote:
> On 9/8/25 12:38 PM, Judith Mendez wrote:
>> Since eMMC HS400 has been descoped for J722s due to errata i2478 [0]
>> and is supported for AM62Px device, remove eMMC HS400 support from
>> common-main.dtsi and include only in am62p-main.dtsi.
>>
>> [0] https://www.ti.com/lit/pdf/sprz575
>> Signed-off-by: Judith Mendez <jm@...com>
>> ---
>> arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi | 3 ---
>> arch/arm64/boot/dts/ti/k3-am62p-main.dtsi | 6 ++++++
>> 2 files changed, 6 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi b/
>> arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi
>> index 4427b12058a6..0c05bcf1d776 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62p-j722s-common-main.dtsi
>> @@ -576,15 +576,12 @@ sdhci0: mmc@...0000 {
>> bus-width = <8>;
>> mmc-ddr-1_8v;
>> mmc-hs200-1_8v;
>> - mmc-hs400-1_8v;
>> ti,clkbuf-sel = <0x7>;
>> - ti,strobe-sel = <0x77>;
>> ti,trm-icp = <0x8>;
>> ti,otap-del-sel-legacy = <0x1>;
>> ti,otap-del-sel-mmc-hs = <0x1>;
>> ti,otap-del-sel-ddr52 = <0x6>;
>> ti,otap-del-sel-hs200 = <0x8>;
>> - ti,otap-del-sel-hs400 = <0x5>;
>> ti,itap-del-sel-legacy = <0x10>;
>> ti,itap-del-sel-mmc-hs = <0xa>;
>> ti,itap-del-sel-ddr52 = <0x3>;
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62p-main.dtsi b/arch/arm64/
>> boot/dts/ti/k3-am62p-main.dtsi
>> index 6aea9d3f134e..90afe21e972b 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62p-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62p-main.dtsi
>> @@ -31,6 +31,12 @@ usb1: usb@...00000 {
>> snps,usb2-lpm-disable;
>> };
>> };
>> +
>> + sdhci0: mmc@...0000 {
>
> The node already has a label where it is defined, so you could just use:
>
> &sdhci0 {
> ...
>
> here at the base level, and not have it inside cbass_main.
>
Oops, missed that detail, thanks Andrew. Will fix for v3.
~ Judith
Powered by blists - more mailing lists