The subsystem prefix isn't right. It should be" [PATCH] Staging: wlan-ng: Fixed line more than 80 characters
On Wed, Jan 30, 2019 at 05:11:27PM +0100, Sandesh Kenjana Ashok wrote: > Lines over 80 characters are adjusted according to standards > > Signed-off-by: Sandesh Kenjana Ashok <kas.sand...@gmail.com> > --- > drivers/staging/wlan-ng/cfg80211.c | 17 +++++++++-------- > 1 file changed, 9 insertions(+), 8 deletions(-) > > diff --git a/drivers/staging/wlan-ng/cfg80211.c > b/drivers/staging/wlan-ng/cfg80211.c > index e5d7def1f366..7687f619ee34 100644 > --- a/drivers/staging/wlan-ng/cfg80211.c > +++ b/drivers/staging/wlan-ng/cfg80211.c > @@ -373,7 +373,8 @@ static int prism2_scan(struct wiphy *wiphy, > msg2.beaconperiod.data, > ie_buf, > ie_len, > - (msg2.signal.data - 65536) * 100, /* Conversion to > signed type */ > + /*Conversion to signed type*/ Put the spaces back: /* Conversion to signed type */ > + (msg2.signal.data - 65536) * 100, > GFP_KERNEL > ); > > @@ -410,7 +411,7 @@ static int prism2_set_wiphy_params(struct wiphy *wiphy, > u32 changed) > data = wiphy->rts_threshold; > > result = prism2_domibset_uint32(wlandev, > - > DIDMIB_DOT11MAC_OPERATIONTABLE_RTSTHRESHOLD, > + DIDMIB_DOT11MAC_OPERATIONTABLE_RTSTHRESHOLD, > data); No. The original code is nicer to look at. regards, dan carpenter _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel