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

From: Judith Mendez
Date: Tue Feb 13 2024 - 11:08:17 EST


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 <jm@xxxxxx>
---
  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;
  };