From: Colin Foster <colin.foster@in-advantage.com> To: linux-renesas-soc@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, netdev@vger.kernel.org Cc: "John Crispin" <john@phrozen.org>, "Alexandre Belloni" <alexandre.belloni@bootlin.com>, "Claudiu Manoil" <claudiu.manoil@nxp.com>, "Marek Vasut" <marex@denx.de>, "Sean Wang" <sean.wang@mediatek.com>, "DENG Qingfang" <dqfext@gmail.com>, "Landen Chao" <Landen.Chao@mediatek.com>, "nç ÜNAL" <arinc.unal@arinc9.com>, "Clément Léger" <clement.leger@bootlin.com>, "Alvin Šipraga" <alsi@bang-olufsen.dk>, "Linus Walleij" <linus.walleij@linaro.org>, UNGLinuxDriver@microchip.com, "Woojung Huh" <woojung.huh@microchip.com>, "Matthias Brugger" <matthias.bgg@gmail.com>, "Kurt Kanzenbach" <kurt@linutronix.de>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>, "Rob Herring" <robh+dt@kernel.org>, "Paolo Abeni" <pabeni@redhat.com>, "Jakub Kicinski" <kuba@kernel.org>, "Eric Dumazet" <edumazet@google.com>, "David S. Miller" <davem@davemloft.net>, "Vladimir Oltean" <olteanv@gmail.com>, "Florian Fainelli" <f.fainelli@gmail.com>, "Andrew Lunn" <andrew@lunn.ch>, "George McCollister" <george.mccollister@gmail.com>, "Rob Herring" <robh@kernel.org> Subject: [PATCH v6 net-next 02/10] dt-bindings: net: dsa: sf2: fix brcm,use-bcm-hdr documentation Date: Mon, 2 Jan 2023 21:13:53 -0800 [thread overview] Message-ID: <20230103051401.2265961-3-colin.foster@in-advantage.com> (raw) In-Reply-To: <20230103051401.2265961-1-colin.foster@in-advantage.com> The property use-bcm-hdr was documented as an entry under the ports node for the bcm_sf2 DSA switch. This property is actually evaluated for each port. Correct the documentation to match the actual behavior and properly reference dsa-port.yaml for additional properties of the node. Suggested-by: Rob Herring <robh@kernel.org> Signed-off-by: Colin Foster <colin.foster@in-advantage.com> Acked-by: Florian Fainelli <f.fainelli@gmail.com> Reviewed-by: Rob Herring <robh@kernel.org> --- v4 -> v6 * No change v3 -> v4 * Add Acked and Reviewed tags v3 * New patch --- .../devicetree/bindings/net/dsa/brcm,sf2.yaml | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml b/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml index d159ac78cec1..eed16e216fb6 100644 --- a/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml +++ b/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml @@ -85,11 +85,16 @@ properties: ports: type: object - properties: - brcm,use-bcm-hdr: - description: if present, indicates that the switch port has Broadcom - tags enabled (per-packet metadata) - type: boolean + patternProperties: + '^port@[0-9a-f]$': + $ref: dsa-port.yaml# + unevaluatedProperties: false + + properties: + brcm,use-bcm-hdr: + description: if present, indicates that the switch port has Broadcom + tags enabled (per-packet metadata) + type: boolean required: - reg -- 2.25.1
WARNING: multiple messages have this Message-ID (diff)
From: Colin Foster <colin.foster@in-advantage.com> To: linux-renesas-soc@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, netdev@vger.kernel.org Cc: "John Crispin" <john@phrozen.org>, "Alexandre Belloni" <alexandre.belloni@bootlin.com>, "Claudiu Manoil" <claudiu.manoil@nxp.com>, "Marek Vasut" <marex@denx.de>, "Sean Wang" <sean.wang@mediatek.com>, "DENG Qingfang" <dqfext@gmail.com>, "Landen Chao" <Landen.Chao@mediatek.com>, "nç ÜNAL" <arinc.unal@arinc9.com>, "Clément Léger" <clement.leger@bootlin.com>, "Alvin Šipraga" <alsi@bang-olufsen.dk>, "Linus Walleij" <linus.walleij@linaro.org>, UNGLinuxDriver@microchip.com, "Woojung Huh" <woojung.huh@microchip.com>, "Matthias Brugger" <matthias.bgg@gmail.com>, "Kurt Kanzenbach" <kurt@linutronix.de>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>, "Rob Herring" <robh+dt@kernel.org>, "Paolo Abeni" <pabeni@redhat.com>, "Jakub Kicinski" <kuba@kernel.org>, "Eric Dumazet" <edumazet@google.com>, "David S. Miller" <davem@davemloft.net>, "Vladimir Oltean" <olteanv@gmail.com>, "Florian Fainelli" <f.fainelli@gmail.com>, "Andrew Lunn" <andrew@lunn.ch>, "George McCollister" <george.mccollister@gmail.com>, "Rob Herring" <robh@kernel.org> Subject: [PATCH v6 net-next 02/10] dt-bindings: net: dsa: sf2: fix brcm,use-bcm-hdr documentation Date: Mon, 2 Jan 2023 21:13:53 -0800 [thread overview] Message-ID: <20230103051401.2265961-3-colin.foster@in-advantage.com> (raw) In-Reply-To: <20230103051401.2265961-1-colin.foster@in-advantage.com> The property use-bcm-hdr was documented as an entry under the ports node for the bcm_sf2 DSA switch. This property is actually evaluated for each port. Correct the documentation to match the actual behavior and properly reference dsa-port.yaml for additional properties of the node. Suggested-by: Rob Herring <robh@kernel.org> Signed-off-by: Colin Foster <colin.foster@in-advantage.com> Acked-by: Florian Fainelli <f.fainelli@gmail.com> Reviewed-by: Rob Herring <robh@kernel.org> --- v4 -> v6 * No change v3 -> v4 * Add Acked and Reviewed tags v3 * New patch --- .../devicetree/bindings/net/dsa/brcm,sf2.yaml | 15 ++++++++++----- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml b/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml index d159ac78cec1..eed16e216fb6 100644 --- a/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml +++ b/Documentation/devicetree/bindings/net/dsa/brcm,sf2.yaml @@ -85,11 +85,16 @@ properties: ports: type: object - properties: - brcm,use-bcm-hdr: - description: if present, indicates that the switch port has Broadcom - tags enabled (per-packet metadata) - type: boolean + patternProperties: + '^port@[0-9a-f]$': + $ref: dsa-port.yaml# + unevaluatedProperties: false + + properties: + brcm,use-bcm-hdr: + description: if present, indicates that the switch port has Broadcom + tags enabled (per-packet metadata) + type: boolean required: - reg -- 2.25.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-01-03 5:14 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-01-03 5:13 [PATCH v6 net-next 00/10] dt-binding preparation for ocelot switches Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 5:13 ` [PATCH v6 net-next 01/10] dt-bindings: dsa: sync with maintainers Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 22:19 ` Florian Fainelli 2023-01-03 22:19 ` Florian Fainelli 2023-01-03 22:25 ` Rob Herring 2023-01-03 22:25 ` Rob Herring 2023-01-03 22:54 ` Colin Foster 2023-01-03 22:54 ` Colin Foster 2023-01-03 5:13 ` Colin Foster [this message] 2023-01-03 5:13 ` [PATCH v6 net-next 02/10] dt-bindings: net: dsa: sf2: fix brcm,use-bcm-hdr documentation Colin Foster 2023-01-03 5:13 ` [PATCH v6 net-next 03/10] dt-bindings: net: dsa: qca8k: remove address-cells and size-cells from switch node Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 5:13 ` [PATCH v6 net-next 04/10] dt-bindings: net: dsa: utilize base definitions for standard dsa switches Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 5:13 ` [PATCH v6 net-next 05/10] dt-bindings: net: dsa: allow additional ethernet-port properties Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 5:13 ` [PATCH v6 net-next 06/10] dt-bindings: net: dsa: qca8k: utilize shared dsa.yaml Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 5:13 ` [PATCH v6 net-next 07/10] dt-bindings: net: dsa: mediatek,mt7530: remove unnecessary dsa-port reference Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 5:13 ` [PATCH v6 net-next 08/10] dt-bindings: net: add generic ethernet-switch Colin Foster 2023-01-03 5:13 ` Colin Foster 2023-01-03 5:14 ` [PATCH v6 net-next 09/10] dt-bindings: net: add generic ethernet-switch-port binding Colin Foster 2023-01-03 5:14 ` Colin Foster 2023-01-03 17:56 ` Vladimir Oltean 2023-01-03 17:56 ` Vladimir Oltean 2023-01-04 15:16 ` Colin Foster 2023-01-04 15:16 ` Colin Foster 2023-01-03 5:14 ` [PATCH v6 net-next 10/10] dt-bindings: net: mscc,vsc7514-switch: utilize generic ethernet-switch.yaml Colin Foster 2023-01-03 5:14 ` Colin Foster
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20230103051401.2265961-3-colin.foster@in-advantage.com \ --to=colin.foster@in-advantage.com \ --cc=Landen.Chao@mediatek.com \ --cc=UNGLinuxDriver@microchip.com \ --cc=alexandre.belloni@bootlin.com \ --cc=alsi@bang-olufsen.dk \ --cc=andrew@lunn.ch \ --cc=arinc.unal@arinc9.com \ --cc=claudiu.manoil@nxp.com \ --cc=clement.leger@bootlin.com \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=dqfext@gmail.com \ --cc=edumazet@google.com \ --cc=f.fainelli@gmail.com \ --cc=george.mccollister@gmail.com \ --cc=john@phrozen.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=kuba@kernel.org \ --cc=kurt@linutronix.de \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=marex@denx.de \ --cc=matthias.bgg@gmail.com \ --cc=netdev@vger.kernel.org \ --cc=olteanv@gmail.com \ --cc=pabeni@redhat.com \ --cc=robh+dt@kernel.org \ --cc=robh@kernel.org \ --cc=sean.wang@mediatek.com \ --cc=woojung.huh@microchip.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.