[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20210311123721.00005913@intel.com>
Date: Thu, 11 Mar 2021 12:37:21 -0800
From: Jesse Brandeburg <jesse.brandeburg@...el.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: Alexander Duyck <alexander.duyck@...il.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"oss-drivers@...ronome.com" <oss-drivers@...ronome.com>,
"simon.horman@...ronome.com" <simon.horman@...ronome.com>,
"yisen.zhuang@...wei.com" <yisen.zhuang@...wei.com>,
"salil.mehta@...wei.com" <salil.mehta@...wei.com>,
"intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>,
"Nguyen, Anthony L" <anthony.l.nguyen@...el.com>,
"drivers@...sando.io" <drivers@...sando.io>,
"snelson@...sando.io" <snelson@...sando.io>,
"netanel@...zon.com" <netanel@...zon.com>,
"akiyano@...zon.com" <akiyano@...zon.com>,
"gtzalik@...zon.com" <gtzalik@...zon.com>,
"saeedb@...zon.com" <saeedb@...zon.com>,
"GR-Linux-NIC-Dev@...vell.com" <GR-Linux-NIC-Dev@...vell.com>,
"skalluru@...vell.com" <skalluru@...vell.com>,
"rmody@...vell.com" <rmody@...vell.com>,
"kys@...rosoft.com" <kys@...rosoft.com>,
"haiyangz@...rosoft.com" <haiyangz@...rosoft.com>,
"sthemmin@...rosoft.com" <sthemmin@...rosoft.com>,
"wei.liu@...nel.org" <wei.liu@...nel.org>,
"mst@...hat.com" <mst@...hat.com>,
"jasowang@...hat.com" <jasowang@...hat.com>,
"pv-drivers@...are.com" <pv-drivers@...are.com>,
"doshir@...are.com" <doshir@...are.com>,
"alexanderduyck@...com" <alexanderduyck@...com>
Subject: Re: [RFC PATCH 01/10] ethtool: Add common function for filling out
strings
Jakub Kicinski wrote:
> On Wed, 10 Mar 2021 17:35:17 -0800 Alexander Duyck wrote:
> > From: Alexander Duyck <alexanderduyck@...com>
> > +/**
> > + * ethtool_gsprintf - Write formatted string to ethtool string data
> > + * @data: Pointer to start of string to update
> > + * @fmt: Format of string to write
> > + *
> > + * Write formatted string to data. Update data to point at start of
> > + * next string.
> > + */
> > +extern __printf(2, 3) void ethtool_gsprintf(u8 **data, const char *fmt, ...);
>
> I'd drop the 'g' TBH, it seems to have made its way from the ethtool
> command ('gstrings') to various places but without the 'string' after
> it - it becomes less and less meaningful. Just ethtool_sprintf() would
> be fine IMHO.
>
> Other than that there is a minor rev xmas tree violation in patch 2 :)
I agree with Jakub, and I really like this series, it's a good clean up.
I'll be glad to add a reviewed by tag to v2.
Powered by blists - more mailing lists