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/[email protected]/
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
Add unit address to form a proper node name.
Signed-off-by: Nishanth Menon <[email protected]>
---
No change since V1
V1: https://lore.kernel.org/r/[email protected]
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
Just use "rtc" as the nodename to better match with the bindings.
Signed-off-by: Nishanth Menon <[email protected]>
---
No change since V1
V1: https://lore.kernel.org/r/[email protected]
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
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 <[email protected]>
---
No change since V1
V1: https://lore.kernel.org/r/[email protected]
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
s/mcan/can to stay in sync with bindings conventions.
Signed-off-by: Nishanth Menon <[email protected]>
---
No change since V1
V1: https://lore.kernel.org/r/[email protected]
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
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 <[email protected]>
> ---
> No change since V1
> V1: https://lore.kernel.org/r/[email protected]
>
> 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 */
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