Re: [PATCH v3 0/2] Fix subject line

From: Dan Carpenter
Date: Thu Jun 12 2014 - 17:31:06 EST


On Thu, Jun 12, 2014 at 01:48:38PM -0700, Davidlohr Bueso wrote:
> On Thu, 2014-06-12 at 13:35 -0700, Greg Kroah-Hartman wrote:
> > On Thu, Jun 12, 2014 at 01:25:34PM -0700, Davidlohr Bueso wrote:
> > > On Thu, 2014-06-12 at 23:40 +0400, Wahib Faizi wrote:
> > > > Fix coding style issues detected by checkpatch.pl in usbip_host_driver.c.
> > >
> > > Sorry but unless bundled with something more meaningful, I really don't
> > > see the value in these changes. I certainly don't want to discourage
> > > folks or anything, but just testing other patches is a lot more helpful
> > > than this.
> >
> > When the staging code is still needing basic fixes like this, it is
> > "meaningful" to do patches that clean up stuff like this. That's what
> > the staging tree is for.
>
> Sure, but "making checkpatch happy just to make checkpatch happy" isn't
> a good justification, even for staging.

It actually is. Fighting against checkpatch is a losers battle. Our
way more efficient.

1) We do need to fix this checkpatch warning before it moves out of
staging.
2) NAKing patches is actually a lot of stress for everyone. It stresses
out submitters and drives them away. It stresses out the
maintainers because we feel bad. That stress is bad when it is
pointless.
3) This patch is ok.
4) If we don't apply this patch then someone else will send the exact
same patch or something worse until we apply something.
5) The v3 of this patch takes under 30 seconds to review and apply.
6) Newbies feel happy when their patch gets merged and that is good.

The other thing is that if you start asking "Is this patch meaningful"
then it makes applying any patch into a big question about meaning and
it tires you out.

In staging we have clear rules for when a patch is going to be applied
and everyone understands the rules. It means that if Greg is traveling
then you know which patches he is going to apply in what order and life
is easier because it is more predictable.

regards,
dan carpenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/