* [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
@ 2020-10-30 11:07 Codrin Ciubotariu
2020-10-30 13:38 ` Nicolas Ferre
0 siblings, 1 reply; 11+ messages in thread
From: Codrin Ciubotariu @ 2020-10-30 11:07 UTC (permalink / raw)
To: devicetree, linux-arm-kernel, linux-kernel
Cc: nicolas.ferre, alexandre.belloni, ludovic.desroches, Codrin Ciubotariu
The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
to match the registered platform device. For this reason, we add a serial
subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
will also remove the boot warning:
"atmel_usart_serial: Failed to locate of_node [id: -2]"
Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
---
arch/arm/boot/dts/at91-sam9x60ek.dts | 3 +++
arch/arm/boot/dts/at91rm9200.dtsi | 15 ++++++++++++
arch/arm/boot/dts/at91sam9261.dtsi | 12 ++++++++++
arch/arm/boot/dts/at91sam9261ek.dts | 3 +++
arch/arm/boot/dts/at91sam9263.dtsi | 12 ++++++++++
arch/arm/boot/dts/at91sam9g45.dtsi | 15 ++++++++++++
arch/arm/boot/dts/at91sam9n12.dtsi | 15 ++++++++++++
arch/arm/boot/dts/at91sam9rl.dtsi | 15 ++++++++++++
arch/arm/boot/dts/at91sam9x5.dtsi | 18 ++++++++++++++
arch/arm/boot/dts/at91sam9x5_usart3.dtsi | 3 +++
arch/arm/boot/dts/sam9x60.dtsi | 3 +++
arch/arm/boot/dts/sama5d2.dtsi | 30 ++++++++++++++++++++++++
arch/arm/boot/dts/sama5d3.dtsi | 18 ++++++++++++++
arch/arm/boot/dts/sama5d3_uart.dtsi | 6 +++++
arch/arm/boot/dts/sama5d4.dtsi | 30 ++++++++++++++++++++++++
15 files changed, 198 insertions(+)
diff --git a/arch/arm/boot/dts/at91-sam9x60ek.dts b/arch/arm/boot/dts/at91-sam9x60ek.dts
index eae28b82c7fd..e317531f7363 100644
--- a/arch/arm/boot/dts/at91-sam9x60ek.dts
+++ b/arch/arm/boot/dts/at91-sam9x60ek.dts
@@ -280,6 +280,9 @@ AT91_XDMAC_DT_PERID(10))>,
atmel,use-dma-rx;
atmel,use-dma-tx;
status = "okay";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
};
diff --git a/arch/arm/boot/dts/at91rm9200.dtsi b/arch/arm/boot/dts/at91rm9200.dtsi
index d1181ead18e5..2abb646f6b68 100644
--- a/arch/arm/boot/dts/at91rm9200.dtsi
+++ b/arch/arm/boot/dts/at91rm9200.dtsi
@@ -602,6 +602,9 @@ dbgu: serial@fffff200 {
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart0: serial@fffc0000 {
@@ -615,6 +618,9 @@ usart0: serial@fffc0000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@fffc4000 {
@@ -628,6 +634,9 @@ usart1: serial@fffc4000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@fffc8000 {
@@ -641,6 +650,9 @@ usart2: serial@fffc8000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart3: serial@fffcc000 {
@@ -654,6 +666,9 @@ usart3: serial@fffcc000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usb1: gadget@fffb0000 {
diff --git a/arch/arm/boot/dts/at91sam9261.dtsi b/arch/arm/boot/dts/at91sam9261.dtsi
index 7adc36ca8a46..e9602378f92c 100644
--- a/arch/arm/boot/dts/at91sam9261.dtsi
+++ b/arch/arm/boot/dts/at91sam9261.dtsi
@@ -187,6 +187,9 @@ usart0: serial@fffb0000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@fffb4000 {
@@ -200,6 +203,9 @@ usart1: serial@fffb4000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@fffb8000{
@@ -213,6 +219,9 @@ usart2: serial@fffb8000{
clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
ssc0: ssc@fffbc000 {
@@ -307,6 +316,9 @@ dbgu: serial@fffff200 {
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
pinctrl@fffff400 {
diff --git a/arch/arm/boot/dts/at91sam9261ek.dts b/arch/arm/boot/dts/at91sam9261ek.dts
index beed819609e8..fae5d408268d 100644
--- a/arch/arm/boot/dts/at91sam9261ek.dts
+++ b/arch/arm/boot/dts/at91sam9261ek.dts
@@ -176,6 +176,9 @@ tsc2046@2 {
dbgu: serial@fffff200 {
status = "okay";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
watchdog@fffffd40 {
diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
index fe45d96239c9..e4d370b582c8 100644
--- a/arch/arm/boot/dts/at91sam9263.dtsi
+++ b/arch/arm/boot/dts/at91sam9263.dtsi
@@ -546,6 +546,9 @@ dbgu: serial@ffffee00 {
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart0: serial@fff8c000 {
@@ -559,6 +562,9 @@ usart0: serial@fff8c000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@fff90000 {
@@ -572,6 +578,9 @@ usart1: serial@fff90000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@fff94000 {
@@ -585,6 +594,9 @@ usart2: serial@fff94000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
ssc0: ssc@fff98000 {
diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi
index 19fc748a87c5..8aad026f10ec 100644
--- a/arch/arm/boot/dts/at91sam9g45.dtsi
+++ b/arch/arm/boot/dts/at91sam9g45.dtsi
@@ -682,6 +682,9 @@ dbgu: serial@ffffee00 {
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart0: serial@fff8c000 {
@@ -695,6 +698,9 @@ usart0: serial@fff8c000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@fff90000 {
@@ -708,6 +714,9 @@ usart1: serial@fff90000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@fff94000 {
@@ -721,6 +730,9 @@ usart2: serial@fff94000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart3: serial@fff98000 {
@@ -734,6 +746,9 @@ usart3: serial@fff98000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 10>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
macb0: ethernet@fffbc000 {
diff --git a/arch/arm/boot/dts/at91sam9n12.dtsi b/arch/arm/boot/dts/at91sam9n12.dtsi
index 0785389f5507..f83d1a313edf 100644
--- a/arch/arm/boot/dts/at91sam9n12.dtsi
+++ b/arch/arm/boot/dts/at91sam9n12.dtsi
@@ -599,6 +599,9 @@ dbgu: serial@fffff200 {
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
ssc0: ssc@f0010000 {
@@ -624,6 +627,9 @@ usart0: serial@f801c000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 5>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@f8020000 {
@@ -635,6 +641,9 @@ usart1: serial@f8020000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@f8024000 {
@@ -646,6 +655,9 @@ usart2: serial@f8024000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart3: serial@f8028000 {
@@ -657,6 +669,9 @@ usart3: serial@f8028000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
i2c0: i2c@f8010000 {
diff --git a/arch/arm/boot/dts/at91sam9rl.dtsi b/arch/arm/boot/dts/at91sam9rl.dtsi
index 5653e70c84b4..5480b6542c6a 100644
--- a/arch/arm/boot/dts/at91sam9rl.dtsi
+++ b/arch/arm/boot/dts/at91sam9rl.dtsi
@@ -183,6 +183,9 @@ usart0: serial@fffb0000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@fffb4000 {
@@ -196,6 +199,9 @@ usart1: serial@fffb4000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@fffb8000 {
@@ -209,6 +215,9 @@ usart2: serial@fffb8000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart3: serial@fffbc000 {
@@ -222,6 +231,9 @@ usart3: serial@fffbc000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
ssc0: ssc@fffc0000 {
@@ -353,6 +365,9 @@ dbgu: serial@fffff200 {
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
pinctrl@fffff400 {
diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi b/arch/arm/boot/dts/at91sam9x5.dtsi
index 4cdb05079cc7..52aa49b007d1 100644
--- a/arch/arm/boot/dts/at91sam9x5.dtsi
+++ b/arch/arm/boot/dts/at91sam9x5.dtsi
@@ -683,6 +683,9 @@ dbgu: serial@fffff200 {
clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart0: serial@f801c000 {
@@ -697,6 +700,9 @@ usart0: serial@f801c000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 5>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@f8020000 {
@@ -711,6 +717,9 @@ usart1: serial@f8020000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@f8024000 {
@@ -725,6 +734,9 @@ usart2: serial@f8024000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
i2c0: i2c@f8010000 {
@@ -781,6 +793,9 @@ uart0: serial@f8040000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 15>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
uart1: serial@f8044000 {
@@ -792,6 +807,9 @@ uart1: serial@f8044000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
adc0: adc@f804c000 {
diff --git a/arch/arm/boot/dts/at91sam9x5_usart3.dtsi b/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
index 098d3fef5c37..a80593188ac1 100644
--- a/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
+++ b/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
@@ -53,6 +53,9 @@ usart3: serial@f8028000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
};
};
diff --git a/arch/arm/boot/dts/sam9x60.dtsi b/arch/arm/boot/dts/sam9x60.dtsi
index 84066c1298df..ab4d80134d86 100644
--- a/arch/arm/boot/dts/sam9x60.dtsi
+++ b/arch/arm/boot/dts/sam9x60.dtsi
@@ -598,6 +598,9 @@ AT91_XDMAC_DT_PERID(28))>,
clocks = <&pmc PMC_TYPE_PERIPHERAL 47>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
pinctrl: pinctrl@fffff400 {
diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
index 2ddc85dff8ce..26f231147d21 100644
--- a/arch/arm/boot/dts/sama5d2.dtsi
+++ b/arch/arm/boot/dts/sama5d2.dtsi
@@ -444,6 +444,9 @@ AT91_XDMAC_DT_PERID(35))>,
clocks = <&pmc PMC_TYPE_PERIPHERAL 24>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
uart1: serial@f8020000 {
@@ -460,6 +463,9 @@ AT91_XDMAC_DT_PERID(37))>,
clocks = <&pmc PMC_TYPE_PERIPHERAL 25>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
uart2: serial@f8024000 {
@@ -476,6 +482,9 @@ AT91_XDMAC_DT_PERID(39))>,
clocks = <&pmc PMC_TYPE_PERIPHERAL 26>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
i2c0: i2c@f8028000 {
@@ -536,6 +545,9 @@ AT91_XDMAC_DT_PER_IF(1) |
dma-names = "tx", "rx";
atmel,fifo-size = <32>;
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
spi2: spi@400 {
@@ -606,6 +618,9 @@ AT91_XDMAC_DT_PER_IF(1) |
dma-names = "tx", "rx";
atmel,fifo-size = <32>;
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
spi3: spi@400 {
@@ -771,6 +786,9 @@ AT91_XDMAC_DT_PERID(41))>,
clocks = <&pmc PMC_TYPE_PERIPHERAL 27>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
uart4: serial@fc00c000 {
@@ -787,6 +805,9 @@ AT91_XDMAC_DT_PERID(43))>,
clocks = <&pmc PMC_TYPE_PERIPHERAL 28>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
flx2: flexcom@fc010000 {
@@ -815,6 +836,9 @@ AT91_XDMAC_DT_PER_IF(1) |
dma-names = "tx", "rx";
atmel,fifo-size = <32>;
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
spi4: spi@400 {
@@ -885,6 +909,9 @@ AT91_XDMAC_DT_PER_IF(1) |
dma-names = "tx", "rx";
atmel,fifo-size = <32>;
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
spi5: spi@400 {
@@ -956,6 +983,9 @@ AT91_XDMAC_DT_PER_IF(1) |
dma-names = "tx", "rx";
atmel,fifo-size = <32>;
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
spi6: spi@400 {
diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
index 86137f8d2b45..1791de52d1c3 100644
--- a/arch/arm/boot/dts/sama5d3.dtsi
+++ b/arch/arm/boot/dts/sama5d3.dtsi
@@ -203,6 +203,9 @@ usart0: serial@f001c000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 12>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@f0020000 {
@@ -217,6 +220,9 @@ usart1: serial@f0020000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 13>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
uart0: serial@f0024000 {
@@ -228,6 +234,9 @@ uart0: serial@f0024000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
pwm0: pwm@f002c000 {
@@ -389,6 +398,9 @@ usart2: serial@f8020000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 14>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart3: serial@f8024000 {
@@ -403,6 +415,9 @@ usart3: serial@f8024000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 15>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
sha@f8034000 {
@@ -497,6 +512,9 @@ dbgu: serial@ffffee00 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 2>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
aic: interrupt-controller@fffff000 {
diff --git a/arch/arm/boot/dts/sama5d3_uart.dtsi b/arch/arm/boot/dts/sama5d3_uart.dtsi
index a3eaba995cf4..b9f83b771a09 100644
--- a/arch/arm/boot/dts/sama5d3_uart.dtsi
+++ b/arch/arm/boot/dts/sama5d3_uart.dtsi
@@ -45,6 +45,9 @@ uart0: serial@f0024000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
uart1: serial@f8028000 {
@@ -56,6 +59,9 @@ uart1: serial@f8028000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 17>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
};
};
diff --git a/arch/arm/boot/dts/sama5d4.dtsi b/arch/arm/boot/dts/sama5d4.dtsi
index 04f24cf752d3..0627fa8cdaf4 100644
--- a/arch/arm/boot/dts/sama5d4.dtsi
+++ b/arch/arm/boot/dts/sama5d4.dtsi
@@ -284,6 +284,9 @@ uart0: serial@f8004000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 27>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
ssc0: ssc@f8008000 {
@@ -443,6 +446,9 @@ usart0: serial@f802c000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart1: serial@f8030000 {
@@ -461,6 +467,9 @@ usart1: serial@f8030000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
mmc1: mmc@fc000000 {
@@ -496,6 +505,9 @@ uart1: serial@fc004000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 28>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart2: serial@fc008000 {
@@ -514,6 +526,9 @@ usart2: serial@fc008000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 29>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart3: serial@fc00c000 {
@@ -532,6 +547,9 @@ usart3: serial@fc00c000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 30>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
usart4: serial@fc010000 {
@@ -550,6 +568,9 @@ usart4: serial@fc010000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 31>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
ssc1: ssc@fc014000 {
@@ -568,6 +589,9 @@ ssc1: ssc@fc014000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 49>;
clock-names = "pclk";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
spi1: spi@fc018000 {
@@ -588,6 +612,9 @@ spi1: spi@fc018000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 38>;
clock-names = "spi_clk";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
spi2: spi@fc01c000 {
@@ -794,6 +821,9 @@ dbgu: serial@fc069000 {
clocks = <&pmc PMC_TYPE_PERIPHERAL 45>;
clock-names = "usart";
status = "disabled";
+ serial {
+ compatible = "atmel,at91rm9200-usart-serial";
+ };
};
--
2.25.1
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-10-30 11:07 [PATCH] ARM: dts: at91: add serial MFD sub-node for usart Codrin Ciubotariu
@ 2020-10-30 13:38 ` Nicolas Ferre
2020-10-30 14:24 ` Codrin.Ciubotariu
2020-11-02 9:01 ` Lee Jones
0 siblings, 2 replies; 11+ messages in thread
From: Nicolas Ferre @ 2020-10-30 13:38 UTC (permalink / raw)
To: Codrin Ciubotariu, devicetree, linux-arm-kernel, linux-kernel,
Richard Genoud, Lee Jones
Cc: alexandre.belloni, ludovic.desroches
On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
> to match the registered platform device. For this reason, we add a serial
> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
> will also remove the boot warning:
> "atmel_usart_serial: Failed to locate of_node [id: -2]"
I don't remember this warning was raised previously even if the MFD
driver was added a while ago (Sept. 2018).
I would say it's due to 466a62d7642f ("mfd: core: Make a best effort
attempt to match devices with the correct of_nodes") which was added on
mid August and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure
disabled devices are ignored without error") but maybe not covering our
case.
So, well, I don't know what's the best option to this change. Moreover,
I would say that all other USART related properties go into the child
not if there is a need for one.
Lee, I suspect that we're not the only ones experiencing this ugly
warning during the boot log: can you point us out how to deal with it
for our existing atmel_serial.c users?
Best regards,
Nicolas
> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
> ---
> arch/arm/boot/dts/at91-sam9x60ek.dts | 3 +++
> arch/arm/boot/dts/at91rm9200.dtsi | 15 ++++++++++++
> arch/arm/boot/dts/at91sam9261.dtsi | 12 ++++++++++
> arch/arm/boot/dts/at91sam9261ek.dts | 3 +++
> arch/arm/boot/dts/at91sam9263.dtsi | 12 ++++++++++
> arch/arm/boot/dts/at91sam9g45.dtsi | 15 ++++++++++++
> arch/arm/boot/dts/at91sam9n12.dtsi | 15 ++++++++++++
> arch/arm/boot/dts/at91sam9rl.dtsi | 15 ++++++++++++
> arch/arm/boot/dts/at91sam9x5.dtsi | 18 ++++++++++++++
> arch/arm/boot/dts/at91sam9x5_usart3.dtsi | 3 +++
> arch/arm/boot/dts/sam9x60.dtsi | 3 +++
> arch/arm/boot/dts/sama5d2.dtsi | 30 ++++++++++++++++++++++++
> arch/arm/boot/dts/sama5d3.dtsi | 18 ++++++++++++++
> arch/arm/boot/dts/sama5d3_uart.dtsi | 6 +++++
> arch/arm/boot/dts/sama5d4.dtsi | 30 ++++++++++++++++++++++++
> 15 files changed, 198 insertions(+)
>
> diff --git a/arch/arm/boot/dts/at91-sam9x60ek.dts b/arch/arm/boot/dts/at91-sam9x60ek.dts
> index eae28b82c7fd..e317531f7363 100644
> --- a/arch/arm/boot/dts/at91-sam9x60ek.dts
> +++ b/arch/arm/boot/dts/at91-sam9x60ek.dts
> @@ -280,6 +280,9 @@ AT91_XDMAC_DT_PERID(10))>,
> atmel,use-dma-rx;
> atmel,use-dma-tx;
> status = "okay";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
> };
>
> diff --git a/arch/arm/boot/dts/at91rm9200.dtsi b/arch/arm/boot/dts/at91rm9200.dtsi
> index d1181ead18e5..2abb646f6b68 100644
> --- a/arch/arm/boot/dts/at91rm9200.dtsi
> +++ b/arch/arm/boot/dts/at91rm9200.dtsi
> @@ -602,6 +602,9 @@ dbgu: serial@fffff200 {
> clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart0: serial@fffc0000 {
> @@ -615,6 +618,9 @@ usart0: serial@fffc0000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@fffc4000 {
> @@ -628,6 +634,9 @@ usart1: serial@fffc4000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@fffc8000 {
> @@ -641,6 +650,9 @@ usart2: serial@fffc8000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart3: serial@fffcc000 {
> @@ -654,6 +666,9 @@ usart3: serial@fffcc000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usb1: gadget@fffb0000 {
> diff --git a/arch/arm/boot/dts/at91sam9261.dtsi b/arch/arm/boot/dts/at91sam9261.dtsi
> index 7adc36ca8a46..e9602378f92c 100644
> --- a/arch/arm/boot/dts/at91sam9261.dtsi
> +++ b/arch/arm/boot/dts/at91sam9261.dtsi
> @@ -187,6 +187,9 @@ usart0: serial@fffb0000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@fffb4000 {
> @@ -200,6 +203,9 @@ usart1: serial@fffb4000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@fffb8000{
> @@ -213,6 +219,9 @@ usart2: serial@fffb8000{
> clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> ssc0: ssc@fffbc000 {
> @@ -307,6 +316,9 @@ dbgu: serial@fffff200 {
> clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> pinctrl@fffff400 {
> diff --git a/arch/arm/boot/dts/at91sam9261ek.dts b/arch/arm/boot/dts/at91sam9261ek.dts
> index beed819609e8..fae5d408268d 100644
> --- a/arch/arm/boot/dts/at91sam9261ek.dts
> +++ b/arch/arm/boot/dts/at91sam9261ek.dts
> @@ -176,6 +176,9 @@ tsc2046@2 {
>
> dbgu: serial@fffff200 {
> status = "okay";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> watchdog@fffffd40 {
> diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
> index fe45d96239c9..e4d370b582c8 100644
> --- a/arch/arm/boot/dts/at91sam9263.dtsi
> +++ b/arch/arm/boot/dts/at91sam9263.dtsi
> @@ -546,6 +546,9 @@ dbgu: serial@ffffee00 {
> clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart0: serial@fff8c000 {
> @@ -559,6 +562,9 @@ usart0: serial@fff8c000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@fff90000 {
> @@ -572,6 +578,9 @@ usart1: serial@fff90000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@fff94000 {
> @@ -585,6 +594,9 @@ usart2: serial@fff94000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> ssc0: ssc@fff98000 {
> diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi
> index 19fc748a87c5..8aad026f10ec 100644
> --- a/arch/arm/boot/dts/at91sam9g45.dtsi
> +++ b/arch/arm/boot/dts/at91sam9g45.dtsi
> @@ -682,6 +682,9 @@ dbgu: serial@ffffee00 {
> clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart0: serial@fff8c000 {
> @@ -695,6 +698,9 @@ usart0: serial@fff8c000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@fff90000 {
> @@ -708,6 +714,9 @@ usart1: serial@fff90000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@fff94000 {
> @@ -721,6 +730,9 @@ usart2: serial@fff94000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart3: serial@fff98000 {
> @@ -734,6 +746,9 @@ usart3: serial@fff98000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 10>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> macb0: ethernet@fffbc000 {
> diff --git a/arch/arm/boot/dts/at91sam9n12.dtsi b/arch/arm/boot/dts/at91sam9n12.dtsi
> index 0785389f5507..f83d1a313edf 100644
> --- a/arch/arm/boot/dts/at91sam9n12.dtsi
> +++ b/arch/arm/boot/dts/at91sam9n12.dtsi
> @@ -599,6 +599,9 @@ dbgu: serial@fffff200 {
> clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> ssc0: ssc@f0010000 {
> @@ -624,6 +627,9 @@ usart0: serial@f801c000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 5>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@f8020000 {
> @@ -635,6 +641,9 @@ usart1: serial@f8020000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@f8024000 {
> @@ -646,6 +655,9 @@ usart2: serial@f8024000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart3: serial@f8028000 {
> @@ -657,6 +669,9 @@ usart3: serial@f8028000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> i2c0: i2c@f8010000 {
> diff --git a/arch/arm/boot/dts/at91sam9rl.dtsi b/arch/arm/boot/dts/at91sam9rl.dtsi
> index 5653e70c84b4..5480b6542c6a 100644
> --- a/arch/arm/boot/dts/at91sam9rl.dtsi
> +++ b/arch/arm/boot/dts/at91sam9rl.dtsi
> @@ -183,6 +183,9 @@ usart0: serial@fffb0000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@fffb4000 {
> @@ -196,6 +199,9 @@ usart1: serial@fffb4000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@fffb8000 {
> @@ -209,6 +215,9 @@ usart2: serial@fffb8000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart3: serial@fffbc000 {
> @@ -222,6 +231,9 @@ usart3: serial@fffbc000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> ssc0: ssc@fffc0000 {
> @@ -353,6 +365,9 @@ dbgu: serial@fffff200 {
> clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> pinctrl@fffff400 {
> diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi b/arch/arm/boot/dts/at91sam9x5.dtsi
> index 4cdb05079cc7..52aa49b007d1 100644
> --- a/arch/arm/boot/dts/at91sam9x5.dtsi
> +++ b/arch/arm/boot/dts/at91sam9x5.dtsi
> @@ -683,6 +683,9 @@ dbgu: serial@fffff200 {
> clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart0: serial@f801c000 {
> @@ -697,6 +700,9 @@ usart0: serial@f801c000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 5>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@f8020000 {
> @@ -711,6 +717,9 @@ usart1: serial@f8020000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@f8024000 {
> @@ -725,6 +734,9 @@ usart2: serial@f8024000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> i2c0: i2c@f8010000 {
> @@ -781,6 +793,9 @@ uart0: serial@f8040000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 15>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> uart1: serial@f8044000 {
> @@ -792,6 +807,9 @@ uart1: serial@f8044000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> adc0: adc@f804c000 {
> diff --git a/arch/arm/boot/dts/at91sam9x5_usart3.dtsi b/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
> index 098d3fef5c37..a80593188ac1 100644
> --- a/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
> +++ b/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
> @@ -53,6 +53,9 @@ usart3: serial@f8028000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
> };
> };
> diff --git a/arch/arm/boot/dts/sam9x60.dtsi b/arch/arm/boot/dts/sam9x60.dtsi
> index 84066c1298df..ab4d80134d86 100644
> --- a/arch/arm/boot/dts/sam9x60.dtsi
> +++ b/arch/arm/boot/dts/sam9x60.dtsi
> @@ -598,6 +598,9 @@ AT91_XDMAC_DT_PERID(28))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 47>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> pinctrl: pinctrl@fffff400 {
> diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
> index 2ddc85dff8ce..26f231147d21 100644
> --- a/arch/arm/boot/dts/sama5d2.dtsi
> +++ b/arch/arm/boot/dts/sama5d2.dtsi
> @@ -444,6 +444,9 @@ AT91_XDMAC_DT_PERID(35))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 24>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> uart1: serial@f8020000 {
> @@ -460,6 +463,9 @@ AT91_XDMAC_DT_PERID(37))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 25>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> uart2: serial@f8024000 {
> @@ -476,6 +482,9 @@ AT91_XDMAC_DT_PERID(39))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 26>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> i2c0: i2c@f8028000 {
> @@ -536,6 +545,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> dma-names = "tx", "rx";
> atmel,fifo-size = <32>;
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> spi2: spi@400 {
> @@ -606,6 +618,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> dma-names = "tx", "rx";
> atmel,fifo-size = <32>;
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> spi3: spi@400 {
> @@ -771,6 +786,9 @@ AT91_XDMAC_DT_PERID(41))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 27>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> uart4: serial@fc00c000 {
> @@ -787,6 +805,9 @@ AT91_XDMAC_DT_PERID(43))>,
> clocks = <&pmc PMC_TYPE_PERIPHERAL 28>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> flx2: flexcom@fc010000 {
> @@ -815,6 +836,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> dma-names = "tx", "rx";
> atmel,fifo-size = <32>;
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> spi4: spi@400 {
> @@ -885,6 +909,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> dma-names = "tx", "rx";
> atmel,fifo-size = <32>;
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> spi5: spi@400 {
> @@ -956,6 +983,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> dma-names = "tx", "rx";
> atmel,fifo-size = <32>;
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> spi6: spi@400 {
> diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
> index 86137f8d2b45..1791de52d1c3 100644
> --- a/arch/arm/boot/dts/sama5d3.dtsi
> +++ b/arch/arm/boot/dts/sama5d3.dtsi
> @@ -203,6 +203,9 @@ usart0: serial@f001c000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 12>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@f0020000 {
> @@ -217,6 +220,9 @@ usart1: serial@f0020000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 13>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> uart0: serial@f0024000 {
> @@ -228,6 +234,9 @@ uart0: serial@f0024000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> pwm0: pwm@f002c000 {
> @@ -389,6 +398,9 @@ usart2: serial@f8020000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 14>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart3: serial@f8024000 {
> @@ -403,6 +415,9 @@ usart3: serial@f8024000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 15>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> sha@f8034000 {
> @@ -497,6 +512,9 @@ dbgu: serial@ffffee00 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 2>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> aic: interrupt-controller@fffff000 {
> diff --git a/arch/arm/boot/dts/sama5d3_uart.dtsi b/arch/arm/boot/dts/sama5d3_uart.dtsi
> index a3eaba995cf4..b9f83b771a09 100644
> --- a/arch/arm/boot/dts/sama5d3_uart.dtsi
> +++ b/arch/arm/boot/dts/sama5d3_uart.dtsi
> @@ -45,6 +45,9 @@ uart0: serial@f0024000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> uart1: serial@f8028000 {
> @@ -56,6 +59,9 @@ uart1: serial@f8028000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 17>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
> };
> };
> diff --git a/arch/arm/boot/dts/sama5d4.dtsi b/arch/arm/boot/dts/sama5d4.dtsi
> index 04f24cf752d3..0627fa8cdaf4 100644
> --- a/arch/arm/boot/dts/sama5d4.dtsi
> +++ b/arch/arm/boot/dts/sama5d4.dtsi
> @@ -284,6 +284,9 @@ uart0: serial@f8004000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 27>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> ssc0: ssc@f8008000 {
> @@ -443,6 +446,9 @@ usart0: serial@f802c000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart1: serial@f8030000 {
> @@ -461,6 +467,9 @@ usart1: serial@f8030000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> mmc1: mmc@fc000000 {
> @@ -496,6 +505,9 @@ uart1: serial@fc004000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 28>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart2: serial@fc008000 {
> @@ -514,6 +526,9 @@ usart2: serial@fc008000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 29>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart3: serial@fc00c000 {
> @@ -532,6 +547,9 @@ usart3: serial@fc00c000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 30>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> usart4: serial@fc010000 {
> @@ -550,6 +568,9 @@ usart4: serial@fc010000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 31>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> ssc1: ssc@fc014000 {
> @@ -568,6 +589,9 @@ ssc1: ssc@fc014000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 49>;
> clock-names = "pclk";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> spi1: spi@fc018000 {
> @@ -588,6 +612,9 @@ spi1: spi@fc018000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 38>;
> clock-names = "spi_clk";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
> spi2: spi@fc01c000 {
> @@ -794,6 +821,9 @@ dbgu: serial@fc069000 {
> clocks = <&pmc PMC_TYPE_PERIPHERAL 45>;
> clock-names = "usart";
> status = "disabled";
> + serial {
> + compatible = "atmel,at91rm9200-usart-serial";
> + };
> };
>
>
>
--
Nicolas Ferre
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-10-30 13:38 ` Nicolas Ferre
@ 2020-10-30 14:24 ` Codrin.Ciubotariu
2020-11-02 9:07 ` Lee Jones
2020-11-02 9:01 ` Lee Jones
1 sibling, 1 reply; 11+ messages in thread
From: Codrin.Ciubotariu @ 2020-10-30 14:24 UTC (permalink / raw)
To: Nicolas.Ferre, devicetree, linux-arm-kernel, linux-kernel,
richard.genoud, lee.jones
Cc: alexandre.belloni, Ludovic.Desroches
On 30.10.2020 15:38, Nicolas Ferre wrote:
> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
>> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs
>> sub-nodes
>> to match the registered platform device. For this reason, we add a serial
>> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
>> will also remove the boot warning:
>> "atmel_usart_serial: Failed to locate of_node [id: -2]"
>
> I don't remember this warning was raised previously even if the MFD
> driver was added a while ago (Sept. 2018).
>
> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort
> attempt to match devices with the correct of_nodes") which was added on
> mid August and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure
> disabled devices are ignored without error") but maybe not covering our
> case.
Well, it's not covering our enabled devices.
>
> So, well, I don't know what's the best option to this change. Moreover,
> I would say that all other USART related properties go into the child
> not if there is a need for one.
>
> Lee, I suspect that we're not the only ones experiencing this ugly
> warning during the boot log: can you point us out how to deal with it
> for our existing atmel_serial.c users?
My understading is that platform devices registered by MFD should have a
correspondig DT node. The parrent properties are also available for the
other usart device (usart-spi), so I think we should keep them in the
parrent.
Best regards,
Codrin
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-10-30 13:38 ` Nicolas Ferre
2020-10-30 14:24 ` Codrin.Ciubotariu
@ 2020-11-02 9:01 ` Lee Jones
2020-11-02 12:11 ` Codrin.Ciubotariu
1 sibling, 1 reply; 11+ messages in thread
From: Lee Jones @ 2020-11-02 9:01 UTC (permalink / raw)
To: Nicolas Ferre
Cc: Codrin Ciubotariu, devicetree, linux-arm-kernel, linux-kernel,
Richard Genoud, alexandre.belloni, ludovic.desroches
On Fri, 30 Oct 2020, Nicolas Ferre wrote:
> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
> > The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
> > to match the registered platform device. For this reason, we add a serial
> > subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
> > will also remove the boot warning:
> > "atmel_usart_serial: Failed to locate of_node [id: -2]"
>
> I don't remember this warning was raised previously even if the MFD driver
> was added a while ago (Sept. 2018).
>
> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort attempt
> to match devices with the correct of_nodes") which was added on mid August
> and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure disabled devices are
> ignored without error") but maybe not covering our case.
>
> So, well, I don't know what's the best option to this change. Moreover, I
> would say that all other USART related properties go into the child not if
> there is a need for one.
>
> Lee, I suspect that we're not the only ones experiencing this ugly warning
> during the boot log: can you point us out how to deal with it for our
> existing atmel_serial.c users?
You should not be instantiating drivers through Device Tree which are
not described there. If the correct representation of the H/W already
exists in Device Tree i.e. no SPI and UART IP really exists, use the
MFD core API to register them utilising the platform API instead.
This should do it:
diff --git a/drivers/mfd/at91-usart.c b/drivers/mfd/at91-usart.c
index 6a8351a4588e2..939bd2332a4f6 100644
--- a/drivers/mfd/at91-usart.c
+++ b/drivers/mfd/at91-usart.c
@@ -17,12 +17,10 @@
static const struct mfd_cell at91_usart_spi_subdev = {
.name = "at91_usart_spi",
- .of_compatible = "microchip,at91sam9g45-usart-spi",
};
static const struct mfd_cell at91_usart_serial_subdev = {
.name = "atmel_usart_serial",
- .of_compatible = "atmel,at91rm9200-usart-serial",
};
static int at91_usart_mode_probe(struct platform_device *pdev)
> > Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com>
> > ---
> > arch/arm/boot/dts/at91-sam9x60ek.dts | 3 +++
> > arch/arm/boot/dts/at91rm9200.dtsi | 15 ++++++++++++
> > arch/arm/boot/dts/at91sam9261.dtsi | 12 ++++++++++
> > arch/arm/boot/dts/at91sam9261ek.dts | 3 +++
> > arch/arm/boot/dts/at91sam9263.dtsi | 12 ++++++++++
> > arch/arm/boot/dts/at91sam9g45.dtsi | 15 ++++++++++++
> > arch/arm/boot/dts/at91sam9n12.dtsi | 15 ++++++++++++
> > arch/arm/boot/dts/at91sam9rl.dtsi | 15 ++++++++++++
> > arch/arm/boot/dts/at91sam9x5.dtsi | 18 ++++++++++++++
> > arch/arm/boot/dts/at91sam9x5_usart3.dtsi | 3 +++
> > arch/arm/boot/dts/sam9x60.dtsi | 3 +++
> > arch/arm/boot/dts/sama5d2.dtsi | 30 ++++++++++++++++++++++++
> > arch/arm/boot/dts/sama5d3.dtsi | 18 ++++++++++++++
> > arch/arm/boot/dts/sama5d3_uart.dtsi | 6 +++++
> > arch/arm/boot/dts/sama5d4.dtsi | 30 ++++++++++++++++++++++++
> > 15 files changed, 198 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/at91-sam9x60ek.dts b/arch/arm/boot/dts/at91-sam9x60ek.dts
> > index eae28b82c7fd..e317531f7363 100644
> > --- a/arch/arm/boot/dts/at91-sam9x60ek.dts
> > +++ b/arch/arm/boot/dts/at91-sam9x60ek.dts
> > @@ -280,6 +280,9 @@ AT91_XDMAC_DT_PERID(10))>,
> > atmel,use-dma-rx;
> > atmel,use-dma-tx;
> > status = "okay";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > };
> > diff --git a/arch/arm/boot/dts/at91rm9200.dtsi b/arch/arm/boot/dts/at91rm9200.dtsi
> > index d1181ead18e5..2abb646f6b68 100644
> > --- a/arch/arm/boot/dts/at91rm9200.dtsi
> > +++ b/arch/arm/boot/dts/at91rm9200.dtsi
> > @@ -602,6 +602,9 @@ dbgu: serial@fffff200 {
> > clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart0: serial@fffc0000 {
> > @@ -615,6 +618,9 @@ usart0: serial@fffc0000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@fffc4000 {
> > @@ -628,6 +634,9 @@ usart1: serial@fffc4000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@fffc8000 {
> > @@ -641,6 +650,9 @@ usart2: serial@fffc8000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart3: serial@fffcc000 {
> > @@ -654,6 +666,9 @@ usart3: serial@fffcc000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usb1: gadget@fffb0000 {
> > diff --git a/arch/arm/boot/dts/at91sam9261.dtsi b/arch/arm/boot/dts/at91sam9261.dtsi
> > index 7adc36ca8a46..e9602378f92c 100644
> > --- a/arch/arm/boot/dts/at91sam9261.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9261.dtsi
> > @@ -187,6 +187,9 @@ usart0: serial@fffb0000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@fffb4000 {
> > @@ -200,6 +203,9 @@ usart1: serial@fffb4000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@fffb8000{
> > @@ -213,6 +219,9 @@ usart2: serial@fffb8000{
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > ssc0: ssc@fffbc000 {
> > @@ -307,6 +316,9 @@ dbgu: serial@fffff200 {
> > clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > pinctrl@fffff400 {
> > diff --git a/arch/arm/boot/dts/at91sam9261ek.dts b/arch/arm/boot/dts/at91sam9261ek.dts
> > index beed819609e8..fae5d408268d 100644
> > --- a/arch/arm/boot/dts/at91sam9261ek.dts
> > +++ b/arch/arm/boot/dts/at91sam9261ek.dts
> > @@ -176,6 +176,9 @@ tsc2046@2 {
> > dbgu: serial@fffff200 {
> > status = "okay";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > watchdog@fffffd40 {
> > diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
> > index fe45d96239c9..e4d370b582c8 100644
> > --- a/arch/arm/boot/dts/at91sam9263.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9263.dtsi
> > @@ -546,6 +546,9 @@ dbgu: serial@ffffee00 {
> > clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart0: serial@fff8c000 {
> > @@ -559,6 +562,9 @@ usart0: serial@fff8c000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@fff90000 {
> > @@ -572,6 +578,9 @@ usart1: serial@fff90000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@fff94000 {
> > @@ -585,6 +594,9 @@ usart2: serial@fff94000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > ssc0: ssc@fff98000 {
> > diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi
> > index 19fc748a87c5..8aad026f10ec 100644
> > --- a/arch/arm/boot/dts/at91sam9g45.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9g45.dtsi
> > @@ -682,6 +682,9 @@ dbgu: serial@ffffee00 {
> > clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart0: serial@fff8c000 {
> > @@ -695,6 +698,9 @@ usart0: serial@fff8c000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@fff90000 {
> > @@ -708,6 +714,9 @@ usart1: serial@fff90000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@fff94000 {
> > @@ -721,6 +730,9 @@ usart2: serial@fff94000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart3: serial@fff98000 {
> > @@ -734,6 +746,9 @@ usart3: serial@fff98000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 10>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > macb0: ethernet@fffbc000 {
> > diff --git a/arch/arm/boot/dts/at91sam9n12.dtsi b/arch/arm/boot/dts/at91sam9n12.dtsi
> > index 0785389f5507..f83d1a313edf 100644
> > --- a/arch/arm/boot/dts/at91sam9n12.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9n12.dtsi
> > @@ -599,6 +599,9 @@ dbgu: serial@fffff200 {
> > clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > ssc0: ssc@f0010000 {
> > @@ -624,6 +627,9 @@ usart0: serial@f801c000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 5>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@f8020000 {
> > @@ -635,6 +641,9 @@ usart1: serial@f8020000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@f8024000 {
> > @@ -646,6 +655,9 @@ usart2: serial@f8024000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart3: serial@f8028000 {
> > @@ -657,6 +669,9 @@ usart3: serial@f8028000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > i2c0: i2c@f8010000 {
> > diff --git a/arch/arm/boot/dts/at91sam9rl.dtsi b/arch/arm/boot/dts/at91sam9rl.dtsi
> > index 5653e70c84b4..5480b6542c6a 100644
> > --- a/arch/arm/boot/dts/at91sam9rl.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9rl.dtsi
> > @@ -183,6 +183,9 @@ usart0: serial@fffb0000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@fffb4000 {
> > @@ -196,6 +199,9 @@ usart1: serial@fffb4000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@fffb8000 {
> > @@ -209,6 +215,9 @@ usart2: serial@fffb8000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart3: serial@fffbc000 {
> > @@ -222,6 +231,9 @@ usart3: serial@fffbc000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 9>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > ssc0: ssc@fffc0000 {
> > @@ -353,6 +365,9 @@ dbgu: serial@fffff200 {
> > clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > pinctrl@fffff400 {
> > diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi b/arch/arm/boot/dts/at91sam9x5.dtsi
> > index 4cdb05079cc7..52aa49b007d1 100644
> > --- a/arch/arm/boot/dts/at91sam9x5.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9x5.dtsi
> > @@ -683,6 +683,9 @@ dbgu: serial@fffff200 {
> > clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart0: serial@f801c000 {
> > @@ -697,6 +700,9 @@ usart0: serial@f801c000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 5>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@f8020000 {
> > @@ -711,6 +717,9 @@ usart1: serial@f8020000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@f8024000 {
> > @@ -725,6 +734,9 @@ usart2: serial@f8024000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > i2c0: i2c@f8010000 {
> > @@ -781,6 +793,9 @@ uart0: serial@f8040000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 15>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > uart1: serial@f8044000 {
> > @@ -792,6 +807,9 @@ uart1: serial@f8044000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > adc0: adc@f804c000 {
> > diff --git a/arch/arm/boot/dts/at91sam9x5_usart3.dtsi b/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
> > index 098d3fef5c37..a80593188ac1 100644
> > --- a/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
> > +++ b/arch/arm/boot/dts/at91sam9x5_usart3.dtsi
> > @@ -53,6 +53,9 @@ usart3: serial@f8028000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 8>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > };
> > };
> > diff --git a/arch/arm/boot/dts/sam9x60.dtsi b/arch/arm/boot/dts/sam9x60.dtsi
> > index 84066c1298df..ab4d80134d86 100644
> > --- a/arch/arm/boot/dts/sam9x60.dtsi
> > +++ b/arch/arm/boot/dts/sam9x60.dtsi
> > @@ -598,6 +598,9 @@ AT91_XDMAC_DT_PERID(28))>,
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 47>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > pinctrl: pinctrl@fffff400 {
> > diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
> > index 2ddc85dff8ce..26f231147d21 100644
> > --- a/arch/arm/boot/dts/sama5d2.dtsi
> > +++ b/arch/arm/boot/dts/sama5d2.dtsi
> > @@ -444,6 +444,9 @@ AT91_XDMAC_DT_PERID(35))>,
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 24>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > uart1: serial@f8020000 {
> > @@ -460,6 +463,9 @@ AT91_XDMAC_DT_PERID(37))>,
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 25>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > uart2: serial@f8024000 {
> > @@ -476,6 +482,9 @@ AT91_XDMAC_DT_PERID(39))>,
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 26>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > i2c0: i2c@f8028000 {
> > @@ -536,6 +545,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> > dma-names = "tx", "rx";
> > atmel,fifo-size = <32>;
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > spi2: spi@400 {
> > @@ -606,6 +618,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> > dma-names = "tx", "rx";
> > atmel,fifo-size = <32>;
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > spi3: spi@400 {
> > @@ -771,6 +786,9 @@ AT91_XDMAC_DT_PERID(41))>,
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 27>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > uart4: serial@fc00c000 {
> > @@ -787,6 +805,9 @@ AT91_XDMAC_DT_PERID(43))>,
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 28>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > flx2: flexcom@fc010000 {
> > @@ -815,6 +836,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> > dma-names = "tx", "rx";
> > atmel,fifo-size = <32>;
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > spi4: spi@400 {
> > @@ -885,6 +909,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> > dma-names = "tx", "rx";
> > atmel,fifo-size = <32>;
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > spi5: spi@400 {
> > @@ -956,6 +983,9 @@ AT91_XDMAC_DT_PER_IF(1) |
> > dma-names = "tx", "rx";
> > atmel,fifo-size = <32>;
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > spi6: spi@400 {
> > diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
> > index 86137f8d2b45..1791de52d1c3 100644
> > --- a/arch/arm/boot/dts/sama5d3.dtsi
> > +++ b/arch/arm/boot/dts/sama5d3.dtsi
> > @@ -203,6 +203,9 @@ usart0: serial@f001c000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 12>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@f0020000 {
> > @@ -217,6 +220,9 @@ usart1: serial@f0020000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 13>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > uart0: serial@f0024000 {
> > @@ -228,6 +234,9 @@ uart0: serial@f0024000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > pwm0: pwm@f002c000 {
> > @@ -389,6 +398,9 @@ usart2: serial@f8020000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 14>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart3: serial@f8024000 {
> > @@ -403,6 +415,9 @@ usart3: serial@f8024000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 15>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > sha@f8034000 {
> > @@ -497,6 +512,9 @@ dbgu: serial@ffffee00 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 2>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > aic: interrupt-controller@fffff000 {
> > diff --git a/arch/arm/boot/dts/sama5d3_uart.dtsi b/arch/arm/boot/dts/sama5d3_uart.dtsi
> > index a3eaba995cf4..b9f83b771a09 100644
> > --- a/arch/arm/boot/dts/sama5d3_uart.dtsi
> > +++ b/arch/arm/boot/dts/sama5d3_uart.dtsi
> > @@ -45,6 +45,9 @@ uart0: serial@f0024000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 16>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > uart1: serial@f8028000 {
> > @@ -56,6 +59,9 @@ uart1: serial@f8028000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 17>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > };
> > };
> > diff --git a/arch/arm/boot/dts/sama5d4.dtsi b/arch/arm/boot/dts/sama5d4.dtsi
> > index 04f24cf752d3..0627fa8cdaf4 100644
> > --- a/arch/arm/boot/dts/sama5d4.dtsi
> > +++ b/arch/arm/boot/dts/sama5d4.dtsi
> > @@ -284,6 +284,9 @@ uart0: serial@f8004000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 27>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > ssc0: ssc@f8008000 {
> > @@ -443,6 +446,9 @@ usart0: serial@f802c000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 6>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart1: serial@f8030000 {
> > @@ -461,6 +467,9 @@ usart1: serial@f8030000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 7>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > mmc1: mmc@fc000000 {
> > @@ -496,6 +505,9 @@ uart1: serial@fc004000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 28>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart2: serial@fc008000 {
> > @@ -514,6 +526,9 @@ usart2: serial@fc008000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 29>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart3: serial@fc00c000 {
> > @@ -532,6 +547,9 @@ usart3: serial@fc00c000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 30>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > usart4: serial@fc010000 {
> > @@ -550,6 +568,9 @@ usart4: serial@fc010000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 31>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > ssc1: ssc@fc014000 {
> > @@ -568,6 +589,9 @@ ssc1: ssc@fc014000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 49>;
> > clock-names = "pclk";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > spi1: spi@fc018000 {
> > @@ -588,6 +612,9 @@ spi1: spi@fc018000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 38>;
> > clock-names = "spi_clk";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> > spi2: spi@fc01c000 {
> > @@ -794,6 +821,9 @@ dbgu: serial@fc069000 {
> > clocks = <&pmc PMC_TYPE_PERIPHERAL 45>;
> > clock-names = "usart";
> > status = "disabled";
> > + serial {
> > + compatible = "atmel,at91rm9200-usart-serial";
> > + };
> > };
> >
>
>
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-10-30 14:24 ` Codrin.Ciubotariu
@ 2020-11-02 9:07 ` Lee Jones
2020-11-02 12:41 ` Codrin.Ciubotariu
0 siblings, 1 reply; 11+ messages in thread
From: Lee Jones @ 2020-11-02 9:07 UTC (permalink / raw)
To: Codrin.Ciubotariu
Cc: Nicolas.Ferre, devicetree, linux-arm-kernel, linux-kernel,
richard.genoud, alexandre.belloni, Ludovic.Desroches
On Fri, 30 Oct 2020, Codrin.Ciubotariu@microchip.com wrote:
> On 30.10.2020 15:38, Nicolas Ferre wrote:
> > On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
> >> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs
> >> sub-nodes
> >> to match the registered platform device. For this reason, we add a serial
> >> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
> >> will also remove the boot warning:
> >> "atmel_usart_serial: Failed to locate of_node [id: -2]"
> >
> > I don't remember this warning was raised previously even if the MFD
> > driver was added a while ago (Sept. 2018).
> >
> > I would say it's due to 466a62d7642f ("mfd: core: Make a best effort
> > attempt to match devices with the correct of_nodes") which was added on
> > mid August and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure
> > disabled devices are ignored without error") but maybe not covering our
> > case.
>
> Well, it's not covering our enabled devices.
>
> >
> > So, well, I don't know what's the best option to this change. Moreover,
> > I would say that all other USART related properties go into the child
> > not if there is a need for one.
> >
> > Lee, I suspect that we're not the only ones experiencing this ugly
> > warning during the boot log: can you point us out how to deal with it
> > for our existing atmel_serial.c users?
>
> My understading is that platform devices registered by MFD should have a
> correspondig DT node. The parrent properties are also available for the
> other usart device (usart-spi), so I think we should keep them in the
> parrent.
Device Tree and MFD are unrelated. MFDs don't even exist - they are a
figment of a Linux Kernel Engineer's imagination - we made them up!
The DT should describe the hardware and nothing else. If we wish to
mess with devices for our own gain i.e. organise them into different
subsystems, we have to do that in software. That's what MFD is for.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-11-02 9:01 ` Lee Jones
@ 2020-11-02 12:11 ` Codrin.Ciubotariu
2020-11-02 12:29 ` Lee Jones
0 siblings, 1 reply; 11+ messages in thread
From: Codrin.Ciubotariu @ 2020-11-02 12:11 UTC (permalink / raw)
To: lee.jones, Nicolas.Ferre
Cc: devicetree, linux-arm-kernel, linux-kernel, richard.genoud,
alexandre.belloni, Ludovic.Desroches
On 02.11.2020 11:01, Lee Jones wrote:
> On Fri, 30 Oct 2020, Nicolas Ferre wrote:
>
>> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
>>> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
>>> to match the registered platform device. For this reason, we add a serial
>>> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
>>> will also remove the boot warning:
>>> "atmel_usart_serial: Failed to locate of_node [id: -2]"
>>
>> I don't remember this warning was raised previously even if the MFD driver
>> was added a while ago (Sept. 2018).
>>
>> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort attempt
>> to match devices with the correct of_nodes") which was added on mid August
>> and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure disabled devices are
>> ignored without error") but maybe not covering our case.
>>
>> So, well, I don't know what's the best option to this change. Moreover, I
>> would say that all other USART related properties go into the child not if
>> there is a need for one.
>>
>> Lee, I suspect that we're not the only ones experiencing this ugly warning
>> during the boot log: can you point us out how to deal with it for our
>> existing atmel_serial.c users?
>
> You should not be instantiating drivers through Device Tree which are
> not described there. If the correct representation of the H/W already
> exists in Device Tree i.e. no SPI and UART IP really exists, use the
> MFD core API to register them utilising the platform API instead.
>
> This should do it:
>
> diff --git a/drivers/mfd/at91-usart.c b/drivers/mfd/at91-usart.c
> index 6a8351a4588e2..939bd2332a4f6 100644
> --- a/drivers/mfd/at91-usart.c
> +++ b/drivers/mfd/at91-usart.c
> @@ -17,12 +17,10 @@
>
> static const struct mfd_cell at91_usart_spi_subdev = {
> .name = "at91_usart_spi",
> - .of_compatible = "microchip,at91sam9g45-usart-spi",
> };
>
> static const struct mfd_cell at91_usart_serial_subdev = {
> .name = "atmel_usart_serial",
> - .of_compatible = "atmel,at91rm9200-usart-serial",
> };
>
> static int at91_usart_mode_probe(struct platform_device *pdev)
[snip]
Hi Lee, thank you for looking through our usart driver and for sharing
your thoughts. Removing the usage of compatible string means that for
similar serial/SPI IPs we would need to create new platform drivers.
This is not ideal, but it's a solution. What I proposed is more
flexible, but, as you pointed out, I am not sure it correctly describes
the HW, because the decision of whether to use this IP as a serial or a
SPI is a configurable one.
Thanks and best regards,
Codrin
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-11-02 12:11 ` Codrin.Ciubotariu
@ 2020-11-02 12:29 ` Lee Jones
2020-11-02 12:55 ` Codrin.Ciubotariu
0 siblings, 1 reply; 11+ messages in thread
From: Lee Jones @ 2020-11-02 12:29 UTC (permalink / raw)
To: Codrin.Ciubotariu
Cc: Nicolas.Ferre, devicetree, linux-arm-kernel, linux-kernel,
richard.genoud, alexandre.belloni, Ludovic.Desroches
On Mon, 02 Nov 2020, Codrin.Ciubotariu@microchip.com wrote:
> On 02.11.2020 11:01, Lee Jones wrote:
> > On Fri, 30 Oct 2020, Nicolas Ferre wrote:
> >
> >> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
> >>> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
> >>> to match the registered platform device. For this reason, we add a serial
> >>> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
> >>> will also remove the boot warning:
> >>> "atmel_usart_serial: Failed to locate of_node [id: -2]"
> >>
> >> I don't remember this warning was raised previously even if the MFD driver
> >> was added a while ago (Sept. 2018).
> >>
> >> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort attempt
> >> to match devices with the correct of_nodes") which was added on mid August
> >> and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure disabled devices are
> >> ignored without error") but maybe not covering our case.
> >>
> >> So, well, I don't know what's the best option to this change. Moreover, I
> >> would say that all other USART related properties go into the child not if
> >> there is a need for one.
> >>
> >> Lee, I suspect that we're not the only ones experiencing this ugly warning
> >> during the boot log: can you point us out how to deal with it for our
> >> existing atmel_serial.c users?
> >
> > You should not be instantiating drivers through Device Tree which are
> > not described there. If the correct representation of the H/W already
> > exists in Device Tree i.e. no SPI and UART IP really exists, use the
> > MFD core API to register them utilising the platform API instead.
> >
> > This should do it:
> >
> > diff --git a/drivers/mfd/at91-usart.c b/drivers/mfd/at91-usart.c
> > index 6a8351a4588e2..939bd2332a4f6 100644
> > --- a/drivers/mfd/at91-usart.c
> > +++ b/drivers/mfd/at91-usart.c
> > @@ -17,12 +17,10 @@
> >
> > static const struct mfd_cell at91_usart_spi_subdev = {
> > .name = "at91_usart_spi",
> > - .of_compatible = "microchip,at91sam9g45-usart-spi",
> > };
> >
> > static const struct mfd_cell at91_usart_serial_subdev = {
> > .name = "atmel_usart_serial",
> > - .of_compatible = "atmel,at91rm9200-usart-serial",
> > };
> >
> > static int at91_usart_mode_probe(struct platform_device *pdev)
>
> [snip]
>
> Hi Lee, thank you for looking through our usart driver and for sharing
> your thoughts. Removing the usage of compatible string means that for
> similar serial/SPI IPs we would need to create new platform drivers.
Why would you need to do that?
> This is not ideal, but it's a solution. What I proposed is more
> flexible, but, as you pointed out, I am not sure it correctly describes
> the HW, because the decision of whether to use this IP as a serial or a
> SPI is a configurable one.
>
> Thanks and best regards,
> Codrin
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-11-02 9:07 ` Lee Jones
@ 2020-11-02 12:41 ` Codrin.Ciubotariu
0 siblings, 0 replies; 11+ messages in thread
From: Codrin.Ciubotariu @ 2020-11-02 12:41 UTC (permalink / raw)
To: lee.jones
Cc: Nicolas.Ferre, devicetree, linux-arm-kernel, linux-kernel,
richard.genoud, alexandre.belloni, Ludovic.Desroches
On 02.11.2020 11:07, Lee Jones wrote:
> On Fri, 30 Oct 2020, Codrin.Ciubotariu@microchip.com wrote:
>
>> On 30.10.2020 15:38, Nicolas Ferre wrote:
>>> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
>>>> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs
>>>> sub-nodes
>>>> to match the registered platform device. For this reason, we add a serial
>>>> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
>>>> will also remove the boot warning:
>>>> "atmel_usart_serial: Failed to locate of_node [id: -2]"
>>>
>>> I don't remember this warning was raised previously even if the MFD
>>> driver was added a while ago (Sept. 2018).
>>>
>>> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort
>>> attempt to match devices with the correct of_nodes") which was added on
>>> mid August and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure
>>> disabled devices are ignored without error") but maybe not covering our
>>> case.
>>
>> Well, it's not covering our enabled devices.
>>
>>>
>>> So, well, I don't know what's the best option to this change. Moreover,
>>> I would say that all other USART related properties go into the child
>>> not if there is a need for one.
>>>
>>> Lee, I suspect that we're not the only ones experiencing this ugly
>>> warning during the boot log: can you point us out how to deal with it
>>> for our existing atmel_serial.c users?
>>
>> My understading is that platform devices registered by MFD should have a
>> correspondig DT node. The parrent properties are also available for the
>> other usart device (usart-spi), so I think we should keep them in the
>> parrent.
>
> Device Tree and MFD are unrelated. MFDs don't even exist - they are a
> figment of a Linux Kernel Engineer's imagination - we made them up!
>
> The DT should describe the hardware and nothing else. If we wish to
> mess with devices for our own gain i.e. organise them into different
> subsystems, we have to do that in software. That's what MFD is for.
You are right, I mixed up things. We are using the MFD here to describe
a hardware USART IP that can also function as an SPI, but not at the
same time. The decision of whether the IP works as a normal USART or an
SPI is DT configurable, at this moment. It is doing more than just
describing the HW, but I don't know how to describe it otherwise.
Best regards,
Codrin
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-11-02 12:29 ` Lee Jones
@ 2020-11-02 12:55 ` Codrin.Ciubotariu
2020-11-02 16:56 ` Codrin.Ciubotariu
0 siblings, 1 reply; 11+ messages in thread
From: Codrin.Ciubotariu @ 2020-11-02 12:55 UTC (permalink / raw)
To: lee.jones
Cc: Nicolas.Ferre, devicetree, linux-arm-kernel, linux-kernel,
richard.genoud, alexandre.belloni, Ludovic.Desroches
On 02.11.2020 14:29, Lee Jones wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> On Mon, 02 Nov 2020, Codrin.Ciubotariu@microchip.com wrote:
>
>> On 02.11.2020 11:01, Lee Jones wrote:
>>> On Fri, 30 Oct 2020, Nicolas Ferre wrote:
>>>
>>>> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
>>>>> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
>>>>> to match the registered platform device. For this reason, we add a serial
>>>>> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
>>>>> will also remove the boot warning:
>>>>> "atmel_usart_serial: Failed to locate of_node [id: -2]"
>>>>
>>>> I don't remember this warning was raised previously even if the MFD driver
>>>> was added a while ago (Sept. 2018).
>>>>
>>>> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort attempt
>>>> to match devices with the correct of_nodes") which was added on mid August
>>>> and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure disabled devices are
>>>> ignored without error") but maybe not covering our case.
>>>>
>>>> So, well, I don't know what's the best option to this change. Moreover, I
>>>> would say that all other USART related properties go into the child not if
>>>> there is a need for one.
>>>>
>>>> Lee, I suspect that we're not the only ones experiencing this ugly warning
>>>> during the boot log: can you point us out how to deal with it for our
>>>> existing atmel_serial.c users?
>>>
>>> You should not be instantiating drivers through Device Tree which are
>>> not described there. If the correct representation of the H/W already
>>> exists in Device Tree i.e. no SPI and UART IP really exists, use the
>>> MFD core API to register them utilising the platform API instead.
>>>
>>> This should do it:
>>>
>>> diff --git a/drivers/mfd/at91-usart.c b/drivers/mfd/at91-usart.c
>>> index 6a8351a4588e2..939bd2332a4f6 100644
>>> --- a/drivers/mfd/at91-usart.c
>>> +++ b/drivers/mfd/at91-usart.c
>>> @@ -17,12 +17,10 @@
>>>
>>> static const struct mfd_cell at91_usart_spi_subdev = {
>>> .name = "at91_usart_spi",
>>> - .of_compatible = "microchip,at91sam9g45-usart-spi",
>>> };
>>>
>>> static const struct mfd_cell at91_usart_serial_subdev = {
>>> .name = "atmel_usart_serial",
>>> - .of_compatible = "atmel,at91rm9200-usart-serial",
>>> };
>>>
>>> static int at91_usart_mode_probe(struct platform_device *pdev)
>>
>> [snip]
>>
>> Hi Lee, thank you for looking through our usart driver and for sharing
>> your thoughts. Removing the usage of compatible string means that for
>> similar serial/SPI IPs we would need to create new platform drivers.
>
> Why would you need to do that?
In the case we will have to support another similar IP, but with a
different set of features. Not a new platform driver from scratch, but
at least a new struct platform_driver for each variant.
>
>> This is not ideal, but it's a solution. What I proposed is more
>> flexible, but, as you pointed out, I am not sure it correctly describes
>> the HW, because the decision of whether to use this IP as a serial or a
>> SPI is a configurable one.
>>
>> Thanks and best regards,
>> Codrin
>
> --
> Lee Jones [李琼斯]
> Senior Technical Lead - Developer Services
> Linaro.org │ Open source software for Arm SoCs
> Follow Linaro: Facebook | Twitter | Blog
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-11-02 12:55 ` Codrin.Ciubotariu
@ 2020-11-02 16:56 ` Codrin.Ciubotariu
2020-11-03 8:56 ` Lee Jones
0 siblings, 1 reply; 11+ messages in thread
From: Codrin.Ciubotariu @ 2020-11-02 16:56 UTC (permalink / raw)
To: lee.jones
Cc: Nicolas.Ferre, devicetree, linux-arm-kernel, linux-kernel,
richard.genoud, alexandre.belloni, Ludovic.Desroches
On 02.11.2020 14:55, Codrin.Ciubotariu@microchip.com wrote:
> On 02.11.2020 14:29, Lee Jones wrote:
>> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>>
>> On Mon, 02 Nov 2020, Codrin.Ciubotariu@microchip.com wrote:
>>
>>> On 02.11.2020 11:01, Lee Jones wrote:
>>>> On Fri, 30 Oct 2020, Nicolas Ferre wrote:
>>>>
>>>>> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
>>>>>> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
>>>>>> to match the registered platform device. For this reason, we add a serial
>>>>>> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
>>>>>> will also remove the boot warning:
>>>>>> "atmel_usart_serial: Failed to locate of_node [id: -2]"
>>>>>
>>>>> I don't remember this warning was raised previously even if the MFD driver
>>>>> was added a while ago (Sept. 2018).
>>>>>
>>>>> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort attempt
>>>>> to match devices with the correct of_nodes") which was added on mid August
>>>>> and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure disabled devices are
>>>>> ignored without error") but maybe not covering our case.
>>>>>
>>>>> So, well, I don't know what's the best option to this change. Moreover, I
>>>>> would say that all other USART related properties go into the child not if
>>>>> there is a need for one.
>>>>>
>>>>> Lee, I suspect that we're not the only ones experiencing this ugly warning
>>>>> during the boot log: can you point us out how to deal with it for our
>>>>> existing atmel_serial.c users?
>>>>
>>>> You should not be instantiating drivers through Device Tree which are
>>>> not described there. If the correct representation of the H/W already
>>>> exists in Device Tree i.e. no SPI and UART IP really exists, use the
>>>> MFD core API to register them utilising the platform API instead.
>>>>
>>>> This should do it:
>>>>
>>>> diff --git a/drivers/mfd/at91-usart.c b/drivers/mfd/at91-usart.c
>>>> index 6a8351a4588e2..939bd2332a4f6 100644
>>>> --- a/drivers/mfd/at91-usart.c
>>>> +++ b/drivers/mfd/at91-usart.c
>>>> @@ -17,12 +17,10 @@
>>>>
>>>> static const struct mfd_cell at91_usart_spi_subdev = {
>>>> .name = "at91_usart_spi",
>>>> - .of_compatible = "microchip,at91sam9g45-usart-spi",
>>>> };
>>>>
>>>> static const struct mfd_cell at91_usart_serial_subdev = {
>>>> .name = "atmel_usart_serial",
>>>> - .of_compatible = "atmel,at91rm9200-usart-serial",
>>>> };
>>>>
>>>> static int at91_usart_mode_probe(struct platform_device *pdev)
>>>
>>> [snip]
>>>
>>> Hi Lee, thank you for looking through our usart driver and for sharing
>>> your thoughts. Removing the usage of compatible string means that for
>>> similar serial/SPI IPs we would need to create new platform drivers.
>>
>> Why would you need to do that?
>
> In the case we will have to support another similar IP, but with a
> different set of features. Not a new platform driver from scratch, but
> at least a new struct platform_driver for each variant.
I guess we could use struct mfd_cell.platform_data to select the
features for the serial/SPI. This platform data can be per compatible of
our MFD driver. I will send a patch with the changes you suggested.
Thank you!
Best regards,
Codrin
>
>>
>>> This is not ideal, but it's a solution. What I proposed is more
>>> flexible, but, as you pointed out, I am not sure it correctly describes
>>> the HW, because the decision of whether to use this IP as a serial or a
>>> SPI is a configurable one.
>>>
>>> Thanks and best regards,
>>> Codrin
>>
>> --
>> Lee Jones [李琼斯]
>> Senior Technical Lead - Developer Services
>> Linaro.org │ Open source software for Arm SoCs
>> Follow Linaro: Facebook | Twitter | Blog
>>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ARM: dts: at91: add serial MFD sub-node for usart
2020-11-02 16:56 ` Codrin.Ciubotariu
@ 2020-11-03 8:56 ` Lee Jones
0 siblings, 0 replies; 11+ messages in thread
From: Lee Jones @ 2020-11-03 8:56 UTC (permalink / raw)
To: Codrin.Ciubotariu
Cc: Nicolas.Ferre, devicetree, linux-arm-kernel, linux-kernel,
richard.genoud, alexandre.belloni, Ludovic.Desroches
On Mon, 02 Nov 2020, Codrin.Ciubotariu@microchip.com wrote:
> On 02.11.2020 14:55, Codrin.Ciubotariu@microchip.com wrote:
> > On 02.11.2020 14:29, Lee Jones wrote:
> >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> >>
> >> On Mon, 02 Nov 2020, Codrin.Ciubotariu@microchip.com wrote:
> >>
> >>> On 02.11.2020 11:01, Lee Jones wrote:
> >>>> On Fri, 30 Oct 2020, Nicolas Ferre wrote:
> >>>>
> >>>>> On 30/10/2020 at 12:07, Codrin Ciubotariu wrote:
> >>>>>> The "atmel,at91sam9260-usart" driver is a MFD driver, so it needs sub-nodes
> >>>>>> to match the registered platform device. For this reason, we add a serial
> >>>>>> subnode to all the "atmel,at91sam9260-usart" serial compatible nods. This
> >>>>>> will also remove the boot warning:
> >>>>>> "atmel_usart_serial: Failed to locate of_node [id: -2]"
> >>>>>
> >>>>> I don't remember this warning was raised previously even if the MFD driver
> >>>>> was added a while ago (Sept. 2018).
> >>>>>
> >>>>> I would say it's due to 466a62d7642f ("mfd: core: Make a best effort attempt
> >>>>> to match devices with the correct of_nodes") which was added on mid August
> >>>>> and corrected with 22380b65dc70 ("mfd: mfd-core: Ensure disabled devices are
> >>>>> ignored without error") but maybe not covering our case.
> >>>>>
> >>>>> So, well, I don't know what's the best option to this change. Moreover, I
> >>>>> would say that all other USART related properties go into the child not if
> >>>>> there is a need for one.
> >>>>>
> >>>>> Lee, I suspect that we're not the only ones experiencing this ugly warning
> >>>>> during the boot log: can you point us out how to deal with it for our
> >>>>> existing atmel_serial.c users?
> >>>>
> >>>> You should not be instantiating drivers through Device Tree which are
> >>>> not described there. If the correct representation of the H/W already
> >>>> exists in Device Tree i.e. no SPI and UART IP really exists, use the
> >>>> MFD core API to register them utilising the platform API instead.
> >>>>
> >>>> This should do it:
> >>>>
> >>>> diff --git a/drivers/mfd/at91-usart.c b/drivers/mfd/at91-usart.c
> >>>> index 6a8351a4588e2..939bd2332a4f6 100644
> >>>> --- a/drivers/mfd/at91-usart.c
> >>>> +++ b/drivers/mfd/at91-usart.c
> >>>> @@ -17,12 +17,10 @@
> >>>>
> >>>> static const struct mfd_cell at91_usart_spi_subdev = {
> >>>> .name = "at91_usart_spi",
> >>>> - .of_compatible = "microchip,at91sam9g45-usart-spi",
> >>>> };
> >>>>
> >>>> static const struct mfd_cell at91_usart_serial_subdev = {
> >>>> .name = "atmel_usart_serial",
> >>>> - .of_compatible = "atmel,at91rm9200-usart-serial",
> >>>> };
> >>>>
> >>>> static int at91_usart_mode_probe(struct platform_device *pdev)
> >>>
> >>> [snip]
> >>>
> >>> Hi Lee, thank you for looking through our usart driver and for sharing
> >>> your thoughts. Removing the usage of compatible string means that for
> >>> similar serial/SPI IPs we would need to create new platform drivers.
> >>
> >> Why would you need to do that?
> >
> > In the case we will have to support another similar IP, but with a
> > different set of features. Not a new platform driver from scratch, but
> > at least a new struct platform_driver for each variant.
>
> I guess we could use struct mfd_cell.platform_data to select the
> features for the serial/SPI. This platform data can be per compatible of
> our MFD driver. I will send a patch with the changes you suggested.
Yes, that is what platform data is for.
> Thank you!
NP.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2020-11-03 8:56 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-30 11:07 [PATCH] ARM: dts: at91: add serial MFD sub-node for usart Codrin Ciubotariu
2020-10-30 13:38 ` Nicolas Ferre
2020-10-30 14:24 ` Codrin.Ciubotariu
2020-11-02 9:07 ` Lee Jones
2020-11-02 12:41 ` Codrin.Ciubotariu
2020-11-02 9:01 ` Lee Jones
2020-11-02 12:11 ` Codrin.Ciubotariu
2020-11-02 12:29 ` Lee Jones
2020-11-02 12:55 ` Codrin.Ciubotariu
2020-11-02 16:56 ` Codrin.Ciubotariu
2020-11-03 8:56 ` Lee Jones
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).