Motorcomm YT921x is a series of ethernet switches developed by Shanghai
Motorcomm Electronic Technology, including:

  - YT9215S / YT9215RB / YT9215SC: 5 GbE phys
  - YT9213NB / YT9214NB: 2 GbE phys
  - YT9218N / YT9218MB: 8 GbE phys

and up to 2 serdes interfaces.

This patch adds basic support for a working DSA switch.

v2: https://lore.kernel.org/r/20250814065032.3766988-1-mmyangfl@gmail.com
  - fix words in dt binding
  - add support for lag and mst
v1: https://lore.kernel.org/r/20250808173808.273774-1-mmyangfl@gmail.com
  - fix coding style
  - add dt binding
  - add support for fdb, vlan and bridge

David Yang (3):
  dt-bindings: net: dsa: yt921x: Add Motorcomm YT921x switch support
  net: dsa: tag_yt921x: add support for Motorcomm YT921x tags
  net: dsa: yt921x: Add support for Motorcomm YT921x

 .../bindings/net/dsa/motorcomm,yt921x.yaml    |  166 +
 drivers/net/dsa/Kconfig                       |    7 +
 drivers/net/dsa/Makefile                      |    1 +
 drivers/net/dsa/yt921x.c                      | 3960 +++++++++++++++++
 include/net/dsa.h                             |    2 +
 net/dsa/Kconfig                               |    6 +
 net/dsa/Makefile                              |    1 +
 net/dsa/tag_yt921x.c                          |  126 +
 8 files changed, 4269 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/net/dsa/motorcomm,yt921x.yaml
 create mode 100644 drivers/net/dsa/yt921x.c
 create mode 100644 net/dsa/tag_yt921x.c

-- 
2.47.2

On Sat, Aug 16, 2025 at 01:23:18PM +0800, David Yang wrote:
> Motorcomm YT921x is a series of ethernet switches developed by Shanghai
> Motorcomm Electronic Technology, including:
> 
>   - YT9215S / YT9215RB / YT9215SC: 5 GbE phys
>   - YT9213NB / YT9214NB: 2 GbE phys
>   - YT9218N / YT9218MB: 8 GbE phys
> 
> and up to 2 serdes interfaces.
> 
> This patch adds basic support for a working DSA switch.
> 
> v2: https://lore.kernel.org/r/20250814065032.3766988-1-mmyangfl@gmail.com
>   - fix words in dt binding
>   - add support for lag and mst

Please don't add new features between revisions. Reviewers spend time
reviewing the code. They assume just the issues raised will be
address, and the rest of the code remains unchanged. It then means
they just need to check the issues raised have been addressed. By
adding new features, they back to the beginning, having to review all
the code again, because you potentially added new issues.

LAG and MST should of been implemented as patches on top of the basic
driver. They can then be reviewed as small increments.

Please put yourself in our position. How would you review this code?
That would make it easy for you to review it?

	Andrew