lkml.org 
[lkml]   [2013]   [Nov]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] x86: boot: Fix mixed indentation in a20.c

* Joe Perches <joe@perches.com> wrote:

> On Tue, 2013-11-19 at 10:24 -0800, H. Peter Anvin wrote:
>
> > I'll take a proper cleanup,
> > but just whitespace... just pointless churn.
>
> What in your view would be "a proper cleanup" for a20.c?

If a newbie does a meaningful, complete, well done cleanup patch then
congratulations and any such help is welcome.

If you as a more experienced kernel developer do a cleanup as part of
some real work then sure, all such cleanups are welcome and they are a
natural part of development work.

So a standalone cleanup patch to a20.c from _you_ would probably not
qualify, almost by definition: your first patch was applied 7 years
ago, you are by far not a newbie anymore, yet you seem to be mostly
stuck on the 'cleanups and trivialities' level! Sheesh!

My message to the buerocrat Joe Perches is: please leave trivial and
printk patches to newbies, you need to raise to the next level of
kernel development already.

FYI, Linux is a meritocracy, not a bureaucracy: creating self-serving
churn and attention-seeking but unimportant patches is not the way to
gain kernel development credibility long term, and eventually people
start protecting against your increasing abuse of the development
process. IMHO.

Thanks,

Ingo


\
 
 \ /
  Last update: 2013-11-25 16:01    [W:0.232 / U:0.800 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site