2017-03-28 19:13:18

by Vivien Didelot

[permalink] [raw]
Subject: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label

Rename the switch2@0 label of the switch2 node to switch@0 to respect
the general unit@address DTS rule, and be consistent with the other
switch nodes of the DTS file.

Signed-off-by: Vivien Didelot <[email protected]>
---
arch/arm/boot/dts/vf610-zii-dev-rev-b.dts | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/boot/dts/vf610-zii-dev-rev-b.dts b/arch/arm/boot/dts/vf610-zii-dev-rev-b.dts
index 7940408838df..5a93f2d875b1 100644
--- a/arch/arm/boot/dts/vf610-zii-dev-rev-b.dts
+++ b/arch/arm/boot/dts/vf610-zii-dev-rev-b.dts
@@ -239,7 +239,7 @@
#size-cells = <0>;
reg = <4>;

- switch2: switch2@0 {
+ switch2: switch@0 {
compatible = "marvell,mv88e6085";
#address-cells = <1>;
#size-cells = <0>;
--
2.12.1


2017-03-28 19:15:13

by Florian Fainelli

[permalink] [raw]
Subject: Re: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label

On 03/28/2017 12:11 PM, Vivien Didelot wrote:
> Rename the switch2@0 label of the switch2 node to switch@0 to respect
> the general unit@address DTS rule, and be consistent with the other
> switch nodes of the DTS file.
>
> Signed-off-by: Vivien Didelot <[email protected]>

This looks good, but you certainly want the ARM SoC maintainer for that
platform to be able to pick that change through his tree, can you
resubmit with the proper CC list?

Thanks!
--
Florian

2017-03-28 19:20:14

by Vivien Didelot

[permalink] [raw]
Subject: Re: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label

Hi Florian,

Florian Fainelli <[email protected]> writes:

> On 03/28/2017 12:11 PM, Vivien Didelot wrote:
>> Rename the switch2@0 label of the switch2 node to switch@0 to respect
>> the general unit@address DTS rule, and be consistent with the other
>> switch nodes of the DTS file.
>>
>> Signed-off-by: Vivien Didelot <[email protected]>
>
> This looks good, but you certainly want the ARM SoC maintainer for that
> platform to be able to pick that change through his tree, can you
> resubmit with the proper CC list?

Hum it looks like I'm still not confortable with who is going to pick
minor changes to DTS files... Will do, thanks!

Vivien