[PATCH v2 09/23] ARM: dts: sh73a0: Fix W=1 dtc warnings

[PATCH v2 09/23] ARM: dts: sh73a0: Fix W=1 dtc warnings

[Geert Uytterhoeven]

Warning (unit_address_vs_reg): Node /cache-controller has a reg or ranges property, but no unit name

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
v2:
  - No changes.
---
 arch/arm/boot/dts/sh73a0.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
index c4f434cdec60f075..032fe2f14b16998f 100644
--- a/arch/arm/boot/dts/sh73a0.dtsi
+++ b/arch/arm/boot/dts/sh73a0.dtsi
@@ -55,7 +55,7 @@
 		      <0xf0000100 0x100>;
 	};
 
-	L2: cache-controller {
+	L2: cache-controller at f0100000 {
 		compatible = "arm,pl310-cache";
 		reg = <0xf0100000 0x1000>;
 		interrupts = <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>;
-- 
1.9.1

[PATCH v2 23/23] arm64: dts: salvator-x: Fix W=1 dtc warnings

[Geert Uytterhoeven]

Warning (unit_address_vs_reg): Node /regulator at 1 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /regulator at 2 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /regulator at 3 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /regulator at 4 has a unit name, but no reg property

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
v2:
  - No changes.
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index 84082922edb219ab..98f02631a0f0c66c 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -62,7 +62,7 @@
 		clock-frequency = <24576000>;
 	};
 
-	vcc_sdhi0: regulator at 1 {
+	vcc_sdhi0: regulator-vcc-sdhi0 {
 		compatible = "regulator-fixed";
 
 		regulator-name = "SDHI0 Vcc";
@@ -73,7 +73,7 @@
 		enable-active-high;
 	};
 
-	vccq_sdhi0: regulator at 2 {
+	vccq_sdhi0: regulator-vccq-sdhi0 {
 		compatible = "regulator-gpio";
 
 		regulator-name = "SDHI0 VccQ";
@@ -86,7 +86,7 @@
 			  1800000 0>;
 	};
 
-	vcc_sdhi3: regulator at 3 {
+	vcc_sdhi3: regulator-vcc-sdhi3 {
 		compatible = "regulator-fixed";
 
 		regulator-name = "SDHI3 Vcc";
@@ -97,7 +97,7 @@
 		enable-active-high;
 	};
 
-	vccq_sdhi3: regulator at 4 {
+	vccq_sdhi3: regulator-vccq-sdhi3 {
 		compatible = "regulator-gpio";
 
 		regulator-name = "SDHI3 VccQ";
-- 
1.9.1

[PATCH v2 11/23] ARM: dts: armadillo800eva: Fix W=1 dtc warnings

[Sergei Shtylyov]

Hello.

On 05/20/2016 10:10 AM, Geert Uytterhoeven wrote:

> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
> Warning (unit_address_vs_reg): Node /regulator at 0 has a unit name, but no reg property
> Warning (unit_address_vs_reg): Node /regulator at 1 has a unit name, but no reg property
> Warning (unit_address_vs_reg): Node /regulator at 2 has a unit name, but no reg property
> Warning (unit_address_vs_reg): Node /regulator at 3 has a unit name, but no reg property
> Warning (unit_address_vs_reg): Node /i2c at 2 has a unit name, but no reg property
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> v2:
>   - No changes.
> ---
>  arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> index 2c82dab2b6f45f9c..b3f0ac4b99bd5a3d 100644
> --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
[...]
> @@ -127,7 +127,7 @@
>  		};
>  	};
>
> -	i2c2: i2c at 2 {
> +	i2c2: i2c2 {

    Not "i2c-2" as in the other patches?

>  		#address-cells = <1>;
>  		#size-cells = <0>;
>  		compatible = "i2c-gpio";

MBR, Sergei

[PATCH v2 11/23] ARM: dts: armadillo800eva: Fix W=1 dtc warnings

[Geert Uytterhoeven]

On Fri, May 20, 2016 at 5:38 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> On 05/20/2016 10:10 AM, Geert Uytterhoeven wrote:
>> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
>> Warning (unit_address_vs_reg): Node /regulator at 0 has a unit name, but no reg property
>> Warning (unit_address_vs_reg): Node /regulator at 1 has a unit name, but no reg property
>> Warning (unit_address_vs_reg): Node /regulator at 2 has a unit name, but no reg property
>> Warning (unit_address_vs_reg): Node /regulator at 3 has a unit name, but no reg property
>> Warning (unit_address_vs_reg): Node /i2c at 2 has a unit name, but no reg property
>>
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> ---
>> v2:
>>   - No changes.
>> ---
>>  arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 12 ++++++------
>>  1 file changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
>> b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
>> index 2c82dab2b6f45f9c..b3f0ac4b99bd5a3d 100644
>> --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
>> +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
>
> [...]
>>
>> @@ -127,7 +127,7 @@
>>                 };
>>         };
>>
>> -       i2c2: i2c at 2 {
>> +       i2c2: i2c2 {
>
>    Not "i2c-2" as in the other patches?

Oops... Thanks for noticing!

Simon: Do you want me to repost the whole series once again, or
can you fix this up while applying?

Thanks!

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

[PATCH v2 11/23] ARM: dts: armadillo800eva: Fix W=1 dtc warnings

[Simon Horman]

On Mon, May 23, 2016 at 02:52:20PM +0200, Geert Uytterhoeven wrote:
> On Fri, May 20, 2016 at 5:38 PM, Sergei Shtylyov
> <sergei.shtylyov@cogentembedded.com> wrote:
> > On 05/20/2016 10:10 AM, Geert Uytterhoeven wrote:
> >> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
> >> Warning (unit_address_vs_reg): Node /regulator at 0 has a unit name, but no reg property
> >> Warning (unit_address_vs_reg): Node /regulator at 1 has a unit name, but no reg property
> >> Warning (unit_address_vs_reg): Node /regulator at 2 has a unit name, but no reg property
> >> Warning (unit_address_vs_reg): Node /regulator at 3 has a unit name, but no reg property
> >> Warning (unit_address_vs_reg): Node /i2c at 2 has a unit name, but no reg property
> >>
> >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >> ---
> >> v2:
> >>   - No changes.
> >> ---
> >>  arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 12 ++++++------
> >>  1 file changed, 6 insertions(+), 6 deletions(-)
> >>
> >> diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> >> b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> >> index 2c82dab2b6f45f9c..b3f0ac4b99bd5a3d 100644
> >> --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> >> +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> >
> > [...]
> >>
> >> @@ -127,7 +127,7 @@
> >>                 };
> >>         };
> >>
> >> -       i2c2: i2c at 2 {
> >> +       i2c2: i2c2 {
> >
> >    Not "i2c-2" as in the other patches?
> 
> Oops... Thanks for noticing!
> 
> Simon: Do you want me to repost the whole series once again, or
> can you fix this up while applying?

I'll just fix it up.
No need to repost for this.

[PATCH v2 11/23] ARM: dts: armadillo800eva: Fix W=1 dtc warnings

[Simon Horman]

On Tue, May 24, 2016 at 09:13:09AM +0900, Simon Horman wrote:
> On Mon, May 23, 2016 at 02:52:20PM +0200, Geert Uytterhoeven wrote:
> > On Fri, May 20, 2016 at 5:38 PM, Sergei Shtylyov
> > <sergei.shtylyov@cogentembedded.com> wrote:
> > > On 05/20/2016 10:10 AM, Geert Uytterhoeven wrote:
> > >> Warning (unit_address_vs_reg): Node /memory has a reg or ranges property, but no unit name
> > >> Warning (unit_address_vs_reg): Node /regulator at 0 has a unit name, but no reg property
> > >> Warning (unit_address_vs_reg): Node /regulator at 1 has a unit name, but no reg property
> > >> Warning (unit_address_vs_reg): Node /regulator at 2 has a unit name, but no reg property
> > >> Warning (unit_address_vs_reg): Node /regulator at 3 has a unit name, but no reg property
> > >> Warning (unit_address_vs_reg): Node /i2c at 2 has a unit name, but no reg property
> > >>
> > >> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > >> ---
> > >> v2:
> > >>   - No changes.
> > >> ---
> > >>  arch/arm/boot/dts/r8a7740-armadillo800eva.dts | 12 ++++++------
> > >>  1 file changed, 6 insertions(+), 6 deletions(-)
> > >>
> > >> diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> > >> b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> > >> index 2c82dab2b6f45f9c..b3f0ac4b99bd5a3d 100644
> > >> --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> > >> +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
> > >
> > > [...]
> > >>
> > >> @@ -127,7 +127,7 @@
> > >>                 };
> > >>         };
> > >>
> > >> -       i2c2: i2c at 2 {
> > >> +       i2c2: i2c2 {
> > >
> > >    Not "i2c-2" as in the other patches?
> > 
> > Oops... Thanks for noticing!
> > 
> > Simon: Do you want me to repost the whole series once again, or
> > can you fix this up while applying?
> 
> I'll just fix it up.
> No need to repost for this.

I have queued this up with the above hunk changed to the following.
Please check the resulting patch and let me know if I've messed things up
somehow.

@@ -127,7 +127,7 @@
 		};
 	};
 
-	i2c2: i2c at 2 {
+	i2c2: i2c-2 {
 		#address-cells = <1>;
 		#size-cells = <0>;
 		compatible = "i2c-gpio";

[PATCH v2 00/23] ARM/arm64: dts: renesas: Fix W=1 dtc warnings

[Simon Horman]

On Fri, May 20, 2016 at 09:09:51AM +0200, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus, Morimoto-san, Rob,
> 
> This patch series fixes the new warnings related to Renesas arm32/arm64
> DTSes that can be seen in -next using "make W=1 dtbs", courtesy of
> commit b993734718c01064 ("scripts/dtc: Update to upstream version
> 53bf130b1cdd") in Rob herring's for-next branch. For testing, you can just
> cherry-pick that commit (showing the warning for W=1 only was a separate
> commit).

Thanks, I have queued this up.

[PATCH v2 11/23] ARM: dts: armadillo800eva: Fix W=1 dtc warnings

[Geert Uytterhoeven]

Hi Simon,

On Tue, May 24, 2016 at 2:33 AM, Simon Horman <horms@verge.net.au> wrote:
>> > >> --- a/arch/arm/boot/dts/r8a7740-armadillo800eva.dts
>> > >> +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva.dts

> I have queued this up with the above hunk changed to the following.
> Please check the resulting patch and let me know if I've messed things up
> somehow.
>
> @@ -127,7 +127,7 @@
>                 };
>         };
>
> -       i2c2: i2c at 2 {
> +       i2c2: i2c-2 {

Thanks, looks fine.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds