[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <63942260-AF46-475D-9963-8A7A10484906@gmail.com>
Date: Wed, 2 Mar 2016 04:48:28 +0100
From: John Holland <jotihojr@...il.com>
To: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
Cc: "Brown, Aaron F" <aaron.f.brown@...el.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>
Subject: Re: [Intel-wired-lan] [next] igb: allow setting MAC address on i211 using a device tree blob V5
> On 02.03.2016, at 00:55, Jeff Kirsher <jeffrey.t.kirsher@...el.com> wrote:
>
>> On Tue, 2016-03-01 at 07:56 +0100, John Holland wrote:
>> On Mar 1, 2016, at 03:52, Brown, Aaron F <aaron.f.brown@...el.com>
>> wrote:
>>
>>> This throws a few checkpatch warnings, but I won't withhold my
>> tested by for these:
>>>
>>> total: 0 errors, 2 warnings, 0 checks, 21 lines checked
>>>
>>> Your patch has style problems, please review.
>>>
>>> NOTE: If any of the errors are false positives, please report
>>> them to the maintainer, see CHECKPATCH in MAINTAINERS.
>>> u1463:[0]/usr/src/kernels/next-queue>
>>
>> Thanks for testing...
>>
>> Do you require me to reformat the patch text? And won't that break
>> the link?
>
> No, that checkpatch.pl warning on the link you provided is fine,
> although I will clean up your signed-off-by when I go to push it
> upstream. :-)
Thank you very much!
Powered by blists - more mailing lists