From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id EAF98C433FE for ; Mon, 3 Oct 2022 17:49:34 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229582AbiJCRtc (ORCPT ); Mon, 3 Oct 2022 13:49:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43190 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229564AbiJCRtX (ORCPT ); Mon, 3 Oct 2022 13:49:23 -0400 Received: from mail-lf1-x12e.google.com (mail-lf1-x12e.google.com [IPv6:2a00:1450:4864:20::12e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C88AB3CBF4 for ; Mon, 3 Oct 2022 10:48:59 -0700 (PDT) Received: by mail-lf1-x12e.google.com with SMTP id d6so3549994lfs.10 for ; Mon, 03 Oct 2022 10:48:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date; bh=6ipOceJJFj+HqQfrKHGaIq1ts4QULB2eTqa5pJ5AnGQ=; b=Pb6u00JY40ihS6lWhfk/8psqfFcrlLbyTM/67KyV5kttLM7RrD9ABhM8bgO6Y+82fW o++6jBccSvlcyn7dtnthJEhrgj3tGxEos3d6Rr464yNqf4l+ToeDY8OsJQLshB5LdsNw SiafljJe0dTksArzYypzHGqVehjBcYRFpZhKvq8azF38Vm/iwVYXQ4rfefk7MpaS9/fP 3O/k45mUMWaaluWL/TLIEC75WiOxhevhXPmUATvp0eccltZYD47iWKZzKUsdf/65vX1Z gO3O0zJhUtPHUSOsU/Q34YDuVGE++QkSNob7FNppTV5sUXw5kku0zuPP2MOvvtPtcc64 vHSQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date; bh=6ipOceJJFj+HqQfrKHGaIq1ts4QULB2eTqa5pJ5AnGQ=; b=qZC3A/hRAlFZsImjgr8VdbqLnQnG+Rct246BUxMs/HdjZRkJnLtZpswwQKqYIEpPB8 H4KNC5gmA8KFJUFIefSi10kir94apQMb3MnevOgRHnE35xNJ2wwhPH1hER++Sur51808 Jr5AW0alQJMj/CKThWllCHWx3Ys3f7YRn0aP9l2Uevwuql8MOPknCYW6n7p4Yof+C5BF Dp2njZRTQL9GZuCDS0F/nA3Gr72zhPiOGtlGMuB/51gR92HOMuQ7xoUHiMZDdxipQB62 xV1e4hLNh89ijB2KYAF70OiQTlE9YE0BO7NWbH55IfOpeM96GShS+ZOo5Sh4jh0Djwko pgig== X-Gm-Message-State: ACrzQf1Kl1Ct109/6HFz6cUHxFNRwMQzc2tveR0OMlsJOdmI7PmXmMrD cXRRaYGvTzRYh3PXYgSw+YMHpQ== X-Google-Smtp-Source: AMsMyM7EWpJ7gFlCh3Jln+/dYe1qMlIoeM2gUsYwbQztUdBfRAFaIXRtE/eAHqbh+hehWvK5O+F+xQ== X-Received: by 2002:a05:6512:22cf:b0:49e:860d:8f4e with SMTP id g15-20020a05651222cf00b0049e860d8f4emr7370759lfu.584.1664819337829; Mon, 03 Oct 2022 10:48:57 -0700 (PDT) Received: from [192.168.0.21] (78-11-189-27.static.ip.netia.com.pl. [78.11.189.27]) by smtp.gmail.com with ESMTPSA id g14-20020a056512118e00b0049e9122bd1bsm1543958lfr.164.2022.10.03.10.48.56 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 03 Oct 2022 10:48:57 -0700 (PDT) Message-ID: Date: Mon, 3 Oct 2022 19:48:56 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.3.0 Subject: Re: [net-next][PATCH v4] dt-bindings: dsa: Add lan9303 yaml Content-Language: en-US To: Jerry Ray , Andrew Lunn , Vivien Didelot , Florian Fainelli , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , UNGLinuxDriver@microchip.com, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20221003164624.4823-1-jerry.ray@microchip.com> From: Krzysztof Kozlowski In-Reply-To: <20221003164624.4823-1-jerry.ray@microchip.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 03/10/2022 18:46, Jerry Ray wrote: > Adding the dt binding yaml for the lan9303 3-port ethernet switch. > The microchip lan9354 3-port ethernet switch will also use the > same binding. > > Signed-off-by: Jerry Ray > --- > v3->v4: > - Addressed v3 community feedback > v2->v3: > - removed cpu labels > - now patching against latest net-next > v1->v2: > - fixed dt_binding_check warning > - added max-speed setting on the switches 10/100 ports. > --- > .../devicetree/bindings/net/dsa/lan9303.txt | 100 ------------- > .../bindings/net/dsa/microchip,lan9303.yaml | 134 ++++++++++++++++++ > MAINTAINERS | 8 ++ > 3 files changed, 142 insertions(+), 100 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/net/dsa/lan9303.txt > create mode 100644 Documentation/devicetree/bindings/net/dsa/microchip,lan9303.yaml > > diff --git a/Documentation/devicetree/bindings/net/dsa/lan9303.txt b/Documentation/devicetree/bindings/net/dsa/lan9303.txt > deleted file mode 100644 > index 46a732087f5c..000000000000 > --- a/Documentation/devicetree/bindings/net/dsa/lan9303.txt > +++ /dev/null > @@ -1,100 +0,0 @@ > -SMSC/MicroChip LAN9303 three port ethernet switch > -------------------------------------------------- > - > -Required properties: > - > -- compatible: should be > - - "smsc,lan9303-i2c" for I2C managed mode > - or > - - "smsc,lan9303-mdio" for mdio managed mode > - > -Optional properties: > - > -- reset-gpios: GPIO to be used to reset the whole device > -- reset-duration: reset duration in milliseconds, defaults to 200 ms > - > -Subnodes: > - > -The integrated switch subnode should be specified according to the binding > -described in dsa/dsa.txt. The CPU port of this switch is always port 0. > - > -Note: always use 'reg = <0/1/2>;' for the three DSA ports, even if the device is > -configured to use 1/2/3 instead. This hardware configuration will be > -auto-detected and mapped accordingly. > - > -Example: > - > -I2C managed mode: > - > - master: masterdevice@X { > - > - fixed-link { /* RMII fixed link to LAN9303 */ > - speed = <100>; > - full-duplex; > - }; > - }; > - > - switch: switch@a { > - compatible = "smsc,lan9303-i2c"; > - reg = <0xa>; > - reset-gpios = <&gpio7 6 GPIO_ACTIVE_LOW>; > - reset-duration = <200>; > - > - ports { > - #address-cells = <1>; > - #size-cells = <0>; > - > - port@0 { /* RMII fixed link to master */ > - reg = <0>; > - ethernet = <&master>; > - }; > - > - port@1 { /* external port 1 */ > - reg = <1>; > - label = "lan1"; > - }; > - > - port@2 { /* external port 2 */ > - reg = <2>; > - label = "lan2"; > - }; > - }; > - }; > - > -MDIO managed mode: > - > - master: masterdevice@X { > - phy-handle = <&switch>; > - > - mdio { > - #address-cells = <1>; > - #size-cells = <0>; > - > - switch: switch-phy@0 { > - compatible = "smsc,lan9303-mdio"; > - reg = <0>; > - reset-gpios = <&gpio7 6 GPIO_ACTIVE_LOW>; > - reset-duration = <100>; > - > - ports { > - #address-cells = <1>; > - #size-cells = <0>; > - > - port@0 { > - reg = <0>; > - ethernet = <&master>; > - }; > - > - port@1 { /* external port 1 */ > - reg = <1>; > - label = "lan1"; > - }; > - > - port@2 { /* external port 2 */ > - reg = <2>; > - label = "lan2"; > - }; > - }; > - }; > - }; > - }; > diff --git a/Documentation/devicetree/bindings/net/dsa/microchip,lan9303.yaml b/Documentation/devicetree/bindings/net/dsa/microchip,lan9303.yaml > new file mode 100644 > index 000000000000..ca6cbe83ba75 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/dsa/microchip,lan9303.yaml > @@ -0,0 +1,134 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/dsa/microchip,lan9303.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: LAN9303 Ethernet Switch Series > + > +allOf: > + - $ref: dsa.yaml# > + > +maintainers: > + - UNGLinuxDriver@microchip.com > + > +properties: > + compatible: > + oneOf: > + - enum: > + - smsc,lan9303-mdio > + - microchip,lan9354-mdio > + - enum: > + - smsc,lan9303-i2c > + - microchip,lan9354-i2c This still does not make sense. It is one enum. Drop oneOf. > + > + reg: > + maxItems: 1 > + > + reset-gpios: > + description: Optional reset line > + maxItems: 1 > + > + reset-duration: > + description: Reset duration in milliseconds > + default: 200 This is a friendly reminder during the review process. It seems my previous comments were not fully addressed. Maybe my feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. Thank you. > + > +required: > + - compatible > + - reg > + Best regards, Krzysztof