Messages in this thread | | | Subject | Re: [PATCH net-next v8 05/19] net: ethtool: add new ETHTOOL_GSETTINGS/SSETTINGS API | From | Ben Hutchings <> | Date | Sat, 13 Feb 2016 01:49:09 +0000 |
| |
On Tue, 2016-02-09 at 16:29 -0800, David Decotigny wrote: > From: David Decotigny <decot@googlers.com> > > This patch defines a new ETHTOOL_GSETTINGS/SSETTINGS API, handled by > the new get_ksettings/set_ksettings callbacks. This API provides > support for most legacy ethtool_cmd fields, adds support for larger > link mode masks (up to 4064 bits, variable length), and removes > ethtool_cmd deprecated fields (transceiver/maxrxpkt/maxtxpkt). [...]
I previously asked you to include 'link' in the command names and structure name. This would clarify that these are now only for link settings and reduce the risk of confusion between old and new commands. However, you didn't reply to that review. Do you have any objection to doing this?
Ben.
-- Ben Hutchings Sturgeon's Law: Ninety percent of everything is crap.[unhandled content-type:application/pgp-signature] | |