[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20250910022230.GA3646514-robh@kernel.org>
Date: Tue, 9 Sep 2025 21:22:30 -0500
From: Rob Herring <robh@...nel.org>
To: Chen-Yu Tsai <wens@...nel.org>
Cc: Andrew Lunn <andrew+netdev@...n.ch>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>, Chen-Yu Tsai <wens@...e.org>,
Jernej Skrabec <jernej@...nel.org>,
Samuel Holland <samuel@...lland.org>, netdev@...r.kernel.org,
devicetree@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-sunxi@...ts.linux.dev, linux-kernel@...r.kernel.org,
Andre Przywara <andre.przywara@....com>
Subject: Re: [PATCH net-next v4 01/10] dt-bindings: net: sun8i-emac: Add A523
GMAC200 compatible
On Tue, Sep 09, 2025 at 02:10:50AM +0800, Chen-Yu Tsai wrote:
> From: Chen-Yu Tsai <wens@...e.org>
>
> The Allwinner A523 SoC family has a second Ethernet controller, called
> the GMAC200 in the BSP and T527 datasheet, and referred to as GMAC1 for
> numbering. This controller, according to BSP sources, is fully
> compatible with a slightly newer version of the Synopsys DWMAC core.
> The glue layer around the controller is the same as found around older
> DWMAC cores on Allwinner SoCs. The only slight difference is that since
> this is the second controller on the SoC, the register for the clock
> delay controls is at a different offset. Last, the integration includes
> a dedicated clock gate for the memory bus and the whole thing is put in
> a separately controllable power domain.
>
> Add a compatible string entry for it, and work in the requirements for
> a second clock and a power domain.
>
> Signed-off-by: Chen-Yu Tsai <wens@...e.org>
> ---
> Changes since v2:
> - Added "select" to avoid matching against all dwmac entries
> Changes since v1:
> - Switch to generic (tx|rx)-internal-delay-ps properties
> ---
> .../net/allwinner,sun8i-a83t-emac.yaml | 96 ++++++++++++++++++-
> 1 file changed, 94 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml b/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml
> index 2ac709a4c472..9d205c5d93ca 100644
> --- a/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml
> +++ b/Documentation/devicetree/bindings/net/allwinner,sun8i-a83t-emac.yaml
> @@ -10,6 +10,21 @@ maintainers:
> - Chen-Yu Tsai <wens@...e.org>
> - Maxime Ripard <mripard@...nel.org>
>
> +# We need a select here so we don't match all nodes with 'snps,dwmac'
> +select:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - allwinner,sun8i-a83t-emac
> + - allwinner,sun8i-h3-emac
> + - allwinner,sun8i-r40-gmac
> + - allwinner,sun8i-v3s-emac
> + - allwinner,sun50i-a64-emac
> + - allwinner,sun55i-a523-gmac200
> + required:
> + - compatible
> +
> properties:
> compatible:
> oneOf:
> @@ -26,6 +41,9 @@ properties:
> - allwinner,sun50i-h616-emac0
> - allwinner,sun55i-a523-gmac0
> - const: allwinner,sun50i-a64-emac
> + - items:
> + - const: allwinner,sun55i-a523-gmac200
> + - const: snps,dwmac-4.20a
>
> reg:
> maxItems: 1
> @@ -37,14 +55,19 @@ properties:
> const: macirq
>
> clocks:
> - maxItems: 1
> + minItems: 1
> + maxItems: 2
>
> clock-names:
> - const: stmmaceth
> + minItems: 1
> + maxItems: 2
minItems: 1
items:
- const: stmmaceth
- const: mbus
>
> phy-supply:
> description: PHY regulator
>
> + power-domains:
> + maxItems: 1
> +
> syscon:
> $ref: /schemas/types.yaml#/definitions/phandle
> description:
> @@ -191,6 +214,45 @@ allOf:
> - mdio-parent-bus
> - mdio@1
>
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: allwinner,sun55i-a523-gmac200
> + then:
> + properties:
> + clocks:
> + minItems: 2
> + clock-names:
> + items:
> + - const: stmmaceth
> + - const: mbus
Just 'minItems: 2' here.
> + tx-internal-delay-ps:
> + default: 0
> + minimum: 0
> + maximum: 700
> + multipleOf: 100
> + description:
> + External RGMII PHY TX clock delay chain value in ps.
> + rx-internal-delay-ps:
> + default: 0
> + minimum: 0
> + maximum: 3100
> + multipleOf: 100
> + description:
> + External RGMII PHY TX clock delay chain value in ps.
> + required:
> + - power-domains
> + else:
> + properties:
> + clocks:
> + maxItems: 1
> + clock-names:
> + items:
> + - const: stmmaceth
maxItems: 1
> + power-domains: false
> +
> +
> unevaluatedProperties: false
>
> examples:
> @@ -323,4 +385,34 @@ examples:
> };
> };
>
> + - |
> + ethernet@...0000 {
> + compatible = "allwinner,sun55i-a523-gmac200",
> + "snps,dwmac-4.20a";
> + reg = <0x04510000 0x10000>;
> + clocks = <&ccu 117>, <&ccu 79>;
> + clock-names = "stmmaceth", "mbus";
> + resets = <&ccu 43>;
> + reset-names = "stmmaceth";
> + interrupts = <0 47 4>;
> + interrupt-names = "macirq";
> + pinctrl-names = "default";
> + pinctrl-0 = <&rgmii1_pins>;
> + power-domains = <&pck600 4>;
> + syscon = <&syscon>;
> + phy-handle = <&ext_rgmii_phy_1>;
> + phy-mode = "rgmii-id";
> + snps,fixed-burst;
> + snps,axi-config = <&gmac1_stmmac_axi_setup>;
> +
> + mdio {
> + compatible = "snps,dwmac-mdio";
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + ext_rgmii_phy_1: ethernet-phy@1 {
> + reg = <1>;
> + };
> + };
> + };
> ...
> --
> 2.39.5
>
Powered by blists - more mailing lists