ARM: dts: omap3: name mdio node properly

Message ID 20191127153212.22209-1-grygorii.strashko@ti.com
State New
Headers show
Series
  • ARM: dts: omap3: name mdio node properly
Related show

Commit Message

Grygorii Strashko Nov. 27, 2019, 3:32 p.m.
Rename davinci_mdio DT node "ethernet"->"mdio"
This fixes the following DT schemas check errors:
am3517-craneboard.dt.yaml: ethernet@5c030000: $nodename:0: 'ethernet@5c030000' does not match '^mdio(@.*)?'

Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

---
 arch/arm/boot/dts/am3517.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.17.1

Comments

Grygorii Strashko Nov. 27, 2019, 4:26 p.m. | #1
On 27/11/2019 17:32, Grygorii Strashko wrote:
> Rename davinci_mdio DT node "ethernet"->"mdio"

> This fixes the following DT schemas check errors:

> am3517-craneboard.dt.yaml: ethernet@5c030000: $nodename:0: 'ethernet@5c030000' does not match '^mdio(@.*)?'

> 

> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

> ---


This can be for next material.

>   arch/arm/boot/dts/am3517.dtsi | 2 +-

>   1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/arch/arm/boot/dts/am3517.dtsi b/arch/arm/boot/dts/am3517.dtsi

> index 76f819f4ba48..84e13778a64a 100644

> --- a/arch/arm/boot/dts/am3517.dtsi

> +++ b/arch/arm/boot/dts/am3517.dtsi

> @@ -74,7 +74,7 @@

>   			clock-names = "ick";

>   		};

>   

> -		davinci_mdio: ethernet@5c030000 {

> +		davinci_mdio: mdio@5c030000 {

>   			compatible = "ti,davinci_mdio";

>   			ti,hwmods = "davinci_mdio";

>   			status = "disabled";

> 


-- 
Best regards,
grygorii
Grygorii Strashko Dec. 12, 2019, 10:08 a.m. | #2
Hi Tony,

On 27/11/2019 18:26, Grygorii Strashko wrote:
> 

> 

> On 27/11/2019 17:32, Grygorii Strashko wrote:

>> Rename davinci_mdio DT node "ethernet"->"mdio"

>> This fixes the following DT schemas check errors:

>> am3517-craneboard.dt.yaml: ethernet@5c030000: $nodename:0: 'ethernet@5c030000' does not match '^mdio(@.*)?'

>>

>> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

>> ---

> 

> This can be for next material.

> 

>>   arch/arm/boot/dts/am3517.dtsi | 2 +-

>>   1 file changed, 1 insertion(+), 1 deletion(-)

>>

>> diff --git a/arch/arm/boot/dts/am3517.dtsi b/arch/arm/boot/dts/am3517.dtsi

>> index 76f819f4ba48..84e13778a64a 100644

>> --- a/arch/arm/boot/dts/am3517.dtsi

>> +++ b/arch/arm/boot/dts/am3517.dtsi

>> @@ -74,7 +74,7 @@

>>               clock-names = "ick";

>>           };

>> -        davinci_mdio: ethernet@5c030000 {

>> +        davinci_mdio: mdio@5c030000 {

>>               compatible = "ti,davinci_mdio";

>>               ti,hwmods = "davinci_mdio";

>>               status = "disabled";

>>

> 


Any comments here?

-- 
Best regards,
grygorii
Tony Lindgren Dec. 12, 2019, 4:19 p.m. | #3
* Grygorii Strashko <grygorii.strashko@ti.com> [191212 10:09]:
> Hi Tony,

> 

> On 27/11/2019 18:26, Grygorii Strashko wrote:

> > 

> > 

> > On 27/11/2019 17:32, Grygorii Strashko wrote:

> > > Rename davinci_mdio DT node "ethernet"->"mdio"

> > > This fixes the following DT schemas check errors:

> > > am3517-craneboard.dt.yaml: ethernet@5c030000: $nodename:0: 'ethernet@5c030000' does not match '^mdio(@.*)?'

> > > 

> > > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

> > > ---

> > 

> > This can be for next material.

> > 

> > >   arch/arm/boot/dts/am3517.dtsi | 2 +-

> > >   1 file changed, 1 insertion(+), 1 deletion(-)

> > > 

> > > diff --git a/arch/arm/boot/dts/am3517.dtsi b/arch/arm/boot/dts/am3517.dtsi

> > > index 76f819f4ba48..84e13778a64a 100644

> > > --- a/arch/arm/boot/dts/am3517.dtsi

> > > +++ b/arch/arm/boot/dts/am3517.dtsi

> > > @@ -74,7 +74,7 @@

> > >               clock-names = "ick";

> > >           };

> > > -        davinci_mdio: ethernet@5c030000 {

> > > +        davinci_mdio: mdio@5c030000 {

> > >               compatible = "ti,davinci_mdio";

> > >               ti,hwmods = "davinci_mdio";

> > >               status = "disabled";

> > > 

> > 

> 

> Any comments here?


No comments looks fine. I just have not yet gotten to applying
patches for v5.6 merge window yet.

Regards,

Tony
Grygorii Strashko Dec. 12, 2019, 5:31 p.m. | #4
On 12/12/2019 18:19, Tony Lindgren wrote:
> * Grygorii Strashko <grygorii.strashko@ti.com> [191212 10:09]:

>> Hi Tony,

>>

>> On 27/11/2019 18:26, Grygorii Strashko wrote:

>>>

>>>

>>> On 27/11/2019 17:32, Grygorii Strashko wrote:

>>>> Rename davinci_mdio DT node "ethernet"->"mdio"

>>>> This fixes the following DT schemas check errors:

>>>> am3517-craneboard.dt.yaml: ethernet@5c030000: $nodename:0: 'ethernet@5c030000' does not match '^mdio(@.*)?'

>>>>

>>>> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

>>>> ---

>>>

>>> This can be for next material.

>>>

>>>>    arch/arm/boot/dts/am3517.dtsi | 2 +-

>>>>    1 file changed, 1 insertion(+), 1 deletion(-)

>>>>

>>>> diff --git a/arch/arm/boot/dts/am3517.dtsi b/arch/arm/boot/dts/am3517.dtsi

>>>> index 76f819f4ba48..84e13778a64a 100644

>>>> --- a/arch/arm/boot/dts/am3517.dtsi

>>>> +++ b/arch/arm/boot/dts/am3517.dtsi

>>>> @@ -74,7 +74,7 @@

>>>>                clock-names = "ick";

>>>>            };

>>>> -        davinci_mdio: ethernet@5c030000 {

>>>> +        davinci_mdio: mdio@5c030000 {

>>>>                compatible = "ti,davinci_mdio";

>>>>                ti,hwmods = "davinci_mdio";

>>>>                status = "disabled";

>>>>

>>>

>>

>> Any comments here?

> 

> No comments looks fine. I just have not yet gotten to applying

> patches for v5.6 merge window yet.


Oh. Ok. Thank you.

-- 
Best regards,
grygorii
Tony Lindgren Dec. 12, 2019, 5:32 p.m. | #5
* Tony Lindgren <tony@atomide.com> [191212 16:19]:
> * Grygorii Strashko <grygorii.strashko@ti.com> [191212 10:09]:

> > Hi Tony,

> > 

> > On 27/11/2019 18:26, Grygorii Strashko wrote:

> > > 

> > > 

> > > On 27/11/2019 17:32, Grygorii Strashko wrote:

> > > > Rename davinci_mdio DT node "ethernet"->"mdio"

> > > > This fixes the following DT schemas check errors:

> > > > am3517-craneboard.dt.yaml: ethernet@5c030000: $nodename:0: 'ethernet@5c030000' does not match '^mdio(@.*)?'

> > > > 

> > > > Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>

> > > > ---

> > > 

> > > This can be for next material.

> > > 

> > > >   arch/arm/boot/dts/am3517.dtsi | 2 +-

> > > >   1 file changed, 1 insertion(+), 1 deletion(-)

> > > > 

> > > > diff --git a/arch/arm/boot/dts/am3517.dtsi b/arch/arm/boot/dts/am3517.dtsi

> > > > index 76f819f4ba48..84e13778a64a 100644

> > > > --- a/arch/arm/boot/dts/am3517.dtsi

> > > > +++ b/arch/arm/boot/dts/am3517.dtsi

> > > > @@ -74,7 +74,7 @@

> > > >               clock-names = "ick";

> > > >           };

> > > > -        davinci_mdio: ethernet@5c030000 {

> > > > +        davinci_mdio: mdio@5c030000 {

> > > >               compatible = "ti,davinci_mdio";

> > > >               ti,hwmods = "davinci_mdio";

> > > >               status = "disabled";

> > > > 

> > > 

> > 

> > Any comments here?

> 

> No comments looks fine. I just have not yet gotten to applying

> patches for v5.6 merge window yet.


Applying into omap-for-v5.6/dt thanks.

Tony

Patch

diff --git a/arch/arm/boot/dts/am3517.dtsi b/arch/arm/boot/dts/am3517.dtsi
index 76f819f4ba48..84e13778a64a 100644
--- a/arch/arm/boot/dts/am3517.dtsi
+++ b/arch/arm/boot/dts/am3517.dtsi
@@ -74,7 +74,7 @@ 
 			clock-names = "ick";
 		};
 
-		davinci_mdio: ethernet@5c030000 {
+		davinci_mdio: mdio@5c030000 {
 			compatible = "ti,davinci_mdio";
 			ti,hwmods = "davinci_mdio";
 			status = "disabled";