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 A16B6C3271E for ; Mon, 8 Jul 2024 21:16:56 +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=Bjix730OORicI09nRR7PRWAIljLkQK44UYc71ax5gUc=; b=b+tjcDWsQ7VQFRMopyjSHf3P/6 Ml2nShDD43EARge8PIH/kv3wJUrFV6rd/Q2ubqFS+r4oqHZ+oVrMRn9s/BzWkIRf+WZo8REsvircs 2dxDpxAQNG3CBL8PO5sTD12fDLc0xB2VLF6ma7z1s/t0mBVB3avdg9ZPjNWq2gMlizm8J8pKp7WSH 4VbaSfG+rADoXixk2iVe/UQ8DfnvVNBLwflZ4JTr+ly5I1PdDuBKVDb0hV9Glc1WE0FpAooXfrutU jfQ6os0+Ne1hEZLh+6Bi6e6m6AJgI3RfbfMIgnQbJixvxGZSOuOL3PiyrXFovVxdXAr0EWTlNp+/9 df0esCDg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQvj5-000000053OZ-14fB; Mon, 08 Jul 2024 21:16:47 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sQvip-000000053J6-3Bxi for linux-arm-kernel@lists.infradead.org; Mon, 08 Jul 2024 21:16:33 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 1370460BBB; Mon, 8 Jul 2024 21:16:31 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 3D5C2C116B1; Mon, 8 Jul 2024 21:16:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1720473390; bh=MiCSvIEK0l/TjhKUxrz4JOVrP2CSGLGatDFwJW1Tdc4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Lsw4pn/mmsF/guRcIjwA+5dZ0MJNt/aIikyJDaj99W6AZaGXGsFaCZ/z4+Vqx5pFO kbf0N0hKKfz6s7//7iIDFzHfOpyaNS3e83HJ0fe8ufpKW4hjgrcRxDLTobrWfDsmi2 b0GUHYhBNKMUzU1MmTUM6MLYF8r3G6LRqhRUZKddj9wd/h4F0HdVq0/IDIIWf+zQ5R SZ9FZ+GcWyEeb9LTZu1ZZvyP+vwD0oZuzrP4RpGHDSIQwnCx2K26MFQC+ViaUZACv1 vE13oqY+8lVEUT3Ldwe0yuBIAqtp5siQ+90gvFArXbdGH6/w+zdJHih9HAieeNLcmD z2qtwg/qNjf6A== Date: Mon, 8 Jul 2024 23:16:27 +0200 From: Lorenzo Bianconi To: Andrew Lunn Cc: Rob Herring , 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, arnd@arndb.de, horms@kernel.org Subject: Re: [PATCH v5 net-next 1/2] dt-bindings: net: airoha: Add EN7581 ethernet controller Message-ID: References: <48dde2595c6ff497a846183b117ac9704537b78c.1720079772.git.lorenzo@kernel.org> <20240708163708.GA3371750-robh@kernel.org> <8f8a1ee1-8c6c-48d6-a794-286464c38712@lunn.ch> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="Cu2L8LuMZ/p8V+GH" Content-Disposition: inline In-Reply-To: <8f8a1ee1-8c6c-48d6-a794-286464c38712@lunn.ch> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240708_141631_903963_F9264EB4 X-CRM114-Status: GOOD ( 13.75 ) 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 --Cu2L8LuMZ/p8V+GH Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable > > > eth0: ethernet@1fb50000 { > > > compatible =3D "airoha,en7581-eth"; > > > reg =3D <0 0x1fb50000 0 0x2600>, > > > <0 0x1fb54000 0 0x2000>, > > > <0 0x1fb56000 0 0x2000>; > > > reg-names =3D "fe", "qdma0", "qdma1"; > > > > > > resets =3D <&scuclk EN7581_FE_RST>, > > > <&scuclk EN7581_FE_PDMA_RST>, > > > <&scuclk EN7581_FE_QDMA_RST>, > > > <&scuclk EN7581_XSI_MAC_RST>, > > > <&scuclk EN7581_DUAL_HSI0_MAC_RST>, > > > <&scuclk EN7581_DUAL_HSI1_MAC_RST>, > > > <&scuclk EN7581_HSI_MAC_RST>, > > > <&scuclk EN7581_XFP_MAC_RST>; > > > reset-names =3D "fe", "pdma", "qdma", "xsi-mac", > > > "hsi0-mac", "hsi1-mac", "hsi-mac", > > > "xfp-mac"; > > > > > > interrupts =3D , > > > , > > > , > > > , > > > , > > > , > > > , > > > , > > > , > > > ; > > > > > > status =3D "disabled"; > > > > > > #address-cells =3D <1>; > > > #size-cells =3D <0>; > > > > > > gdm1: mac@1 { > > > compatible =3D "airoha,eth-mac"; > > > reg =3D <1>; > > > phy-mode =3D "internal"; > > > status =3D "disabled"; > > > > > > fixed-link { > > > speed =3D <1000>; > > > full-duplex; > > > pause; > > > }; > > > }; > > > }; > > > > > > I am using phy related binding for gdm1:mac@1 node. >=20 > Hi Lorenzo Hi Andrew, >=20 > phy-mode is a MAC property, not a PHY property. Same for > fixed-link. These are in ethernet-controller.yaml. ack >=20 > You sometimes have an network controller IP which has multiple MACs > and some shared infrastructure. You would typically describe the > shared infrastructure at the top level. The MACs are then listed as > children, and they make use of ethernet-controller.yaml, and that is > where all the network specific properties are placed. Is that what you > are trying to do here? yep, exactly. Here we have multiple mac nodes that represent the real ethernet controllers (e.g. used as DSA cpu port). I will use the ethernet-controller.yaml just for the mac childs. Regards, Lorenzo >=20 > Andrew --Cu2L8LuMZ/p8V+GH Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQTquNwa3Txd3rGGn7Y6cBh0uS2trAUCZoxXKgAKCRA6cBh0uS2t rDwOAQCrVya7sfgRJqZGKdFPLB748PsT7YO1VxAuzpHbXbbveQD/Stiv17jgUWbv jlIJtfQCPrAPQJ5j+1P+uhoEh/5bTAk= =1oOg -----END PGP SIGNATURE----- --Cu2L8LuMZ/p8V+GH--