Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756419AbdIHNcl (ORCPT ); Fri, 8 Sep 2017 09:32:41 -0400 Received: from mail-wm0-f44.google.com ([74.125.82.44]:43251 "EHLO mail-wm0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755541AbdIHNch (ORCPT ); Fri, 8 Sep 2017 09:32:37 -0400 X-Google-Smtp-Source: ADKCNb7OKrI5mpKo6DipYn5R8g5hzGAhDiyT9bVyuqhua7Lhptm1R5J9Sd4v93Ew2K5BARj1Ye3Nr9iga7jmKk8fmEA= MIME-Version: 1.0 In-Reply-To: <20170907215417.GU11248@lunn.ch> References: <93AF473E2DA327428DE3D46B72B1E9FD41121A5B@CHN-SV-EXMX02.mchp-main.com> <20170907215417.GU11248@lunn.ch> From: Maxim Uvarov Date: Fri, 8 Sep 2017 16:32:35 +0300 Message-ID: Subject: Re: [PATCH RFC] Update documentation for KSZ DSA drivers so that new drivers can be added To: Andrew Lunn Cc: Tristram.Ha@microchip.com, Pavel Machek , Nathan Conrad , Vivien Didelot , Florian Fainelli , netdev , linux-kernel@vger.kernel.org, Woojung.Huh@microchip.com Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 8307 Lines: 150 2017-09-08 0:54 GMT+03:00 Andrew Lunn : >> -- compatible: For external switch chips, compatible string must be exactly one >> - of: "microchip,ksz9477" >> +- compatible: Should be "microchip,ksz9477" for KSZ9477 chip, >> + "microchip,ksz8795" for KSZ8795 chip, >> + "microchip,ksz8794" for KSZ8794 chip, >> + "microchip,ksz8765" for KSZ8765 chip, >> + "microchip,ksz8895" for KSZ8895 chip, >> + "microchip,ksz8864" for KSZ8864 chip, >> + "microchip,ksz8873" for KSZ8873 chip, >> + "microchip,ksz8863" for KSZ8863 chip, >> + "microchip,ksz8463" for KSZ8463 chip > all that chips have the same spi access to get chip id on probe(). I prefer common microship,ksz-spi rather than somebody will always maintain that list. Maxim. > This part of this patch should be in a patch of the series that > actually adds support for these chips. Don't document chips until you > actually support them. > >> See Documentation/devicetree/bindings/dsa/dsa.txt for a list of additional required and optional properties. >> @@ -13,60 +20,60 @@ Examples: >> >> Ethernet switch connected via SPI to the host, CPU port wired to eth0: >> >> - eth0: ethernet@10001000 { >> - 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>; >> - status = "okay"; >> + spi1: spi@f8008000 { >> + cs-gpios = <&pioC 25 0>; >> + id = <1>; >> + status = "okay"; >> >> - ksz9477: ksz9477@0 { >> - compatible = "microchip,ksz9477"; >> - reg = <0>; >> + ksz9477: ksz9477@0 { >> + compatible = "microchip,ksz9477"; >> + reg = <0>; >> >> - spi-max-frequency = <44000000>; >> - spi-cpha; >> - spi-cpol; >> + 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 = <ð0>; >> + fixed-link { >> + speed = <1000>; >> + full-duplex; >> + }; >> + }; >> + }; >> + }; >> + }; >> >> - 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 = <ð0>; >> - fixed-link { >> - speed = <1000>; >> - full-duplex; >> - }; >> - }; >> - }; >> - }; >> - }; > > This part however is a nice cleanup. You can submit this patch as a > separate patch, once netdev has opened again in about 10 days time. > > Andrew -- Best regards, Maxim Uvarov