Add sub address support to SPI driver enabling multiple devices to be be accessed using the minimum of GPIO #221
pelwellcheckpatch.yml
on: pull_request
checkpatch review
1m 52s
Annotations
10 errors
|
checkpatch review:
include/linux/spi/spi.h#L476
WARNING: Block comments should align the * on each line
|
|
checkpatch review
ERROR: Missing Signed-off-by: line(s)
|
|
checkpatch review:
drivers/spi/spi.c#L639
ERROR: that open brace { should be on the previous line
|
|
checkpatch review:
drivers/spi/spi.c#L639
ERROR: space required before the open parenthesis '('
|
|
checkpatch review:
drivers/spi/spi.c#L640
WARNING: please, no spaces at the start of a line
|
|
checkpatch review:
drivers/spi/spi.c#L642
WARNING: please, no spaces at the start of a line
|
|
checkpatch review:
drivers/spi/spi.c#L687
ERROR: that open brace { should be on the previous line
|
|
checkpatch review:
drivers/spi/spi.c#L687
WARNING: suspect code indent for conditional statements (8, 8)
|
|
checkpatch review:
drivers/spi/spi.c#L687
ERROR: space required before the open parenthesis '('
|
|
checkpatch review:
drivers/spi/spi.c#L689
ERROR: code indent should use tabs where possible
|