* [PATCH 01/12] ARM: dts: sunxi: h3/h5: Fix simple-bus unit address format error
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 02/12] ARM: dts: sunxi: h3/h5: Fix i2c2 register address Corentin Labbe
` (10 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch remove leading 0 of unit address and so remove
lots of warning when building DT with W=1.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sunxi-h3-h5.dtsi | 72 +++++++++++++++++++-------------------
1 file changed, 36 insertions(+), 36 deletions(-)
diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
index a8e9b8f378ba..59ddac9f65a7 100644
--- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
+++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
@@ -91,7 +91,7 @@
reg = <0x01c00000 0x1000>;
};
- dma: dma-controller at 01c02000 {
+ dma: dma-controller at 1c02000 {
compatible = "allwinner,sun8i-h3-dma";
reg = <0x01c02000 0x1000>;
interrupts = <GIC_SPI 50 IRQ_TYPE_LEVEL_HIGH>;
@@ -100,7 +100,7 @@
#dma-cells = <1>;
};
- mmc0: mmc at 01c0f000 {
+ mmc0: mmc at 1c0f000 {
/* compatible and clocks are in per SoC .dtsi file */
reg = <0x01c0f000 0x1000>;
resets = <&ccu RST_BUS_MMC0>;
@@ -111,7 +111,7 @@
#size-cells = <0>;
};
- mmc1: mmc at 01c10000 {
+ mmc1: mmc at 1c10000 {
/* compatible and clocks are in per SoC .dtsi file */
reg = <0x01c10000 0x1000>;
resets = <&ccu RST_BUS_MMC1>;
@@ -122,7 +122,7 @@
#size-cells = <0>;
};
- mmc2: mmc at 01c11000 {
+ mmc2: mmc at 1c11000 {
/* compatible and clocks are in per SoC .dtsi file */
reg = <0x01c11000 0x1000>;
resets = <&ccu RST_BUS_MMC2>;
@@ -133,7 +133,7 @@
#size-cells = <0>;
};
- usb_otg: usb at 01c19000 {
+ usb_otg: usb at 1c19000 {
compatible = "allwinner,sun8i-h3-musb";
reg = <0x01c19000 0x400>;
clocks = <&ccu CLK_BUS_OTG>;
@@ -158,7 +158,7 @@
status = "disabled";
};
- usbphy: phy at 01c19400 {
+ usbphy: phy at 1c19400 {
compatible = "allwinner,sun8i-h3-usb-phy";
reg = <0x01c19400 0x2c>,
<0x01c1a800 0x4>,
@@ -190,7 +190,7 @@
#phy-cells = <1>;
};
- ehci0: usb at 01c1a000 {
+ ehci0: usb at 1c1a000 {
compatible = "allwinner,sun8i-h3-ehci", "generic-ehci";
reg = <0x01c1a000 0x100>;
interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_HIGH>;
@@ -199,7 +199,7 @@
status = "disabled";
};
- ohci0: usb at 01c1a400 {
+ ohci0: usb at 1c1a400 {
compatible = "allwinner,sun8i-h3-ohci", "generic-ohci";
reg = <0x01c1a400 0x100>;
interrupts = <GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH>;
@@ -209,7 +209,7 @@
status = "disabled";
};
- ehci1: usb at 01c1b000 {
+ ehci1: usb at 1c1b000 {
compatible = "allwinner,sun8i-h3-ehci", "generic-ehci";
reg = <0x01c1b000 0x100>;
interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>;
@@ -220,7 +220,7 @@
status = "disabled";
};
- ohci1: usb at 01c1b400 {
+ ohci1: usb at 1c1b400 {
compatible = "allwinner,sun8i-h3-ohci", "generic-ohci";
reg = <0x01c1b400 0x100>;
interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>;
@@ -232,7 +232,7 @@
status = "disabled";
};
- ehci2: usb at 01c1c000 {
+ ehci2: usb at 1c1c000 {
compatible = "allwinner,sun8i-h3-ehci", "generic-ehci";
reg = <0x01c1c000 0x100>;
interrupts = <GIC_SPI 76 IRQ_TYPE_LEVEL_HIGH>;
@@ -243,7 +243,7 @@
status = "disabled";
};
- ohci2: usb at 01c1c400 {
+ ohci2: usb at 1c1c400 {
compatible = "allwinner,sun8i-h3-ohci", "generic-ohci";
reg = <0x01c1c400 0x100>;
interrupts = <GIC_SPI 77 IRQ_TYPE_LEVEL_HIGH>;
@@ -255,7 +255,7 @@
status = "disabled";
};
- ehci3: usb at 01c1d000 {
+ ehci3: usb at 1c1d000 {
compatible = "allwinner,sun8i-h3-ehci", "generic-ehci";
reg = <0x01c1d000 0x100>;
interrupts = <GIC_SPI 78 IRQ_TYPE_LEVEL_HIGH>;
@@ -266,7 +266,7 @@
status = "disabled";
};
- ohci3: usb at 01c1d400 {
+ ohci3: usb at 1c1d400 {
compatible = "allwinner,sun8i-h3-ohci", "generic-ohci";
reg = <0x01c1d400 0x100>;
interrupts = <GIC_SPI 79 IRQ_TYPE_LEVEL_HIGH>;
@@ -278,7 +278,7 @@
status = "disabled";
};
- ccu: clock at 01c20000 {
+ ccu: clock at 1c20000 {
/* compatible is in per SoC .dtsi file */
reg = <0x01c20000 0x400>;
clocks = <&osc24M>, <&osc32k>;
@@ -287,7 +287,7 @@
#reset-cells = <1>;
};
- pio: pinctrl at 01c20800 {
+ pio: pinctrl at 1c20800 {
/* compatible is in per SoC .dtsi file */
reg = <0x01c20800 0x400>;
interrupts = <GIC_SPI 11 IRQ_TYPE_LEVEL_HIGH>,
@@ -400,7 +400,7 @@
};
};
- timer at 01c20c00 {
+ timer at 1c20c00 {
compatible = "allwinner,sun4i-a10-timer";
reg = <0x01c20c00 0xa0>;
interrupts = <GIC_SPI 18 IRQ_TYPE_LEVEL_HIGH>,
@@ -453,7 +453,7 @@
};
};
- spi0: spi at 01c68000 {
+ spi0: spi at 1c68000 {
compatible = "allwinner,sun8i-h3-spi";
reg = <0x01c68000 0x1000>;
interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
@@ -469,7 +469,7 @@
#size-cells = <0>;
};
- spi1: spi at 01c69000 {
+ spi1: spi at 1c69000 {
compatible = "allwinner,sun8i-h3-spi";
reg = <0x01c69000 0x1000>;
interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
@@ -485,13 +485,13 @@
#size-cells = <0>;
};
- wdt0: watchdog at 01c20ca0 {
+ wdt0: watchdog at 1c20ca0 {
compatible = "allwinner,sun6i-a31-wdt";
reg = <0x01c20ca0 0x20>;
interrupts = <GIC_SPI 25 IRQ_TYPE_LEVEL_HIGH>;
};
- spdif: spdif at 01c21000 {
+ spdif: spdif at 1c21000 {
#sound-dai-cells = <0>;
compatible = "allwinner,sun8i-h3-spdif";
reg = <0x01c21000 0x400>;
@@ -504,7 +504,7 @@
status = "disabled";
};
- pwm: pwm at 01c21400 {
+ pwm: pwm at 1c21400 {
compatible = "allwinner,sun8i-h3-pwm";
reg = <0x01c21400 0x8>;
clocks = <&osc24M>;
@@ -512,7 +512,7 @@
status = "disabled";
};
- i2s0: i2s at 01c22000 {
+ i2s0: i2s at 1c22000 {
#sound-dai-cells = <0>;
compatible = "allwinner,sun8i-h3-i2s";
reg = <0x01c22000 0x400>;
@@ -525,7 +525,7 @@
status = "disabled";
};
- i2s1: i2s at 01c22400 {
+ i2s1: i2s at 1c22400 {
#sound-dai-cells = <0>;
compatible = "allwinner,sun8i-h3-i2s";
reg = <0x01c22400 0x400>;
@@ -538,7 +538,7 @@
status = "disabled";
};
- codec: codec at 01c22c00 {
+ codec: codec at 1c22c00 {
#sound-dai-cells = <0>;
compatible = "allwinner,sun8i-h3-codec";
reg = <0x01c22c00 0x400>;
@@ -552,7 +552,7 @@
status = "disabled";
};
- uart0: serial at 01c28000 {
+ uart0: serial at 1c28000 {
compatible = "snps,dw-apb-uart";
reg = <0x01c28000 0x400>;
interrupts = <GIC_SPI 0 IRQ_TYPE_LEVEL_HIGH>;
@@ -565,7 +565,7 @@
status = "disabled";
};
- uart1: serial at 01c28400 {
+ uart1: serial at 1c28400 {
compatible = "snps,dw-apb-uart";
reg = <0x01c28400 0x400>;
interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
@@ -578,7 +578,7 @@
status = "disabled";
};
- uart2: serial at 01c28800 {
+ uart2: serial at 1c28800 {
compatible = "snps,dw-apb-uart";
reg = <0x01c28800 0x400>;
interrupts = <GIC_SPI 2 IRQ_TYPE_LEVEL_HIGH>;
@@ -591,7 +591,7 @@
status = "disabled";
};
- uart3: serial at 01c28c00 {
+ uart3: serial at 1c28c00 {
compatible = "snps,dw-apb-uart";
reg = <0x01c28c00 0x400>;
interrupts = <GIC_SPI 3 IRQ_TYPE_LEVEL_HIGH>;
@@ -604,7 +604,7 @@
status = "disabled";
};
- i2c0: i2c at 01c2ac00 {
+ i2c0: i2c at 1c2ac00 {
compatible = "allwinner,sun6i-a31-i2c";
reg = <0x01c2ac00 0x400>;
interrupts = <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
@@ -617,7 +617,7 @@
#size-cells = <0>;
};
- i2c1: i2c at 01c2b000 {
+ i2c1: i2c at 1c2b000 {
compatible = "allwinner,sun6i-a31-i2c";
reg = <0x01c2b000 0x400>;
interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
@@ -643,7 +643,7 @@
#size-cells = <0>;
};
- gic: interrupt-controller at 01c81000 {
+ gic: interrupt-controller at 1c81000 {
compatible = "arm,gic-400";
reg = <0x01c81000 0x1000>,
<0x01c82000 0x2000>,
@@ -654,7 +654,7 @@
interrupts = <GIC_PPI 9 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>;
};
- rtc: rtc at 01f00000 {
+ rtc: rtc at 1f00000 {
compatible = "allwinner,sun6i-a31-rtc";
reg = <0x01f00000 0x54>;
interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>,
@@ -671,12 +671,12 @@
#reset-cells = <1>;
};
- codec_analog: codec-analog at 01f015c0 {
+ codec_analog: codec-analog at 1f015c0 {
compatible = "allwinner,sun8i-h3-codec-analog";
reg = <0x01f015c0 0x4>;
};
- ir: ir at 01f02000 {
+ ir: ir at 1f02000 {
compatible = "allwinner,sun5i-a13-ir";
clocks = <&r_ccu CLK_APB0_IR>, <&r_ccu CLK_IR>;
clock-names = "apb", "ir";
@@ -686,7 +686,7 @@
status = "disabled";
};
- r_pio: pinctrl at 01f02c00 {
+ r_pio: pinctrl at 1f02c00 {
compatible = "allwinner,sun8i-h3-r-pinctrl";
reg = <0x01f02c00 0x400>;
interrupts = <GIC_SPI 45 IRQ_TYPE_LEVEL_HIGH>;
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 02/12] ARM: dts: sunxi: h3/h5: Fix i2c2 register address
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
2017-09-21 17:21 ` [PATCH 01/12] ARM: dts: sunxi: h3/h5: Fix simple-bus unit address format error Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-22 7:49 ` Maxime Ripard
2017-09-21 17:21 ` [PATCH 03/12] ARM: dts: sunxi: h3/h5: Fix node with unit name and no reg property Corentin Labbe
` (9 subsequent siblings)
11 siblings, 1 reply; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
The unit address and register address does not match.
This patch fix the register address with the good one.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sunxi-h3-h5.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
index 59ddac9f65a7..37464d4d07f6 100644
--- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
+++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
@@ -630,9 +630,9 @@
#size-cells = <0>;
};
- i2c2: i2c at 01c2b400 {
+ i2c2: i2c at 1c2b400 {
compatible = "allwinner,sun6i-a31-i2c";
- reg = <0x01c2b000 0x400>;
+ reg = <0x1c2b400 0x400>;
interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&ccu CLK_BUS_I2C2>;
resets = <&ccu RST_BUS_I2C2>;
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 02/12] ARM: dts: sunxi: h3/h5: Fix i2c2 register address
2017-09-21 17:21 ` [PATCH 02/12] ARM: dts: sunxi: h3/h5: Fix i2c2 register address Corentin Labbe
@ 2017-09-22 7:49 ` Maxime Ripard
2017-09-22 8:02 ` Corentin Labbe
0 siblings, 1 reply; 15+ messages in thread
From: Maxime Ripard @ 2017-09-22 7:49 UTC (permalink / raw)
To: linux-arm-kernel
On Thu, Sep 21, 2017 at 05:21:41PM +0000, Corentin Labbe wrote:
> The unit address and register address does not match.
> This patch fix the register address with the good one.
>
> Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
> ---
> arch/arm/boot/dts/sunxi-h3-h5.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> index 59ddac9f65a7..37464d4d07f6 100644
> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> @@ -630,9 +630,9 @@
> #size-cells = <0>;
> };
>
> - i2c2: i2c at 01c2b400 {
> + i2c2: i2c at 1c2b400 {
This one should be merged into the previous patch.
The serie looks good otherwise, onec it's fixed, you can add my
Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>
Maxime
--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170922/62b866d5/attachment.sig>
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH 02/12] ARM: dts: sunxi: h3/h5: Fix i2c2 register address
2017-09-22 7:49 ` Maxime Ripard
@ 2017-09-22 8:02 ` Corentin Labbe
0 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-22 8:02 UTC (permalink / raw)
To: linux-arm-kernel
On Fri, Sep 22, 2017 at 09:49:45AM +0200, Maxime Ripard wrote:
> On Thu, Sep 21, 2017 at 05:21:41PM +0000, Corentin Labbe wrote:
> > The unit address and register address does not match.
> > This patch fix the register address with the good one.
> >
> > Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
> > ---
> > arch/arm/boot/dts/sunxi-h3-h5.dtsi | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> > index 59ddac9f65a7..37464d4d07f6 100644
> > --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> > +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> > @@ -630,9 +630,9 @@
> > #size-cells = <0>;
> > };
> >
> > - i2c2: i2c at 01c2b400 {
> > + i2c2: i2c at 1c2b400 {
>
> This one should be merged into the previous patch.
>
> The serie looks good otherwise, onec it's fixed, you can add my
> Acked-by: Maxime Ripard <maxime.ripard@free-electrons.com>
>
> Maxime
>
For me this patch could be backported (since it fixes a bug) so I have put it appart.
But as requsted, I will merge it.
Regards
^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH 03/12] ARM: dts: sunxi: h3/h5: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
2017-09-21 17:21 ` [PATCH 01/12] ARM: dts: sunxi: h3/h5: Fix simple-bus unit address format error Corentin Labbe
2017-09-21 17:21 ` [PATCH 02/12] ARM: dts: sunxi: h3/h5: Fix i2c2 register address Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 04/12] ARM: dts: nanopi: " Corentin Labbe
` (8 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sunxi-h3-h5.dtsi | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
index 37464d4d07f6..917e8c7434b2 100644
--- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
+++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
@@ -322,7 +322,7 @@
function = "i2c2";
};
- mmc0_pins_a: mmc0 at 0 {
+ mmc0_pins_a: mmc0 {
pins = "PF0", "PF1", "PF2", "PF3",
"PF4", "PF5";
function = "mmc0";
@@ -330,13 +330,13 @@
bias-pull-up;
};
- mmc0_cd_pin: mmc0_cd_pin at 0 {
+ mmc0_cd_pin: mmc0_cd_pin {
pins = "PF6";
function = "gpio_in";
bias-pull-up;
};
- mmc1_pins_a: mmc1 at 0 {
+ mmc1_pins_a: mmc1 {
pins = "PG0", "PG1", "PG2", "PG3",
"PG4", "PG5";
function = "mmc1";
@@ -354,7 +354,7 @@
bias-pull-up;
};
- spdif_tx_pins_a: spdif at 0 {
+ spdif_tx_pins_a: spdif {
pins = "PA17";
function = "spdif";
};
@@ -369,7 +369,7 @@
function = "spi1";
};
- uart0_pins_a: uart0 at 0 {
+ uart0_pins_a: uart0 {
pins = "PA4", "PA5";
function = "uart0";
};
@@ -697,7 +697,7 @@
interrupt-controller;
#interrupt-cells = <3>;
- ir_pins_a: ir at 0 {
+ ir_pins_a: ir {
pins = "PL11";
function = "s_cir_rx";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 04/12] ARM: dts: nanopi: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (2 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 03/12] ARM: dts: sunxi: h3/h5: Fix node with unit name and no reg property Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 05/12] ARM: dts: orangepi2: " Corentin Labbe
` (7 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-h3-nanopi.dtsi | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/sun8i-h3-nanopi.dtsi b/arch/arm/boot/dts/sun8i-h3-nanopi.dtsi
index c6decee41a27..7646e331bd29 100644
--- a/arch/arm/boot/dts/sun8i-h3-nanopi.dtsi
+++ b/arch/arm/boot/dts/sun8i-h3-nanopi.dtsi
@@ -81,7 +81,7 @@
pinctrl-names = "default";
pinctrl-0 = <&sw_r_npi>;
- k1 at 0 {
+ k1 {
label = "k1";
linux,code = <KEY_POWER>;
gpios = <&r_pio 0 3 GPIO_ACTIVE_LOW>;
@@ -108,19 +108,19 @@
};
&pio {
- leds_npi: led_pins at 0 {
+ leds_npi: led_pins {
pins = "PA10";
function = "gpio_out";
};
};
&r_pio {
- leds_r_npi: led_pins at 0 {
+ leds_r_npi: led_pins {
pins = "PL10";
function = "gpio_out";
};
- sw_r_npi: key_pins at 0 {
+ sw_r_npi: key_pins {
pins = "PL3";
function = "gpio_in";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 05/12] ARM: dts: orangepi2: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (3 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 04/12] ARM: dts: nanopi: " Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 06/12] ARM: dts: sun8i: orangepi-lite: " Corentin Labbe
` (6 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-h3-orangepi-2.dts | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-2.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-2.dts
index 17cdeae19c6f..b20be95b49d5 100644
--- a/arch/arm/boot/dts/sun8i-h3-orangepi-2.dts
+++ b/arch/arm/boot/dts/sun8i-h3-orangepi-2.dts
@@ -160,24 +160,24 @@
};
&pio {
- leds_opc: led_pins at 0 {
+ leds_opc: led_pins {
pins = "PA15";
function = "gpio_out";
};
};
&r_pio {
- leds_r_opc: led_pins at 0 {
+ leds_r_opc: led_pins {
pins = "PL10";
function = "gpio_out";
};
- sw_r_opc: key_pins at 0 {
+ sw_r_opc: key_pins {
pins = "PL3", "PL4";
function = "gpio_in";
};
- wifi_pwrseq_pin_orangepi: wifi_pwrseq_pin at 0 {
+ wifi_pwrseq_pin_orangepi: wifi_pwrseq_pin {
pins = "PL7";
function = "gpio_out";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 06/12] ARM: dts: sun8i: orangepi-lite: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (4 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 05/12] ARM: dts: orangepi2: " Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 07/12] ARM: dts: sun8i: orangepi one: " Corentin Labbe
` (5 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-h3-orangepi-lite.dts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-lite.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-lite.dts
index 9b47a0def740..a70a1daf4e2c 100644
--- a/arch/arm/boot/dts/sun8i-h3-orangepi-lite.dts
+++ b/arch/arm/boot/dts/sun8i-h3-orangepi-lite.dts
@@ -141,19 +141,19 @@
};
&pio {
- leds_opc: led_pins at 0 {
+ leds_opc: led_pins {
pins = "PA15";
function = "gpio_out";
};
};
&r_pio {
- leds_r_opc: led_pins at 0 {
+ leds_r_opc: led_pins {
pins = "PL10";
function = "gpio_out";
};
- sw_r_opc: key_pins at 0 {
+ sw_r_opc: key_pins {
pins = "PL3";
function = "gpio_in";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 07/12] ARM: dts: sun8i: orangepi one: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (5 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 06/12] ARM: dts: sun8i: orangepi-lite: " Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 08/12] ARM: dts: sun8i: orangepipc: " Corentin Labbe
` (4 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-h3-orangepi-one.dts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-one.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-one.dts
index 6880268e8b87..82e5d28cd698 100644
--- a/arch/arm/boot/dts/sun8i-h3-orangepi-one.dts
+++ b/arch/arm/boot/dts/sun8i-h3-orangepi-one.dts
@@ -124,19 +124,19 @@
};
&pio {
- leds_opc: led_pins at 0 {
+ leds_opc: led_pins {
pins = "PA15";
function = "gpio_out";
};
};
&r_pio {
- leds_r_opc: led_pins at 0 {
+ leds_r_opc: led_pins {
pins = "PL10";
function = "gpio_out";
};
- sw_r_opc: key_pins at 0 {
+ sw_r_opc: key_pins {
pins = "PL3";
function = "gpio_in";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 08/12] ARM: dts: sun8i: orangepipc: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (6 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 07/12] ARM: dts: sun8i: orangepi one: " Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 09/12] ARM: dts: sun8i: orangepi-plus: " Corentin Labbe
` (3 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts
index f5f0f15a2088..6d98bcfbe877 100644
--- a/arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts
+++ b/arch/arm/boot/dts/sun8i-h3-orangepi-pc.dts
@@ -158,19 +158,19 @@
};
&pio {
- leds_opc: led_pins at 0 {
+ leds_opc: led_pins {
pins = "PA15";
function = "gpio_out";
};
};
&r_pio {
- leds_r_opc: led_pins at 0 {
+ leds_r_opc: led_pins {
pins = "PL10";
function = "gpio_out";
};
- sw_r_opc: key_pins at 0 {
+ sw_r_opc: key_pins {
pins = "PL3";
function = "gpio_in";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 09/12] ARM: dts: sun8i: orangepi-plus: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (7 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 08/12] ARM: dts: sun8i: orangepipc: " Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 10/12] arm64: allwinner: a64: Fix simple-bus unit address format error Corentin Labbe
` (2 subsequent siblings)
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts b/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts
index 331ed683ac62..119732505844 100644
--- a/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts
+++ b/arch/arm/boot/dts/sun8i-h3-orangepi-plus.dts
@@ -114,7 +114,7 @@
};
&pio {
- usb3_vbus_pin_a: usb3_vbus_pin at 0 {
+ usb3_vbus_pin_a: usb3_vbus_pin {
pins = "PG11";
function = "gpio_out";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 10/12] arm64: allwinner: a64: Fix simple-bus unit address format error
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (8 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 09/12] ARM: dts: sun8i: orangepi-plus: " Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 11/12] arm64: allwinner: a64: Fix node with unit name and no reg property Corentin Labbe
2017-09-21 17:21 ` [PATCH 12/12] ARM: sun8i: a83t: Fix simple-bus unit address format error Corentin Labbe
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch remove leading 0 of unit address and so remove
lots of warning when building DT with W=1.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
index e30476f05802..662e8b7981b5 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
@@ -190,7 +190,7 @@
status = "disabled";
};
- usb_otg: usb at 01c19000 {
+ usb_otg: usb at 1c19000 {
compatible = "allwinner,sun8i-a33-musb";
reg = <0x01c19000 0x0400>;
clocks = <&ccu CLK_BUS_OTG>;
@@ -203,7 +203,7 @@
status = "disabled";
};
- usbphy: phy at 01c19400 {
+ usbphy: phy at 1c19400 {
compatible = "allwinner,sun50i-a64-usb-phy";
reg = <0x01c19400 0x14>,
<0x01c1a800 0x4>,
@@ -223,7 +223,7 @@
#phy-cells = <1>;
};
- ehci0: usb at 01c1a000 {
+ ehci0: usb at 1c1a000 {
compatible = "allwinner,sun50i-a64-ehci", "generic-ehci";
reg = <0x01c1a000 0x100>;
interrupts = <GIC_SPI 72 IRQ_TYPE_LEVEL_HIGH>;
@@ -235,7 +235,7 @@
status = "disabled";
};
- ohci0: usb at 01c1a400 {
+ ohci0: usb at 1c1a400 {
compatible = "allwinner,sun50i-a64-ohci", "generic-ohci";
reg = <0x01c1a400 0x100>;
interrupts = <GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH>;
@@ -245,7 +245,7 @@
status = "disabled";
};
- ehci1: usb at 01c1b000 {
+ ehci1: usb at 1c1b000 {
compatible = "allwinner,sun50i-a64-ehci", "generic-ehci";
reg = <0x01c1b000 0x100>;
interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>;
@@ -259,7 +259,7 @@
status = "disabled";
};
- ohci1: usb at 01c1b400 {
+ ohci1: usb at 1c1b400 {
compatible = "allwinner,sun50i-a64-ohci", "generic-ohci";
reg = <0x01c1b400 0x100>;
interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>;
@@ -271,7 +271,7 @@
status = "disabled";
};
- ccu: clock at 01c20000 {
+ ccu: clock at 1c20000 {
compatible = "allwinner,sun50i-a64-ccu";
reg = <0x01c20000 0x400>;
clocks = <&osc24M>, <&osc32k>;
@@ -472,7 +472,7 @@
};
- spi0: spi at 01c68000 {
+ spi0: spi at 1c68000 {
compatible = "allwinner,sun8i-h3-spi";
reg = <0x01c68000 0x1000>;
interrupts = <GIC_SPI 65 IRQ_TYPE_LEVEL_HIGH>;
@@ -487,7 +487,7 @@
#size-cells = <0>;
};
- spi1: spi at 01c69000 {
+ spi1: spi at 1c69000 {
compatible = "allwinner,sun8i-h3-spi";
reg = <0x01c69000 0x1000>;
interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
@@ -560,7 +560,7 @@
#reset-cells = <1>;
};
- r_pio: pinctrl at 01f02c00 {
+ r_pio: pinctrl at 1f02c00 {
compatible = "allwinner,sun50i-a64-r-pinctrl";
reg = <0x01f02c00 0x400>;
interrupts = <GIC_SPI 45 IRQ_TYPE_LEVEL_HIGH>;
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 11/12] arm64: allwinner: a64: Fix node with unit name and no reg property
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (9 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 10/12] arm64: allwinner: a64: Fix simple-bus unit address format error Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
2017-09-21 17:21 ` [PATCH 12/12] ARM: sun8i: a83t: Fix simple-bus unit address format error Corentin Labbe
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch fix the warning "xxx has a unit name, but no reg property" by
removing "@0" from such node
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
index 662e8b7981b5..b02a8476b0c8 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
+++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi
@@ -347,7 +347,7 @@
function = "spi1";
};
- uart0_pins_a: uart0 at 0 {
+ uart0_pins_a: uart0 {
pins = "PB8", "PB9";
function = "uart0";
};
@@ -571,7 +571,7 @@
interrupt-controller;
#interrupt-cells = <3>;
- r_rsb_pins: rsb at 0 {
+ r_rsb_pins: rsb {
pins = "PL0", "PL1";
function = "s_rsb";
};
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread* [PATCH 12/12] ARM: sun8i: a83t: Fix simple-bus unit address format error
2017-09-21 17:21 [PATCH 00/12] ARM: dts: sunxi: Fix DT build warnings Corentin Labbe
` (10 preceding siblings ...)
2017-09-21 17:21 ` [PATCH 11/12] arm64: allwinner: a64: Fix node with unit name and no reg property Corentin Labbe
@ 2017-09-21 17:21 ` Corentin Labbe
11 siblings, 0 replies; 15+ messages in thread
From: Corentin Labbe @ 2017-09-21 17:21 UTC (permalink / raw)
To: linux-arm-kernel
This patch remove leading 0 of unit address and so remove
lots of warning when building DT with W=1.
Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
---
arch/arm/boot/dts/sun8i-a83t.dtsi | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index 7f99ef37f61f..dd6a393725c6 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -248,7 +248,7 @@
status = "disabled";
};
- usb_otg: usb at 01c19000 {
+ usb_otg: usb at 1c19000 {
compatible = "allwinner,sun8i-a83t-musb",
"allwinner,sun8i-a33-musb";
reg = <0x01c19000 0x0400>;
@@ -436,7 +436,7 @@
status = "disabled";
};
- uart0: serial at 01c28000 {
+ uart0: serial at 1c28000 {
compatible = "snps,dw-apb-uart";
reg = <0x01c28000 0x400>;
interrupts = <GIC_SPI 0 IRQ_TYPE_LEVEL_HIGH>;
@@ -447,7 +447,7 @@
status = "disabled";
};
- uart1: serial at 01c28400 {
+ uart1: serial at 1c28400 {
compatible = "snps,dw-apb-uart";
reg = <0x01c28400 0x400>;
interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
--
2.13.5
^ permalink raw reply related [flat|nested] 15+ messages in thread