[Intel-wired-lan] [PATCH] ethtool: stop the line wrapping madness
pmenzel at molgen.mpg.de
Tue May 15 17:37:30 UTC 2018
On 05/15/18 19:32, Mitch Williams wrote:
> Folks, we have a hard limit of 80 characters per line in the kernel,
> mostly due to Linus' insistence on printing out each release on greenbar
> with his Decwriter. So why do we have function and macro names that are
> over 30 characters long? Add a tab or two and a few parameters and boom!
> you're wrapping lines.
> This patch is a search-n-replace of the newly-added ethtool link
> settings API with shorter names. In general, I replaced 'ksettings' with
> 'ks' and elided some unnecessary verbiage. In nearly every instance I
> unwrapped lines and made the code easier to read, especially on a VT102.
Nice, let’s see what the others think.
> In the case of the Amazon Ethernet driver, I found a bug where they were
> setting bits in the 'settings' field twice. Almost certainly this
> was supposed to set bits in the 'advertising' field instead. So I fixed
Please split this out into a separate patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 5174 bytes
Desc: S/MIME Cryptographic Signature
More information about the Intel-wired-lan