Hi David,
On Wed, Nov 13, 2013 at 9:13 PM, David Miller <davem@xxxxxxxxxxxxx> wrote:
On Wed, Nov 13, 2013 at 12:16 AM, David Miller <davem@xxxxxxxxxxxxx> 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.
Sorry, I didn't get that message. I was puzzled by your request to correct
the whitespace of the lines I touched only, which causes them to stand out
in a block of old non-compliant whitespace.
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.
Nevermind, it's not urgent nor critical. I will resubmit for the next merge
window.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
