* Re: [PATCH net-next 10/13] dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx switches
@ 2025-10-25 7:14 kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2025-10-25 7:14 UTC (permalink / raw)
To: oe-kbuild; +Cc: lkp
::::::
:::::: Manual check reason: "dtcheck: binding changes may go via different trees"
::::::
BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <c8580eb023b99447e33d5b95624440c0af602f89.1761324950.git.daniel@makrotopia.org>
References: <c8580eb023b99447e33d5b95624440c0af602f89.1761324950.git.daniel@makrotopia.org>
TO: Daniel Golle <daniel@makrotopia.org>
TO: Hauke Mehrtens <hauke@hauke-m.de>
TO: Andrew Lunn <andrew@lunn.ch>
TO: Vladimir Oltean <olteanv@gmail.com>
TO: "David S. Miller" <davem@davemloft.net>
CC: netdev@vger.kernel.org
TO: Eric Dumazet <edumazet@google.com>
TO: Jakub Kicinski <kuba@kernel.org>
TO: Paolo Abeni <pabeni@redhat.com>
TO: Rob Herring <robh@kernel.org>
TO: Krzysztof Kozlowski <krzk@kernel.org>
TO: Conor Dooley <conor+dt@kernel.org>
TO: Simon Horman <horms@kernel.org>
TO: Russell King <linux@armlinux.org.uk>
TO: devicetree@vger.kernel.org
TO: linux-kernel@vger.kernel.org
CC: Andreas Schirm <andreas.schirm@siemens.com>
CC: Lukas Stockmann <lukas.stockmann@siemens.com>
CC: Alexander Sverdlin <alexander.sverdlin@siemens.com>
CC: Peter Christen <peter.christen@siemens.com>
CC: Avinash Jayaraman <ajayaraman@maxlinear.com>
CC: Bing tao Xu <bxu@maxlinear.com>
CC: Liang Xu <lxu@maxlinear.com>
CC: Juraj Povazanec <jpovazanec@maxlinear.com>
CC: "Fanni (Fang-Yi) Chan" <fchan@maxlinear.com>
CC: "Benny (Ying-Tsan) Weng" <yweng@maxlinear.com>
CC: "Livia M. Rosu" <lrosu@maxlinear.com>
CC: John Crispin <john@phrozen.org>
Hi Daniel,
kernel test robot noticed the following build warnings:
[auto build test WARNING on net-next/main]
url: https://github.com/intel-lab-lkp/linux/commits/Daniel-Golle/net-dsa-lantiq_gswip-split-into-common-and-MMIO-parts/20251025-010937
base: net-next/main
patch link: https://lore.kernel.org/r/c8580eb023b99447e33d5b95624440c0af602f89.1761324950.git.daniel%40makrotopia.org
patch subject: [PATCH net-next 10/13] dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx switches
:::::: branch date: 14 hours ago
:::::: commit date: 14 hours ago
config: csky-randconfig-051-20251025 (https://download.01.org/0day-ci/archive/20251025/202510251557.FS2kslve-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 15.1.0
dtschema version: 2025.9.dev7+g3b6fdac89
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20251025/202510251557.FS2kslve-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/r/202510251557.FS2kslve-lkp@intel.com/
dtcheck warnings: (new ones prefixed by >>)
>> Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml: ignoring, error parsing file
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread* [PATCH net-next 00/13] net: dsa: lantiq_gswip: Add support for MaxLinear GSW1xx switch family
@ 2025-10-24 17:01 Daniel Golle
2025-10-24 17:04 ` [PATCH net-next 10/13] dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx switches Daniel Golle
0 siblings, 1 reply; 2+ messages in thread
From: Daniel Golle @ 2025-10-24 17:01 UTC (permalink / raw)
To: Hauke Mehrtens, Andrew Lunn, Vladimir Oltean, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Simon Horman, Russell King,
netdev, devicetree, linux-kernel
Cc: Andreas Schirm, Lukas Stockmann, Alexander Sverdlin,
Peter Christen, Avinash Jayaraman, Bing tao Xu, Liang Xu,
Juraj Povazanec, Fanni (Fang-Yi) Chan, Benny (Ying-Tsan) Weng,
Livia M. Rosu, John Crispin
This patch series extends the existing lantiq_gswip DSA driver to support
the MaxLinear GSW1xx family of dedicated Ethernet switch ICs. These switches
are based on the same IP as the Lantiq/Intel GSWIP found in VR9 and xRX
MIPS router SoCs, but are connected via MDIO instead of memory-mapped I/O.
The series includes several improvements and refactoring to prepare for the
new hardware support.
The GSW1xx family includes several variants:
- GSW120: 4 ports, 2 PHYs, RGMII & SGMII/2500Base-X
- GSW125: 4 ports, 2 PHYs, RGMII & SGMII/2500Base-X, industrial temperature
- GSW140: 6 ports, 4 PHYs, RGMII & SGMII/2500Base-X
- GSW141: 6 ports, 4 PHYs, RGMII & SGMII
- GSW145: 6 ports, 4 PHYs, RGMII & SGMII/2500Base-X, industrial temperature
Key features implemented:
- MDIO-based register access using regmap
- Support for SGMII/1000Base-X/2500Base-X SerDes interfaces
- Configurable MII delays via device tree properties
- Energy Efficient Ethernet (EEE) support
- enabling/disabling learning
Daniel Golle (13):
net: dsa: lantiq_gswip: split into common and MMIO parts
net: dsa: lantiq_gswip: support enable/disable learning
net: dsa: lantiq_gswip: support Energy Efficient Ethernet
net: dsa: lantiq_gswip: set link parameters also for CPU port
net: dsa: lantiq_gswip: define and use
GSWIP_TABLE_MAC_BRIDGE_VAL1_VALID
dt-bindings: net: dsa: lantiq,gswip: add support for MII delay
properties
net: dsa: lantiq_gswip: allow adjusting MII delays
dt-bindings: net: dsa: lantiq,gswip: add MaxLinear RMII refclk output
property
net: dsa: lantiq_gswip: add vendor property to setup MII refclk output
dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx
switches
net: dsa: add tagging driver for MaxLinear GSW1xx switch family
net: dsa: lantiq_gswip: add registers specific for MaxLinear GSW1xx
net: dsa: add driver for MaxLinear GSW1xx switch family
.../bindings/net/dsa/lantiq,gswip.yaml | 290 ++-
MAINTAINERS | 3 +-
drivers/net/dsa/lantiq/Kconfig | 18 +-
drivers/net/dsa/lantiq/Makefile | 2 +
drivers/net/dsa/lantiq/lantiq_gswip.c | 1617 +--------------
drivers/net/dsa/lantiq/lantiq_gswip.h | 128 ++
drivers/net/dsa/lantiq/lantiq_gswip_common.c | 1747 +++++++++++++++++
drivers/net/dsa/lantiq/mxl-gsw1xx.c | 685 +++++++
drivers/net/dsa/lantiq/mxl-gsw1xx_pce.h | 154 ++
include/net/dsa.h | 2 +
net/dsa/Kconfig | 8 +
net/dsa/Makefile | 1 +
net/dsa/tag_mxl-gsw1xx.c | 138 ++
13 files changed, 3127 insertions(+), 1666 deletions(-)
create mode 100644 drivers/net/dsa/lantiq/lantiq_gswip_common.c
create mode 100644 drivers/net/dsa/lantiq/mxl-gsw1xx.c
create mode 100644 drivers/net/dsa/lantiq/mxl-gsw1xx_pce.h
create mode 100644 net/dsa/tag_mxl-gsw1xx.c
--
2.51.0
^ permalink raw reply [flat|nested] 2+ messages in thread* [PATCH net-next 10/13] dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx switches 2025-10-24 17:01 [PATCH net-next 00/13] net: dsa: lantiq_gswip: Add support for MaxLinear GSW1xx switch family Daniel Golle @ 2025-10-24 17:04 ` Daniel Golle 0 siblings, 0 replies; 2+ messages in thread From: Daniel Golle @ 2025-10-24 17:04 UTC (permalink / raw) To: Hauke Mehrtens, Andrew Lunn, Vladimir Oltean, David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni, Rob Herring, Krzysztof Kozlowski, Conor Dooley, Simon Horman, Russell King, netdev, devicetree, linux-kernel Cc: Andreas Schirm, Lukas Stockmann, Alexander Sverdlin, Peter Christen, Avinash Jayaraman, Bing tao Xu, Liang Xu, Juraj Povazanec, Fanni (Fang-Yi) Chan, Benny (Ying-Tsan) Weng, Livia M. Rosu, John Crispin Extend the Lantiq GSWIP device tree binding to also cover MaxLinear GSW1xx switches which are based on the same hardware IP but connected via MDIO instead of being memory-mapped. Add compatible strings for MaxLinear GSW120, GSW125, GSW140, GSW141, and GSW145 switches and adjust the schema to handle the different connection methods with conditional properties. Add MaxLinear GSW125 example showing MDIO-connected configuration. Signed-off-by: Daniel Golle <daniel@makrotopia.org> --- .../bindings/net/dsa/lantiq,gswip.yaml | 263 +++++++++++++----- 1 file changed, 190 insertions(+), 73 deletions(-) diff --git a/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml b/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml index 48641c27da10..eb6a4c30a8be 100644 --- a/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml +++ b/Documentation/devicetree/bindings/net/dsa/lantiq,gswip.yaml @@ -4,7 +4,12 @@ $id: http://devicetree.org/schemas/net/dsa/lantiq,gswip.yaml# $schema: http://devicetree.org/meta-schemas/core.yaml# -title: Lantiq GSWIP Ethernet switches +title: Lantiq GSWIP and MaxLinear GSW1xx Ethernet switches + +description: + Lantiq GSWIP and MaxLinear GSW1xx switches share the same hardware IP. + Lantiq switches are embedded in SoCs and accessed via memory-mapped I/O, + while MaxLinear switches are standalone ICs connected via MDIO. allOf: - $ref: dsa.yaml#/$defs/ethernet-ports @@ -33,6 +38,98 @@ allOf: description: Configure the RMII reference clock to be a clock output rather than an input. Only applicable for RMII mode. + - if: + properties: + compatible: + contains: + enum: + - lantiq,xrx200-gswip + - lantiq,xrx300-gswip + - lantiq,xrx330-gswip + then: + properties: + reg: + minItems: 3 + maxItems: 3 + description: Memory-mapped register regions (switch, mdio, mii) + reg-names: + items: + - const: switch + - const: mdio + - const: mii + mdio: + $ref: /schemas/net/mdio.yaml# + unevaluatedProperties: false + + properties: + compatible: + const: lantiq,xrx200-mdio + + required: + - compatible + gphy-fw: + type: object + properties: + '#address-cells': + const: 1 + + '#size-cells': + const: 0 + + compatible: + items: + - enum: + - lantiq,xrx200-gphy-fw + - lantiq,xrx300-gphy-fw + - lantiq,xrx330-gphy-fw + - const: lantiq,gphy-fw + + lantiq,rcu: + $ref: /schemas/types.yaml#/definitions/phandle + description: phandle to the RCU syscon + + patternProperties: + "^gphy@[0-9a-f]{1,2}$": + type: object + + additionalProperties: false + + properties: + reg: + minimum: 0 + maximum: 255 + description: + Offset of the GPHY firmware register in the RCU register range + + resets: + items: + - description: GPHY reset line + + reset-names: + items: + - const: gphy + + required: + - reg + + required: + - compatible + - lantiq,rcu + + additionalProperties: false + required: + - reg-names + else: + properties: + reg: + maxItems: 1 + description: MDIO bus address + reg-names: false + gphy-fw: false + mdio: + $ref: /schemas/net/mdio.yaml# + unevaluatedProperties: false +>>>>>>> f34c3d144cf3 (dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx switches) maintainers: - Hauke Mehrtens <hauke@hauke-m.de> @@ -43,78 +140,11 @@ properties: - lantiq,xrx200-gswip - lantiq,xrx300-gswip - lantiq,xrx330-gswip - - reg: - minItems: 3 - maxItems: 3 - - reg-names: - items: - - const: switch - - const: mdio - - const: mii - - mdio: - $ref: /schemas/net/mdio.yaml# - unevaluatedProperties: false - - properties: - compatible: - const: lantiq,xrx200-mdio - - required: - - compatible - - gphy-fw: - type: object - properties: - '#address-cells': - const: 1 - - '#size-cells': - const: 0 - - compatible: - items: - - enum: - - lantiq,xrx200-gphy-fw - - lantiq,xrx300-gphy-fw - - lantiq,xrx330-gphy-fw - - const: lantiq,gphy-fw - - lantiq,rcu: - $ref: /schemas/types.yaml#/definitions/phandle - description: phandle to the RCU syscon - - patternProperties: - "^gphy@[0-9a-f]{1,2}$": - type: object - - additionalProperties: false - - properties: - reg: - minimum: 0 - maximum: 255 - description: - Offset of the GPHY firmware register in the RCU register range - - resets: - items: - - description: GPHY reset line - - reset-names: - items: - - const: gphy - - required: - - reg - - required: - - compatible - - lantiq,rcu - - additionalProperties: false + - maxlinear,gsw120 + - maxlinear,gsw125 + - maxlinear,gsw140 + - maxlinear,gsw141 + - maxlinear,gsw145 required: - compatible @@ -129,6 +159,7 @@ examples: reg = <0xe108000 0x3100>, /* switch */ <0xe10b100 0xd8>, /* mdio */ <0xe10b1d8 0x130>; /* mii */ + reg-names = "switch", "mdio", "mii"; dsa,member = <0 0>; ports { @@ -227,3 +258,89 @@ examples: }; }; }; + + - | + #include <dt-bindings/leds/common.h> + + mdio { + #address-cells = <1>; + #size-cells = <0>; + + switch@1f { + compatible = "maxlinear,gsw125"; + reg = <0x1f>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + label = "lan0"; + phy-handle = <&switchphy0>; + phy-mode = "internal"; + }; + + port@1 { + reg = <1>; + label = "lan1"; + phy-handle = <&switchphy1>; + phy-mode = "internal"; + }; + + port@4 { + reg = <4>; + label = "wan"; + phy-mode = "sgmii"; + }; + + port@5 { + reg = <5>; + phy-mode = "rgmii-id"; + tx-internal-delay-ps = <2000>; + rx-internal-delay-ps = <2000>; + ethernet = <ð0>; + + fixed-link { + speed = <1000>; + full-duplex; + }; + }; + }; + + mdio { + #address-cells = <1>; + #size-cells = <0>; + + switchphy0: switchphy@0 { + reg = <0>; + + leds { + #address-cells = <1>; + #size-cells = <0>; + + led@0 { + reg = <0>; + color = <LED_COLOR_ID_GREEN>; + function = LED_FUNCTION_LAN; + }; + }; + }; + + switchphy1: switchphy@1 { + reg = <1>; + + leds { + #address-cells = <1>; + #size-cells = <0>; + + led@0 { + reg = <0>; + color = <LED_COLOR_ID_GREEN>; + function = LED_FUNCTION_LAN; + }; + }; + }; + }; + }; + }; -- 2.51.0 ^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-10-25 7:14 UTC | newest] Thread overview: 2+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-10-25 7:14 [PATCH net-next 10/13] dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx switches kernel test robot -- strict thread matches above, loose matches on Subject: below -- 2025-10-24 17:01 [PATCH net-next 00/13] net: dsa: lantiq_gswip: Add support for MaxLinear GSW1xx switch family Daniel Golle 2025-10-24 17:04 ` [PATCH net-next 10/13] dt-bindings: net: dsa: lantiq,gswip: add support for MaxLinear GSW1xx switches Daniel Golle
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.