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-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
