[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <Y+1MTXX1+uRsPnNO@shell.armlinux.org.uk>
Date: Wed, 15 Feb 2023 21:19:09 +0000
From: "Russell King (Oracle)" <linux@...linux.org.uk>
To: Daniel Golle <daniel@...rotopia.org>
Cc: devicetree@...r.kernel.org, Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
netdev@...r.kernel.org, linux-mediatek@...ts.infradead.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
Heiner Kallweit <hkallweit1@...il.com>,
Lorenzo Bianconi <lorenzo@...nel.org>,
Mark Lee <Mark-MC.Lee@...iatek.com>,
John Crispin <john@...ozen.org>, Felix Fietkau <nbd@....name>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@...labora.com>,
Matthias Brugger <matthias.bgg@...il.com>,
DENG Qingfang <dqfext@...il.com>,
Landen Chao <Landen.Chao@...iatek.com>,
Sean Wang <sean.wang@...iatek.com>,
Paolo Abeni <pabeni@...hat.com>,
Jakub Kicinski <kuba@...nel.org>,
Eric Dumazet <edumazet@...gle.com>,
"David S. Miller" <davem@...emloft.net>,
Vladimir Oltean <olteanv@...il.com>,
Florian Fainelli <f.fainelli@...il.com>,
Andrew Lunn <andrew@...n.ch>,
Jianhui Zhao <zhaojh329@...il.com>,
Bjørn Mork <bjorn@...k.no>
Subject: Re: [PATCH v7 00/12] net: ethernet: mtk_eth_soc: various enhancements
Hi Daniel,
Please don't be so quick to send the next revision as soon as you
receive comments on a series - as has just been demonstrated, I've
replied to Rob's comment about moving the binding to clocks which
I don't agree with having read the documentation.
Posting a new series so quickly cuts off any hope of a sensible
discussion, and leads to people having to scramble to say "no,
we're still discussing the previous series" to possibly prevent
netdev applying the patches before the discussion has reached
a conclusion.
Thanks.
On Wed, Feb 15, 2023 at 08:59:37PM +0000, Daniel Golle wrote:
> This series brings a variety of fixes and enhancements for mtk_eth_soc,
> adds support for the MT7981 SoC and facilitates sharing the SGMII PCS
> code between mtk_eth_soc and mt7530.
>
> Note that this series depends on commit 697c3892d825
> ("regmap: apply reg_base and reg_downshift for single register ops") to
> not break mt7530 pcs register access.
>
> The whole series has been tested on MT7622+MT7531 (BPi-R64),
> MT7623+MT7530 (BPi-R2) and MT7981+GPY211 (GL.iNet GL-MT3000).
>
> Changes since v6:
> * label MAC MCR bit 12 in 08/12, MediaTek replied explaining its function
> * move mediatek,sgmiisys bindings to
> Documentation/devicetree/bindings/clock
>
> Changes since v5:
> * drop dev pointer also from struct mtk_sgmii, pass it as function
> paramter instead
> * address comments left for dt-bindings
> * minor improvements to commit messages
>
> Changes since v4:
> * remove unused dev pointer in struct pcs_mtk_lynxi
> * squash link timer check into correct follow-up patch
>
> Changes since v3:
> * remove unused #define's
> * use BMCR_* instead of #define'ing our own constants
> * return before changing registers in case of invalid link timer
>
> Changes since v2:
> * improve dt-bindings, convert sgmisys bindings to dt-schema yaml
> * fix typo
>
> Changes since v1:
> * apply reverse xmas tree everywhere
> * improve commit descriptions
> * add dt binding documentation
> * various small changes addressing all comments received for v1
>
>
> Daniel Golle (12):
> net: ethernet: mtk_eth_soc: add support for MT7981 SoC
> dt-bindings: net: mediatek,net: add mt7981-eth binding
> dt-bindings: arm: mediatek: sgmiisys: Convert to DT schema
> dt-bindings: arm: mediatek: sgmiisys: add MT7981 SoC
> net: ethernet: mtk_eth_soc: set MDIO bus clock frequency
> net: ethernet: mtk_eth_soc: reset PCS state
> net: ethernet: mtk_eth_soc: only write values if needed
> net: ethernet: mtk_eth_soc: fix RX data corruption issue
> net: ethernet: mtk_eth_soc: ppe: add support for flow accounting
> net: pcs: add driver for MediaTek SGMII PCS
> net: ethernet: mtk_eth_soc: switch to external PCS driver
> net: dsa: mt7530: use external PCS driver
>
> .../arm/mediatek/mediatek,sgmiisys.txt | 27 --
> .../bindings/clock/mediatek,sgmiisys.yaml | 55 ++++
> .../devicetree/bindings/net/mediatek,net.yaml | 52 ++-
> MAINTAINERS | 7 +
> drivers/net/dsa/Kconfig | 1 +
> drivers/net/dsa/mt7530.c | 277 ++++------------
> drivers/net/dsa/mt7530.h | 47 +--
> drivers/net/ethernet/mediatek/Kconfig | 2 +
> drivers/net/ethernet/mediatek/mtk_eth_path.c | 14 +-
> drivers/net/ethernet/mediatek/mtk_eth_soc.c | 67 +++-
> drivers/net/ethernet/mediatek/mtk_eth_soc.h | 105 +++---
> drivers/net/ethernet/mediatek/mtk_ppe.c | 114 ++++++-
> drivers/net/ethernet/mediatek/mtk_ppe.h | 25 +-
> .../net/ethernet/mediatek/mtk_ppe_debugfs.c | 9 +-
> .../net/ethernet/mediatek/mtk_ppe_offload.c | 8 +
> drivers/net/ethernet/mediatek/mtk_ppe_regs.h | 14 +
> drivers/net/ethernet/mediatek/mtk_sgmii.c | 192 ++---------
> drivers/net/pcs/Kconfig | 7 +
> drivers/net/pcs/Makefile | 1 +
> drivers/net/pcs/pcs-mtk-lynxi.c | 303 ++++++++++++++++++
> include/linux/pcs/pcs-mtk-lynxi.h | 13 +
> 21 files changed, 802 insertions(+), 538 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,sgmiisys.txt
> create mode 100644 Documentation/devicetree/bindings/clock/mediatek,sgmiisys.yaml
> create mode 100644 drivers/net/pcs/pcs-mtk-lynxi.c
> create mode 100644 include/linux/pcs/pcs-mtk-lynxi.h
>
>
> base-commit: 9d9019bcea1aac7eed64a1a4966282b6b7b141c8
> --
> 2.39.1
>
>
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!
Powered by blists - more mailing lists