Vasanthakumar Thiagarajan <vthiagar@xxxxxxxxxxxxxxxx> writes: >> This added new ath10k-check warnings, I fixed those in the pending >> branch: >> >> drivers/net/wireless/ath/ath10k/debug.h:203: Alignment should match open parenthesis >> drivers/net/wireless/ath/ath10k/debug.h:226: Alignment should match open parenthesis >> drivers/net/wireless/ath/ath10k/debugfs_sta.c:24: Alignment should >> match open parenthesis >> drivers/net/wireless/ath/ath10k/debugfs_sta.c:40: Alignment should >> match open parenthesis >> drivers/net/wireless/ath/ath10k/debugfs_sta.c:60: Alignment should >> match open parenthesis >> >> Also I fixed the typo "rages" in debug.h. My changes: > > Thanks for fixing it. In most of the cases, fixing above warnings > would result in lines > 80 chars. > May be we need not be very strict about > 80 char limit? In ath10k-check I set the limit to 90 chars, but I guess it could be even higher than that. But on the other it shouldn't be too long, not everyone like to have one huge window for editor :) -- Kalle Valo
