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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 31145C76196 for ; Mon, 3 Apr 2023 17:43:26 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231816AbjDCRnY (ORCPT ); Mon, 3 Apr 2023 13:43:24 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48856 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231771AbjDCRnV (ORCPT ); Mon, 3 Apr 2023 13:43:21 -0400 Received: from fudo.makrotopia.org (fudo.makrotopia.org [IPv6:2a07:2ec0:3002::71]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0ECB91FFD; Mon, 3 Apr 2023 10:43:14 -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 1pjOCs-0001Lc-1w; Mon, 03 Apr 2023 19:43:02 +0200 Date: Mon, 3 Apr 2023 18:42:59 +0100 From: Daniel Golle To: =?utf-8?B?QXLEsW7DpyDDnE5BTA==?= Cc: devicetree@vger.kernel.org, netdev@vger.kernel.org, linux-mediatek@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rob Herring , Krzysztof Kozlowski , 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 , Russell King , Sam Shih , Lorenzo Bianconi , John Crispin , Felix Fietkau Subject: Re: [PATCH net-next v2 00/14] net: dsa: add support for MT7988 Message-ID: References: <53d89480-936d-25b1-6422-cda7769de369@arinc9.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <53d89480-936d-25b1-6422-cda7769de369@arinc9.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Arınç, On Mon, Apr 03, 2023 at 08:08:19PM +0300, Arınç ÜNAL wrote: > On 3.04.2023 04:16, Daniel Golle wrote: > > The MediaTek MT7988 SoC comes with a built-in switch very similar to > > previous MT7530 and MT7531. However, the switch address space is mapped > > into the SoCs memory space rather than being connected via MDIO. > > Using MMIO simplifies register access and also removes the need for a bus > > lock, and for that reason also makes interrupt handling more light-weight. > > > > Note that this is different from previous SoCs like MT7621 and MT7623N > > which also came with an integrated MT7530-like switch which yet had to be > > accessed via MDIO. > > > > Split-off the part of the driver registering an MDIO driver, then add > > another module acting as MMIO/platform driver. > > > > The whole series has been tested on various MediaTek boards: > > * MT7623A + MT7530 (BPi-R2) > > * MT7986A + MT7531 (BPi-R3) > > * MT7988A reference board > > You did not address the incorrect information I pointed out here. Now that I'm sorry, that was certainly not intentional and I may have missed your comments. Actually it doesn't look like they have made it to the netdev list archive or patchwork either. > the patch series is applied, people reading this on the merge branch commit > will be misled by the misinformation. I've changed Kconfig stuff according to your recommendation and also addressed possible misleading USXGMII and 10GBase-KR support by introducing MT7988-specific functions and using 'internal' PHY mode. So which of your comments have not been addressed? > > > > > Changes since v1: > > * use 'internal' PHY mode where appropriate > > * use regmap_update_bits in mt7530_rmw > > * improve dt-bindings > > As a maintainer of the said dt-bindings, I pointed out almost 7 things for > you to change. Of those 7 points, you only did one, a trivial grammar > change. The patch series is applied now so one of us maintainers (you are > one too now) need to fix it with additional patches. I was also surprised the series made it to net-next so quickly, but it wasn't me applying it, I merly posted v2 with all comments I received addressed. Me and supposedly also netdevbpf maintainers use patchwork to track patches and whether comments have been addressed. Can you point me to emails with the comments which haven't been addressed there? Looking in patchwork for the dt-bindings patch [1] I don't see any comments there. Thank you for reviewing! Daniel [1]: See patchwork tracking for RFCv3, v1 and v2. Prior to RFCv3 the series didn't have the dt-bindings addition, I introduced it with RFCv3 when splitting the series into many small changes: https://patchwork.kernel.org/project/netdevbpf/patch/9b504e3e88807bfb62022c0877451933d30abeb5.1680105013.git.daniel@makrotopia.org/ https://patchwork.kernel.org/project/netdevbpf/patch/fef2cb2fe3d2b70fa46e93107a0c862f53bb3bfa.1680180959.git.daniel@makrotopia.org/ https://patchwork.kernel.org/project/netdevbpf/patch/dffacdb59aea462c9f7d4242cf9563a04cf79807.1680483896.git.daniel@makrotopia.org/