Re: [PATCH] x86: boot: Fix mixed indentation in a20.c

[Ingo Molnar <mingo@kernel.org>]

* 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