[PATCH v2 2/3] ARM: dts: stm32: Fix DT dtc warnings for stm32f7
From: Patrice Chotard
Fix the following DT dtc warnings for stm32f746 and stm32f769
boards:
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_a@0: Character
'_' not recommended in node name
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_od_a@0:
Character '_' not recommended in node name
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_b@0: Character
'_' not recommended in node name
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_od_b@0:
Character '_' not recommended in node name
Warning (node_name_chars_strict): /gpio_keys: Character '_' not recommended in
node name
Warning (node_name_chars_strict): /mmc_vcard: Character '_' not recommended in
node name
Warning (unit_address_vs_reg): /memory: node has a reg or ranges property, but
no unit name
Warning (unit_address_vs_reg): /soc/pin-controller: node has a reg or ranges
property, but no unit name
Warning (unit_address_vs_reg): /soc/pin-controller/cec@0: node has a unit name,
but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usart1@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/i2c1@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usbotg-hs@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usbotg-hs@1: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usbotg-fs@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_a@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_od_a@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_b@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_od_b@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /gpio_keys/button@0: node has a unit name, but
no reg property
Warning (avoid_unnecessary_addr_size): /soc/timers@40001c00: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Warning (avoid_unnecessary_addr_size): /soc/timers@40002000: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Warning (avoid_unnecessary_addr_size): /soc/timers@40014400: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Warning (avoid_unnecessary_addr_size): /soc/timers@40014800: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Signed-off-by: Patrice Chotard
---
Change since v1:
- update nodes name as following: @0 to -0
arch/arm/boot/dts/stm32746g-eval.dts | 9 -
arch/arm/boot/dts/stm32f7-pinctrl.dtsi | 24
arch/arm/boot/dts/stm32f746-disco.dts | 6 +++---
arch/arm/boot/dts/stm32f746.dtsi | 8 +++-
arch/arm/boot/dts/stm32f769-disco.dts | 11 +--
5 files changed, 27 insertions(+), 31 deletions(-)
diff --git a/arch/arm/boot/dts/stm32746g-eval.dts
b/arch/arm/boot/dts/stm32746g-eval.dts
index 8c081eaf20fe..b8a8e06d1427 100644
--- a/arch/arm/boot/dts/stm32746g-eval.dts
+++ b/arch/arm/boot/dts/stm32746g-eval.dts
@@ -54,7 +54,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@c000 {
reg = <0xc000 0x200>;
};
@@ -73,12 +73,11 @@
};
};
- gpio_keys {
+ gpio-keys {
compatible = "gpio-keys";
- #address-cells = <1>;
#size-cells = <0>;
autorepeat;
- button@0 {
+ button-0 {
label = "Wake up";
linux,code = ;
gpios = < 13 0>;
@@ -92,7 +91,7 @@
clock-names = "main_clk";
};
- mmc_vcard: mmc_vcard {
+ mmc_vcard: mmc-vcard {
compatible = "regulator-fixed";
regulator-name = "mmc_vcard";
regulator-min-microvolt = <330>;
diff --git a/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
b/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
index 9314128df185..1cf8a23c2644 100644
--- a/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
+++ b/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
@@ -9,7 +9,7 @@
/ {
soc {
- pinctrl: pin-controller {
+ pinctrl: pin-controller@4002 {
#address-cells = <1>;
#size-cells = <1>;
ranges = <0 0x4002 0x3000>;
@@ -127,7 +127,7 @@
st,bank-name = "GPIOK";
};
- cec_pins_a: cec@0 {
+ cec_pins_a: cec-0 {
pins {
pinmux = ;
/* HDMI
[PATCH v2 2/3] ARM: dts: stm32: Fix DT dtc warnings for stm32f7
From: Patrice Chotard
Fix the following DT dtc warnings for stm32f746 and stm32f769
boards:
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_a@0: Character
'_' not recommended in node name
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_od_a@0:
Character '_' not recommended in node name
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_b@0: Character
'_' not recommended in node name
Warning (node_name_chars_strict): /soc/pin-controller/sdio_pins_od_b@0:
Character '_' not recommended in node name
Warning (node_name_chars_strict): /gpio_keys: Character '_' not recommended in
node name
Warning (node_name_chars_strict): /mmc_vcard: Character '_' not recommended in
node name
Warning (unit_address_vs_reg): /memory: node has a reg or ranges property, but
no unit name
Warning (unit_address_vs_reg): /soc/pin-controller: node has a reg or ranges
property, but no unit name
Warning (unit_address_vs_reg): /soc/pin-controller/cec@0: node has a unit name,
but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usart1@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/i2c1@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usbotg-hs@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usbotg-hs@1: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/usbotg-fs@0: node has a unit
name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_a@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_od_a@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_b@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /soc/pin-controller/sdio_pins_od_b@0: node has a
unit name, but no reg property
Warning (unit_address_vs_reg): /gpio_keys/button@0: node has a unit name, but
no reg property
Warning (avoid_unnecessary_addr_size): /soc/timers@40001c00: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Warning (avoid_unnecessary_addr_size): /soc/timers@40002000: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Warning (avoid_unnecessary_addr_size): /soc/timers@40014400: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Warning (avoid_unnecessary_addr_size): /soc/timers@40014800: unnecessary
#address-cells/#size-cells without "ranges" or child "reg" property
Signed-off-by: Patrice Chotard
---
Change since v1:
- update nodes name as following: @0 to -0
arch/arm/boot/dts/stm32746g-eval.dts | 9 -
arch/arm/boot/dts/stm32f7-pinctrl.dtsi | 24
arch/arm/boot/dts/stm32f746-disco.dts | 6 +++---
arch/arm/boot/dts/stm32f746.dtsi | 8 +++-
arch/arm/boot/dts/stm32f769-disco.dts | 11 +--
5 files changed, 27 insertions(+), 31 deletions(-)
diff --git a/arch/arm/boot/dts/stm32746g-eval.dts
b/arch/arm/boot/dts/stm32746g-eval.dts
index 8c081eaf20fe..b8a8e06d1427 100644
--- a/arch/arm/boot/dts/stm32746g-eval.dts
+++ b/arch/arm/boot/dts/stm32746g-eval.dts
@@ -54,7 +54,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@c000 {
reg = <0xc000 0x200>;
};
@@ -73,12 +73,11 @@
};
};
- gpio_keys {
+ gpio-keys {
compatible = "gpio-keys";
- #address-cells = <1>;
#size-cells = <0>;
autorepeat;
- button@0 {
+ button-0 {
label = "Wake up";
linux,code = ;
gpios = < 13 0>;
@@ -92,7 +91,7 @@
clock-names = "main_clk";
};
- mmc_vcard: mmc_vcard {
+ mmc_vcard: mmc-vcard {
compatible = "regulator-fixed";
regulator-name = "mmc_vcard";
regulator-min-microvolt = <330>;
diff --git a/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
b/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
index 9314128df185..1cf8a23c2644 100644
--- a/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
+++ b/arch/arm/boot/dts/stm32f7-pinctrl.dtsi
@@ -9,7 +9,7 @@
/ {
soc {
- pinctrl: pin-controller {
+ pinctrl: pin-controller@4002 {
#address-cells = <1>;
#size-cells = <1>;
ranges = <0 0x4002 0x3000>;
@@ -127,7 +127,7 @@
st,bank-name = "GPIOK";
};
- cec_pins_a: cec@0 {
+ cec_pins_a: cec-0 {
pins {
pinmux = ;
/* HDMI
