* [PATCH 1/4] ARM: dts: stm32: fix memory nodes to match with DT validation tool
2019-10-07 14:33 [PATCH 0/4] ARM: dts: stm32: Add fixes to be compliant with DT validation tool Alexandre Torgue
@ 2019-10-07 14:33 ` Alexandre Torgue
2019-10-07 14:34 ` [PATCH 2/4] ARM: dts: stm32: fix joystick node on stm32f746 and stm32mp157c eval boards Alexandre Torgue
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Alexandre Torgue @ 2019-10-07 14:33 UTC (permalink / raw)
To: Maxime Coquelin, arnd, robh+dt, mark.rutland
Cc: linux-arm-kernel, devicetree, Alexandre Torgue, linux-stm32,
linux-kernel
DT validation ("make dtbs_check") has shown that some memory nodes were not
correctly written. This commit fixes this kind of issue:
"stm32f746-disco.dt.yaml: /: memory: False schema does not allow
{'device_type': ['memory'], 'reg': [[3221225472, 8388608]]}"
Signed-off-by: Alexandre Torgue <alexandre.torgue@st.com>
diff --git a/arch/arm/boot/dts/stm32429i-eval.dts b/arch/arm/boot/dts/stm32429i-eval.dts
index ba08624c6237..36ac61d0828c 100644
--- a/arch/arm/boot/dts/stm32429i-eval.dts
+++ b/arch/arm/boot/dts/stm32429i-eval.dts
@@ -60,7 +60,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@00000000 {
device_type = "memory";
reg = <0x00000000 0x2000000>;
};
diff --git a/arch/arm/boot/dts/stm32746g-eval.dts b/arch/arm/boot/dts/stm32746g-eval.dts
index 2b1664884ae7..d7bb2027cfaa 100644
--- a/arch/arm/boot/dts/stm32746g-eval.dts
+++ b/arch/arm/boot/dts/stm32746g-eval.dts
@@ -55,7 +55,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@c0000000 {
device_type = "memory";
reg = <0xc0000000 0x2000000>;
};
diff --git a/arch/arm/boot/dts/stm32f429-disco.dts b/arch/arm/boot/dts/stm32f429-disco.dts
index e19d0fe7dbda..30c0f6717871 100644
--- a/arch/arm/boot/dts/stm32f429-disco.dts
+++ b/arch/arm/boot/dts/stm32f429-disco.dts
@@ -59,7 +59,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@90000000 {
device_type = "memory";
reg = <0x90000000 0x800000>;
};
diff --git a/arch/arm/boot/dts/stm32f469-disco.dts b/arch/arm/boot/dts/stm32f469-disco.dts
index a3ff04940aec..539aa5903fdd 100644
--- a/arch/arm/boot/dts/stm32f469-disco.dts
+++ b/arch/arm/boot/dts/stm32f469-disco.dts
@@ -60,7 +60,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@00000000 {
device_type = "memory";
reg = <0x00000000 0x1000000>;
};
diff --git a/arch/arm/boot/dts/stm32f746-disco.dts b/arch/arm/boot/dts/stm32f746-disco.dts
index 0ba9c5b08ab9..569d23cc61e5 100644
--- a/arch/arm/boot/dts/stm32f746-disco.dts
+++ b/arch/arm/boot/dts/stm32f746-disco.dts
@@ -55,7 +55,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@c0000000 {
device_type = "memory";
reg = <0xC0000000 0x800000>;
};
diff --git a/arch/arm/boot/dts/stm32f769-disco.dts b/arch/arm/boot/dts/stm32f769-disco.dts
index 6f1d0ac8c31c..1626e00bb2cb 100644
--- a/arch/arm/boot/dts/stm32f769-disco.dts
+++ b/arch/arm/boot/dts/stm32f769-disco.dts
@@ -55,7 +55,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@c0000000 {
device_type = "memory";
reg = <0xC0000000 0x1000000>;
};
diff --git a/arch/arm/boot/dts/stm32h743i-disco.dts b/arch/arm/boot/dts/stm32h743i-disco.dts
index 3acd2e9c434e..e446d311c520 100644
--- a/arch/arm/boot/dts/stm32h743i-disco.dts
+++ b/arch/arm/boot/dts/stm32h743i-disco.dts
@@ -53,7 +53,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@d0000000 {
device_type = "memory";
reg = <0xd0000000 0x2000000>;
};
diff --git a/arch/arm/boot/dts/stm32h743i-eval.dts b/arch/arm/boot/dts/stm32h743i-eval.dts
index e4d3c58f3d97..8f398178f5e5 100644
--- a/arch/arm/boot/dts/stm32h743i-eval.dts
+++ b/arch/arm/boot/dts/stm32h743i-eval.dts
@@ -53,7 +53,7 @@
stdout-path = "serial0:115200n8";
};
- memory {
+ memory@d0000000 {
device_type = "memory";
reg = <0xd0000000 0x2000000>;
};
diff --git a/arch/arm/boot/dts/stm32mp157a-dk1.dts b/arch/arm/boot/dts/stm32mp157a-dk1.dts
index 0615d1c8a6fc..e2aa45e9ba57 100644
--- a/arch/arm/boot/dts/stm32mp157a-dk1.dts
+++ b/arch/arm/boot/dts/stm32mp157a-dk1.dts
@@ -25,6 +25,7 @@
};
memory@c0000000 {
+ device_type = "memory";
reg = <0xc0000000 0x20000000>;
};
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/4] ARM: dts: stm32: fix joystick node on stm32f746 and stm32mp157c eval boards
2019-10-07 14:33 [PATCH 0/4] ARM: dts: stm32: Add fixes to be compliant with DT validation tool Alexandre Torgue
2019-10-07 14:33 ` [PATCH 1/4] ARM: dts: stm32: fix memory nodes to match " Alexandre Torgue
@ 2019-10-07 14:34 ` Alexandre Torgue
2019-10-07 14:34 ` [PATCH 3/4] ARM: dts: stm32: remove usb phy-names entries on stm32mp157c-ev1 Alexandre Torgue
2019-10-07 14:34 ` [PATCH 4/4] ARM: dts: stm32: fix regulator-sd_switch node on stm32mp157c-ed1 board Alexandre Torgue
3 siblings, 0 replies; 5+ messages in thread
From: Alexandre Torgue @ 2019-10-07 14:34 UTC (permalink / raw)
To: Maxime Coquelin, arnd, robh+dt, mark.rutland
Cc: linux-arm-kernel, devicetree, Alexandre Torgue, linux-stm32,
linux-kernel
"#size-cells" entry is not needed for "gpio-keys" driver. Indeed "reg"
entry is not used. This commit will fix a warnings seen by DT validation
tool.
Signed-off-by: Alexandre Torgue <alexandre.torgue@st.com>
diff --git a/arch/arm/boot/dts/stm32746g-eval.dts b/arch/arm/boot/dts/stm32746g-eval.dts
index d7bb2027cfaa..fcc804e3c158 100644
--- a/arch/arm/boot/dts/stm32746g-eval.dts
+++ b/arch/arm/boot/dts/stm32746g-eval.dts
@@ -95,7 +95,6 @@
joystick {
compatible = "gpio-keys";
- #size-cells = <0>;
pinctrl-0 = <&joystick_pins>;
pinctrl-names = "default";
button-0 {
diff --git a/arch/arm/boot/dts/stm32mp157c-ev1.dts b/arch/arm/boot/dts/stm32mp157c-ev1.dts
index 89d29b50c3f4..6287db532e7d 100644
--- a/arch/arm/boot/dts/stm32mp157c-ev1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ev1.dts
@@ -32,7 +32,6 @@
joystick {
compatible = "gpio-keys";
- #size-cells = <0>;
pinctrl-0 = <&joystick_pins>;
pinctrl-names = "default";
button-0 {
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 3/4] ARM: dts: stm32: remove usb phy-names entries on stm32mp157c-ev1
2019-10-07 14:33 [PATCH 0/4] ARM: dts: stm32: Add fixes to be compliant with DT validation tool Alexandre Torgue
2019-10-07 14:33 ` [PATCH 1/4] ARM: dts: stm32: fix memory nodes to match " Alexandre Torgue
2019-10-07 14:34 ` [PATCH 2/4] ARM: dts: stm32: fix joystick node on stm32f746 and stm32mp157c eval boards Alexandre Torgue
@ 2019-10-07 14:34 ` Alexandre Torgue
2019-10-07 14:34 ` [PATCH 4/4] ARM: dts: stm32: fix regulator-sd_switch node on stm32mp157c-ed1 board Alexandre Torgue
3 siblings, 0 replies; 5+ messages in thread
From: Alexandre Torgue @ 2019-10-07 14:34 UTC (permalink / raw)
To: Maxime Coquelin, arnd, robh+dt, mark.rutland
Cc: linux-arm-kernel, devicetree, Alexandre Torgue, linux-stm32,
linux-kernel
"phy-names" entries are not used. To be compliant with DT validation
tool, those entries have to be remove.
Signed-off-by: Alexandre Torgue <alexandre.torgue@st.com>
diff --git a/arch/arm/boot/dts/stm32mp157c-ev1.dts b/arch/arm/boot/dts/stm32mp157c-ev1.dts
index 6287db532e7d..2baae5f25e2c 100644
--- a/arch/arm/boot/dts/stm32mp157c-ev1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ev1.dts
@@ -343,14 +343,12 @@
&usbh_ehci {
phys = <&usbphyc_port0>;
- phy-names = "usb";
status = "okay";
};
&usbotg_hs {
dr_mode = "peripheral";
phys = <&usbphyc_port1 0>;
- phy-names = "usb2-phy";
status = "okay";
};
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 4/4] ARM: dts: stm32: fix regulator-sd_switch node on stm32mp157c-ed1 board
2019-10-07 14:33 [PATCH 0/4] ARM: dts: stm32: Add fixes to be compliant with DT validation tool Alexandre Torgue
` (2 preceding siblings ...)
2019-10-07 14:34 ` [PATCH 3/4] ARM: dts: stm32: remove usb phy-names entries on stm32mp157c-ev1 Alexandre Torgue
@ 2019-10-07 14:34 ` Alexandre Torgue
3 siblings, 0 replies; 5+ messages in thread
From: Alexandre Torgue @ 2019-10-07 14:34 UTC (permalink / raw)
To: Maxime Coquelin, arnd, robh+dt, mark.rutland
Cc: linux-arm-kernel, devicetree, Alexandre Torgue, linux-stm32,
linux-kernel
This commit fixes regulator-sd_switch node in order to be compliant to
DT validation schema.
Signed-off-by: Alexandre Torgue <alexandre.torgue@st.com>
diff --git a/arch/arm/boot/dts/stm32mp157c-ed1.dts b/arch/arm/boot/dts/stm32mp157c-ed1.dts
index 1d426ea8bdaf..329853d9b1de 100644
--- a/arch/arm/boot/dts/stm32mp157c-ed1.dts
+++ b/arch/arm/boot/dts/stm32mp157c-ed1.dts
@@ -100,7 +100,8 @@
gpios = <&gpiof 14 GPIO_ACTIVE_HIGH>;
gpios-states = <0>;
- states = <1800000 0x1 2900000 0x0>;
+ states = <1800000 0x1>,
+ <2900000 0x0>;
};
};
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread