All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: Colin Foster <colin.foster@in-advantage.com>
Cc: Vladimir Oltean <vladimir.oltean@nxp.com>,
	netdev@vger.kernel.org, "David S. Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Claudiu Manoil <claudiu.manoil@nxp.com>,
	Alexandre Belloni <alexandre.belloni@bootlin.com>,
	UNGLinuxDriver@microchip.com, Andrew Lunn <andrew@lunn.ch>,
	Florian Fainelli <f.fainelli@gmail.com>
Subject: Re: [PATCH net-next 0/8] Probing cleanup for the Felix DSA driver
Date: Fri, 31 May 2024 08:47:59 +0100	[thread overview]
Message-ID: <ZlmAry5lo/Mrrlhl@shell.armlinux.org.uk> (raw)
In-Reply-To: <Zlk/GmpxUq/iOqs4@colin-ia-desktop>

On Thu, May 30, 2024 at 10:08:10PM -0500, Colin Foster wrote:
> Hi Vladimir,
> 
> On Thu, May 30, 2024 at 07:33:25PM +0300, Vladimir Oltean wrote:
> > This is a follow-up to Russell King's request for code consolidation
> > among felix_vsc9959, seville_vsc9953 and ocelot_ext, stated here:
> > https://lore.kernel.org/all/Zh1GvcOTXqb7CpQt@shell.armlinux.org.uk/
> > 
> > Details are in individual patches. Testing was done on NXP LS1028A
> > (felix_vsc9959).
> > 
> > Vladimir Oltean (8):
> >   net: dsa: ocelot: use devres in ocelot_ext_probe()
> >   net: dsa: ocelot: use devres in seville_probe()
> >   net: dsa: ocelot: delete open coded status = "disabled" parsing
> >   net: dsa: ocelot: consistently use devres in felix_pci_probe()
> >   net: dsa: ocelot: move devm_request_threaded_irq() to felix_setup()
> >   net: dsa: ocelot: use ds->num_tx_queues = OCELOT_NUM_TC for all models
> >   net: dsa: ocelot: common probing code
> >   net: dsa: ocelot: unexport felix_phylink_mac_ops and felix_switch_ops
> > 
> >  drivers/net/dsa/ocelot/felix.c           |  62 ++++++++++++-
> >  drivers/net/dsa/ocelot/felix.h           |  10 +-
> >  drivers/net/dsa/ocelot/felix_vsc9959.c   | 113 +++++++----------------
> >  drivers/net/dsa/ocelot/ocelot_ext.c      |  55 +----------
> 
> Just FYI I tried testing this but hit an unrelated regression in 6.10,
> and a `git b4` on 6.9.3 has conflicts. So I'm still alive, but probably
> won't get to testing this tonight. Looks good though.

You will need... the net-next tree because it's dependent on a patch I
submitted earlier this week.

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

  reply	other threads:[~2024-05-31  7:48 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-30 16:33 [PATCH net-next 0/8] Probing cleanup for the Felix DSA driver Vladimir Oltean
2024-05-30 16:33 ` [PATCH net-next 1/8] net: dsa: ocelot: use devres in ocelot_ext_probe() Vladimir Oltean
2024-05-30 16:40   ` Colin Foster
2024-06-03  2:00   ` Colin Foster
2024-06-03 11:01     ` Vladimir Oltean
2024-05-30 16:33 ` [PATCH net-next 2/8] net: dsa: ocelot: use devres in seville_probe() Vladimir Oltean
2024-05-30 16:33 ` [PATCH net-next 3/8] net: dsa: ocelot: delete open coded status = "disabled" parsing Vladimir Oltean
2024-05-30 16:33 ` [PATCH net-next 4/8] net: dsa: ocelot: consistently use devres in felix_pci_probe() Vladimir Oltean
2024-05-30 16:33 ` [PATCH net-next 5/8] net: dsa: ocelot: move devm_request_threaded_irq() to felix_setup() Vladimir Oltean
2024-05-30 16:33 ` [PATCH net-next 6/8] net: dsa: ocelot: use ds->num_tx_queues = OCELOT_NUM_TC for all models Vladimir Oltean
2024-05-30 16:47   ` Colin Foster
2024-06-03  2:02   ` Colin Foster
2024-05-30 16:33 ` [PATCH net-next 7/8] net: dsa: ocelot: common probing code Vladimir Oltean
2024-06-03  2:04   ` Colin Foster
2024-05-30 16:33 ` [PATCH net-next 8/8] net: dsa: ocelot: unexport felix_phylink_mac_ops and felix_switch_ops Vladimir Oltean
2024-05-30 18:12   ` Sai Krishna Gajula
2024-05-31  3:08 ` [PATCH net-next 0/8] Probing cleanup for the Felix DSA driver Colin Foster
2024-05-31  7:47   ` Russell King (Oracle) [this message]
2024-06-03 12:10 ` patchwork-bot+netdevbpf

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=ZlmAry5lo/Mrrlhl@shell.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=UNGLinuxDriver@microchip.com \
    --cc=alexandre.belloni@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=claudiu.manoil@nxp.com \
    --cc=colin.foster@in-advantage.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=f.fainelli@gmail.com \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.