- 08 Mar, 2014 4 commits
-
-
Ana Rey authored
This patch fixes whitespaces as the codingStyle of the kernel recommends. Thus, this patch fixes these errors found by checkpatch.pl in r8180.h ERROR: space required after that close brace '}' ERROR: space required after that ',' (ctx:VxV) ERROR: spaces required around that '==' (ctx:VxV) ERROR: spaces required around that '=' (ctx:VxV) ERROR: space required after that ',' (ctx:VxV) Signed-off-by:
Ana Rey <anarey@gmail.com> Acked-by:
Bob Copeland <me@bobcopeland.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Raluca Oncioiu authored
Remove brackets from single line branches of if statements. Break long lines. Signed-off-by:
Raluca Oncioiu <raluca.oncioiu91@gmail.com> Acked-by:
Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by:
-
Raluca Oncioiu authored
Break up long lines. Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following sparse warnings in ieee80211_rx.c: drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:2201:34: warning: cast to restricted __le32 drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:2202:34: warning: cast to restricted __le32 drivers/staging/rtl8192u/ieee80211/ieee80211_rx.c:2203:36: warning: cast to restricted __le32 by changing the annotation in the structure header file for struct ieee80211_probe_response for time_stamp and altering the cast for beacon_interval to le16. Signed-off-by:
Himangi Saraogi <himangi774@gmail.com> Reviewed-by:
-
- 07 Mar, 2014 36 commits
-
-
Himangi Saraogi authored
This patch fixes the following sparse warnings: drivers/staging/rtl8712/rtl871x_cmd.c:471:30: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_cmd.c:471:30: expected unsigned int [unsigned] [usertype] Length drivers/staging/rtl8712/rtl871x_cmd.c:471:30: got restricted __le32 [usertype] <noident> drivers/staging/rtl8712/rtl871x_cmd.c:472:32: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_cmd.c:472:32: expected unsigned int [unsigned] [usertype] IELength drivers/staging/rtl8712/rtl871x_cmd.c:472:32: got restricted __le32 [usertype] <noident> drivers/staging/rtl8712/rtl871x_cmd.c:473:39: warning: incorrect type in assignment (different base types) drivers/staging/rtl8712/rtl871x_cmd.c:473:39: expected unsigned int [unsigned] [usertype] SsidLength drivers/staging/rtl8712/rtl871x_cmd.c:473:39: got restricted __le32 [usertype] <noident> Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the checkpatch.pl warning: line over 80 charcters. Signed-off-by:
Sarah Sharp <sarah.a.sharp@linux.intel.com> Signed-off-by:
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with __packed is preferred over __attribute__((packed)) in ft1000_ioctl.h Signed-off-by:
Ebru Akagunduz <ebru.akagunduz@gmail.com> Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warnings: drivers/staging/gdm72xx/gdm_wimax.c:543:37: warning: incorrect type in argument 1 (different address spaces) drivers/staging/gdm72xx/gdm_wimax.c:543:37: expected void [noderef] <asn:1>*to drivers/staging/gdm72xx/gdm_wimax.c:543:37: got void *buf drivers/staging/gdm72xx/gdm_wimax.c:566:41: warning: incorrect type in argument 2 (different address spaces) drivers/staging/gdm72xx/gdm_wimax.c:566:41: expected void const [noderef] <asn:1>*from drivers/staging/gdm72xx/gdm_wimax.c:566:41: got void *buf Signed-off-by:
Monam Agarwal <monamagarwal123@gmail.com> Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following warning for gdm_wimax.c WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: -
Ebru Akagunduz authored
Fix checkpatch.pl issues with line over 80 characters in wb35rx.c Signed-off-by:
-
Himangi Saraogi authored
This patch moves shared private data kzalloc to managed devm_kzalloc and cleans now unneccessary kfree in probe and remove functions. Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following Sparse warnings : drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:27:4: warning: symbol 'rsn_authen_cipher_suite' was not declared. Should it be static? drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:352:6: warning: symbol 'ext_ieee80211_send_beacon_wq' was not declared. Should it be static? Signed-off-by:
-
Himangi Saraogi authored
This patch fixes sparse warnings like : (1) drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:331:31: warning: incorrect type in assignment (different base types) drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:331:31: expected unsigned short [unsigned] [usertype] frame_ctl drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:331:31: got restricted __le16 [usertype] <noident> (2) drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:1091:32: warning: incorrect type in assignment (different base types) drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:1091:32: expected restricted __le16 [usertype] duration_id drivers/staging/rtl8187se/ieee80211/ieee80211_softmac.c:1091:32: got int Signed-off-by:
-
Keerthimai Janarthanan authored
Unnecessary space between function pointer name. Signed-off-by:
Keerthimai Janarthanan <keerthimaipb@gmail.com> Acked-by:
-
Monam Agarwal authored
This patch fixes the following sparse warning in rts51x.c drivers/staging/rts5139/rts51x.c:218:5: warning: symbol 'rts51x_pre_reset' was not declared. Should it be static? drivers/staging/rts5139/rts51x.c:229:5: warning: symbol 'rts51x_post_reset' was not declared. Should it be static? Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following Sparse warnings : formatter_inquiry_str should be static queuecommand_lck should be static Signed-off-by:
-
Georgiana Rodica Chelu authored
This patch fixes the quoted string split across lines in staging/rts5139/ms.c Signed-off-by:
Georgiana Rodica Chelu <georgiana.chelu93@gmail.com> Acked-by:
-
Georgiana Rodica Chelu authored
Remove unnecessary parentheses from staging/rts5208/rtsx_scsi.c Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warning in rts51x_fop.c Warning: incorrect type in assignment Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following sparse warning : drivers/staging/serqt_usb2/serqt_usb2.c:727:40: warning: restricted __le16 degrades to integer Signed-off-by:
-
Himangi Saraogi authored
This patch moves shared private data kzalloc to managed devm_kzalloc. There are no kfree statements associated with this data, so using devm adds the possibility of the data being freed, if the proble function fails (if that is possible) or if the remove function succeeds. Signed-off-by:
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with quoted string split across lines in nvec.c Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warning in crystalhd_lnx.c WARNING: incorrect type in argument Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warning in crystalhd_misc.c WARNING: incorrect type in argument Signed-off-by:
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with "foo * bar" should be "foo *bar" in dgap.h Signed-off-by:
-
Ebru Akagunduz authored
Fix checkpatch.pl issues with open brace '{' following struct go on the same line in dgap.h Signed-off-by: -
Ebru Akagunduz authored
This patch fixes the Sparse Warnings "symbol was not declared. Should it be static?" Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warnings: drivers/staging/wlags49_h2/wl_wext.c:3357:28: warning: incorrect type in assignment (different address spaces) drivers/staging/wlags49_h2/wl_wext.c:3357:28: expected void [noderef] <asn:1>*[addressable] [assigned] pointer drivers/staging/wlags49_h2/wl_wext.c:3357:28: got void *<noident> drivers/staging/wlags49_h2/wl_wext.c:3422:47: warning: incorrect type in assignment (different address spaces) drivers/staging/wlags49_h2/wl_wext.c:3422:47: expected void [noderef] <asn:1>*[addressable] [assigned] pointer drivers/staging/wlags49_h2/wl_wext.c:3422:47: got char *[usertype] <noident> drivers/staging/wlags49_h2/wl_wext.c:3781:21: warning: symbol 'wl_priv_args' was not declared. Should it be static? Signed-off-by:
-
Monam Agarwal authored
This patch fixes the following sparse warnings: drivers/staging/wlags49_h2/dhf.c:109:6: warning: symbol 'signature' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:126:25: warning: symbol 'mfi_sup' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:127:25: warning: symbol 'cfi_sup' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:142:17: warning: symbol 'ltv_info' was not declared. Should it be static? drivers/staging/wlags49_h2/dhf.c:173:1: warning: symbol 'check_comp_fw' was not declared. Should it be static? Signed-off-by:
-
Monam Agarwal authored
This patch fixes following sparse warnings: drivers/staging/wlags49_h2/hcf.c:253:26: warning: symbol 'cfg_drv_identity' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:262:24: warning: symbol 'cfg_drv_sup_range' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:274:31: warning: symbol 'cfg_drv_act_ranges_pri' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:291:31: warning: symbol 'cfg_drv_act_ranges_sta' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:336:31: warning: symbol 'cfg_drv_act_ranges_hsi' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:373:24: warning: symbol 'cfg_drv_act_ranges_apf' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:3103:1: warning: symbol 'calc_mic' was not declared. Should it be static? drivers/staging/wlags49_h2/hcf.c:3419:1: warning: symbol 'check_mic' was not declared. Should it be static? Signed-off-by:
-
Monam Agarwal authored
This patch removes the unused function "percent" from wl_util.c Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the checkpatch warnings: WARNING: braces {} are not necessary for any arm of this statement Signed-off-by: -
Himangi Saraogi authored
This patch fixes the following checkpatch errors: ERROR: space prohibited after that open parenthesis '(' ERROR: space prohibited before that close parenthesis ')' Signed-off-by: -
Himangi Saraogi authored
This patch fixes the following warnings: WARNING: that open brace { should be on the previous line WARNING: suspect code indent for conditional statements (16, 16) Signed-off-by: -
Himangi Saraogi authored
This patch fixes the following issue detected by checkpatch.pl. WARNING: Use #include <linux/uaccess.h> instead of <asm/uaccess.h> Signed-off-by:
-
Himangi Saraogi authored
Coding style requires that comments use the standard /* */ style instead of C99 style. The commented #include statements were also removed as they are not required. Signed-off-by:
-
Monam Agarwal authored
This patch fixes sparse warnings "Using plain integer as NULL pointer" in sta_h2.c Signed-off-by:
-
Monam Agarwal authored
Staging: iio/accel: Fix Prefer netdev_warn(netdev, ...then dev_warn(dev, ... then pr_warn(... to printk(KERN_INFO ... in sca3000_ring.c This patch fixes the following checkpatch.pl warning in sca3000_ring.c WARNING: Prefer netdev_info(netdev, ... then dev_info(dev, ... then pr_info(... to printk(KERN_INFO ... Signed-off-by:
-
Monam Agarwal authored
This patch fixes following checkpatch.pl warning WARNING: Prefer pr_warn(... to pr_warning(... Signed-off-by:
-
Elena Oat authored
This patch fixes the warning of "Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2)" in file rtw_cmd.c. Pahole shows that the addresses are 2-byte aligned. Signed-off-by:
Elena Oat <oat.elena@gmail.com> Signed-off-by:
-
