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

Nishanth Menon posted 4 patches 2 years, 8 months ago
There is a newer version of this series
[PATCH V2 1/4] arm64: dts: ti: k3-am65-main: Fix mux controller node name
Posted by Nishanth Menon 2 years, 8 months ago
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
Posted by Andrew Davis 2 years, 8 months ago
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
Posted by Nishanth Menon 2 years, 8 months ago
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