Re: [PATCH net-next] net: marvell: prestera: fix FEC error message for SFP ports

[Jakub Kicinski <kuba@kernel.org>]

On Mon, 2 Feb 2026 11:26:00 +0100 Jonas Gorski wrote:
> On 02/02/2026 06:28, Alok Tiwari wrote:
> > In prestera_ethtool_set_fecparam(), the error message is opposite of
> > the condition checking PRESTERA_PORT_TCVR_SFP. FEC configuration is
> > not allowed on SFP ports, but the message says "non-SFP ports", which
> > does not match the condition.
> >
> > diff --git a/drivers/net/ethernet/marvell/prestera/prestera_ethtool.c b/drivers/net/ethernet/marvell/prestera/prestera_ethtool.c
> > index 2f52daba58e6..a259da9f30f4 100644
> > --- a/drivers/net/ethernet/marvell/prestera/prestera_ethtool.c
> > +++ b/drivers/net/ethernet/marvell/prestera/prestera_ethtool.c
> > @@ -718,7 +718,7 @@ static int prestera_ethtool_set_fecparam(struct net_device *dev,
> >  	}
> >  
> >  	if (port->caps.transceiver == PRESTERA_PORT_TCVR_SFP) {
> > -		netdev_err(dev, "FEC set is not allowed on non-SFP ports\n");
> > +		netdev_err(dev, "FEC set is not allowed on SFP ports\n");
> >  		return -EINVAL;
> >  	}
> >    
> 
> I'm pretty sure that the condition here is wrong, not the error message.
> 
> AFAIK it doesn't make sense to configure FEC on ethernet/copper ports,
> but for SFP ports, it may be required depending on the transceiver or
> cable used, especially for fiber modules.
> 
> I may be wrong though.
> 
> This check was introduced with bb5dbf2cc64d ("net: marvell: prestera:
> add firmware v4.0 support"), but that commit does not offer any
> explanation for it in the commit message.

Elad, could you clarify this? A bit hard to tell whether CR is included
in "SFP" from glancing at the code.