On Mon, Dec 11, 2017 at 03:16:25PM +0300, Igor Russkikh wrote: > Signed-off-by: Igor Russkikh <igor.russk...@aquantia.com> > --- > drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c > b/drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c > index 70efb74..f2d8063 100644 > --- a/drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c > +++ b/drivers/net/ethernet/aquantia/atlantic/aq_ethtool.c > @@ -66,14 +66,14 @@ static const char > aq_ethtool_stat_names[][ETH_GSTRING_LEN] = { > "OutUCast", > "OutMCast", > "OutBCast", > - "InUCastOctects", > - "OutUCastOctects", > - "InMCastOctects", > - "OutMCastOctects", > - "InBCastOctects", > - "OutBCastOctects", > - "InOctects", > - "OutOctects", > + "InUCastOctets", > + "OutUCastOctets", > + "InMCastOctets", > + "OutMCastOctets", > + "InBCastOctets", > + "OutBCastOctets", > + "InOctets", > + "OutOctets", > "InPacketsDma", > "OutPacketsDma", > "InOctetsDma",
Hi Igor I know it is not nice, but maybe we need to consider this is part of the kernel ABI? You could be breaking applications by making this change. What do others think? Andrew