2024-02-13 01:09:29

by Judith Mendez

[permalink] [raw]
Subject: [PATCH v2 6/9] arm64: dts: ti: k3-am6*: Remove DLL properties for soft PHYs

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 <[email protected]>
---
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 -
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@fa10000 {
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@fa00000 {
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@fa20000 {
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 = <&reg_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 = <&reg_sdhc1_vmmc>;
vqmmc-supply = <&reg_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@fa00000 {
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@fa00000 {
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;
};

--
2.43.0



2024-02-13 05:26:48

by Wadim Egorov

[permalink] [raw]
Subject: Re: [PATCH v2 6/9] arm64: dts: ti: k3-am6*: Remove DLL properties for soft PHYs

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 <[email protected]>
> ---
> 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

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.

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@fa10000 {
> 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@fa00000 {
> 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@fa20000 {
> 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 = <&reg_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 = <&reg_sdhc1_vmmc>;
> vqmmc-supply = <&reg_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@fa00000 {
> 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@fa00000 {
> 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;
> };
>

2024-02-13 08:17:40

by Francesco Dolcini

[permalink] [raw]
Subject: Re: [PATCH v2 6/9] arm64: dts: ti: k3-am6*: Remove DLL properties for soft PHYs

Judith, you asked for tests, but if this property does nothing
on AM62 there is no much to test, these changes will just have zero
effect given your explanation.

On Mon, Feb 12, 2024 at 06:24:13PM -0600, Judith Mendez wrote:
> 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 <[email protected]>

Acked-by: Francesco Dolcini <[email protected]> # Verdin AM62


2024-02-13 15:53:12

by Judith Mendez

[permalink] [raw]
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 <[email protected]>
>> ---
>>   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@fa10000 {
>>           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@fa00000 {
>>           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@fa20000 {
>>           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 = <&reg_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 = <&reg_sdhc1_vmmc>;
>>       vqmmc-supply = <&reg_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@fa00000 {
>>           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@fa00000 {
>>           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;
>>   };


2024-02-13 16:08:14

by Judith Mendez

[permalink] [raw]
Subject: Re: [PATCH v2 6/9] arm64: dts: ti: k3-am6*: Remove DLL properties for soft PHYs

Hi Wadim,

On 2/13/24 9:52 AM, Judith Mendez wrote:
> 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 <[email protected]>
>>> ---
>>>   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.

I will add for v3.

>
>> 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@fa10000 {
>>>           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@fa00000 {
>>>           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@fa20000 {
>>>           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 = <&reg_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 = <&reg_sdhc1_vmmc>;
>>>       vqmmc-supply = <&reg_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@fa00000 {
>>>           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@fa00000 {
>>>           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;
>>>   };
>
>