Received: by 2002:a05:6358:11c7:b0:104:8066:f915 with SMTP id i7csp850191rwl; Fri, 31 Mar 2023 03:24:24 -0700 (PDT) X-Google-Smtp-Source: AKy350aPEwFTz+e3fTKfpcvmBmtQdPHAWfroCfleC3AUraq0e8+4IegGCxkddOb/9pD9hD3CJhqy X-Received: by 2002:a05:6a20:3d91:b0:e3:e236:bbcb with SMTP id s17-20020a056a203d9100b000e3e236bbcbmr6555141pzi.0.1680258263830; Fri, 31 Mar 2023 03:24:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1680258263; cv=none; d=google.com; s=arc-20160816; b=iWEQsPwBKFd90R24EQJoTor0rAYkEfTLHNUZqGK2tq+B7ObWShgoP3YooTl/Kfya8X 45W1qVsbt1oj0X0LjVJj+kt7Mq4IPpIxS/4e7agesT1KSSR+kI18MSsCY2Nc95QCWsQ+ tDB0BUT9kaFs8XDBY2e5+T97Hll74qFjB7XzIxcmbDvScv8Vbt/sZDUPmftHiqrXn6tM MOJB135dXdK+014QjvN3ixe/xHgJo/kkaS8pc2lTEhUX68V0YTH/NLU14NCC5pr3rrlD dAHkK3YzfIiiWvr6olGAkfZW0fsaJ7vii1F2/n+ptOhuXT7Oyjm5asrxJ42wXqBNI1ER ihWA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=5P91d6PC4Ir4m6v+hoMMV1mzQVmPLodC7Mby9lNyx3E=; b=CSVDVIgTM18FhhZFOiPTts/QEhZxMV89IHjhKBv28/7tAy6pEOWRYprZnM2IdxASnz j6JKn0aONPa/dp8PxsAUjRPkYKbq0wfbJssnHnqedwJBXH2EfNHjoR9MR68vJfZUT0+2 m38t1fknfXilVMLBWo/6nkjQmAmsVHUpfxtyaxhFvQFJRQpCimnWm625oNdfSzH883jR x3ftkknDPZGDcQeFoVKcZ2qDsZ+RJX3l/2vhRFWdFuDNLj3jR8omPK9d+QJi0Qdd8sik GeuLpIne2RxgOYVpyhAs4jgMsyfmS6IwtwpIBx5WMASvuBQt3pMG+6UASH7bGq4g592t Y1Qg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id n7-20020a6543c7000000b0050bcb8e3a8csi1924959pgp.828.2023.03.31.03.24.11; Fri, 31 Mar 2023 03:24:23 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230058AbjCaKVl (ORCPT + 99 others); Fri, 31 Mar 2023 06:21:41 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52968 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230385AbjCaKU4 (ORCPT ); Fri, 31 Mar 2023 06:20:56 -0400 Received: from fudo.makrotopia.org (fudo.makrotopia.org [IPv6:2a07:2ec0:3002::71]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CE3581F7B4; Fri, 31 Mar 2023 03:17:09 -0700 (PDT) Received: from local by fudo.makrotopia.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.96) (envelope-from ) id 1piBoc-0001PC-0s; Fri, 31 Mar 2023 12:17:02 +0200 Date: Fri, 31 Mar 2023 11:16:44 +0100 From: Daniel Golle To: =?utf-8?B?QXLEsW7DpyDDnE5BTA==?= Cc: netdev@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux@armlinux.org.uk, linux-kernel@vger.kernel.org, Andrew Lunn , Florian Fainelli , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , AngeloGioacchino Del Regno , Sean Wang , Landen Chao , DENG Qingfang , Philipp Zabel , Sam Shih , Lorenzo Bianconi , John Crispin , Felix Fietkau , Luiz Angelo Daros de Luca Subject: Re: [PATCH net-next 14/15] net: dsa: mt7530: introduce driver for MT7988 built-in switch Message-ID: References: <6a7c5f81-a8a3-27b5-4af3-7175a3313f9a@arinc9.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <6a7c5f81-a8a3-27b5-4af3-7175a3313f9a@arinc9.com> X-Spam-Status: No, score=0.0 required=5.0 tests=SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Mar 31, 2023 at 08:50:28AM +0300, Arınç ÜNAL wrote: > On 30.03.2023 18:23, Daniel Golle wrote: > > Add driver for the built-in Gigabit Ethernet switch which can be found > > in the MediaTek MT7988 SoC. > > > > The switch shares most of its design with MT7530 and MT7531, but has > > it's registers mapped into the SoCs register space rather than being > > connected externally or internally via MDIO. > > > > Introduce a new platform driver to support that. > > > > Signed-off-by: Daniel Golle > > --- > > MAINTAINERS | 2 + > > drivers/net/dsa/Kconfig | 12 ++++ > > drivers/net/dsa/Makefile | 1 + > > drivers/net/dsa/mt7530-mmio.c | 101 ++++++++++++++++++++++++++++++++++ > > drivers/net/dsa/mt7530.c | 86 ++++++++++++++++++++++++++++- > > drivers/net/dsa/mt7530.h | 12 ++-- > > 6 files changed, 206 insertions(+), 8 deletions(-) > > create mode 100644 drivers/net/dsa/mt7530-mmio.c > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 14924aed15ca7..674673dbdfd8b 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -13174,9 +13174,11 @@ MEDIATEK SWITCH DRIVER > > M: Sean Wang > > M: Landen Chao > > M: DENG Qingfang > > +M: Daniel Golle > > L: netdev@vger.kernel.org > > S: Maintained > > F: drivers/net/dsa/mt7530-mdio.c > > +F: drivers/net/dsa/mt7530-mmio.c > > F: drivers/net/dsa/mt7530.* > > F: net/dsa/tag_mtk.c > > diff --git a/drivers/net/dsa/Kconfig b/drivers/net/dsa/Kconfig > > index c2551b13324c2..de4d86e37973f 100644 > > --- a/drivers/net/dsa/Kconfig > > +++ b/drivers/net/dsa/Kconfig > > @@ -52,6 +52,18 @@ config NET_DSA_MT7530 > > Multi-chip module MT7530 in MT7621AT, MT7621DAT, MT7621ST and > > MT7623AI SoCs is supported as well. > > +config NET_DSA_MT7988 > > + tristate "MediaTek MT7988 built-in Ethernet switch support" > > + select NET_DSA_MT7530_COMMON > > + depends on HAS_IOMEM > > + help > > + This enables support for the built-in Ethernet switch found > > + in the MediaTek MT7988 SoC. > > + The switch is a similar design as MT7531, however, unlike > > + other MT7530 and MT7531 the switch registers are directly > > + mapped into the SoCs register space rather than being accessible > > + via MDIO. > > + > > config NET_DSA_MV88E6060 > > tristate "Marvell 88E6060 ethernet switch chip support" > > select NET_DSA_TAG_TRAILER > > diff --git a/drivers/net/dsa/Makefile b/drivers/net/dsa/Makefile > > index 71250d7dd41af..103a33e20de4b 100644 > > --- a/drivers/net/dsa/Makefile > > +++ b/drivers/net/dsa/Makefile > > @@ -8,6 +8,7 @@ endif > > obj-$(CONFIG_NET_DSA_LANTIQ_GSWIP) += lantiq_gswip.o > > obj-$(CONFIG_NET_DSA_MT7530_COMMON) += mt7530.o > > obj-$(CONFIG_NET_DSA_MT7530) += mt7530-mdio.o > > +obj-$(CONFIG_NET_DSA_MT7988) += mt7530-mmio.o > > I'm not fond of this way. Wouldn't it be better if we split the mdio and > mmio drivers to separate modules and kept switch hardware support on > mt7530.c? You mean this in terms of Kconfig symbols? Because the way you describe is basically what I'm doing here: * mt7530.c is the shared/common switch hardware driver * mt7530-mdio.c contains the MDIO accessors and MDIO device drivers for MT7530, MT7531, MT7621, MT7623, ... * mt7530-mmio.c contains the platform device driver for in-SoC switches which are accessed via MMIO, ie. MT7988 (and yes, this could be extended to also support MT7620A/N). In early drafts I also named the Kconfig symbols CONFIG_NET_DSA_MT7530 for mt7530.c (ie. the common part) CONFIG_NET_DSA_MT7530_MDIO for the MDIO driver CONFIG_NET_DSA_MT7530_MMIO for the MMIO driver However, as existing kernel configurations expect CONFIG_NET_DSA_MT7530 to select the MDIO driver, I decided it would be better to hide the symbol of the common part and have CONFIG_NET_DSA_MT7530 select the MDIO driver like it was before. Hence I decided to go with CONFIG_NET_DSA_MT7530 selects the MDIO driver, just like before CONFIG_NET_DSA_MT7988 selects the new MMIO driver CONFIG_NET_DSA_MT7530_COMMON is hidden, selected by both of the above > > The mmio driver could be useful in the future for the MT7530 on the MT7620 > SoCs or generally new hardware that would use MMIO to be controlled. > Sure, it would be a bit confusing once we add support for MT7620A/N (if that ever happens...), then CONFIG_NET_DSA_MT7988 would need to be selected to support this ancient MIPS SoC... If you are planning to work on support for MT7620A/N feel free to suggest a better way to name the Kconfig symbols. > Luiz did this for the Realtek switches that use MDIO and SMI to be > controlled. > > https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/tree/drivers/net/dsa/realtek/Kconfig > > https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/tree/drivers/net/dsa/realtek/Makefile Are you suggesting to split-off a device-specific driver which would then select the access-method driver (MDIO vs. MMIO) and the common/shared driver? To me this looks like overkill for MT7530, given that the designs of all MT7530 are pretty similar, ie. same tag format and also otherwise very similar. Thank you for reviewing! Daniel