* [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label
@ 2017-03-28 19:11 Vivien Didelot
2017-03-28 19:14 ` Florian Fainelli
0 siblings, 1 reply; 3+ messages in thread
From: Vivien Didelot @ 2017-03-28 19:11 UTC (permalink / raw)
To: netdev
Cc: linux-kernel, kernel, David S. Miller, Florian Fainelli,
Andrew Lunn, Vivien Didelot
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 <vivien.didelot@savoirfairelinux.com>
---
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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label
2017-03-28 19:11 [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label Vivien Didelot
@ 2017-03-28 19:14 ` Florian Fainelli
2017-03-28 19:18 ` Vivien Didelot
0 siblings, 1 reply; 3+ messages in thread
From: Florian Fainelli @ 2017-03-28 19:14 UTC (permalink / raw)
To: Vivien Didelot, netdev; +Cc: linux-kernel, kernel, David S. Miller, Andrew Lunn
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 <vivien.didelot@savoirfairelinux.com>
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label
2017-03-28 19:14 ` Florian Fainelli
@ 2017-03-28 19:18 ` Vivien Didelot
0 siblings, 0 replies; 3+ messages in thread
From: Vivien Didelot @ 2017-03-28 19:18 UTC (permalink / raw)
To: Florian Fainelli, netdev
Cc: linux-kernel, kernel, David S. Miller, Andrew Lunn
Hi Florian,
Florian Fainelli <f.fainelli@gmail.com> 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 <vivien.didelot@savoirfairelinux.com>
>
> 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
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-03-28 19:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-28 19:11 [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label Vivien Didelot
2017-03-28 19:14 ` Florian Fainelli
2017-03-28 19:18 ` Vivien Didelot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).