From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 60092C2BD09 for ; Mon, 1 Jul 2024 18:22:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=8Rn3pm5yzJs4fAYUbn+sHYN/XyRiJ+ii7LrKBUYo4FU=; b=GWwWN/pxpliAcQbuVFeyBRMgXy gJz/KDjokfnZ9HY0FH//X8ETwDXKjoOuelkJauznYeldqmrWqHoaGkiK6PXHg3VVhmbykgkBsZb+q vdGtj/FRJcQQwfEIiHzrMYKXpz1md9wRaYgIQ3DwYVW7FCEXz164OhyFGDsjSy6/EZlgwY/UCdqoR PbWtDimQbB91cxpolDn7JRBeoYwB2qtxzttgxZvHWX9BVpQQI2wMxEs/VnmnkHyuyK5U5EGhkpZHz tucnjRhex+7fpjvizZFRTv1b5LfFybrq9YRR8wnWnI8ttgxf3wyxj7zhZXbwCU+21Bl5nsvzdv/hA Bok7acEg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOLfA-00000004Qtw-2CTC; Mon, 01 Jul 2024 18:22:04 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOLez-00000004Qrh-2z92 for linux-arm-kernel@lists.infradead.org; Mon, 01 Jul 2024 18:21:55 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id EA92A6173D; Mon, 1 Jul 2024 18:21:52 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 75DD1C116B1; Mon, 1 Jul 2024 18:21:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1719858112; bh=ZS1M09pR2e89i2/x0GjF/RUkGIJBxm9gVRdcxXpYdMY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=X01SANG/MsX/TCJKLM73Ob4M+abTasVmecgoFTT0yOI9RGboFcJsf7Ak3InrgXlyk PLHYxcs8mBODuBG1+hIVBIa7QIW8v7v9KLD0omu48Pb13tk0uyKf2QraE6tdPiwmsl n/xr4TlzW2TTgzP5T74oa3Q+2I4hOSy/zJn7hhpvhuG4+Zxo9C3nMD6cA9e26fysxt Qi4MXITPKP+aZ4bcImNxXg9dO2fD1KqCvPTf/GhRJU0FQjAhOONHRBVy+wmcCE+Rw2 AIvBsgOiB6V9n+EFnVf/iPUI1eBe/WJxu68ey9tzeyQJ3tLZ+qg+smyBYlPFIiLReN AwtEE4OQfvAEA== Date: Mon, 1 Jul 2024 12:21:51 -0600 From: Rob Herring To: Lorenzo Bianconi Cc: netdev@vger.kernel.org, nbd@nbd.name, lorenzo.bianconi83@gmail.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, conor@kernel.org, linux-arm-kernel@lists.infradead.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, devicetree@vger.kernel.org, catalin.marinas@arm.com, will@kernel.org, upstream@airoha.com, angelogioacchino.delregno@collabora.com, benjamin.larsson@genexis.eu, rkannoth@marvell.com, sgoutham@marvell.com, andrew@lunn.ch, arnd@arndb.de, horms@kernel.org Subject: Re: [PATCH v4 1/2] dt-bindings: net: airoha: Add EN7581 ethernet controller Message-ID: <20240701182151.GA320555-robh@kernel.org> References: <20d103799a20d9d61a1c378eb27e61748859e978.1719672695.git.lorenzo@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20d103799a20d9d61a1c378eb27e61748859e978.1719672695.git.lorenzo@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240701_112153_908372_C61AC7B5 X-CRM114-Status: GOOD ( 19.31 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Sat, Jun 29, 2024 at 05:01:37PM +0200, Lorenzo Bianconi wrote: > Introduce device-tree binding documentation for Airoha EN7581 ethernet > mac controller. > > Signed-off-by: Lorenzo Bianconi > --- > .../bindings/net/airoha,en7581-eth.yaml | 171 ++++++++++++++++++ > 1 file changed, 171 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml > > diff --git a/Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml b/Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml > new file mode 100644 > index 000000000000..e2c0da02ccf2 > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/airoha,en7581-eth.yaml > @@ -0,0 +1,171 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/net/airoha,en7581-eth.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Airoha EN7581 Frame Engine Ethernet controller > + > +allOf: > + - $ref: ethernet-controller.yaml# > + > +maintainers: > + - Lorenzo Bianconi > + > +description: > + The frame engine ethernet controller can be found on Airoha SoCs. > + These SoCs have dual GMAC ports. > + > +properties: > + compatible: > + enum: > + - airoha,en7581-eth > + > + reg: > + items: > + - description: Frame engine base address > + - description: QDMA0 base address > + - description: QDMA1 base address > + > + reg-names: > + items: > + - const: fe > + - const: qdma0 > + - const: qdma1 > + > + interrupts: > + items: > + - description: QDMA lan irq0 > + - description: QDMA lan irq1 > + - description: QDMA lan irq2 > + - description: QDMA lan irq3 > + - description: QDMA wan irq0 > + - description: QDMA wan irq1 > + - description: QDMA wan irq2 > + - description: QDMA wan irq3 > + - description: FE error irq > + - description: PDMA irq > + > + resets: > + maxItems: 8 > + > + reset-names: > + items: > + - const: fe > + - const: pdma > + - const: qdma > + - const: xsi-mac > + - const: hsi0-mac > + - const: hsi1-mac > + - const: hsi-mac > + - const: xfp-mac > + > + "#address-cells": > + const: 1 > + > + "#size-cells": > + const: 0 > + > +patternProperties: > + "^mac@[1-4]$": ethernet@ > + type: object > + unevaluatedProperties: false > + allOf: Can drop 'allOf' > + - $ref: ethernet-controller.yaml# Which node represents an ethernet controller? This one or the parent? Most likely it is not both. > + description: > + Ethernet MAC node > + properties: > + compatible: > + const: airoha,eth-mac > + > + reg: > + maxItems: 1 Based on the unit-address, you need instead: minimum: 1 maximum: 4 But what does 1-4 represent? There are no MMIO registers associated with the MACs? Please describe. > + > + required: > + - reg > + - compatible > + > +required: > + - compatible > + - reg > + - interrupts > + - resets > + - reset-names > + > +unevaluatedProperties: false > + > +examples: > + - | > + #include > + #include > + #include > + > + soc { > + #address-cells = <2>; > + #size-cells = <2>; > + > + eth0: ethernet@1fb50000 { > + compatible = "airoha,en7581-eth"; > + reg = <0 0x1fb50000 0 0x2600>, > + <0 0x1fb54000 0 0x2000>, > + <0 0x1fb56000 0 0x2000>; > + reg-names = "fe", "qdma0", "qdma1"; > + > + resets = <&scuclk 44>, > + <&scuclk 30>, > + <&scuclk 31>, > + <&scuclk 6>, > + <&scuclk 15>, > + <&scuclk 16>, > + <&scuclk 17>, > + <&scuclk 26>; > + reset-names = "fe", "pdma", "qdma", "xsi-mac", > + "hsi0-mac", "hsi1-mac", "hsi-mac", > + "xfp-mac"; > + > + interrupts = , > + , > + , > + , > + , > + , > + , > + , > + , > + ; > + > + #address-cells = <1>; > + #size-cells = <0>; > + > + mac1: mac@1 { > + compatible = "airoha,eth-mac"; > + reg = <1>; > + phy-mode = "2500base-x"; > + phy-handle = <&phy0>; > + }; > + > + mac2: mac@2 { > + compatible = "airoha,eth-mac"; > + reg = <2>; > + phy-mode = "2500base-x"; > + phy-handle = <&phy1>; > + }; > + }; > + > + mdio: mdio-bus { mdio { But really, drop this if it is not part of this device (binding). What is the control interface for this MDIO bus? If it is part of this device, then the mdio node needs to be within the device's node. > + #address-cells = <1>; > + #size-cells = <0>; > + > + phy0: ethernet-phy@0 { > + compatible = "ethernet-phy-id67c9.de0a"; > + reg = <0>; > + phy-mode = "2500base-x"; > + }; > + > + phy1: ethernet-phy@1 { > + compatible = "ethernet-phy-id67c9.de0a"; > + reg = <1>; > + phy-mode = "2500base-x"; > + }; > + }; > + }; > -- > 2.45.2 >