2018-12-18 14:40:59

by Felix Brack

[permalink] [raw]
Subject: [PATCH: linux-next] ARM: dts: am33xx: Remove unnecessary properties

Remove the unnecessary properties #address-cells and #size-cells
of node pinmux as there are no child-nodes with property reg.

Signed-off-by: Felix Brack <[email protected]>
---
arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
1 file changed, 2 deletions(-)

diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
index 7b818d9d2eab..e957370f8aec 100644
--- a/arch/arm/boot/dts/am33xx-l4.dtsi
+++ b/arch/arm/boot/dts/am33xx-l4.dtsi
@@ -288,8 +288,6 @@
am33xx_pinmux: pinmux@800 {
compatible = "pinctrl-single";
reg = <0x800 0x238>;
- #address-cells = <1>;
- #size-cells = <0>;
#pinctrl-cells = <1>;
pinctrl-single,register-width = <32>;
pinctrl-single,function-mask = <0x7f>;
--
2.17.1



2019-01-19 17:40:10

by Felix Brack

[permalink] [raw]
Subject: Re: [PATCH: linux-next] ARM: dts: am33xx: Remove unnecessary properties

Hi Tony,
On 18.12.18 15:39, Felix Brack wrote:
> Remove the unnecessary properties #address-cells and #size-cells
> of node pinmux as there are no child-nodes with property reg.
>
> Signed-off-by: Felix Brack <[email protected]>
> ---
> arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
> index 7b818d9d2eab..e957370f8aec 100644
> --- a/arch/arm/boot/dts/am33xx-l4.dtsi
> +++ b/arch/arm/boot/dts/am33xx-l4.dtsi
> @@ -288,8 +288,6 @@
> am33xx_pinmux: pinmux@800 {
> compatible = "pinctrl-single";
> reg = <0x800 0x238>;
> - #address-cells = <1>;
> - #size-cells = <0>;
> #pinctrl-cells = <1>;
> pinctrl-single,register-width = <32>;
> pinctrl-single,function-mask = <0x7f>;
>

Just a gentle ping. BTW: the patch applies not just to linux-next
correctly but now also to linux.

regards Felix

2019-01-21 17:15:04

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH: linux-next] ARM: dts: am33xx: Remove unnecessary properties

* Felix Brack <[email protected]> [190119 17:38]:
> Hi Tony,
> On 18.12.18 15:39, Felix Brack wrote:
> > Remove the unnecessary properties #address-cells and #size-cells
> > of node pinmux as there are no child-nodes with property reg.
> >
> > Signed-off-by: Felix Brack <[email protected]>
> > ---
> > arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
> > index 7b818d9d2eab..e957370f8aec 100644
> > --- a/arch/arm/boot/dts/am33xx-l4.dtsi
> > +++ b/arch/arm/boot/dts/am33xx-l4.dtsi
> > @@ -288,8 +288,6 @@
> > am33xx_pinmux: pinmux@800 {
> > compatible = "pinctrl-single";
> > reg = <0x800 0x238>;
> > - #address-cells = <1>;
> > - #size-cells = <0>;
> > #pinctrl-cells = <1>;
> > pinctrl-single,register-width = <32>;
> > pinctrl-single,function-mask = <0x7f>;
> >
>
> Just a gentle ping. BTW: the patch applies not just to linux-next correctly
> but now also to linux.

Sorry I'm still struggling with pending regressions for
the -rc cycle.. I should get to applying patches for next
merge window this week though.

Regards,

Tony

2019-01-24 16:17:05

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH: linux-next] ARM: dts: am33xx: Remove unnecessary properties

* Tony Lindgren <[email protected]> [190121 17:13]:
> * Felix Brack <[email protected]> [190119 17:38]:
> > Hi Tony,
> > On 18.12.18 15:39, Felix Brack wrote:
> > > Remove the unnecessary properties #address-cells and #size-cells
> > > of node pinmux as there are no child-nodes with property reg.
> > >
> > > Signed-off-by: Felix Brack <[email protected]>
> > > ---
> > > arch/arm/boot/dts/am33xx-l4.dtsi | 2 --
> > > 1 file changed, 2 deletions(-)
> > >
> > > diff --git a/arch/arm/boot/dts/am33xx-l4.dtsi b/arch/arm/boot/dts/am33xx-l4.dtsi
> > > index 7b818d9d2eab..e957370f8aec 100644
> > > --- a/arch/arm/boot/dts/am33xx-l4.dtsi
> > > +++ b/arch/arm/boot/dts/am33xx-l4.dtsi
> > > @@ -288,8 +288,6 @@
> > > am33xx_pinmux: pinmux@800 {
> > > compatible = "pinctrl-single";
> > > reg = <0x800 0x238>;
> > > - #address-cells = <1>;
> > > - #size-cells = <0>;
> > > #pinctrl-cells = <1>;
> > > pinctrl-single,register-width = <32>;
> > > pinctrl-single,function-mask = <0x7f>;
> > >
> >
> > Just a gentle ping. BTW: the patch applies not just to linux-next correctly
> > but now also to linux.
>
> Sorry I'm still struggling with pending regressions for
> the -rc cycle.. I should get to applying patches for next
> merge window this week though.

Applying into omap-for-v5.1/dt finally today thanks.

Tony