On Fri, Mar 10, 2023 at 11:56:59PM +0500, Khadija Kamran wrote: > On Sat, Mar 11, 2023 at 12:05:54AM +0530, Deepak R Varma wrote: > > On Fri, Mar 10, 2023 at 10:09:47PM +0500, Khadija Kamran wrote: > > > Length of line 182 exceeds 100 columns in file > > > drivers/staging/grebus/arche-platform.c, fix by removing tabs from the > > > line. > > > > Hi Khadija, > > I think if you also include merging the if condition and the call to > > spin_unlock...() on single lines, it should make the code more human. > Hi Deepak! Sorry I am unable to understand how to merge the if condition > and spin_unlock...() together. Can you please elaborate. > Thank you Hah.. my bad. The if condition is split on two lines. Join them to form a single line if evaluation. Similarly, join the spin_un..() call that is split on two lines into a single line. Hope that clarifies it. Deepak. > _______________________________________________ greybus-dev mailing list -- greybus-dev@xxxxxxxxxxxxxxxx To unsubscribe send an email to greybus-dev-leave@xxxxxxxxxxxxxxxx
|