Hi Viresh, So does greybus not follow the coding style from checkpatch.pl ? On 07/26/2017 10:32 PM, Viresh Kumar wrote: > On 26-07-17, 17:13, Deb McLemore wrote: >> checkpatch.pl line over 80 characters so fix the formatting >> for coding style compliance. >> >> Signed-off-by: Deb McLemore <debmc@xxxxxxxxxxxxxxxxxx> >> --- >> drivers/staging/greybus/spilib.h | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/staging/greybus/spilib.h b/drivers/staging/greybus/spilib.h >> index 566d0dd..cb60925 100644 >> --- a/drivers/staging/greybus/spilib.h >> +++ b/drivers/staging/greybus/spilib.h >> @@ -18,7 +18,8 @@ struct spilib_ops { >> void (*unprepare_transfer_hardware)(struct device *dev); >> }; >> >> -int gb_spilib_master_init(struct gb_connection *connection, struct device *dev, struct spilib_ops *ops); >> +int gb_spilib_master_init(struct gb_connection *connection, >> + struct device *dev, struct spilib_ops *ops); > > It was intentionally written so and you will find tons of examples of > such declarations in the .h files specially. > _______________________________________________ greybus-dev mailing list greybus-dev@xxxxxxxxxxxxxxxx https://lists.linaro.org/mailman/listinfo/greybus-dev