[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20131113.151301.100218641838867631.davem@davemloft.net>
Date: Wed, 13 Nov 2013 15:13:01 -0500 (EST)
From: David Miller <davem@...emloft.net>
To: geert@...ux-m68k.org
Cc: sergei.shtylyov@...entembedded.com, netdev@...r.kernel.org,
linux-m68k@...r.kernel.org
Subject: Re: [PATCH] net/7990: Make lance_private.name const
From: Geert Uytterhoeven <geert@...ux-m68k.org>
Date: Wed, 13 Nov 2013 09:54:18 +0100
> On Wed, Nov 13, 2013 at 12:16 AM, David Miller <davem@...emloft.net> wrote:
>>>> And remember about checkpatch.pl which was hardly content with the patch.
>>>
>>> Only because checkpatch looks at the _new_ lines, and doesn't compare the
>>> styles of the old and new lines.
>>
>> You really should fix the line you are changing to use tabs, please respin
>> with this in mind, thanks.
>
> As that would add more to the TAB/space mess^H^H^Hix, I prepended a
> patch to fix all whitespace errors.
I implicitly asked you not to do this, now things are more difficult
and your original change will take longer to integrate.
Becuase now it isn't a patch set I can easily just apply quickly in
the current merge window, and it's therefore a series you'll have to
resubmit later when the merge window closes and the net-next tree
opens up again.
Sorry.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists