ARM: dts: omap3: name mdio node properly
diff mbox series

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. UTC
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(-)

Comments

Grygorii Strashko Nov. 27, 2019, 4:26 p.m. UTC | #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";
>
Grygorii Strashko Dec. 12, 2019, 10:08 a.m. UTC | #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?
Tony Lindgren Dec. 12, 2019, 4:19 p.m. UTC | #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. UTC | #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.
Tony Lindgren Dec. 12, 2019, 5:32 p.m. UTC | #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 mbox series

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";