netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings
@ 2018-12-22  7:39 Sergio Paracuellos
  2018-12-23 22:16 ` Florian Fainelli
  2018-12-24 22:15 ` David Miller
  0 siblings, 2 replies; 6+ messages in thread
From: Sergio Paracuellos @ 2018-12-22  7:39 UTC (permalink / raw)
  To: Woojung.Huh
  Cc: andrew, f.fainelli, vivien.didelot, netdev, driverdev-devel,
	UNGLinuxDriver, devicetree, davem

Switch bindings for spi managed mode are using spaces instead of tabs.
Fix them to get a file with a proper kernel indentation style.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
---
 .../devicetree/bindings/net/dsa/ksz.txt       | 110 +++++++++---------
 1 file changed, 55 insertions(+), 55 deletions(-)

diff --git a/Documentation/devicetree/bindings/net/dsa/ksz.txt b/Documentation/devicetree/bindings/net/dsa/ksz.txt
index 2f276aa42f0b..e8d33d008355 100644
--- a/Documentation/devicetree/bindings/net/dsa/ksz.txt
+++ b/Documentation/devicetree/bindings/net/dsa/ksz.txt
@@ -19,61 +19,61 @@ Examples:
 
 Ethernet switch connected via SPI to the host, CPU port wired to eth0:
 
-                             eth0: ethernet@10001000 {
-                                             fixed-link {
-                                                             speed = <1000>;
-                                                             full-duplex;
-                                             };
-                             };
-
-                             spi1: spi@f8008000 {
-                                             pinctrl-0 = <&pinctrl_spi_ksz>;
-                                             cs-gpios = <&pioC 25 0>;
-                                             id = <1>;
-
-                                             ksz9477: ksz9477@0 {
-                                                             compatible = "microchip,ksz9477";
-                                                             reg = <0>;
-
-                                                             spi-max-frequency = <44000000>;
-                                                             spi-cpha;
-                                                             spi-cpol;
-
-                                                             ports {
-                                                                             #address-cells = <1>;
-                                                                             #size-cells = <0>;
-                                                                             port@0 {
-                                                                                             reg = <0>;
-                                                                                             label = "lan1";
-                                                                             };
-                                                                             port@1 {
-                                                                                             reg = <1>;
-                                                                                             label = "lan2";
-                                                                             };
-                                                                             port@2 {
-                                                                                             reg = <2>;
-                                                                                             label = "lan3";
-                                                                             };
-                                                                             port@3 {
-                                                                                             reg = <3>;
-                                                                                             label = "lan4";
-                                                                             };
-                                                                             port@4 {
-                                                                                             reg = <4>;
-                                                                                             label = "lan5";
-                                                                             };
-                                                                             port@5 {
-                                                                                             reg = <5>;
-                                                                                             label = "cpu";
-                                                                                             ethernet = <&eth0>;
-                                                                                             fixed-link {
-                                                                                                             speed = <1000>;
-                                                                                                             full-duplex;
-                                                                                             };
-                                                                             };
-                                                             };
-                                             };
-                             };
+	eth0: ethernet@10001000 {
+		fixed-link {
+			speed = <1000>;
+			full-duplex;
+		};
+	};
+
+	spi1: spi@f8008000 {
+		pinctrl-0 = <&pinctrl_spi_ksz>;
+		cs-gpios = <&pioC 25 0>;
+		id = <1>;
+
+		ksz9477: ksz9477@0 {
+			compatible = "microchip,ksz9477";
+			reg = <0>;
+
+			spi-max-frequency = <44000000>;
+			spi-cpha;
+			spi-cpol;
+
+			ports {
+				#address-cells = <1>;
+				#size-cells = <0>;
+				port@0 {
+					reg = <0>;
+					label = "lan1";
+				};
+				port@1 {
+					reg = <1>;
+					label = "lan2";
+				};
+				port@2 {
+					reg = <2>;
+					label = "lan3";
+				};
+				port@3 {
+					reg = <3>;
+					label = "lan4";
+				};
+				port@4 {
+					reg = <4>;
+					label = "lan5";
+				};
+				port@5 {
+					reg = <5>;
+					label = "cpu";
+					ethernet = <&eth0>;
+					fixed-link {
+						speed = <1000>;
+						full-duplex;
+					};
+				};
+			};
+		};
+	};
 
 Ethernet switch connected via I2C to the host, CPU port wired to eth0:
 
-- 
2.19.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings
  2018-12-22  7:39 [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings Sergio Paracuellos
@ 2018-12-23 22:16 ` Florian Fainelli
  2018-12-24 22:15 ` David Miller
  1 sibling, 0 replies; 6+ messages in thread
From: Florian Fainelli @ 2018-12-23 22:16 UTC (permalink / raw)
  To: Sergio Paracuellos, Woojung.Huh
  Cc: UNGLinuxDriver, andrew, vivien.didelot, driverdev-devel, davem,
	netdev, devicetree

Le 12/21/18 à 11:39 PM, Sergio Paracuellos a écrit :
> Switch bindings for spi managed mode are using spaces instead of tabs.
> Fix them to get a file with a proper kernel indentation style.
> 
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>

Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
-- 
Florian

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings
  2018-12-22  7:39 [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings Sergio Paracuellos
  2018-12-23 22:16 ` Florian Fainelli
@ 2018-12-24 22:15 ` David Miller
  2018-12-25 11:45   ` Sergio Paracuellos
  1 sibling, 1 reply; 6+ messages in thread
From: David Miller @ 2018-12-24 22:15 UTC (permalink / raw)
  To: sergio.paracuellos
  Cc: Woojung.Huh, UNGLinuxDriver, andrew, vivien.didelot, f.fainelli,
	driverdev-devel, netdev, devicetree

From: Sergio Paracuellos <sergio.paracuellos@gmail.com>
Date: Sat, 22 Dec 2018 08:39:09 +0100

> Switch bindings for spi managed mode are using spaces instead of tabs.
> Fix them to get a file with a proper kernel indentation style.
> 
> Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>

This doesn't apply to any of my trees so I'm going to assume this was
targetting the devicetree GIT tree or something like that.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings
  2018-12-24 22:15 ` David Miller
@ 2018-12-25 11:45   ` Sergio Paracuellos
  2018-12-25 12:38     ` Andrew Lunn
  0 siblings, 1 reply; 6+ messages in thread
From: Sergio Paracuellos @ 2018-12-25 11:45 UTC (permalink / raw)
  To: David Miller
  Cc: Woojung.Huh, Andrew Lunn, Florian Fainelli, vivien.didelot,
	netdev, driverdev-devel, UNGLinuxDriver,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS

Hi David,

On Mon, Dec 24, 2018 at 11:15 PM David Miller <davem@davemloft.net> wrote:
>
> From: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> Date: Sat, 22 Dec 2018 08:39:09 +0100
>
> > Switch bindings for spi managed mode are using spaces instead of tabs.
> > Fix them to get a file with a proper kernel indentation style.
> >
> > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
>
> This doesn't apply to any of my trees so I'm going to assume this was
> targetting the devicetree GIT tree or something like that.

Actually, this was rebased onto linux-next. Which tree do you want me
to rebase onto?

Best regards,
    Sergio Paracuellos

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings
  2018-12-25 11:45   ` Sergio Paracuellos
@ 2018-12-25 12:38     ` Andrew Lunn
  2018-12-25 18:41       ` Sergio Paracuellos
  0 siblings, 1 reply; 6+ messages in thread
From: Andrew Lunn @ 2018-12-25 12:38 UTC (permalink / raw)
  To: Sergio Paracuellos
  Cc: David Miller, Woojung.Huh, UNGLinuxDriver, vivien.didelot,
	Florian Fainelli, driverdev-devel, netdev,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS

On Tue, Dec 25, 2018 at 12:45:11PM +0100, Sergio Paracuellos wrote:
> Hi David,
> 
> On Mon, Dec 24, 2018 at 11:15 PM David Miller <davem@davemloft.net> wrote:
> >
> > From: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > Date: Sat, 22 Dec 2018 08:39:09 +0100
> >
> > > Switch bindings for spi managed mode are using spaces instead of tabs.
> > > Fix them to get a file with a proper kernel indentation style.
> > >
> > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> >
> > This doesn't apply to any of my trees so I'm going to assume this was
> > targetting the devicetree GIT tree or something like that.
> 
> Actually, this was rebased onto linux-next. Which tree do you want me
> to rebase onto?

Hi Sergio

It should be based on net-next. However, that is closed now, so please
wait until it reopens in about two weeks time.

     Thanks
	Andrew

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings
  2018-12-25 12:38     ` Andrew Lunn
@ 2018-12-25 18:41       ` Sergio Paracuellos
  0 siblings, 0 replies; 6+ messages in thread
From: Sergio Paracuellos @ 2018-12-25 18:41 UTC (permalink / raw)
  To: Andrew Lunn
  Cc: Woojung.Huh,
	open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
	Florian Fainelli, vivien.didelot, netdev, driverdev-devel,
	UNGLinuxDriver, David Miller

On Tue, Dec 25, 2018 at 1:39 PM Andrew Lunn <andrew@lunn.ch> wrote:
>
> On Tue, Dec 25, 2018 at 12:45:11PM +0100, Sergio Paracuellos wrote:
> > Hi David,
> >
> > On Mon, Dec 24, 2018 at 11:15 PM David Miller <davem@davemloft.net> wrote:
> > >
> > > From: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > > Date: Sat, 22 Dec 2018 08:39:09 +0100
> > >
> > > > Switch bindings for spi managed mode are using spaces instead of tabs.
> > > > Fix them to get a file with a proper kernel indentation style.
> > > >
> > > > Signed-off-by: Sergio Paracuellos <sergio.paracuellos@gmail.com>
> > >
> > > This doesn't apply to any of my trees so I'm going to assume this was
> > > targetting the devicetree GIT tree or something like that.
> >
> > Actually, this was rebased onto linux-next. Which tree do you want me
> > to rebase onto?
>
> Hi Sergio
>
> It should be based on net-next. However, that is closed now, so please
> wait until it reopens in about two weeks time.

Hi Andrew,

Ok, I'll resend v2 rebased onto net-next when merge window is close in
about two weeks.

Thanks for let me know.

>
>      Thanks
>         Andrew

Best regards,
    Sergio Paracuellos

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-12-25 18:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-22  7:39 [PATCH] dt-bindings: net: dsa: ksz9477: fix indentation for switch spi bindings Sergio Paracuellos
2018-12-23 22:16 ` Florian Fainelli
2018-12-24 22:15 ` David Miller
2018-12-25 11:45   ` Sergio Paracuellos
2018-12-25 12:38     ` Andrew Lunn
2018-12-25 18:41       ` Sergio Paracuellos

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