Thread (1 message) 1 message, 1 author, 2011-04-28

Re: [PATCHv3 4/7] ethtool: cosmetic: Use ethtool ethtool_cmd_speed API

From: Ben Hutchings <hidden>
Date: 2011-04-28 02:12:59
Also in: lkml

On Wed, 2011-04-27 at 18:27 -0700, David Decotigny wrote:
This updates the network drivers so that they don't access the
ethtool_cmd::speed field directly, but use ethtoo_cmd_speed() instead.

For most drivers, these changes are purely cosmetic and don't
fix any problem, such as for those 1GbE/10GbE drivers that indirectly
call their own ethtool get_settings()/mii_ethtool_gset(). The changes
are meant to enforce code consistency and provide robustness with
future larger throughputs, at the expense of a few CPU cycles for each
ethtool operation.

All the drivers compiled with make allyesconfig ion x86_64 have been
updated.

Tested: make allyesconfig on x86_64 + e1000e/bnx2x work
Signed-off-by: David Decotigny <redacted>
Reviewed-by: Ben Hutchings <redacted>

-- 
Ben Hutchings, Senior Software Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help