From: "Arınç ÜNAL" <arinc.unal@arinc9.com> To: "Andrew Lunn" <andrew@lunn.ch>, "Vivien Didelot" <vivien.didelot@gmail.com>, "Florian Fainelli" <f.fainelli@gmail.com>, "Vladimir Oltean" <olteanv@gmail.com>, "David S . Miller" <davem@davemloft.net>, "Eric Dumazet" <edumazet@google.com>, "Jakub Kicinski" <kuba@kernel.org>, "Paolo Abeni" <pabeni@redhat.com>, "Rob Herring" <robh+dt@kernel.org>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>, "Matthias Brugger" <matthias.bgg@gmail.com>, "Sean Wang" <sean.wang@mediatek.com>, "Landen Chao" <Landen.Chao@mediatek.com>, "DENG Qingfang" <dqfext@gmail.com>, "Frank Wunderlich" <frank-w@public-files.de>, "Luiz Angelo Daros de Luca" <luizluca@gmail.com>, "Sander Vanheule" <sander@svanheule.net>, "René van Dorst" <opensource@vdorst.com>, "Daniel Golle" <daniel@makrotopia.org>, erkin.bozoglu@xeront.com, "Sergio Paracuellos" <sergio.paracuellos@gmail.com> Cc: netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, "Arınç ÜNAL" <arinc.unal@arinc9.com> Subject: [PATCH v2 2/7] dt-bindings: net: dsa: mediatek,mt7530: fix reset lines Date: Sat, 13 Aug 2022 18:44:10 +0300 [thread overview] Message-ID: <20220813154415.349091-3-arinc.unal@arinc9.com> (raw) In-Reply-To: <20220813154415.349091-1-arinc.unal@arinc9.com> - Fix description of mediatek,mcm. mediatek,mcm is not used on MT7623NI. - Add description for reset-gpios. - Invalidate reset-gpios if mediatek,mcm is used. - Invalidate mediatek,mcm if the compatible device is mediatek,mt7531. - Require mediatek,mcm for the described MT7621 SoCs as the compatible string is only used for MT7530 which is a part of the multi-chip module. Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com> --- .../bindings/net/dsa/mediatek,mt7530.yaml | 31 +++++++++++++++++-- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml index edf48e917173..4c99266ce82a 100644 --- a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml +++ b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml @@ -110,11 +110,15 @@ properties: mediatek,mcm: type: boolean description: - if defined, indicates that either MT7530 is the part on multi-chip - module belong to MT7623A has or the remotely standalone chip as the - function MT7623N reference board provided for. + Used for MT7621AT, MT7621DAT, MT7621ST and MT7623AI SoCs which the MT7530 + switch is a part of the multi-chip module. reset-gpios: + description: + GPIO to reset the switch. Use this if mediatek,mcm is not used. + This property is optional because some boards share the reset line with + other components which makes it impossible to probe the switch if the + reset line is used. maxItems: 1 reset-names: @@ -165,6 +169,9 @@ allOf: required: - mediatek,mcm then: + properties: + reset-gpios: false + required: - resets - reset-names @@ -182,6 +189,24 @@ allOf: - core-supply - io-supply + - if: + properties: + compatible: + items: + - const: mediatek,mt7531 + then: + properties: + mediatek,mcm: false + + - if: + properties: + compatible: + items: + - const: mediatek,mt7621 + then: + required: + - mediatek,mcm + unevaluatedProperties: false examples: -- 2.34.1
WARNING: multiple messages have this Message-ID (diff)
From: "Arınç ÜNAL" <arinc.unal@arinc9.com> To: "Andrew Lunn" <andrew@lunn.ch>, "Vivien Didelot" <vivien.didelot@gmail.com>, "Florian Fainelli" <f.fainelli@gmail.com>, "Vladimir Oltean" <olteanv@gmail.com>, "David S . Miller" <davem@davemloft.net>, "Eric Dumazet" <edumazet@google.com>, "Jakub Kicinski" <kuba@kernel.org>, "Paolo Abeni" <pabeni@redhat.com>, "Rob Herring" <robh+dt@kernel.org>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>, "Matthias Brugger" <matthias.bgg@gmail.com>, "Sean Wang" <sean.wang@mediatek.com>, "Landen Chao" <Landen.Chao@mediatek.com>, "DENG Qingfang" <dqfext@gmail.com>, "Frank Wunderlich" <frank-w@public-files.de>, "Luiz Angelo Daros de Luca" <luizluca@gmail.com>, "Sander Vanheule" <sander@svanheule.net>, "René van Dorst" <opensource@vdorst.com>, "Daniel Golle" <daniel@makrotopia.org>, erkin.bozoglu@xeront.com, "Sergio Paracuellos" <sergio.paracuellos@gmail.com> Cc: netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, "Arınç ÜNAL" <arinc.unal@arinc9.com> Subject: [PATCH v2 2/7] dt-bindings: net: dsa: mediatek,mt7530: fix reset lines Date: Sat, 13 Aug 2022 18:44:10 +0300 [thread overview] Message-ID: <20220813154415.349091-3-arinc.unal@arinc9.com> (raw) In-Reply-To: <20220813154415.349091-1-arinc.unal@arinc9.com> - Fix description of mediatek,mcm. mediatek,mcm is not used on MT7623NI. - Add description for reset-gpios. - Invalidate reset-gpios if mediatek,mcm is used. - Invalidate mediatek,mcm if the compatible device is mediatek,mt7531. - Require mediatek,mcm for the described MT7621 SoCs as the compatible string is only used for MT7530 which is a part of the multi-chip module. Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com> --- .../bindings/net/dsa/mediatek,mt7530.yaml | 31 +++++++++++++++++-- 1 file changed, 28 insertions(+), 3 deletions(-) diff --git a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml index edf48e917173..4c99266ce82a 100644 --- a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml +++ b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml @@ -110,11 +110,15 @@ properties: mediatek,mcm: type: boolean description: - if defined, indicates that either MT7530 is the part on multi-chip - module belong to MT7623A has or the remotely standalone chip as the - function MT7623N reference board provided for. + Used for MT7621AT, MT7621DAT, MT7621ST and MT7623AI SoCs which the MT7530 + switch is a part of the multi-chip module. reset-gpios: + description: + GPIO to reset the switch. Use this if mediatek,mcm is not used. + This property is optional because some boards share the reset line with + other components which makes it impossible to probe the switch if the + reset line is used. maxItems: 1 reset-names: @@ -165,6 +169,9 @@ allOf: required: - mediatek,mcm then: + properties: + reset-gpios: false + required: - resets - reset-names @@ -182,6 +189,24 @@ allOf: - core-supply - io-supply + - if: + properties: + compatible: + items: + - const: mediatek,mt7531 + then: + properties: + mediatek,mcm: false + + - if: + properties: + compatible: + items: + - const: mediatek,mt7621 + then: + required: + - mediatek,mcm + unevaluatedProperties: false examples: -- 2.34.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:[~2022-08-13 15:45 UTC|newest] Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-13 15:44 [PATCH v2 0/7] completely rework mediatek,mt7530 binding Arınç ÜNAL 2022-08-13 15:44 ` Arınç ÜNAL 2022-08-13 15:44 ` [PATCH v2 1/7] dt-bindings: net: dsa: mediatek,mt7530: make trivial changes Arınç ÜNAL 2022-08-13 15:44 ` Arınç ÜNAL 2022-08-16 20:49 ` Rob Herring 2022-08-16 20:49 ` Rob Herring 2022-08-16 20:49 ` Rob Herring 2022-08-13 15:44 ` Arınç ÜNAL [this message] 2022-08-13 15:44 ` [PATCH v2 2/7] dt-bindings: net: dsa: mediatek,mt7530: fix reset lines Arınç ÜNAL 2022-08-16 20:52 ` Rob Herring 2022-08-16 20:52 ` Rob Herring 2022-08-16 20:52 ` Rob Herring 2022-08-16 22:25 ` Arınç ÜNAL 2022-08-16 22:25 ` Arınç ÜNAL 2022-08-16 22:25 ` Arınç ÜNAL 2022-08-13 15:44 ` [PATCH v2 3/7] dt-bindings: net: dsa: mediatek,mt7530: update examples Arınç ÜNAL 2022-08-13 15:44 ` Arınç ÜNAL 2022-08-16 21:02 ` Rob Herring 2022-08-16 21:02 ` Rob Herring 2022-08-16 21:02 ` Rob Herring 2022-08-16 22:15 ` Arınç ÜNAL 2022-08-16 22:15 ` Arınç ÜNAL 2022-08-16 22:15 ` Arınç ÜNAL 2022-08-13 15:44 ` [PATCH v2 4/7] dt-bindings: net: dsa: mediatek,mt7530: define port binding per compatible Arınç ÜNAL 2022-08-13 15:44 ` Arınç ÜNAL 2022-08-19 12:43 ` Krzysztof Kozlowski 2022-08-19 12:43 ` Krzysztof Kozlowski 2022-08-20 7:34 ` Arınç ÜNAL 2022-08-20 7:34 ` Arınç ÜNAL 2022-08-23 10:48 ` Krzysztof Kozlowski 2022-08-23 10:48 ` Krzysztof Kozlowski 2022-08-13 15:44 ` [PATCH v2 5/7] dt-bindings: net: dsa: mediatek,mt7530: remove unnecesary lines Arınç ÜNAL 2022-08-13 15:44 ` Arınç ÜNAL 2022-08-16 21:14 ` Rob Herring 2022-08-16 21:14 ` Rob Herring 2022-08-16 21:14 ` Rob Herring 2022-08-16 22:21 ` Arınç ÜNAL 2022-08-16 22:21 ` Arınç ÜNAL 2022-08-16 22:21 ` Arınç ÜNAL 2022-08-13 15:44 ` [PATCH v2 6/7] dt-bindings: net: dsa: mediatek,mt7530: define phy-mode for each compatible Arınç ÜNAL 2022-08-13 15:44 ` Arınç ÜNAL 2022-08-16 21:21 ` Rob Herring 2022-08-16 21:21 ` Rob Herring 2022-08-16 21:21 ` Rob Herring 2022-08-16 22:09 ` Arınç ÜNAL 2022-08-16 22:09 ` Arınç ÜNAL 2022-08-16 22:09 ` Arınç ÜNAL 2022-08-13 15:44 ` [PATCH v2 7/7] dt-bindings: net: dsa: mediatek,mt7530: update binding description Arınç ÜNAL 2022-08-13 15:44 ` Arınç ÜNAL 2022-08-16 21:25 ` Rob Herring 2022-08-16 21:25 ` Rob Herring 2022-08-16 21:25 ` Rob Herring 2022-08-16 21:53 ` Arınç ÜNAL 2022-08-16 21:53 ` Arınç ÜNAL 2022-08-16 21:53 ` Arınç ÜNAL
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=20220813154415.349091-3-arinc.unal@arinc9.com \ --to=arinc.unal@arinc9.com \ --cc=Landen.Chao@mediatek.com \ --cc=andrew@lunn.ch \ --cc=daniel@makrotopia.org \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=dqfext@gmail.com \ --cc=edumazet@google.com \ --cc=erkin.bozoglu@xeront.com \ --cc=f.fainelli@gmail.com \ --cc=frank-w@public-files.de \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=kuba@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=luizluca@gmail.com \ --cc=matthias.bgg@gmail.com \ --cc=netdev@vger.kernel.org \ --cc=olteanv@gmail.com \ --cc=opensource@vdorst.com \ --cc=pabeni@redhat.com \ --cc=robh+dt@kernel.org \ --cc=sander@svanheule.net \ --cc=sean.wang@mediatek.com \ --cc=sergio.paracuellos@gmail.com \ --cc=vivien.didelot@gmail.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.