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 8FF5AC3ABCB for ; Sun, 11 May 2025 21:58:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: Content-Type:MIME-Version:References:Message-ID:Subject: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=y0vf+r4SgVS8P3BRP1UuumCtts1FPTgBpsRtvD5rM5s=; b=ZB1JeOFAj3G9bul2qZoKbUSo3X v6a3RqGrsU5CDiaEYlOnr8m6JOK7ph/EQlCw3D2dRpkfIdkjRIgjmotvnJfPFvuPD9I4a3KomXQxb HrMV9gZF3jdu0uTbZ2AN/4uwIIzTrMrF/9RocEUREiAL++I2Tmh2zbvcQj08/hvUfyg508ElmNzEu Bm6WeOnnZm/6K7777Vqb83UXrLQxurwVNFo+Tha3NXQkjkqp54+83+LI9SULIK6G/AVuR7rGFDFXL acra+tpiYbX7qUXtibytRznKWzmze3KMGySPrUQ8rJh7VE6RZeZPc2cddFnIoTiJVl3YMhdbffz3H 56M8Bwmw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1uEEgn-00000007vsM-1gF0; Sun, 11 May 2025 21:58:29 +0000 Received: from pidgin.makrotopia.org ([2a07:2ec0:3002::65]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1uEEem-00000007vou-42rq; Sun, 11 May 2025 21:56:26 +0000 Received: from local by pidgin.makrotopia.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.98.2) (envelope-from ) id 1uEEYY-000000006sM-1X1I; Sun, 11 May 2025 21:55:54 +0000 Date: Sun, 11 May 2025 22:55:50 +0100 From: Daniel Golle To: Andrew Lunn Subject: Re: Re: [PATCH v1 09/14] arm64: dts: mediatek: mt7988: add switch node Message-ID: References: <20250511141942.10284-1-linux@fw-web.de> <20250511141942.10284-10-linux@fw-web.de> <74afe286-2adb-42d3-9491-881379053e36@lunn.ch> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <74afe286-2adb-42d3-9491-881379053e36@lunn.ch> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250511_145625_003671_232471EF X-CRM114-Status: GOOD ( 10.71 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Landen.Chao@mediatek.com, edumazet@google.com, linux@fw-web.de, robh@kernel.org, kuba@kernel.org, pabeni@redhat.com, lorenzo@kernel.org, devicetree@vger.kernel.org, conor+dt@kernel.org, sean.wang@mediatek.com, dqfext@gmail.com, linux-mediatek@lists.infradead.org, matthias.bgg@gmail.com, linux-arm-kernel@lists.infradead.org, angelogioacchino.delregno@collabora.com, arinc.unal@arinc9.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, krzk+dt@kernel.org, olteanv@gmail.com, davem@davemloft.net, nbd@nbd.name Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Sun, May 11, 2025 at 11:25:27PM +0200, Andrew Lunn wrote: > > i will move that into the board dtsi file in v2 because "normal" bpi-r4 and 2g5 variant are same here. > > Maybe you just need to expand the commit message? What does this .dtsi > actually represent? The SoC, or what is common across a number of > boards? mt7988a.dtsi represents the SoC, and thus there shouldn't be any labels assigned to the DSA ports. In case of the BananaPi R4 there are two different boards, one with 2x SFP+ and one with 1x SFP+ + 1x 2500Base-T (with PoE-in), hence there is a dtsi files for all the parts shared among the two boards. I suppose Frank meant to move the labels to that board dtsi file, and that's correct imho as the labels of the 1G switch ports are the same on BPi-R4 and BPi-R4-PoE.