[PATCH V2 0/4] arm64: dts: ti: k3-am65: dtbs_check warnings fixups

[PATCH V2 0/4] arm64: dts: ti: k3-am65: dtbs_check warnings fixups

[Nishanth Menon]

Hi,

Series of minor fixups for AM65x device tree to cleanup some of the
dtbs_check warnings.

Changes Since V1:
* Dropped syscon conversion and FSS node rename changes.

V1: https://lore.kernel.org/all/20230424144949.244135-1-nm@ti.com/

Nishanth Menon (4):
  arm64: dts: ti: k3-am65-main: Fix mux controller node name
  arm64: dts: ti: k3-am65-main: Fix mcan node name
  arm64: dts: ti: k3-am65-main: Drop deprecated ti,otap-del-sel property
  arm64: dts: ti: k3-am65-iot2050-common: Rename rtc8564 nodename

 arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 2 +-
 arch/arm64/boot/dts/ti/k3-am65-main.dtsi           | 3 +--
 arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi            | 4 ++--
 3 files changed, 4 insertions(+), 5 deletions(-)

-- 
2.40.0

[PATCH V2 1/4] arm64: dts: ti: k3-am65-main: Fix mux controller node name

[Nishanth Menon]

Add unit address to form a proper node name.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
No change since V1
V1: https://lore.kernel.org/r/20230424144949.244135-3-nm@ti.com

 arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index 1adba2f2c153..f64b4e782ec3 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -506,7 +506,7 @@ serdes1_clk: clock@4090 {
 			reg = <0x00004090 0x4>;
 		};
 
-		serdes_mux: mux-controller {
+		serdes_mux: mux-controller@4080 {
 			compatible = "mmio-mux";
 			#mux-control-cells = <1>;
 			mux-reg-masks = <0x4080 0x3>, /* SERDES0 lane select */
-- 
2.40.0

Re: [PATCH V2 1/4] arm64: dts: ti: k3-am65-main: Fix mux controller node name

[Andrew Davis]

On 6/6/23 11:06 AM, Nishanth Menon wrote:
> Add unit address to form a proper node name.
> 

This is only needed when the node has a reg property, which
we have not added yet to this node. This patch instead adds a
warning:

> node has a unit name, but no reg or ranges property

Andrew

> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> No change since V1
> V1: https://lore.kernel.org/r/20230424144949.244135-3-nm@ti.com
> 
>   arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> index 1adba2f2c153..f64b4e782ec3 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
> @@ -506,7 +506,7 @@ serdes1_clk: clock@4090 {
>   			reg = <0x00004090 0x4>;
>   		};
>   
> -		serdes_mux: mux-controller {
> +		serdes_mux: mux-controller@4080 {
>   			compatible = "mmio-mux";
>   			#mux-control-cells = <1>;
>   			mux-reg-masks = <0x4080 0x3>, /* SERDES0 lane select */

Re: [PATCH V2 1/4] arm64: dts: ti: k3-am65-main: Fix mux controller node name

[Nishanth Menon]

On 11:29-20230606, Andrew Davis wrote:
> On 6/6/23 11:06 AM, Nishanth Menon wrote:
> > Add unit address to form a proper node name.
> > 
> 
> This is only needed when the node has a reg property, which
> we have not added yet to this node. This patch instead adds a
> warning:
> 
> > node has a unit name, but no reg or ranges property

/me bangs head.. I missed it. Thanks for catching it. Will drop
this patch from the series. will wait till tomorrow in case of any
additional comments.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

[PATCH V2 2/4] arm64: dts: ti: k3-am65-main: Fix mcan node name

[Nishanth Menon]

s/mcan/can to stay in sync with bindings conventions.

Signed-off-by: Nishanth Menon <nm@ti.com>
---

No change since V1
V1: https://lore.kernel.org/r/20230424144949.244135-5-nm@ti.com

 arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
index 5dfa31840e9c..c2519d4607a5 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-mcu.dtsi
@@ -227,7 +227,7 @@ mcu_udmap: dma-controller@285c0000 {
 		};
 	};
 
-	m_can0: mcan@40528000 {
+	m_can0: can@40528000 {
 		compatible = "bosch,m_can";
 		reg = <0x0 0x40528000 0x0 0x400>,
 		      <0x0 0x40500000 0x0 0x4400>;
@@ -243,7 +243,7 @@ m_can0: mcan@40528000 {
 		status = "disabled";
 	};
 
-	m_can1: mcan@40568000 {
+	m_can1: can@40568000 {
 		compatible = "bosch,m_can";
 		reg = <0x0 0x40568000 0x0 0x400>,
 		      <0x0 0x40540000 0x0 0x4400>;
-- 
2.40.0

[PATCH V2 3/4] arm64: dts: ti: k3-am65-main: Drop deprecated ti,otap-del-sel property

[Nishanth Menon]

ti,otap-del-sel has been deprecated in favor of ti,otap-del-sel-legacy.

Drop the duplicate and misleading ti,otap-del-sel property.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
No change since V1
V1: https://lore.kernel.org/r/20230424144949.244135-6-nm@ti.com
 arch/arm64/boot/dts/ti/k3-am65-main.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
index f64b4e782ec3..c2fe2ada3f0a 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-main.dtsi
@@ -469,7 +469,6 @@ sdhci1: mmc@4fa0000 {
 		ti,otap-del-sel-ddr52 = <0x4>;
 		ti,otap-del-sel-hs200 = <0x7>;
 		ti,clkbuf-sel = <0x7>;
-		ti,otap-del-sel = <0x2>;
 		ti,trm-icp = <0x8>;
 		dma-coherent;
 	};
-- 
2.40.0

[PATCH V2 4/4] arm64: dts: ti: k3-am65-iot2050-common: Rename rtc8564 nodename

[Nishanth Menon]

Just use "rtc" as the nodename to better match with the bindings.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
No change since V1
V1: https://lore.kernel.org/r/20230424144949.244135-8-nm@ti.com
 arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
index 96ac2b476b11..679857ca95ea 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
@@ -483,7 +483,7 @@ &main_i2c0 {
 	pinctrl-0 = <&main_i2c0_pins_default>;
 	clock-frequency = <400000>;
 
-	rtc: rtc8564@51 {
+	rtc: rtc@51 {
 		compatible = "nxp,pcf8563";
 		reg = <0x51>;
 	};
-- 
2.40.0