Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1611289 > unrolled thread

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

Started byVivien Didelot <vivien.didelot@savoirfairelinux.com>
First post2017-03-28 21:20 +0200
Last post2017-03-28 21:30 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-03-28 21:20 +0200
    Re: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2  label Florian Fainelli <f.fainelli@gmail.com> - 2017-03-28 21:20 +0200
      Re: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-03-28 21:30 +0200

#1611289 — [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label

FromVivien Didelot <vivien.didelot@savoirfairelinux.com>
Date2017-03-28 21:20 +0200
Subject[PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label
Message-ID<tq4OR-3L1-1@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1611291 — Re: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label

FromFlorian Fainelli <f.fainelli@gmail.com>
Date2017-03-28 21:20 +0200
SubjectRe: [PATCH net-next] ARM: dts: vf610-zii-dev-rev-b: change switch2 label
Message-ID<tq4OS-3L1-17@gated-at.bofh.it>
In reply to#1611289
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

[toc] | [prev] | [next] | [standalone]


#1611297

FromVivien Didelot <vivien.didelot@savoirfairelinux.com>
Date2017-03-28 21:30 +0200
Message-ID<tq4Yx-3SP-3@gated-at.bofh.it>
In reply to#1611291
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

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web