| Message ID | 9235D6609DB808459E95D78E17F2E43D40A5C00B@CHN-SV-EXMX02.mchp-main.com |
|---|---|
| State | Changes Requested, archived |
| Delegated to: | David Miller |
| Headers | show |
On Fri, May 05, 2017 at 11:18:05PM +0000, Woojung.Huh@microchip.com wrote: > From: Woojung Huh <Woojung.Huh@microchip.com> > > A sample SPI configuration for Microchip KSZ switches. > > Signed-off-by: Woojung Huh <Woojung.Huh@microchip.com> > --- > Documentation/devicetree/bindings/net/dsa/ksz.txt | 73 +++++++++++++++++++++++ > 1 file changed, 73 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/dsa/ksz.txt > > diff --git a/Documentation/devicetree/bindings/net/dsa/ksz.txt b/Documentation/devicetree/bindings/net/dsa/ksz.txt > new file mode 100644 > index 0000000..9cca7d4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/dsa/ksz.txt > @@ -0,0 +1,73 @@ > +Microchip KSZ Series Ethernet switches > +================================== > + > +Required properties: > + > +- compatible: For external switch chips, compatible string must be exactly one > + of: "microchip,ksz9477" > + > +See Documentation/devicetree/bindings/dsa/dsa.txt for a list of additional > +required and optional properties. > + > +Examples: > + > +Ethernet switch connected via SPI to the host, CPU port wired to eth0: > + > + eth0: ethernet@10001000 { > + fixed-link { > + reg = <7> > + speed = <1000>; > + duplex-full; > + }; > + }; > + .. > + port@5 { > + reg = <5>; > + label = "cpu"; > + ethernet = <&macb0>; I think this should be ethernet = <ð0>; Andrew
Hello! On 5/6/2017 2:18 AM, Woojung.Huh@microchip.com wrote: > From: Woojung Huh <Woojung.Huh@microchip.com> > > A sample SPI configuration for Microchip KSZ switches. > > Signed-off-by: Woojung Huh <Woojung.Huh@microchip.com> > --- > Documentation/devicetree/bindings/net/dsa/ksz.txt | 73 +++++++++++++++++++++++ > 1 file changed, 73 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/dsa/ksz.txt > > diff --git a/Documentation/devicetree/bindings/net/dsa/ksz.txt b/Documentation/devicetree/bindings/net/dsa/ksz.txt > new file mode 100644 > index 0000000..9cca7d4 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/dsa/ksz.txt > @@ -0,0 +1,73 @@ > +Microchip KSZ Series Ethernet switches > +================================== > + > +Required properties: > + > +- compatible: For external switch chips, compatible string must be exactly one > + of: "microchip,ksz9477" > + > +See Documentation/devicetree/bindings/dsa/dsa.txt for a list of additional > +required and optional properties. > + > +Examples: > + > +Ethernet switch connected via SPI to the host, CPU port wired to eth0: > + > + eth0: ethernet@10001000 { > + fixed-link { > + reg = <7> > + speed = <1000>; > + duplex-full; > + }; > + }; > + > + spi1: spi@f8008000 { > + pinctrl-0 = <&pinctrl_spi_ksz>; > + cs-gpios = <&pioC 25 0>; > + id = <1>; > + status = "okay"; 4 lines above should be indented more to the right. > + > + ksz9477: ksz9477@0 { > + compatible = "microchip,ksz9477"; > + reg = <0>; > + > + spi-max-frequency = <44000000>; > + spi-cpha; > + spi-cpol; > + > + status = "okay"; > + 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 = <&macb0>; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + }; > + }; > + }; Unmatched }? MBR, Sergei
Hi Sergei >> + spi1: spi@f8008000 { >> + pinctrl-0 = <&pinctrl_spi_ksz>; >> + cs-gpios = <&pioC 25 0>; >> + id = <1>; >> + status = "okay"; > > 4 lines above should be indented more to the right. >> + }; >> + }; > > Unmatched }? Will fix both. - Woojung
diff --git a/Documentation/devicetree/bindings/net/dsa/ksz.txt b/Documentation/devicetree/bindings/net/dsa/ksz.txt new file mode 100644 index 0000000..9cca7d4 --- /dev/null +++ b/Documentation/devicetree/bindings/net/dsa/ksz.txt @@ -0,0 +1,73 @@ +Microchip KSZ Series Ethernet switches +================================== + +Required properties: + +- compatible: For external switch chips, compatible string must be exactly one + of: "microchip,ksz9477" + +See Documentation/devicetree/bindings/dsa/dsa.txt for a list of additional +required and optional properties. + +Examples: + +Ethernet switch connected via SPI to the host, CPU port wired to eth0: + + eth0: ethernet@10001000 { + fixed-link { + reg = <7> + speed = <1000>; + duplex-full; + }; + }; + + spi1: spi@f8008000 { + pinctrl-0 = <&pinctrl_spi_ksz>; + cs-gpios = <&pioC 25 0>; + id = <1>; + status = "okay"; + + ksz9477: ksz9477@0 { + compatible = "microchip,ksz9477"; + reg = <0>; + + spi-max-frequency = <44000000>; + spi-cpha; + spi-cpol; + + status = "okay"; + 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 = <&macb0>; + fixed-link { + speed = <1000>; + full-duplex; + }; + }; + }; + }; + };