On Fri, Mar 20, 2015 at 07:26:29AM +0100, Martin Sperl wrote: > > On 20.03.2015, at 05:58, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote: > > Do you have a list of the warnings this fixes and why? The extra > > wrapping here looks a bit odd, since the lines before your patch are > > well under 80 columns. > scripts/checkpatch.pl --file --terse --strict drivers/spi/spi-bcm2835.c > drivers/spi/spi-bcm2835.c:182: CHECK: Alignment should match open parenthesis This is telling you to add more indentation to the second argument not wrap the first. Though frankly I don't think it matters - it's certainly not a routinely followed thing and it frequently makes things worse.
Attachment:
signature.asc
Description: Digital signature
