| 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
| ||
|
Message-ID: <e805b72a-17a2-4a54-bb90-dab44a551711@ti.com>
Date: Tue, 13 Feb 2024 09:52:33 -0600
From: Judith Mendez <jm@...com>
To: Wadim Egorov <w.egorov@...tec.de>, Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley
<conor+dt@...nel.org>
CC: Nishanth Menon <nm@...com>, Vignesh Raghavendra <vigneshr@...com>,
Tero
Kristo <kristo@...nel.org>,
<linux-arm-kernel@...ts.infradead.org>, <devicetree@...r.kernel.org>,
<linux-kernel@...r.kernel.org>,
Francesco
Dolcini <francesco.dolcini@...adex.com>
Subject: Re: [PATCH v2 6/9] arm64: dts: ti: k3-am6*: Remove DLL properties for
soft PHYs
Hi Wadim,
On 2/12/24 11:26 PM, Wadim Egorov wrote:
> Hi Judith,
>
> Am 13.02.24 um 01:24 schrieb Judith Mendez:
>> Remove DLL properties which are not applicable for soft PHYs
>> since these PHYs do not have a DLL to enable.
>>
>> Signed-off-by: Judith Mendez <jm@...com>
>> ---
>> arch/arm64/boot/dts/ti/k3-am62-main.dtsi | 3 ---
>> arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi | 1 -
>> arch/arm64/boot/dts/ti/k3-am62-verdin-dahlia.dtsi | 1 -
>> arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi | 1 -
>> arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi | 1 -
>> arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi | 2 --
>> arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts | 3 ---
>> arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts | 1 -
>> arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 1 -
>> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 1 -
>> arch/arm64/boot/dts/ti/k3-am62p5-sk.dts | 1 -
>> arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 2 --
>> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 1 -
>> arch/arm64/boot/dts/ti/k3-am642-evm.dts | 1 -
>> arch/arm64/boot/dts/ti/k3-am642-sk.dts | 1 -
>
> what about sdhci1 node updates on
> - k3-am642-phyboard-electra-rdk.dts
> - k3-am642-tqma64xxl-mbax4xxl.dts
>
The same patch applies for this boards.
> Also does this apply only for am64 and sdhci1 (and not sdhci0)?
> In your v1 you are describing that only AM64x and AM62p devices have a
> DLL to update the drive strength.
> Trying to understand why only one of the interfaces gets updated.
This patch only applies for AM64x sdhci1 node, since sdhci1
is a soft PHY and does not have a DLL.
~ Judith
>
> Regards,
> Wadim
>
>> 15 files changed, 21 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
>> index fe0cc4a9a501..79ed5cbbbda1 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62-main.dtsi
>> @@ -561,7 +561,6 @@ sdhci0: mmc@...0000 {
>> assigned-clock-parents = <&k3_clks 57 8>;
>> mmc-ddr-1_8v;
>> mmc-hs200-1_8v;
>> - ti,trm-icp = <0x2>;
>> bus-width = <8>;
>> ti,clkbuf-sel = <0x7>;
>> ti,otap-del-sel-legacy = <0x0>;
>> @@ -580,7 +579,6 @@ sdhci1: mmc@...0000 {
>> power-domains = <&k3_pds 58 TI_SCI_PD_EXCLUSIVE>;
>> clocks = <&k3_clks 58 5>, <&k3_clks 58 6>;
>> clock-names = "clk_ahb", "clk_xin";
>> - ti,trm-icp = <0x2>;
>> ti,otap-del-sel-legacy = <0x8>;
>> ti,otap-del-sel-sd-hs = <0x0>;
>> ti,otap-del-sel-sdr12 = <0x0>;
>> @@ -604,7 +602,6 @@ sdhci2: mmc@...0000 {
>> power-domains = <&k3_pds 184 TI_SCI_PD_EXCLUSIVE>;
>> clocks = <&k3_clks 184 5>, <&k3_clks 184 6>;
>> clock-names = "clk_ahb", "clk_xin";
>> - ti,trm-icp = <0x2>;
>> ti,otap-del-sel-legacy = <0x8>;
>> ti,otap-del-sel-sd-hs = <0x0>;
>> ti,otap-del-sel-sdr12 = <0x0>;
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
>> index 3372a256c90f..43488cc8bcb1 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62-phycore-som.dtsi
>> @@ -317,7 +317,6 @@ serial_flash: flash@0 {
>> &sdhci0 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&main_mmc0_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> non-removable;
>> status = "okay";
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin-dahlia.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62-verdin-dahlia.dtsi
>> index bf6d27e70bc4..6c4cec8728e4 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62-verdin-dahlia.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin-dahlia.dtsi
>> @@ -185,7 +185,6 @@ &ospi0 {
>> /* Verdin SD_1 */
>> &sdhci1 {
>> - ti,driver-strength-ohm = <33>;
>> status = "okay";
>> };
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi
>> index 680071688dcb..be62648e7818 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi
>> @@ -206,7 +206,6 @@ &ospi0 {
>> /* Verdin SD_1 */
>> &sdhci1 {
>> - ti,driver-strength-ohm = <33>;
>> status = "okay";
>> };
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi
>> index a6808b10c7b2..4768ef42c4fc 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin-wifi.dtsi
>> @@ -26,7 +26,6 @@ &sdhci2 {
>> mmc-pwrseq = <&wifi_pwrseq>;
>> non-removable;
>> ti,fails-without-test-cd;
>> - ti,driver-strength-ohm = <50>;
>> vmmc-supply = <®_3v3>;
>> status = "okay";
>> };
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>> index 6a06724b6d16..d68310444bcb 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
>> @@ -1407,7 +1407,6 @@ &sdhci0 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&pinctrl_sdhci0>;
>> non-removable;
>> - ti,driver-strength-ohm = <50>;
>> status = "okay";
>> };
>> @@ -1416,7 +1415,6 @@ &sdhci1 {
>> pinctrl-names = "default";
>> pinctrl-0 = <&pinctrl_sdhci1>;
>> disable-wp;
>> - ti,driver-strength-ohm = <50>;
>> vmmc-supply = <®_sdhc1_vmmc>;
>> vqmmc-supply = <®_sdhc1_vqmmc>;
>> status = "disabled";
>> diff --git a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
>> b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
>> index 3b4246ce49de..bb6a5837bcb3 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
>> @@ -819,7 +819,6 @@ &sdhci0 {
>> bootph-all;
>> pinctrl-names = "default";
>> pinctrl-0 = <&emmc_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> status = "okay";
>> };
>> @@ -832,7 +831,6 @@ &sdhci1 {
>> vmmc-supply = <&vdd_3v3_sd>;
>> vqmmc-supply = <&vdd_sd_dv>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> cd-gpios = <&main_gpio1 48 GPIO_ACTIVE_LOW>;
>> cd-debounce-delay-ms = <100>;
>> @@ -849,7 +847,6 @@ &sdhci2 {
>> ti,fails-without-test-cd;
>> cap-power-off-card;
>> keep-power-in-suspend;
>> - ti,driver-strength-ohm = <50>;
>> assigned-clocks = <&k3_clks 157 158>;
>> assigned-clock-parents = <&k3_clks 157 160>;
>> #address-cells = <1>;
>> diff --git a/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts
>> b/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts
>> index 5c31f0453def..a83a90497857 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am625-phyboard-lyra-rdk.dts
>> @@ -334,7 +334,6 @@ &sdhci1 {
>> vqmmc-supply = <&vddshv5_sdio>;
>> pinctrl-names = "default";
>> pinctrl-0 = <&main_mmc1_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> no-1-8-v;
>> status = "okay";
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
>> index 6806288ec227..f283777d54b4 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62a-main.dtsi
>> @@ -561,7 +561,6 @@ sdhci1: mmc@...0000 {
>> power-domains = <&k3_pds 58 TI_SCI_PD_EXCLUSIVE>;
>> clocks = <&k3_clks 58 5>, <&k3_clks 58 6>;
>> clock-names = "clk_ahb", "clk_xin";
>> - ti,trm-icp = <0x2>;
>> ti,otap-del-sel-legacy = <0x0>;
>> ti,otap-del-sel-sd-hs = <0x0>;
>> ti,otap-del-sel-sdr12 = <0xf>;
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> index c99b2e90f76d..f241637a5642 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts
>> @@ -582,7 +582,6 @@ &sdhci1 {
>> vmmc-supply = <&vdd_mmc1>;
>> pinctrl-names = "default";
>> pinctrl-0 = <&main_mmc1_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> };
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
>> b/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
>> index 8c73587b0b62..5c9b73726ebd 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am62p5-sk.dts
>> @@ -422,7 +422,6 @@ &sdhci1 {
>> vqmmc-supply = <&vddshv_sdio>;
>> pinctrl-names = "default";
>> pinctrl-0 = <&main_mmc1_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> bootph-all;
>> };
>> diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
>> index 6dd496cd459a..3c45782ab2b7 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
>> @@ -411,7 +411,6 @@ &sdhci0 {
>> status = "okay";
>> pinctrl-names = "default";
>> pinctrl-0 = <&main_mmc0_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> };
>> @@ -421,7 +420,6 @@ &sdhci1 {
>> status = "okay";
>> pinctrl-names = "default";
>> pinctrl-0 = <&main_mmc1_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> };
>> diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
>> b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
>> index 9bfa0a969bfc..a29847735c6e 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
>> @@ -646,7 +646,6 @@ sdhci1: mmc@...0000 {
>> power-domains = <&k3_pds 58 TI_SCI_PD_EXCLUSIVE>;
>> clocks = <&k3_clks 58 3>, <&k3_clks 58 4>;
>> clock-names = "clk_ahb", "clk_xin";
>> - ti,trm-icp = <0x2>;
>> ti,otap-del-sel-legacy = <0x0>;
>> ti,otap-del-sel-sd-hs = <0x0>;
>> ti,otap-del-sel-sdr12 = <0xf>;
>> diff --git a/arch/arm64/boot/dts/ti/k3-am642-evm.dts
>> b/arch/arm64/boot/dts/ti/k3-am642-evm.dts
>> index 5c546ae76d3e..f308076d608a 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am642-evm.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am642-evm.dts
>> @@ -508,7 +508,6 @@ &sdhci1 {
>> pinctrl-names = "default";
>> bus-width = <4>;
>> pinctrl-0 = <&main_mmc1_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> };
>> diff --git a/arch/arm64/boot/dts/ti/k3-am642-sk.dts
>> b/arch/arm64/boot/dts/ti/k3-am642-sk.dts
>> index cce04e188ff6..b286eaa02ada 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am642-sk.dts
>> +++ b/arch/arm64/boot/dts/ti/k3-am642-sk.dts
>> @@ -471,7 +471,6 @@ &sdhci1 {
>> pinctrl-names = "default";
>> bus-width = <4>;
>> pinctrl-0 = <&main_mmc1_pins_default>;
>> - ti,driver-strength-ohm = <50>;
>> disable-wp;
>> };
Powered by blists - more mailing lists