netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: Andrew Lunn <andrew@lunn.ch>, Heiner Kallweit <hkallweit1@gmail.com>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Daniel Scally <djrscally@gmail.com>,
	"David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Florian Fainelli <f.fainelli@gmail.com>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Heikki Krogerus <heikki.krogerus@linux.intel.com>,
	Jakub Kicinski <kuba@kernel.org>,
	linux-acpi@vger.kernel.org, netdev@vger.kernel.org,
	Paolo Abeni <pabeni@redhat.com>,
	"Rafael J. Wysocki" <rafael@kernel.org>,
	Sakari Ailus <sakari.ailus@linux.intel.com>,
	Vladimir Oltean <olteanv@gmail.com>
Subject: [PATCH RFC net-next 0/7] Another attempt at moving mv88e6xxx forward
Date: Wed, 22 Mar 2023 11:59:36 +0000	[thread overview]
Message-ID: <ZBrtqPW29NnxVoEc@shell.armlinux.org.uk> (raw)

Hi,

This is another attempt to move the mv88e6xxx driver forward so that we
can eventually switch it to use phylink_pcs and become a non-legacy
driver.

The issue is that in order to switch to phylink_pcs, we need DSA and CPU
ports to be known to phylink, otherwise the PCS code will not be called.
In order for such ports to be known to phylink, we need to provide
phylink with a configuration, and mv88e6xxx has a history of not
specifying the configuration in firmware, but the driver internally
handling that. This is fine, but it means we can't use phylink for such
ports - and thus converting them to phylink_pcs can cause regressions.

Therefore, this series provides a way for a software-node configuration
to be provided to DSA by the driver, which will then be used only for
phylink to parse.

Some of this patch set comes from an idea from Vladimir, but
re-implemented in a substantially different way.

 drivers/base/swnode.c            |  14 +++-
 drivers/net/dsa/mv88e6xxx/chip.c | 157 ++++++++++++++++++++++++++++-----------
 drivers/net/phy/phylink.c        |  32 ++++++++
 include/linux/phylink.h          |   1 +
 include/linux/property.h         |   4 +
 include/net/dsa.h                |   3 +
 net/dsa/port.c                   |  33 ++++++--
 7 files changed, 191 insertions(+), 53 deletions(-)

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!

             reply	other threads:[~2023-03-22 11:59 UTC|newest]

Thread overview: 56+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-03-22 11:59 Russell King (Oracle) [this message]
2023-03-22 11:59 ` [PATCH RFC net-next 1/7] software node: allow named software node to be created Russell King
2023-03-23 13:59   ` Andy Shevchenko
2023-03-23 14:29     ` Russell King (Oracle)
2023-03-23 14:39       ` Andy Shevchenko
2023-03-22 12:00 ` [PATCH RFC net-next 2/7] net: phylink: provide phylink_find_max_speed() Russell King (Oracle)
2023-03-22 18:44   ` Andrew Lunn
2023-03-22 12:00 ` [PATCH RFC net-next 3/7] net: dsa: use fwnode_get_phy_mode() to get phy interface mode Russell King (Oracle)
2023-03-22 18:42   ` Andrew Lunn
2023-03-23 14:03   ` Andy Shevchenko
2023-03-23 14:31     ` Russell King (Oracle)
2023-03-23 14:38       ` Andy Shevchenko
2023-03-23 14:49         ` Russell King (Oracle)
2023-03-23 15:00           ` Andy Shevchenko
2023-03-23 15:23             ` Russell King (Oracle)
2023-03-23 15:33               ` Andy Shevchenko
2023-03-23 16:29                 ` Russell King (Oracle)
2023-03-23 16:18               ` Russell King (Oracle)
2023-03-23 16:34                 ` Andy Shevchenko
2023-03-23 16:39                   ` Andy Shevchenko
2023-03-23 17:06                   ` Russell King (Oracle)
2023-03-23 17:28                     ` Andy Shevchenko
2023-03-23 17:53             ` Russell King (Oracle)
2023-03-23 18:04               ` Andy Shevchenko
2023-03-23 20:46                 ` Russell King (Oracle)
2023-03-22 12:00 ` [PATCH RFC net-next 4/7] net: dsa: add ability for switch driver to provide a swnode Russell King (Oracle)
2023-03-22 12:00 ` [PATCH RFC net-next 5/7] net: dsa: avoid DT validation for drivers which provide default config Russell King (Oracle)
2023-03-22 18:51   ` Andrew Lunn
2023-03-22 20:09     ` Russell King (Oracle)
2023-03-22 20:14       ` Andrew Lunn
2023-03-22 20:20         ` Russell King (Oracle)
2023-03-22 12:00 ` [PATCH RFC net-next 6/7] net: dsa: mv88e6xxx: provide software node for default settings Russell King (Oracle)
2023-03-22 18:57   ` Andrew Lunn
2023-03-22 20:13     ` Russell King (Oracle)
2023-03-22 20:17       ` Andrew Lunn
2023-03-22 20:22         ` Russell King (Oracle)
2023-03-22 21:40           ` Andrew Lunn
2023-03-23  8:41             ` Russell King (Oracle)
2023-03-23 18:17               ` Andrew Lunn
2023-03-23 18:25                 ` Russell King (Oracle)
2023-03-23 18:34                   ` Andrew Lunn
2023-03-24 14:49   ` Heikki Krogerus
2023-03-24 17:04     ` Russell King (Oracle)
2023-03-27 10:28       ` Heikki Krogerus
2023-03-27 10:55         ` Russell King (Oracle)
2023-03-27 14:13           ` Heikki Krogerus
2023-03-27 14:32             ` Russell King (Oracle)
2023-03-27 15:45               ` Russell King (Oracle)
2023-03-28 12:09                 ` Heikki Krogerus
2023-03-28 13:23                   ` Russell King (Oracle)
2023-03-29 14:07                     ` Heikki Krogerus
2023-03-29 14:33                       ` Russell King (Oracle)
2023-03-30 13:54                         ` Heikki Krogerus
2023-04-03 13:02                           ` Russell King (Oracle)
2023-04-05 17:51                             ` Greg Kroah-Hartman
2023-03-22 12:00 ` [PATCH RFC net-next 7/7] net: dsa: mv88e6xxx: remove handling for DSA and CPU ports Russell King (Oracle)

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=ZBrtqPW29NnxVoEc@shell.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=andrew@lunn.ch \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=davem@davemloft.net \
    --cc=djrscally@gmail.com \
    --cc=edumazet@google.com \
    --cc=f.fainelli@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=heikki.krogerus@linux.intel.com \
    --cc=hkallweit1@gmail.com \
    --cc=kuba@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=olteanv@gmail.com \
    --cc=pabeni@redhat.com \
    --cc=rafael@kernel.org \
    --cc=sakari.ailus@linux.intel.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).