[Intel-wired-lan] [PATCH] ethtool: stop the line wrapping madness
jeffrey.t.kirsher at intel.com
Tue May 15 17:47:48 UTC 2018
On Tue, 2018-05-15 at 19:37 +0200, Paul Menzel wrote:
> On 05/15/18 19:32, Mitch Williams wrote:
> > Folks, we have a hard limit of 80 characters per line in the
> > mostly due to Linus' insistence on printing out each release on
> > with his Decwriter. So why do we have function and macro names that
> > over 30 characters long? Add a tab or two and a few parameters and
> > 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'
> > 'ks' and elided some unnecessary verbiage. In nearly every instance
> > unwrapped lines and made the code easier to read, especially on a
> Nice, let’s see what the others think.
> > In the case of the Amazon Ethernet driver, I found a bug where they
> > setting bits in the 'settings' field twice. Almost certainly this
> > was supposed to set bits in the 'advertising' field instead. So I
> > it.
> Please split this out into a separate patch.
Actually, I dusted the cobwebs off Mitch's original patch and in this
version of the patch, the Amazon driver was not fixed. So I need to
fix up the patch description to remove the reference to the fix and I
will put together a patch to fix the Amazon driver.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 833 bytes
Desc: This is a digitally signed message part
More information about the Intel-wired-lan