Message ID | 20221128115958.4049431-1-o.rempel@pengutronix.de |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:f944:0:0:0:0:0 with SMTP id q4csp5609383wrr; Mon, 28 Nov 2022 04:03:38 -0800 (PST) X-Google-Smtp-Source: AA0mqf7Cklv16F9IzHKeYhNyye+UQnVPuEpspJ+O/Uy+TQ+V13f796y3OG/TcQ8viJ7wUBjfoJ8a X-Received: by 2002:a63:5548:0:b0:476:c39b:ab5b with SMTP id f8-20020a635548000000b00476c39bab5bmr34015261pgm.565.1669637018090; Mon, 28 Nov 2022 04:03:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1669637018; cv=none; d=google.com; s=arc-20160816; b=teE4iMIf/hoZ+OPwqktbHiRFN7Nyc0vTCM/XOkezPFdjXem/s08NGn8N9Fh5L/Gai/ Bx48Esdcor7LraAsVq4yHIZSYnpMnIQyQRP7C18ZhA2SnK/2LMhJsSlraQ0gNtntkCT2 oYJOkwQ91bjR0+hP2KQiuhTtaLvkO/yiaYwYyVVU0K3Js+aOvK+UEIN+jWWuPvwz5HDK KMC8wo6F9/y6KLaokg6cgehNC0Qb5GKLgMoJacmYT8BfhbbcnCnd4gcyuL5Ybxb9I0Z+ y68C1YVBdKDyrklzyPFken6n1S6JG6KsVmB3+wMx7sWh0ISiwoge+JWRIqu+y/Eg8t1A vY9g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=T8vhrrgYyff7gUUyUoX1i45VIZKA+Fp8MsNtsdAQgJY=; b=VMPz3PI/1zqpXaHSe268mlY/kW/G/OyyJG+du/xONh0QkjHtwM2WqBXGsKUOQElZN2 Afc5L9loZaeXc+wa99wDQ8WgzsVG72CYmjCC62HP53XqdEJ6tg//vKzGuHGg68q2rXay znKJeHvWovTveY1ybH5NWcjZaizKQzrTiBiDYlRBnRFC9JdKjcvRLEzEGpuaWQXGgAPK DV8svNsGJQUVPox608v/EdSkGvELB6v7WT90gO5afFbAy4yFtxzEHBIxs/S8P1cdF4R5 T6hGcvVxrd6fEMe41EZfgSn0klYWXH22EIeRiLJs9qn3WMZb24dpPyasM6TjNVR27Go9 DX6Q== 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 Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c20-20020a6566d4000000b004781187a491si3359385pgw.73.2022.11.28.04.03.11; Mon, 28 Nov 2022 04:03:38 -0800 (PST) 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 S231487AbiK1MBj (ORCPT <rfc822;gah0developer@gmail.com> + 99 others); Mon, 28 Nov 2022 07:01:39 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36104 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231377AbiK1MAR (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 28 Nov 2022 07:00:17 -0500 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 53B3218B30 for <linux-kernel@vger.kernel.org>; Mon, 28 Nov 2022 04:00:15 -0800 (PST) Received: from drehscheibe.grey.stw.pengutronix.de ([2a0a:edc0:0:c01:1d::a2]) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from <ore@pengutronix.de>) id 1ozcns-0003nJ-B9; Mon, 28 Nov 2022 13:00:04 +0100 Received: from [2a0a:edc0:0:1101:1d::ac] (helo=dude04.red.stw.pengutronix.de) by drehscheibe.grey.stw.pengutronix.de with esmtp (Exim 4.94.2) (envelope-from <ore@pengutronix.de>) id 1ozcnq-000o91-LC; Mon, 28 Nov 2022 13:00:03 +0100 Received: from ore by dude04.red.stw.pengutronix.de with local (Exim 4.94.2) (envelope-from <ore@pengutronix.de>) id 1ozcnn-00GzcS-Lw; Mon, 28 Nov 2022 12:59:59 +0100 From: Oleksij Rempel <o.rempel@pengutronix.de> To: Woojung Huh <woojung.huh@microchip.com>, UNGLinuxDriver@microchip.com, Andrew Lunn <andrew@lunn.ch>, Vivien Didelot <vivien.didelot@gmail.com>, Florian Fainelli <f.fainelli@gmail.com>, Vladimir Oltean <olteanv@gmail.com>, "David S. Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com> Cc: Oleksij Rempel <o.rempel@pengutronix.de>, kernel@pengutronix.de, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Arun.Ramadoss@microchip.com Subject: [PATCH v1 00/26] net: dsa: microchip: stats64, fdb, error Date: Mon, 28 Nov 2022 12:59:32 +0100 Message-Id: <20221128115958.4049431-1-o.rempel@pengutronix.de> X-Mailer: git-send-email 2.30.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SA-Exim-Connect-IP: 2a0a:edc0:0:c01:1d::a2 X-SA-Exim-Mail-From: ore@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_NONE,SPF_PASS autolearn=ham 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: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1750741305650509836?= X-GMAIL-MSGID: =?utf-8?q?1750741305650509836?= |
Series |
net: dsa: microchip: stats64, fdb, error
|
|
Message
Oleksij Rempel
Nov. 28, 2022, 11:59 a.m. UTC
This patch series is a result of maintaining work on ksz8 part of microchip driver. It includes stats64 and fdb support. Error handling. Loopback fix and so on... Oleksij Rempel (26): net: dsa: microchip: add stats64 support for ksz8 series of switches net: dsa: microchip: ksz8: ksz8_fdb_dump: fix port validation and VID information net: dsa: microchip: ksz8: ksz8_fdb_dump: fix not complete fdb extraction net: dsa: microchip: ksz8: ksz8_fdb_dump: fix time stamp extraction net: dsa: microchip: ksz8: ksz8_fdb_dump: do not extract ghost entry from empty table net: dsa: microchip: ksz8863_smi: fix bulk access net: dsa: microchip: ksz8_r_dyn_mac_table(): remove timestamp support net: dsa: microchip: make ksz8_r_dyn_mac_table() static net: dsa: microchip: ksz8_r_dyn_mac_table(): remove fid support net: dsa: microchip: ksz8: refactor ksz8_fdb_dump() net: dsa: microchip: ksz8: ksz8_fdb_dump: dump static MAC table net: dsa: microchip: ksz8: move static mac table operations to a separate functions net: dsa: microchip: ksz8: add fdb_add/del support net: dsa: microchip: KSZ88x3 fix loopback support net: dsa: microchip: ksz8_r_dyn_mac_table(): move main part of the code out of if statement net: dsa: microchip: ksz8_r_dyn_mac_table(): use ret instead of rc net: dsa: microchip: ksz8_r_dyn_mac_table(): ksz: do not return EAGAIN on timeout net: dsa: microchip: ksz8_r_dyn_mac_table(): return read/write error if we got any net: dsa: microchip: ksz8_r_dyn_mac_table(): use entries variable to signal 0 entries net: dsa: microchip: make ksz8_r_sta_mac_table() static net: dsa: microchip: ksz8_r_sta_mac_table(): do not use error code for empty entries net: dsa: microchip: ksz8_r_sta_mac_table(): make use of error values provided by read/write functions net: dsa: microchip: make ksz8_w_sta_mac_table() static net: dsa: microchip: ksz8_w_sta_mac_table(): make use of error values provided by read/write functions net: dsa: microchip: remove ksz_port:on variable net: dsa: microchip: ksz8: do not force flow control by default drivers/net/dsa/microchip/ksz8.h | 14 +- drivers/net/dsa/microchip/ksz8795.c | 440 +++++++++++++++--------- drivers/net/dsa/microchip/ksz8795_reg.h | 2 + drivers/net/dsa/microchip/ksz8863_smi.c | 10 +- drivers/net/dsa/microchip/ksz_common.c | 100 +++++- drivers/net/dsa/microchip/ksz_common.h | 2 +- 6 files changed, 377 insertions(+), 191 deletions(-)
Comments
On Mon, Nov 28, 2022 at 12:59:32PM +0100, Oleksij Rempel wrote: > This patch series is a result of maintaining work on ksz8 part of > microchip driver. It includes stats64 and fdb support. Error handling. > Loopback fix and so on... > > Oleksij Rempel (26): The netdev FAQ says: * don’t post large series (> 15 patches), break them up This seems like something which could easily be broken up. Andrew
On Mon, 28 Nov 2022 14:51:47 +0100 Andrew Lunn wrote: > * don’t post large series (> 15 patches), break them up > > This seems like something which could easily be broken up. And name the target tree in the subject tag. And make sure it applies cleanly to that tree (last patch is to blame AFAICT). And don't repost within 24 hours.
On 11/28/2022 3:59 AM, Oleksij Rempel wrote: > This patch series is a result of maintaining work on ksz8 part of > microchip driver. It includes stats64 and fdb support. Error handling. > Loopback fix and so on... > > Oleksij Rempel (26): > net: dsa: microchip: add stats64 support for ksz8 series of switches > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix port validation and VID > information > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix not complete fdb > extraction > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix time stamp extraction > net: dsa: microchip: ksz8: ksz8_fdb_dump: do not extract ghost entry > from empty table > net: dsa: microchip: ksz8863_smi: fix bulk access > net: dsa: microchip: ksz8_r_dyn_mac_table(): remove timestamp support > net: dsa: microchip: make ksz8_r_dyn_mac_table() static > net: dsa: microchip: ksz8_r_dyn_mac_table(): remove fid support > net: dsa: microchip: ksz8: refactor ksz8_fdb_dump() > net: dsa: microchip: ksz8: ksz8_fdb_dump: dump static MAC table > net: dsa: microchip: ksz8: move static mac table operations to a > separate functions > net: dsa: microchip: ksz8: add fdb_add/del support > net: dsa: microchip: KSZ88x3 fix loopback support > net: dsa: microchip: ksz8_r_dyn_mac_table(): move main part of the > code out of if statement > net: dsa: microchip: ksz8_r_dyn_mac_table(): use ret instead of rc > net: dsa: microchip: ksz8_r_dyn_mac_table(): ksz: do not return EAGAIN > on timeout > net: dsa: microchip: ksz8_r_dyn_mac_table(): return read/write error > if we got any > net: dsa: microchip: ksz8_r_dyn_mac_table(): use entries variable to > signal 0 entries > net: dsa: microchip: make ksz8_r_sta_mac_table() static > net: dsa: microchip: ksz8_r_sta_mac_table(): do not use error code for > empty entries > net: dsa: microchip: ksz8_r_sta_mac_table(): make use of error values > provided by read/write functions > net: dsa: microchip: make ksz8_w_sta_mac_table() static > net: dsa: microchip: ksz8_w_sta_mac_table(): make use of error values > provided by read/write functions > net: dsa: microchip: remove ksz_port:on variable > net: dsa: microchip: ksz8: do not force flow control by default > My understanding is that we typically limit series to 15 patches. Do you have some justification for why this goes over 15 and can't reasonably be split into two series? At a glance it seems like a bunch of smaller cleanups.
On Mon, Nov 28, 2022 at 03:09:19PM -0800, Jacob Keller wrote: > > > On 11/28/2022 3:59 AM, Oleksij Rempel wrote: > > This patch series is a result of maintaining work on ksz8 part of > > microchip driver. It includes stats64 and fdb support. Error handling. > > Loopback fix and so on... > > > > Oleksij Rempel (26): > > net: dsa: microchip: add stats64 support for ksz8 series of switches > > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix port validation and VID > > information > > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix not complete fdb > > extraction > > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix time stamp extraction > > net: dsa: microchip: ksz8: ksz8_fdb_dump: do not extract ghost entry > > from empty table > > net: dsa: microchip: ksz8863_smi: fix bulk access > > net: dsa: microchip: ksz8_r_dyn_mac_table(): remove timestamp support > > net: dsa: microchip: make ksz8_r_dyn_mac_table() static > > net: dsa: microchip: ksz8_r_dyn_mac_table(): remove fid support > > net: dsa: microchip: ksz8: refactor ksz8_fdb_dump() > > net: dsa: microchip: ksz8: ksz8_fdb_dump: dump static MAC table > > net: dsa: microchip: ksz8: move static mac table operations to a > > separate functions > > net: dsa: microchip: ksz8: add fdb_add/del support > > net: dsa: microchip: KSZ88x3 fix loopback support > > net: dsa: microchip: ksz8_r_dyn_mac_table(): move main part of the > > code out of if statement > > net: dsa: microchip: ksz8_r_dyn_mac_table(): use ret instead of rc > > net: dsa: microchip: ksz8_r_dyn_mac_table(): ksz: do not return EAGAIN > > on timeout > > net: dsa: microchip: ksz8_r_dyn_mac_table(): return read/write error > > if we got any > > net: dsa: microchip: ksz8_r_dyn_mac_table(): use entries variable to > > signal 0 entries > > net: dsa: microchip: make ksz8_r_sta_mac_table() static > > net: dsa: microchip: ksz8_r_sta_mac_table(): do not use error code for > > empty entries > > net: dsa: microchip: ksz8_r_sta_mac_table(): make use of error values > > provided by read/write functions > > net: dsa: microchip: make ksz8_w_sta_mac_table() static > > net: dsa: microchip: ksz8_w_sta_mac_table(): make use of error values > > provided by read/write functions > > net: dsa: microchip: remove ksz_port:on variable > > net: dsa: microchip: ksz8: do not force flow control by default > > > > > My understanding is that we typically limit series to 15 patches. Do you > have some justification for why this goes over 15 and can't reasonably be > split into two series? > > At a glance it seems like a bunch of smaller cleanups. The previous patch set got request to do more clean ups: https://lore.kernel.org/all/20221124101458.3353902-1-o.rempel@pengutronix.de/ I need to show, there are already more patches in the queue. Regards, Oleksij
On Tue, Nov 29, 2022 at 06:35:39AM +0100, Oleksij Rempel wrote: > On Mon, Nov 28, 2022 at 03:09:19PM -0800, Jacob Keller wrote: > > > > > > On 11/28/2022 3:59 AM, Oleksij Rempel wrote: > > > This patch series is a result of maintaining work on ksz8 part of > > > microchip driver. It includes stats64 and fdb support. Error handling. > > > Loopback fix and so on... > > > > > > Oleksij Rempel (26): > > > net: dsa: microchip: add stats64 support for ksz8 series of switches > > > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix port validation and VID > > > information > > > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix not complete fdb > > > extraction > > > net: dsa: microchip: ksz8: ksz8_fdb_dump: fix time stamp extraction > > > net: dsa: microchip: ksz8: ksz8_fdb_dump: do not extract ghost entry > > > from empty table > > > net: dsa: microchip: ksz8863_smi: fix bulk access > > > net: dsa: microchip: ksz8_r_dyn_mac_table(): remove timestamp support > > > net: dsa: microchip: make ksz8_r_dyn_mac_table() static > > > net: dsa: microchip: ksz8_r_dyn_mac_table(): remove fid support > > > net: dsa: microchip: ksz8: refactor ksz8_fdb_dump() > > > net: dsa: microchip: ksz8: ksz8_fdb_dump: dump static MAC table > > > net: dsa: microchip: ksz8: move static mac table operations to a > > > separate functions > > > net: dsa: microchip: ksz8: add fdb_add/del support > > > net: dsa: microchip: KSZ88x3 fix loopback support > > > net: dsa: microchip: ksz8_r_dyn_mac_table(): move main part of the > > > code out of if statement > > > net: dsa: microchip: ksz8_r_dyn_mac_table(): use ret instead of rc > > > net: dsa: microchip: ksz8_r_dyn_mac_table(): ksz: do not return EAGAIN > > > on timeout > > > net: dsa: microchip: ksz8_r_dyn_mac_table(): return read/write error > > > if we got any > > > net: dsa: microchip: ksz8_r_dyn_mac_table(): use entries variable to > > > signal 0 entries > > > net: dsa: microchip: make ksz8_r_sta_mac_table() static > > > net: dsa: microchip: ksz8_r_sta_mac_table(): do not use error code for > > > empty entries > > > net: dsa: microchip: ksz8_r_sta_mac_table(): make use of error values > > > provided by read/write functions > > > net: dsa: microchip: make ksz8_w_sta_mac_table() static > > > net: dsa: microchip: ksz8_w_sta_mac_table(): make use of error values > > > provided by read/write functions > > > net: dsa: microchip: remove ksz_port:on variable > > > net: dsa: microchip: ksz8: do not force flow control by default > > > > > > > > > My understanding is that we typically limit series to 15 patches. Do you > > have some justification for why this goes over 15 and can't reasonably be > > split into two series? > > > > At a glance it seems like a bunch of smaller cleanups. > > The previous patch set got request to do more clean ups: > https://lore.kernel.org/all/20221124101458.3353902-1-o.rempel@pengutronix.de/ > > I need to show, there are already more patches in the queue. There is some psychology involved here. I see 26 patches and decide i need to allocate 30 minutes to this sometime, and put the review off until later, without even looking at them. If i get 5 patches, i probably just do it, knowing i will be finished pretty quickly. My guess is, 5 patches a day for 5 days will be merged faster than 26 patches in one go. Andrew
On Tue, Nov 29, 2022 at 03:15:03PM +0100, Andrew Lunn wrote: > On Tue, Nov 29, 2022 at 06:35:39AM +0100, Oleksij Rempel wrote: > > On Mon, Nov 28, 2022 at 03:09:19PM -0800, Jacob Keller wrote: > > > > > > My understanding is that we typically limit series to 15 patches. Do you > > > have some justification for why this goes over 15 and can't reasonably be > > > split into two series? > > > > > > At a glance it seems like a bunch of smaller cleanups. > > > > The previous patch set got request to do more clean ups: > > https://lore.kernel.org/all/20221124101458.3353902-1-o.rempel@pengutronix.de/ > > > > I need to show, there are already more patches in the queue. > > There is some psychology involved here. I see 26 patches and decide i > need to allocate 30 minutes to this sometime, and put the review off > until later, without even looking at them. If i get 5 patches, i > probably just do it, knowing i will be finished pretty quickly. My > guess is, 5 patches a day for 5 days will be merged faster than 26 > patches in one go. Good point. Thx! I'll split this patch set. Regards, Oleksij