On Tue, 2016-02-09 at 16:29 -0800, David Decotigny wrote: > From: David Decotigny <decot@xxxxxxxxxxxx> > > 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.
Attachment:
signature.asc
Description: This is a digitally signed message part
