On Sun, May 21, 2017 at 01:03:55AM +0530, Ranjit W wrote: > fixing following checkpatch warning wherever possible: > WARNING: line over 80 characters > > Signed-off-by: Ranjit W <programlinuxstuff@xxxxxxxxx> > --- > drivers/staging/wlan-ng/cfg80211.c | 11 ++++++++--- > drivers/staging/wlan-ng/hfa384x.h | 21 ++++++++++++++------- > drivers/staging/wlan-ng/p80211netdev.c | 3 ++- > 3 files changed, 24 insertions(+), 11 deletions(-) > > diff --git a/drivers/staging/wlan-ng/cfg80211.c b/drivers/staging/wlan-ng/cfg80211.c > index 178f6f5d4613..8915c9240d1c 100644 > --- a/drivers/staging/wlan-ng/cfg80211.c > +++ b/drivers/staging/wlan-ng/cfg80211.c > @@ -163,7 +163,8 @@ static int prism2_add_key(struct wiphy *wiphy, struct net_device *dev, > goto exit; > > /* send key to driver */ > - did = DIDmib_dot11smt_dot11WEPDefaultKeysTable_key(key_index + 1); > + did = DIDmib_dot11smt_dot11WEPDefaultKeysTable_key(key_index > + + 1); Ick, that looks horrible now, don't you think? checkpatch is a guide, not a rule-that-must-always-be-followed. There are other ways to fix this up, look at that horrible function name for one such way to resolve it. thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
