Subject: Re: [PATCH net-next v2 3/3] net: txgbe: add FDIR info to ethtool ops

From: Jakub Kicinski <[email-masked]>
Date: Wed, 5 Jun 2024 17:52:54 -0700

On Wed, 5 Jun 2024 07:33:32 +0000 Hariprasad Kelam wrote:
> > +		if (wx->mac.type == wx_mac_sp)
> > +			return WX_STATS_LEN + WX_FDIR_STATS_LEN;
> >  		return WX_STATS_LEN;  
> 
>              Better way is to use ternary operator.
>                  Return (wx->mac.type == wx_mac_sp) ? WX_STATS_LEN + WX_FDIR_STATS_LEN : WX_STATS_LEN;

I'd leave it be. We prefer lines to be shorter than 80 chars.