From: Vladimir Oltean <olteanv@gmail.com>
To: Jakub Kicinski <kuba@kernel.org>,
"David S. Miller" <davem@davemloft.net>,
netdev@vger.kernel.org
Cc: Wong Vee Khee <vee.khee.wong@linux.intel.com>,
Ong Boon Leong <boon.leong.ong@intel.com>,
Michael Sit Wei Hong <michael.wei.hong.sit@intel.com>,
Giuseppe Cavallaro <peppe.cavallaro@st.com>,
Alexandre Torgue <alexandre.torgue@foss.st.com>,
Jose Abreu <joabreu@synopsys.com>,
Jose Abreu <Jose.Abreu@synopsys.com>,
Maxime Coquelin <mcoquelin.stm32@gmail.com>,
Heiner Kallweit <hkallweit1@gmail.com>,
Russell King - ARM Linux admin <linux@armlinux.org.uk>,
Florian Fainelli <f.fainelli@gmail.com>,
Andrew Lunn <andrew@lunn.ch>,
Vivien Didelot <vivien.didelot@gmail.com>,
Vladimir Oltean <vladimir.oltean@nxp.com>
Subject: Re: [PATCH v2 net-next 00/13] Port the SJA1105 DSA driver to XPCS
Date: Fri, 11 Jun 2021 14:44:12 +0300 [thread overview]
Message-ID: <20210611114412.kgapahacqyz72il4@skbuf> (raw)
In-Reply-To: <20210610181410.1886658-1-olteanv@gmail.com>
On Thu, Jun 10, 2021 at 09:13:57PM +0300, Vladimir Oltean wrote:
> From: Vladimir Oltean <vladimir.oltean@nxp.com>
>
> As requested when adding support for the NXP SJA1110, the SJA1105 driver
> could make use of the common XPCS driver, to eliminate some hardware
> specific code duplication.
>
> This series modifies the XPCS driver so that it can accommodate the XPCS
> instantiation from NXP switches, and the SJA1105 driver so it can expose
> what the XPCS driver expects.
>
> Tested on NXP SJA1105S and SJA1110A.
>
> Changes in v2:
> - fix module build (pcs-xpcs-nxp.c is not a different module so this
> means that we need to change the name of pcs-xpcs.ko to pcs_xpcs.ko).
> - delete sja1105_sgmii.h
> - just check for priv->pcs[port] instead of checking the PHY interface
> mode each time.
> - add the 2500base-x check in one place where it was missing (before
> mdio_device_create)
> - remove it from a few places where it is no longer necessary now that
> we check more generically for the presence of priv->xpcs[port]
>
> Vladimir Oltean (13):
> net: pcs: xpcs: rename mdio_xpcs_args to dw_xpcs
> net: stmmac: reverse Christmas tree notation in stmmac_xpcs_setup
> net: stmmac: reduce indentation when calling stmmac_xpcs_setup
> net: pcs: xpcs: move register bit descriptions to a header file
> net: pcs: xpcs: add support for sgmii with no inband AN
> net: pcs: xpcs: also ignore phy id if it's all ones
> net: pcs: xpcs: add support for NXP SJA1105
> net: pcs: xpcs: add support for NXP SJA1110
> net: pcs: xpcs: export xpcs_do_config and xpcs_link_up
> net: dsa: sja1105: migrate to xpcs for SGMII
> net: dsa: sja1105: register the PCS MDIO bus for SJA1110
> net: dsa: sja1105: SGMII and 2500base-x on the SJA1110 are 'special'
> net: dsa: sja1105: plug in support for 2500base-x
>
> MAINTAINERS | 2 +
> drivers/net/dsa/sja1105/Kconfig | 1 +
> drivers/net/dsa/sja1105/sja1105.h | 9 +
> drivers/net/dsa/sja1105/sja1105_main.c | 186 +++----------
> drivers/net/dsa/sja1105/sja1105_mdio.c | 255 +++++++++++++++++
> drivers/net/dsa/sja1105/sja1105_sgmii.h | 53 ----
> drivers/net/dsa/sja1105/sja1105_spi.c | 17 ++
> drivers/net/ethernet/stmicro/stmmac/common.h | 2 +-
> .../net/ethernet/stmicro/stmmac/stmmac_main.c | 10 +-
> .../net/ethernet/stmicro/stmmac/stmmac_mdio.c | 6 +-
> drivers/net/pcs/Makefile | 4 +-
> drivers/net/pcs/pcs-xpcs-nxp.c | 185 ++++++++++++
> drivers/net/pcs/pcs-xpcs.c | 263 +++++++++---------
> drivers/net/pcs/pcs-xpcs.h | 115 ++++++++
> include/linux/pcs/pcs-xpcs.h | 21 +-
> 15 files changed, 772 insertions(+), 357 deletions(-)
> delete mode 100644 drivers/net/dsa/sja1105/sja1105_sgmii.h
> create mode 100644 drivers/net/pcs/pcs-xpcs-nxp.c
> create mode 100644 drivers/net/pcs/pcs-xpcs.h
>
> --
> 2.25.1
>
I see in patchwork this has "changes requested" but when I look through
the patches I see no feedback on them?
https://patchwork.kernel.org/project/netdevbpf/list/?series=498301&state=%2A&archive=both
The checkpatch warnings are along the lines of 'line length of 81/82
characters exceeds limit of 80', which I deliberately ignored, and 'you
didn't CC some random mailing list', which again was more or less
deliberate since I don't know if it would make any difference. Is that
the reason? There are no build failures with this version.
Thanks.
prev parent reply other threads:[~2021-06-11 11:45 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-10 18:13 [PATCH v2 net-next 00/13] Port the SJA1105 DSA driver to XPCS Vladimir Oltean
2021-06-10 18:13 ` [PATCH v2 net-next 01/13] net: pcs: xpcs: rename mdio_xpcs_args to dw_xpcs Vladimir Oltean
2021-06-10 18:13 ` [PATCH v2 net-next 02/13] net: stmmac: reverse Christmas tree notation in stmmac_xpcs_setup Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 03/13] net: stmmac: reduce indentation when calling stmmac_xpcs_setup Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 04/13] net: pcs: xpcs: move register bit descriptions to a header file Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 05/13] net: pcs: xpcs: add support for sgmii with no inband AN Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 06/13] net: pcs: xpcs: also ignore phy id if it's all ones Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 07/13] net: pcs: xpcs: add support for NXP SJA1105 Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 08/13] net: pcs: xpcs: add support for NXP SJA1110 Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 09/13] net: pcs: xpcs: export xpcs_do_config and xpcs_link_up Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 10/13] net: dsa: sja1105: migrate to xpcs for SGMII Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 11/13] net: dsa: sja1105: register the PCS MDIO bus for SJA1110 Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 12/13] net: dsa: sja1105: SGMII and 2500base-x on the SJA1110 are 'special' Vladimir Oltean
2021-06-10 18:14 ` [PATCH v2 net-next 13/13] net: dsa: sja1105: plug in support for 2500base-x Vladimir Oltean
2021-06-11 11:44 ` Vladimir Oltean [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210611114412.kgapahacqyz72il4@skbuf \
--to=olteanv@gmail.com \
--cc=Jose.Abreu@synopsys.com \
--cc=alexandre.torgue@foss.st.com \
--cc=andrew@lunn.ch \
--cc=boon.leong.ong@intel.com \
--cc=davem@davemloft.net \
--cc=f.fainelli@gmail.com \
--cc=hkallweit1@gmail.com \
--cc=joabreu@synopsys.com \
--cc=kuba@kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mcoquelin.stm32@gmail.com \
--cc=michael.wei.hong.sit@intel.com \
--cc=netdev@vger.kernel.org \
--cc=peppe.cavallaro@st.com \
--cc=vee.khee.wong@linux.intel.com \
--cc=vivien.didelot@gmail.com \
--cc=vladimir.oltean@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).