lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <44a55012-32e9-4ecd-8643-d9c0008bc5d2@ti.com>
Date: Thu, 4 Sep 2025 10:27:02 -0500
From: Judith Mendez <jm@...com>
To: 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 1/2] arm64: dts: ti: k3-am62p/j722s: Move sdhci0 from
 common

Hi Nishanth,

On 9/3/25 10:38 PM, Nishanth Menon wrote:
> On 19:47-20250903, Judith Mendez wrote:
>> Since eMMC HS400 has been descoped for j722s due to errata i2478 [0]
>> and is supported for am62p SR1.2 device, remove sdhci0 node from
>> common-main.dtsi and include instead in each device's main.dtsi
>> appropriately.
>>
>> [0] https://www.ti.com/lit/pdf/sprz575
>> Signed-off-by: Judith Mendez <jm@...com>
>> ---
>>   .../dts/ti/k3-am62p-j722s-common-main.dtsi    | 25 -------------------
>>   arch/arm64/boot/dts/ti/k3-am62p-main.dtsi     | 25 +++++++++++++++++++
>>   arch/arm64/boot/dts/ti/k3-j722s-main.dtsi     | 22 ++++++++++++++++
>>   3 files changed, 47 insertions(+), 25 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..84083f5125df 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
>> @@ -566,31 +566,6 @@ main_gpio1: gpio@...000 {
>>   		clock-names = "gpio";
>>   	};
>>   
>> -	sdhci0: mmc@...0000 {
>> -		compatible = "ti,am64-sdhci-8bit";
>> -		reg = <0x00 0x0fa10000 0x00 0x1000>, <0x00 0x0fa18000 0x00 0x400>;
>> -		interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>;
>> -		power-domains = <&k3_pds 57 TI_SCI_PD_EXCLUSIVE>;
>> -		clocks = <&k3_clks 57 1>, <&k3_clks 57 2>;
>> -		clock-names = "clk_ahb", "clk_xin";
>> -		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>;
> 
> would'nt it be sufficient to provide this in am62p and keep the common
> stuff here?

Either way works, I can keep a common no problem.

> 
> Additionally handling of SR1.2 should be documented in am62p

WYM? Why document anything on  SR1.2? For am62p, we support HS400 mode
which is the default, all other silicon revision will automatically
be reduced to HS200, that logic is abstracted away in the driver.

There is nothing to document here IMO.

~ Judith



> 
>> -		ti,itap-del-sel-legacy = <0x10>;
>> -		ti,itap-del-sel-mmc-hs = <0xa>;
>> -		ti,itap-del-sel-ddr52 = <0x3>;
>> -		status = "disabled";
>> -	};
>> -
>>   	sdhci1: mmc@...0000 {
>>   		compatible = "ti,am62-sdhci";
>>   		reg = <0x00 0x0fa00000 0x00 0x1000>, <0x00 0x0fa08000 0x00 0x400>;
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62p-main.dtsi b/arch/arm64/boot/dts/ti/k3-am62p-main.dtsi
>> index 6aea9d3f134e..fb8473ce403a 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62p-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62p-main.dtsi
>> @@ -31,6 +31,31 @@ usb1: usb@...00000 {
>>   			snps,usb2-lpm-disable;
>>   		};
>>   	};
>> +
>> +	sdhci0: mmc@...0000 {
>> +		compatible = "ti,am64-sdhci-8bit";
>> +		reg = <0x00 0x0fa10000 0x00 0x1000>, <0x00 0x0fa18000 0x00 0x400>;
>> +		interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>;
>> +		power-domains = <&k3_pds 57 TI_SCI_PD_EXCLUSIVE>;
>> +		clocks = <&k3_clks 57 1>, <&k3_clks 57 2>;
>> +		clock-names = "clk_ahb", "clk_xin";
>> +		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>;
>> +		status = "disabled";
>> +	};
>>   };
>>   
>>   &oc_sram {
>> diff --git a/arch/arm64/boot/dts/ti/k3-j722s-main.dtsi b/arch/arm64/boot/dts/ti/k3-j722s-main.dtsi
>> index 993828872dfb..2978fe1a151e 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j722s-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-j722s-main.dtsi
>> @@ -404,6 +404,28 @@ e5010: jpeg-encoder@...0000 {
>>   		power-domains = <&k3_pds 201 TI_SCI_PD_EXCLUSIVE>;
>>   		interrupts = <GIC_SPI 98 IRQ_TYPE_LEVEL_HIGH>;
>>   	};
>> +
>> +	sdhci0: mmc@...0000 {
>> +		compatible = "ti,am64-sdhci-8bit";
>> +		reg = <0x00 0x0fa10000 0x00 0x1000>, <0x00 0x0fa18000 0x00 0x400>;
>> +		interrupts = <GIC_SPI 133 IRQ_TYPE_LEVEL_HIGH>;
>> +		power-domains = <&k3_pds 57 TI_SCI_PD_EXCLUSIVE>;
>> +		clocks = <&k3_clks 57 1>, <&k3_clks 57 2>;
>> +		clock-names = "clk_ahb", "clk_xin";
>> +		bus-width = <8>;
>> +		mmc-ddr-1_8v;
>> +		mmc-hs200-1_8v;
>> +		ti,clkbuf-sel = <0x7>;
>> +		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,itap-del-sel-legacy = <0x10>;
>> +		ti,itap-del-sel-mmc-hs = <0xa>;
>> +		ti,itap-del-sel-ddr52 = <0x3>;
>> +		status = "disabled";
>> +	};
>>   };
>>   
>>   &main_bcdma_csi {
>> -- 
>> 2.51.0
>>
> 


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ