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 X-Spam-Level: X-Spam-Status: No, score=-6.6 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EE483C388F7 for ; Tue, 10 Nov 2020 22:43:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 9E161206B6 for ; Tue, 10 Nov 2020 22:43:16 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="D6gF6EqS" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1732500AbgKJWnQ (ORCPT ); Tue, 10 Nov 2020 17:43:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53396 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1731713AbgKJWnP (ORCPT ); Tue, 10 Nov 2020 17:43:15 -0500 Received: from mail-ej1-x644.google.com (mail-ej1-x644.google.com [IPv6:2a00:1450:4864:20::644]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D5671C0613D1; Tue, 10 Nov 2020 14:43:14 -0800 (PST) Received: by mail-ej1-x644.google.com with SMTP id dk16so8222ejb.12; Tue, 10 Nov 2020 14:43:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=N/PQEE+/FPJ0NMebPS1QLSFSGVBrmG2LTD+JvZjZhrA=; b=D6gF6EqSrlCZd90BEdgFKvy5HfV0oHNA1w+wBVLraTax8HCvBdJE6z9M00jaoOkkNH sS7dIbvHM2HtO5yVicnl6VkVyJ9AJ0cC3aL5UKRY7BPqGZVutnzSjNEAf66tC90ogd6m xWkt0UNK+X6rk95fmVaUesY2Tw8w+zI7+rkTcv2TaIN/DBXN8wZEd0JyMTrbdi/3OBNe n334YMTwwyzsp//8LGhwXxfW+h0NKpiQpxj/Lo2zbVAvIFGPvCuhy9MqHj0FZYlsMX0Z 4btSPdh9UiWJdr6Spkgu2KLZ9HbHVH+/S86OOiluAcLy0U+xbHRH16N3qa2tNd2Seqqd X9jA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=N/PQEE+/FPJ0NMebPS1QLSFSGVBrmG2LTD+JvZjZhrA=; b=tRHYhouZWr4lW9p3d8TmODUfMCNEQSOFWQD74anTH+Wf+7mp7AYPdvcWxUgoH6GRFT 32dJwnRpUg7/oeZnsbjH93ROWu0xC9mOqEJVcLYQPvb9NSXaAp90Kf0QUl4rEqdvKLUZ PFR+CHIycYvTcxWpCmJifpeVLn+8yCnksyhDHjGxMNeWyvtcZRAuItN6+CVp4rYQ34zI V5TCLLSeqN0W/bsDIqAwdJWjdPq8QhuP2vfkcaKpTpIHb1lsw1s45p2xIjqEBVcijG5Z lV50xC8sW2SsCmWmpILL/ojLnIsEEu7azWI/4zeuJIYuDVf32o1xfVtiMcYLwV/c3YF5 xcKA== X-Gm-Message-State: AOAM532b6rkNlAKWPUjXJBhfvzB5pVKcOEx85DpHU+mSdCvvD1mnTczF JQl0Tb48bxgmrbcz8jmDi9Y= X-Google-Smtp-Source: ABdhPJwQ4CDUWTmtukolhtYeoKhgrRiaz/vIih5rgiJyldK+BAVzvoXeE6dMZSSEIgfRS7KaK0sXQA== X-Received: by 2002:a17:906:a195:: with SMTP id s21mr21482422ejy.146.1605048193570; Tue, 10 Nov 2020 14:43:13 -0800 (PST) Received: from skbuf ([188.25.2.177]) by smtp.gmail.com with ESMTPSA id mj17sm74969ejb.59.2020.11.10.14.43.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Nov 2020 14:43:13 -0800 (PST) Date: Wed, 11 Nov 2020 00:43:11 +0200 From: Vladimir Oltean To: Florian Fainelli Cc: netdev@vger.kernel.org, Kurt Kanzenbach , Andrew Lunn , Vivien Didelot , "David S. Miller" , Jakub Kicinski , Rob Herring , Ray Jui , Scott Branden , "maintainer:BROADCOM IPROC ARM ARCHITECTURE" , Hauke Mehrtens , =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , "moderated list:BROADCOM IPROC ARM ARCHITECTURE" Subject: Re: [PATCH 10/10] dt-bindings: net: dsa: b53: Add YAML bindings Message-ID: <20201110224311.xtgv6wqqzmg77uny@skbuf> References: <20201110033113.31090-1-f.fainelli@gmail.com> <20201110033113.31090-11-f.fainelli@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201110033113.31090-11-f.fainelli@gmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Nov 09, 2020 at 07:31:13PM -0800, Florian Fainelli wrote: > diff --git a/Documentation/devicetree/bindings/net/dsa/b53.yaml b/Documentation/devicetree/bindings/net/dsa/b53.yaml > new file mode 100644 > index 000000000000..4fcbac1de95b > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/dsa/b53.yaml > @@ -0,0 +1,249 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/dsa/b53.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Broadcom BCM53xx Ethernet switches > + > +allOf: > + - $ref: dsa.yaml# > + > +maintainers: > + - Florian Fainelli > + > +description: > + Broadcom BCM53xx Ethernet switches > + > +properties: > + compatible: > + oneOf: > + - const: brcm,bcm5325 > + - const: brcm,bcm53115 > + - const: brcm,bcm53125 > + - const: brcm,bcm53128 > + - const: brcm,bcm5365 > + - const: brcm,bcm5395 > + - const: brcm,bcm5389 > + - const: brcm,bcm5397 > + - const: brcm,bcm5398 > + - items: > + - const: brcm,bcm11360-srab > + - const: brcm,cygnus-srab > + - items: > + - enum: > + - brcm,bcm53010-srab > + - brcm,bcm53011-srab > + - brcm,bcm53012-srab > + - brcm,bcm53018-srab > + - brcm,bcm53019-srab > + - const: brcm,bcm5301x-srab > + - items: > + - enum: > + - brcm,bcm11404-srab > + - brcm,bcm11407-srab > + - brcm,bcm11409-srab > + - brcm,bcm58310-srab > + - brcm,bcm58311-srab > + - brcm,bcm58313-srab > + - const: brcm,omega-srab > + - items: > + - enum: > + - brcm,bcm58522-srab > + - brcm,bcm58523-srab > + - brcm,bcm58525-srab > + - brcm,bcm58622-srab > + - brcm,bcm58623-srab > + - brcm,bcm58625-srab > + - brcm,bcm88312-srab > + - const: brcm,nsp-srab > + - items: > + - enum: > + - brcm,bcm3384-switch > + - brcm,bcm6328-switch > + - brcm,bcm6368-switch > + - const: brcm,bcm63xx-switch > + > +required: > + - compatible > + - reg > + > +# BCM585xx/586xx/88312 SoCs > +if: > + properties: > + compatible: > + contains: > + enum: > + - brcm,bcm58522-srab > + - brcm,bcm58523-srab > + - brcm,bcm58525-srab > + - brcm,bcm58622-srab > + - brcm,bcm58623-srab > + - brcm,bcm58625-srab > + - brcm,bcm88312-srab > +then: > + properties: > + reg: > + minItems: 3 > + maxItems: 3 > + reg-names: > + items: > + - const: srab > + - const: mux_config > + - const: sgmii_config I am only reading these with a human eye, I don't parse YAML syntax. Does the syntax enforce that these reg-names are declared in this precise order, which is necessary for the proper operation of the driver? > + interrupts: > + minItems: 13 > + maxItems: 13 > + interrupt-names: > + items: > + - const: link_state_p0 > + - const: link_state_p1 > + - const: link_state_p2 > + - const: link_state_p3 > + - const: link_state_p4 > + - const: link_state_p5 > + - const: link_state_p7 > + - const: link_state_p8 > + - const: phy > + - const: ts > + - const: imp_sleep_timer_p5 > + - const: imp_sleep_timer_p7 > + - const: imp_sleep_timer_p8 > + required: > + - interrupts > +else: > + properties: > + reg: > + maxItems: 1 > + > +unevaluatedProperties: false > + > +examples: > + - | > + mdio { > + #address-cells = <1>; > + #size-cells = <0>; > + > + switch@1e { You have renamed a node called 'ethernet-switch' into one called 'switch'. Was it deliberate? > + compatible = "brcm,bcm53125"; > + reg = <30>; > + > + ethernet-ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + label = "lan1"; > + }; > + > + port@1 { > + reg = <1>; > + label = "lan2"; > + }; > + > + port@5 { > + reg = <5>; > + label = "cable-modem"; > + phy-mode = "rgmii-txid"; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + > + port@8 { > + reg = <8>; > + label = "cpu"; > + phy-mode = "rgmii-txid"; > + ethernet = <ð0>; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + }; > + }; > + }; > + - | > + #include > + #include > + > + axi { > + #address-cells = <1>; > + #size-cells = <1>; > + > + switch@36000 { > + compatible = "brcm,bcm58623-srab", "brcm,nsp-srab"; > + reg = <0x36000 0x1000>, > + <0x3f308 0x8>, > + <0x3f410 0xc>; > + reg-names = "srab", "mux_config", "sgmii_config"; > + interrupts = , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + ; > + interrupt-names = "link_state_p0", > + "link_state_p1", > + "link_state_p2", > + "link_state_p3", > + "link_state_p4", > + "link_state_p5", > + "link_state_p7", > + "link_state_p8", > + "phy", > + "ts", > + "imp_sleep_timer_p5", > + "imp_sleep_timer_p7", > + "imp_sleep_timer_p8"; > + > + ethernet-ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + label = "port0"; > + reg = <0>; > + }; > + > + port@1 { > + label = "port1"; > + reg = <1>; > + }; > + > + port@2 { > + label = "port2"; > + reg = <2>; > + }; > + > + port@3 { > + label = "port3"; > + reg = <3>; > + }; > + > + port@4 { > + label = "port4"; > + reg = <4>; > + }; > + > + port@8 { > + ethernet = <&amac2>; > + label = "cpu"; > + reg = <8>; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + }; > + }; > + }; 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 X-Spam-Level: X-Spam-Status: No, score=-6.5 required=3.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 901B9C388F7 for ; Tue, 10 Nov 2020 22:46:36 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 1F8AE2076E for ; Tue, 10 Nov 2020 22:46:36 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="WiRelFtg"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="D6gF6EqS" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1F8AE2076E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Om2g2atl4yY3yUzDIXYRyBipeb8Rt03MXcuGN2HicD8=; b=WiRelFtgqYZ9U5tMbQQmS0rbU x6BLACzJTaOmQX2fO0O2uSQWV9k4/ldEOA5RL4FznaaupYaGMIc1uumgerO5j6SdDpAeXpyNOvxRC L0FbZ9SsCLvgwixBhwOFZZX9Qk+SQcxgjL7TvufmFEcL9Y2oelz+UmV/yFgkTPiOBPijnF4wkc8yO D/FAZ/Csx8zfq8LBnLkJC82ptxitCArjjJV4MoazGsJe01jW1ebmuNMCW4CD0tVvv23+Z7UNnB2rM ql5qIcM5ctzY9/4R1QoOIgAODeqhp4QbAMi7dh5BPY2hyXLAtfuvarBdFn1oE6SK2ncaOJ1YUFBIX 1ErXDCdyA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kccNo-0002qJ-Vd; Tue, 10 Nov 2020 22:45:01 +0000 Received: from mail-ej1-x641.google.com ([2a00:1450:4864:20::641]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1kccM7-0001vY-6x for linux-arm-kernel@lists.infradead.org; Tue, 10 Nov 2020 22:43:39 +0000 Received: by mail-ej1-x641.google.com with SMTP id oq3so39629ejb.7 for ; Tue, 10 Nov 2020 14:43:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=N/PQEE+/FPJ0NMebPS1QLSFSGVBrmG2LTD+JvZjZhrA=; b=D6gF6EqSrlCZd90BEdgFKvy5HfV0oHNA1w+wBVLraTax8HCvBdJE6z9M00jaoOkkNH sS7dIbvHM2HtO5yVicnl6VkVyJ9AJ0cC3aL5UKRY7BPqGZVutnzSjNEAf66tC90ogd6m xWkt0UNK+X6rk95fmVaUesY2Tw8w+zI7+rkTcv2TaIN/DBXN8wZEd0JyMTrbdi/3OBNe n334YMTwwyzsp//8LGhwXxfW+h0NKpiQpxj/Lo2zbVAvIFGPvCuhy9MqHj0FZYlsMX0Z 4btSPdh9UiWJdr6Spkgu2KLZ9HbHVH+/S86OOiluAcLy0U+xbHRH16N3qa2tNd2Seqqd X9jA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=N/PQEE+/FPJ0NMebPS1QLSFSGVBrmG2LTD+JvZjZhrA=; b=ZoiD9M5QgS6oSfPbCT5E/fascWqZUFJuAN0gsuM9T+KEJhMv0tu4nVAfge60kSHM/i CPW/efnYs7UMo+ijZswFKIVAho5VGW0i3MK6ySh5JzOEkzpGoA5QAWm6sDkaludd89VO jnlQ1xD1yfAoX3RPL/d4i4AEsp6EAdD219eTQTfADGYlZuJvLpn04+AA9+cBMUBUbXKu SdcTuN1WLQa9Cbaw66j8UnKU2il2WbO1SfLiSczZIHdPYgvAih02a8ZVbLAsD6kHQPyZ re+i7AWFyGY6q9J+teTy83fKUdBB/u+LAewO70W/0E4pMR+lATWyTwtcBby9VTY8WgsL Y1og== X-Gm-Message-State: AOAM532tKtoIecNw1MFh8xOd2w4s6O7XoIqz6iIswVTEdADk56oScTI1 95YM6Hec6SuAP4gpkK0HmB8= X-Google-Smtp-Source: ABdhPJwQ4CDUWTmtukolhtYeoKhgrRiaz/vIih5rgiJyldK+BAVzvoXeE6dMZSSEIgfRS7KaK0sXQA== X-Received: by 2002:a17:906:a195:: with SMTP id s21mr21482422ejy.146.1605048193570; Tue, 10 Nov 2020 14:43:13 -0800 (PST) Received: from skbuf ([188.25.2.177]) by smtp.gmail.com with ESMTPSA id mj17sm74969ejb.59.2020.11.10.14.43.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 10 Nov 2020 14:43:13 -0800 (PST) Date: Wed, 11 Nov 2020 00:43:11 +0200 From: Vladimir Oltean To: Florian Fainelli Subject: Re: [PATCH 10/10] dt-bindings: net: dsa: b53: Add YAML bindings Message-ID: <20201110224311.xtgv6wqqzmg77uny@skbuf> References: <20201110033113.31090-1-f.fainelli@gmail.com> <20201110033113.31090-11-f.fainelli@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20201110033113.31090-11-f.fainelli@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201110_174315_332225_1B14163E X-CRM114-Status: GOOD ( 16.43 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Andrew Lunn , Scott Branden , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , netdev@vger.kernel.org, =?utf-8?B?UmFmYcWCIE1pxYJlY2tp?= , open list , Vivien Didelot , Rob Herring , "maintainer:BROADCOM IPROC ARM ARCHITECTURE" , Hauke Mehrtens , Ray Jui , Jakub Kicinski , Kurt Kanzenbach , "David S. Miller" , "moderated list:BROADCOM IPROC ARM ARCHITECTURE" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Nov 09, 2020 at 07:31:13PM -0800, Florian Fainelli wrote: > diff --git a/Documentation/devicetree/bindings/net/dsa/b53.yaml b/Documentation/devicetree/bindings/net/dsa/b53.yaml > new file mode 100644 > index 000000000000..4fcbac1de95b > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/dsa/b53.yaml > @@ -0,0 +1,249 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/dsa/b53.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Broadcom BCM53xx Ethernet switches > + > +allOf: > + - $ref: dsa.yaml# > + > +maintainers: > + - Florian Fainelli > + > +description: > + Broadcom BCM53xx Ethernet switches > + > +properties: > + compatible: > + oneOf: > + - const: brcm,bcm5325 > + - const: brcm,bcm53115 > + - const: brcm,bcm53125 > + - const: brcm,bcm53128 > + - const: brcm,bcm5365 > + - const: brcm,bcm5395 > + - const: brcm,bcm5389 > + - const: brcm,bcm5397 > + - const: brcm,bcm5398 > + - items: > + - const: brcm,bcm11360-srab > + - const: brcm,cygnus-srab > + - items: > + - enum: > + - brcm,bcm53010-srab > + - brcm,bcm53011-srab > + - brcm,bcm53012-srab > + - brcm,bcm53018-srab > + - brcm,bcm53019-srab > + - const: brcm,bcm5301x-srab > + - items: > + - enum: > + - brcm,bcm11404-srab > + - brcm,bcm11407-srab > + - brcm,bcm11409-srab > + - brcm,bcm58310-srab > + - brcm,bcm58311-srab > + - brcm,bcm58313-srab > + - const: brcm,omega-srab > + - items: > + - enum: > + - brcm,bcm58522-srab > + - brcm,bcm58523-srab > + - brcm,bcm58525-srab > + - brcm,bcm58622-srab > + - brcm,bcm58623-srab > + - brcm,bcm58625-srab > + - brcm,bcm88312-srab > + - const: brcm,nsp-srab > + - items: > + - enum: > + - brcm,bcm3384-switch > + - brcm,bcm6328-switch > + - brcm,bcm6368-switch > + - const: brcm,bcm63xx-switch > + > +required: > + - compatible > + - reg > + > +# BCM585xx/586xx/88312 SoCs > +if: > + properties: > + compatible: > + contains: > + enum: > + - brcm,bcm58522-srab > + - brcm,bcm58523-srab > + - brcm,bcm58525-srab > + - brcm,bcm58622-srab > + - brcm,bcm58623-srab > + - brcm,bcm58625-srab > + - brcm,bcm88312-srab > +then: > + properties: > + reg: > + minItems: 3 > + maxItems: 3 > + reg-names: > + items: > + - const: srab > + - const: mux_config > + - const: sgmii_config I am only reading these with a human eye, I don't parse YAML syntax. Does the syntax enforce that these reg-names are declared in this precise order, which is necessary for the proper operation of the driver? > + interrupts: > + minItems: 13 > + maxItems: 13 > + interrupt-names: > + items: > + - const: link_state_p0 > + - const: link_state_p1 > + - const: link_state_p2 > + - const: link_state_p3 > + - const: link_state_p4 > + - const: link_state_p5 > + - const: link_state_p7 > + - const: link_state_p8 > + - const: phy > + - const: ts > + - const: imp_sleep_timer_p5 > + - const: imp_sleep_timer_p7 > + - const: imp_sleep_timer_p8 > + required: > + - interrupts > +else: > + properties: > + reg: > + maxItems: 1 > + > +unevaluatedProperties: false > + > +examples: > + - | > + mdio { > + #address-cells = <1>; > + #size-cells = <0>; > + > + switch@1e { You have renamed a node called 'ethernet-switch' into one called 'switch'. Was it deliberate? > + compatible = "brcm,bcm53125"; > + reg = <30>; > + > + ethernet-ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + label = "lan1"; > + }; > + > + port@1 { > + reg = <1>; > + label = "lan2"; > + }; > + > + port@5 { > + reg = <5>; > + label = "cable-modem"; > + phy-mode = "rgmii-txid"; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + > + port@8 { > + reg = <8>; > + label = "cpu"; > + phy-mode = "rgmii-txid"; > + ethernet = <ð0>; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + }; > + }; > + }; > + - | > + #include > + #include > + > + axi { > + #address-cells = <1>; > + #size-cells = <1>; > + > + switch@36000 { > + compatible = "brcm,bcm58623-srab", "brcm,nsp-srab"; > + reg = <0x36000 0x1000>, > + <0x3f308 0x8>, > + <0x3f410 0xc>; > + reg-names = "srab", "mux_config", "sgmii_config"; > + interrupts = , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + ; > + interrupt-names = "link_state_p0", > + "link_state_p1", > + "link_state_p2", > + "link_state_p3", > + "link_state_p4", > + "link_state_p5", > + "link_state_p7", > + "link_state_p8", > + "phy", > + "ts", > + "imp_sleep_timer_p5", > + "imp_sleep_timer_p7", > + "imp_sleep_timer_p8"; > + > + ethernet-ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + label = "port0"; > + reg = <0>; > + }; > + > + port@1 { > + label = "port1"; > + reg = <1>; > + }; > + > + port@2 { > + label = "port2"; > + reg = <2>; > + }; > + > + port@3 { > + label = "port3"; > + reg = <3>; > + }; > + > + port@4 { > + label = "port4"; > + reg = <4>; > + }; > + > + port@8 { > + ethernet = <&amac2>; > + label = "cpu"; > + reg = <8>; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + }; > + }; > + }; _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel