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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 

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

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;
  };
[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]

  Powered by Linux