diff mbox series

[RFC,net-next,v2,2/3] net: dsa: microchip: Add SGMII port support to KSZ9477 switch

Message ID 20250207024316.25334-3-Tristram.Ha@microchip.com (mailing list archive)
State Superseded
Delegated to: Netdev Maintainers
Headers show
Series Add SGMII port support to KSZ9477 switch | expand

Checks

Context Check Description
netdev/tree_selection success Clearly marked for net-next
netdev/apply fail Patch does not apply to net-next-1

Commit Message

Tristram.Ha@microchip.com Feb. 7, 2025, 2:43 a.m. UTC
From: Tristram Ha <tristram.ha@microchip.com>

The KSZ9477 DSA driver uses XPCS driver to operate its SGMII port.

Signed-off-by: Tristram Ha <tristram.ha@microchip.com>
---
v2
 - update Kconfig to pass compilation test

 drivers/net/dsa/microchip/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Vladimir Oltean Feb. 7, 2025, 4:52 p.m. UTC | #1
On Thu, Feb 06, 2025 at 06:43:15PM -0800, Tristram.Ha@microchip.com wrote:
> From: Tristram Ha <tristram.ha@microchip.com>
> 
> The KSZ9477 DSA driver uses XPCS driver to operate its SGMII port.
> 
> Signed-off-by: Tristram Ha <tristram.ha@microchip.com>
> ---
> v2
>  - update Kconfig to pass compilation test
> 
>  drivers/net/dsa/microchip/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/net/dsa/microchip/Kconfig b/drivers/net/dsa/microchip/Kconfig
> index 12a86585a77f..c71d3fd5dfeb 100644
> --- a/drivers/net/dsa/microchip/Kconfig
> +++ b/drivers/net/dsa/microchip/Kconfig
> @@ -6,6 +6,7 @@ menuconfig NET_DSA_MICROCHIP_KSZ_COMMON
>  	select NET_DSA_TAG_NONE
>  	select NET_IEEE8021Q_HELPERS
>  	select DCB
> +	select PCS_XPCS
>  	help
>  	  This driver adds support for Microchip KSZ8, KSZ9 and
>  	  LAN937X series switch chips, being KSZ8863/8873,
> -- 
> 2.34.1
> 

I'm not sure if you split this change intentionally or by mistake, but
either way, you need to squash this patch into 3/3.
Tristram.Ha@microchip.com Feb. 7, 2025, 7:03 p.m. UTC | #2
> -----Original Message-----
> From: Vladimir Oltean <olteanv@gmail.com>
> Sent: Friday, February 7, 2025 8:52 AM
> To: Tristram Ha - C24268 <Tristram.Ha@microchip.com>
> Cc: Russell King <linux@armlinux.org.uk>; Woojung Huh - C21699
> <Woojung.Huh@microchip.com>; Andrew Lunn <andrew@lunn.ch>; Heiner Kallweit
> <hkallweit1@gmail.com>; Maxime Chevallier <maxime.chevallier@bootlin.com>;
> David S. Miller <davem@davemloft.net>; Eric Dumazet <edumazet@google.com>;
> Jakub Kicinski <kuba@kernel.org>; Paolo Abeni <pabeni@redhat.com>;
> UNGLinuxDriver <UNGLinuxDriver@microchip.com>; netdev@vger.kernel.org; linux-
> kernel@vger.kernel.org
> Subject: Re: [PATCH RFC net-next v2 2/3] net: dsa: microchip: Add SGMII port support
> to KSZ9477 switch
> 
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content
> is safe
> 
> On Thu, Feb 06, 2025 at 06:43:15PM -0800, Tristram.Ha@microchip.com wrote:
> > From: Tristram Ha <tristram.ha@microchip.com>
> >
> > The KSZ9477 DSA driver uses XPCS driver to operate its SGMII port.
> >
> > Signed-off-by: Tristram Ha <tristram.ha@microchip.com>
> > ---
> > v2
> >  - update Kconfig to pass compilation test
> >
> >  drivers/net/dsa/microchip/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/net/dsa/microchip/Kconfig b/drivers/net/dsa/microchip/Kconfig
> > index 12a86585a77f..c71d3fd5dfeb 100644
> > --- a/drivers/net/dsa/microchip/Kconfig
> > +++ b/drivers/net/dsa/microchip/Kconfig
> > @@ -6,6 +6,7 @@ menuconfig NET_DSA_MICROCHIP_KSZ_COMMON
> >       select NET_DSA_TAG_NONE
> >       select NET_IEEE8021Q_HELPERS
> >       select DCB
> > +     select PCS_XPCS
> >       help
> >         This driver adds support for Microchip KSZ8, KSZ9 and
> >         LAN937X series switch chips, being KSZ8863/8873,
> > --
> > 2.34.1
> >
> 
> I'm not sure if you split this change intentionally or by mistake, but
> either way, you need to squash this patch into 3/3.

Will update that.
diff mbox series

Patch

diff --git a/drivers/net/dsa/microchip/Kconfig b/drivers/net/dsa/microchip/Kconfig
index 12a86585a77f..c71d3fd5dfeb 100644
--- a/drivers/net/dsa/microchip/Kconfig
+++ b/drivers/net/dsa/microchip/Kconfig
@@ -6,6 +6,7 @@  menuconfig NET_DSA_MICROCHIP_KSZ_COMMON
 	select NET_DSA_TAG_NONE
 	select NET_IEEE8021Q_HELPERS
 	select DCB
+	select PCS_XPCS
 	help
 	  This driver adds support for Microchip KSZ8, KSZ9 and
 	  LAN937X series switch chips, being KSZ8863/8873,