On Sun, Jul 23, 2017 at 03:10:53PM +0300, Dan Carpenter wrote: > On Sun, Jul 23, 2017 at 10:47:43AM +0200, Johan Hovold wrote: > > On Sun, Jul 23, 2017 at 02:09:57PM +0530, Shreeya Patel wrote: > > > Alignment should match with open parenthesis. > > > This fixes the coding style issue. > > > > No, it need not. This is a choice left up to the author, even if > > checkpatch happens to complain about it with the --strict option. > > > > This is staging code. Don't try to fight checkpatch.pl in staging... > You're just making work for everyone else (me) who already just reviewed > this patch and I don't want to review it a few more times when the next > person sends the exact same patch. You can just ignore any greybus patches if you prefer. I've considered patching checkpatch to drop --strict for at least this subsystem, but not sure it's worth it. Johan _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
