The Motorcomm YT921x series is a family of Ethernet switches with up to
8 internal GbE PHYs and up to 2 GMACs.
Signed-off-by: David Yang <mmyangfl@gmail.com>
---
.../bindings/net/dsa/motorcomm,yt921x.yaml | 150 ++++++++++++++++++
1 file changed, 150 insertions(+)
create mode 100644 Documentation/devicetree/bindings/net/dsa/motorcomm,yt921x.yaml
diff --git a/Documentation/devicetree/bindings/net/dsa/motorcomm,yt921x.yaml b/Documentation/devicetree/bindings/net/dsa/motorcomm,yt921x.yaml
new file mode 100644
index 000000000000..3f625de49910
--- /dev/null
+++ b/Documentation/devicetree/bindings/net/dsa/motorcomm,yt921x.yaml
@@ -0,0 +1,150 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/net/dsa/motorcomm,yt921x.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Motorcomm YT921x Ethernet switch family
+
+maintainers:
+ - David Yang <mmyangfl@gmail.com>
+
+description: |
+ The Motorcomm YT921x series is a family of Ethernet switches with up to 8
+ internal GbE PHYs and up to 2 GMACs, including:
+
+ - YT9215S / YT9215RB / YT9215SC: 5 GbE PHYs (Port 0-4) + 2 GMACs (Port 8-9)
+ - YT9213NB: 2 GbE PHYs (Port 1/3) + 1 GMAC (Port 9)
+ - YT9214NB: 2 GbE PHYs (Port 1/3) + 2 GMACs (Port 8-9)
+ - YT9218N: 8 GbE PHYs (Port 0-7)
+ - YT9218MB: 8 GbE PHYs (Port 0-7) + 2 GMACs (Port 8-9)
+
+ Any port can be used as the CPU port.
+
+properties:
+ compatible:
+ const: motorcomm,yt9215
+
+ reg:
+ enum: [0x0, 0x1d]
+
+ reset-gpios:
+ maxItems: 1
+
+ mdio:
+ $ref: /schemas/net/mdio.yaml#
+ unevaluatedProperties: false
+ description: |
+ Internal MDIO bus for the internal GbE PHYs. PHYs 0-7 are used for Port
+ 0-7 respectively.
+
+ mdio-external:
+ $ref: /schemas/net/mdio.yaml#
+ unevaluatedProperties: false
+ description: |
+ External MDIO bus to access external components. External PHYs for GMACs
+ (Port 8-9) are expected to be connected to the external MDIO bus in
+ vendor's reference design, but that is not a hard limitation from the
+ chip.
+
+required:
+ - compatible
+ - reg
+
+allOf:
+ - $ref: dsa.yaml#/$defs/ethernet-ports
+
+unevaluatedProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ switch@1d {
+ compatible = "motorcomm,yt9215";
+ /* default 0x1d, alternate 0x0 */
+ reg = <0x1d>;
+ reset-gpios = <&tlmm 39 GPIO_ACTIVE_LOW>;
+
+ mdio {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ sw_phy0: phy@0 {
+ reg = <0x0>;
+ };
+ };
+
+ mdio-external {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ phy1: phy@b {
+ reg = <0xb>;
+ };
+ };
+
+ ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ /* phy-handle is optional for internal PHYs */
+ port@0 {
+ reg = <0>;
+ label = "lan1";
+ phy-mode = "internal";
+ phy-handle = <&sw_phy0>;
+ };
+
+ port@1 {
+ reg = <1>;
+ label = "lan2";
+ phy-mode = "internal";
+ };
+
+ port@2 {
+ reg = <2>;
+ label = "lan3";
+ phy-mode = "internal";
+ };
+
+ port@3 {
+ reg = <3>;
+ label = "lan4";
+ phy-mode = "internal";
+ };
+
+ port@4 {
+ reg = <4>;
+ label = "lan5";
+ phy-mode = "internal";
+ };
+
+ /* CPU port */
+ port@8 {
+ reg = <8>;
+ phy-mode = "sgmii";
+ ethernet = <ð0>;
+
+ fixed-link {
+ speed = <1000>;
+ full-duplex;
+ pause;
+ asym-pause;
+ };
+ };
+
+ /* if external phy is connected to a MAC */
+ port@9 {
+ reg = <9>;
+ label = "wan";
+ phy-mode = "rgmii";
+ phy-handle = <&phy1>;
+ };
+ };
+ };
+ };
--
2.50.1On Sun, Aug 24, 2025 at 08:51:09AM +0800, David Yang wrote: > The Motorcomm YT921x series is a family of Ethernet switches with up to > 8 internal GbE PHYs and up to 2 GMACs. > > Signed-off-by: David Yang <mmyangfl@gmail.com> > --- <form letter> This is a friendly reminder during the review process. It looks like you received a tag and forgot to add it. If you do not know the process, here is a short explanation: Please add Acked-by/Reviewed-by/Tested-by tags when posting new versions of patchset, under or above your Signed-off-by tag, unless patch changed significantly (e.g. new properties added to the DT bindings). Tag is "received", when provided in a message replied to you on the mailing list. Tools like b4 can help here. However, there's no need to repost patches *only* to add the tags. The upstream maintainer will do that for tags received on the version they apply. Please read: https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 *If a tag was not added on purpose, please state why* and what changed. </form letter> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On Sun, Aug 24, 2025 at 5:20 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > On Sun, Aug 24, 2025 at 08:51:09AM +0800, David Yang wrote: > > The Motorcomm YT921x series is a family of Ethernet switches with up to > > 8 internal GbE PHYs and up to 2 GMACs. > > > > Signed-off-by: David Yang <mmyangfl@gmail.com> > > --- > > <form letter> > This is a friendly reminder during the review process. > > It looks like you received a tag and forgot to add it. > > If you do not know the process, here is a short explanation: > Please add Acked-by/Reviewed-by/Tested-by tags when posting new > versions of patchset, under or above your Signed-off-by tag, unless > patch changed significantly (e.g. new properties added to the DT > bindings). Tag is "received", when provided in a message replied to you > on the mailing list. Tools like b4 can help here. However, there's no > need to repost patches *only* to add the tags. The upstream maintainer > will do that for tags received on the version they apply. > > Please read: > https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 > > *If a tag was not added on purpose, please state why* and what changed. > </form letter> > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > Best regards, > Krzysztof > Thanks. > - use enum for reg in dt binding I made a change in dt binding. If you are fine with that change, I'll add the tag in the following versions (if any).
On 24/08/2025 11:25, Yangfl wrote: > On Sun, Aug 24, 2025 at 5:20 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: >> >> On Sun, Aug 24, 2025 at 08:51:09AM +0800, David Yang wrote: >>> The Motorcomm YT921x series is a family of Ethernet switches with up to >>> 8 internal GbE PHYs and up to 2 GMACs. >>> >>> Signed-off-by: David Yang <mmyangfl@gmail.com> >>> --- >> >> <form letter> >> This is a friendly reminder during the review process. >> >> It looks like you received a tag and forgot to add it. >> >> If you do not know the process, here is a short explanation: >> Please add Acked-by/Reviewed-by/Tested-by tags when posting new >> versions of patchset, under or above your Signed-off-by tag, unless >> patch changed significantly (e.g. new properties added to the DT >> bindings). Tag is "received", when provided in a message replied to you >> on the mailing list. Tools like b4 can help here. However, there's no >> need to repost patches *only* to add the tags. The upstream maintainer >> will do that for tags received on the version they apply. >> >> Please read: >> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 >> >> *If a tag was not added on purpose, please state why* and what changed. >> </form letter> >> >> >> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> >> Best regards, >> Krzysztof >> > > Thanks. > >> - use enum for reg in dt binding > > I made a change in dt binding. If you are fine with that change, I'll > add the tag in the following versions (if any). Cover letter must state the reason. Best regards, Krzysztof
On Mon, Aug 25, 2025 at 2:09 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > On 24/08/2025 11:25, Yangfl wrote: > > On Sun, Aug 24, 2025 at 5:20 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: > >> > >> On Sun, Aug 24, 2025 at 08:51:09AM +0800, David Yang wrote: > >>> The Motorcomm YT921x series is a family of Ethernet switches with up to > >>> 8 internal GbE PHYs and up to 2 GMACs. > >>> > >>> Signed-off-by: David Yang <mmyangfl@gmail.com> > >>> --- > >> > >> <form letter> > >> This is a friendly reminder during the review process. > >> > >> It looks like you received a tag and forgot to add it. > >> > >> If you do not know the process, here is a short explanation: > >> Please add Acked-by/Reviewed-by/Tested-by tags when posting new > >> versions of patchset, under or above your Signed-off-by tag, unless > >> patch changed significantly (e.g. new properties added to the DT > >> bindings). Tag is "received", when provided in a message replied to you > >> on the mailing list. Tools like b4 can help here. However, there's no > >> need to repost patches *only* to add the tags. The upstream maintainer > >> will do that for tags received on the version they apply. > >> > >> Please read: > >> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 > >> > >> *If a tag was not added on purpose, please state why* and what changed. > >> </form letter> > >> > >> > >> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > >> > >> Best regards, > >> Krzysztof > >> > > > > Thanks. > > > >> - use enum for reg in dt binding > > > > I made a change in dt binding. If you are fine with that change, I'll > > add the tag in the following versions (if any). > > > Cover letter must state the reason. > > > Best regards, > Krzysztof as requested in the previous version https://lore.kernel.org/r/f76df98e-f743-4dc2-9f10-93b97f69addb@lunn.ch
On 25/08/2025 04:39, Yangfl wrote: > On Mon, Aug 25, 2025 at 2:09 AM Krzysztof Kozlowski <krzk@kernel.org> wrote: >> >> On 24/08/2025 11:25, Yangfl wrote: >>> On Sun, Aug 24, 2025 at 5:20 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: >>>> >>>> On Sun, Aug 24, 2025 at 08:51:09AM +0800, David Yang wrote: >>>>> The Motorcomm YT921x series is a family of Ethernet switches with up to >>>>> 8 internal GbE PHYs and up to 2 GMACs. >>>>> >>>>> Signed-off-by: David Yang <mmyangfl@gmail.com> >>>>> --- >>>> >>>> <form letter> >>>> This is a friendly reminder during the review process. >>>> >>>> It looks like you received a tag and forgot to add it. >>>> >>>> If you do not know the process, here is a short explanation: >>>> Please add Acked-by/Reviewed-by/Tested-by tags when posting new >>>> versions of patchset, under or above your Signed-off-by tag, unless >>>> patch changed significantly (e.g. new properties added to the DT >>>> bindings). Tag is "received", when provided in a message replied to you >>>> on the mailing list. Tools like b4 can help here. However, there's no >>>> need to repost patches *only* to add the tags. The upstream maintainer >>>> will do that for tags received on the version they apply. >>>> >>>> Please read: >>>> https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 >>>> >>>> *If a tag was not added on purpose, please state why* and what changed. >>>> </form letter> >>>> >>>> >>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>>> >>>> Best regards, >>>> Krzysztof >>>> >>> >>> Thanks. >>> >>>> - use enum for reg in dt binding >>> >>> I made a change in dt binding. If you are fine with that change, I'll >>> add the tag in the following versions (if any). >> >> >> Cover letter must state the reason. >> >> >> Best regards, >> Krzysztof > > as requested in the previous version > > https://lore.kernel.org/r/f76df98e-f743-4dc2-9f10-93b97f69addb@lunn.ch Hm? Where is there a statement that you remove review because of this and that? Best regards, Krzysztof
© 2016 - 2025 Red Hat, Inc.