- 17 Mar, 2014 9 commits
-
-
Monam Agarwal authored
This patch fixes following smatch warning: /drivers/staging/ft1000/ft1000-usb/ft1000_hw.c:1148 ft1000_proc_drvmsg() warn: returning -1 instead of -ENOMEM is sloppy Signed-off-by:
Monam Agarwal <monamagarwal123@gmail.com> Acked-by:
Nick Kossifidis <mickflemm@gmail.com> Signed-off-by:
Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
-
Ana Rey authored
Fix the following warnings from sparse in drivers/staging/rts5139/sd_cprm.c due to the missing include of sd_cprm.h Signed-off-by:
Ana Rey <anarey@gmail.com> Acked-by:
Paul E. McKenney <paulmck@linux.vnet.ibm.com> Signed-off-by:
-
Ana Rey authored
Make the local variable rts5139_usb_ids static Fix the following sparse warnings in rts51x.c CHECK drivers/staging/rts5139/rts51x.c drivers/staging/rts5139/rts51x.c:835:22: warning: symbol 'rts5139_usb_ids' was not declared. Should it be static? Signed-off-by:
-
Ana Rey authored
Fix the following warnings from sparse in drivers/staging/rts5139/ms_mg.c due to the missing include of ms_mg.h drivers/staging/rts5139/ms_mg.c:81:5: warning: symbol 'mg_set_tpc_para_sub' was not declared. Should it be static? drivers/staging/rts5139/ms_mg.c:122:5: warning: symbol 'rts51x_mg_set_leaf_id' was not declared. Should it be static? drivers/staging/rts5139/ms_mg.c:173:5: warning: symbol 'rts51x_mg_get_local_EKB' was not declared. Should it be static? drivers/staging/rts5139/ms_mg.c:232:5: warning: symbol 'rts51x_mg_chg' was not declared. Should it be static? drivers/staging/rts5139/ms_mg.c:323:5: warning: symbol 'rts51x_mg_get_rsp_chg' was not declared. Should it be static? drivers/staging/rts5139/ms_mg.c:392:5: warning: symbol 'rts51x_mg_rsp' was not declared. Should it be static? drivers/staging/rts5139/ms_mg.c:450:5: warning: symbol 'rts51x_mg_get_ICV' was not declared. Should it be static? drivers/staging/rts5139/ms_mg.c:514:5: warning: symbol 'rts51x_mg_set_ICV' was not declared. Should it be static? Signed-off-by:
-
Ana Rey authored
Make the local variable (media_not_present and invalid_cmd_field) static Fix the following sparse warnings in rts51x_transport.c drivers/staging/rts5139/rts51x_transport.c:649:4: warning: symbol 'media_not_present' was not declared. Should it be static? drivers/staging/rts5139/rts51x_transport.c:651:4: warning: symbol 'invalid_cmd_field' was not declared. Should it be static? Signed-off-by:
-
Ana Rey authored
Make some local functions (eprom_cs, eprom_ck_cycle, eprom_w, eprom_r and eprom_send_bits_string) static Fix the following sparse warnings in r8180_93cx6.c drivers/staging/rtl8192u/r8180_93cx6.c:23:6: warning: symbol 'eprom_cs' was not declared. Should it be static? drivers/staging/rtl8192u/r8180_93cx6.c:40:6: warning: symbol 'eprom_ck_cycle' was not declared. Should it be static? drivers/staging/rtl8192u/r8180_93cx6.c:56:6: warning: symbol 'eprom_w' was not declared. Should it be static? drivers/staging/rtl8192u/r8180_93cx6.c:71:7: warning: symbol 'eprom_r' was not declared. Should it be static? drivers/staging/rtl8192u/r8180_93cx6.c:85:6: warning: symbol 'eprom_send_bits_string' was not declared. Should it be static? Signed-off-by:
-
Ana Rey authored
Make some local functions static. The local functions are CamResetAllEntry, tx_timeout, rtl8192_data_hard_stop, rtl8192_data_hard_resume, rtl8192_hard_data_xmit, rtl8192_hard_start_xmit, rtl8192_config_rate, rtl8192_update_cap, rtl8192_net_update, MRateToHwRate8190Pci, tl8192_link_change, rtl8192_update_beacon, WDCAPARA_ADD, rtl8192_qos_activate, GetHalfNmodeSupportByAPs819xUsb, rtl8192_refresh_supportrate, rtl8192_getSupportedWireleeMode, rtl8192_SetWirelessMode, rtl8192_get_channel_map, rtl8192_init, rtl8192_hwconfig, rtl8192_adapter_start, TxCheckStuck, rtl819x_ifcheck_resetornot, CamRestoreAllEntry, rtl819x_ifsilentreset, rtl819x_update_rxcounts, rtl8192_open, r8192_set_mac_adr, rtl8192_ioctl, HwRateToMRate90, rtl819x_translate_todbm, rtl8192_process_phyinfo, rtl8192_record_rxdesc_forlateruse, rtl8192_rx_nomal, rtl819xusb_process_received_packet, rtl8192_rx_cmd. Fix the following sparse warnings in r8192U_core.c drivers/staging/rtl8192u/r8192U_core.c:241:6: warning: symbol 'CamResetAllEntry' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:676:6: warning: symbol 'tx_timeout' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1014:6: warning: symbol 'rtl8192_data_hard_stop' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1020:6: warning: symbol 'rtl8192_data_hard_resume' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1028:6: warning: symbol 'rtl8192_hard_data_xmit' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1056:5: warning: symbol 'rtl8192_hard_start_xmit' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1380:6: warning: symbol 'rtl8192_config_rate' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1427:6: warning: symbol 'rtl8192_update_cap' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1449:6: warning: symbol 'rtl8192_net_update' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:1677:4: warning: symbol 'MRateToHwRate8190Pci' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2043:6: warning: symbol 'rtl8192_link_change' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2075:6: warning: symbol 'rtl8192_update_beacon' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2090:5: warning: symbol 'WDCAPARA_ADD' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2091:6: warning: symbol 'rtl8192_qos_activate' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2319:6: warning: symbol 'GetHalfNmodeSupportByAPs819xUsb' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2333:6: warning: symbol 'rtl8192_refresh_supportrate' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2344:4: warning: symbol 'rtl8192_getSupportedWireleeMode' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2363:6: warning: symbol 'rtl8192_SetWirelessMode' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2783:7: warning: symbol 'rtl8192_get_channel_map' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2796:7: warning: symbol 'rtl8192_init' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2844:6: warning: symbol 'rtl8192_hwconfig' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:2927:6: warning: symbol 'rtl8192_adapter_start' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3183:12: warning: symbol 'TxCheckStuck' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3286:12: warning: symbol 'rtl819x_ifcheck_resetornot' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3325:6: warning: symbol 'CamRestoreAllEntry' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3401:6: warning: symbol 'rtl819x_ifsilentreset' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3521:6: warning: symbol 'rtl819x_update_rxcounts' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3638:5: warning: symbol 'rtl8192_open' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3758:5: warning: symbol 'r8192_set_mac_adr' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3774:5: warning: symbol 'rtl8192_ioctl' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3865:4: warning: symbol 'HwRateToMRate90' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3951:6: warning: symbol 'rtl819x_translate_todbm' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:3967:6: warning: symbol 'rtl8192_process_phyinfo' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:4453:6: warning: symbol 'rtl8192_record_rxdesc_forlateruse' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:4725:6: warning: symbol 'rtl8192_rx_nomal' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:4875:6: warning: symbol 'rtl819xusb_process_received_packet' was not declared. Should it be static? drivers/staging/rtl8192u/r8192U_core.c:4936:6: warning: symbol 'rtl8192_rx_cmd' was not declared. Should it be static? Signed-off-by:
-
Aybuke Ozdemir authored
Fix checkpatch.pl issues with line over 80 characters in solo6x10-v4l2-enc.c Signed-off-by:
Aybuke Ozdemir <aybuke.147@gmail.com> Signed-off-by:
-
Iulia Manda authored
This patch fixes the following sparse warnings: drivers/staging/vt6656/usbpipe.c:147:24: warning: context imbalance in 'PIPEnsControlOut' - unexpected unlock drivers/staging/vt6656/usbpipe.c:209:24: warning: context imbalance in 'PIPEnsControlIn' - unexpected unlock Signed-off-by:
Iulia Manda <iulia.manda21@gmail.com> Signed-off-by:
-
- 16 Mar, 2014 1 commit
-
-
Gulsah Kose authored
Removed unnecessary line continuations in slicoss.c Signed-off-by:
Gulsah Kose <gulsah.1004@gmail.com> Acked-by:
-
- 14 Mar, 2014 10 commits
-
-
Chi Pham authored
Assignments removed from if statements. Fixed checkpatch warning such as indentation and negative error returns in adjacent code. Coccinelle was used for this patch. The following script found the match: @simple@ expression E1, E2; statement S1, S2; @@ + E1 = E2; if ( - (E1 = E2) + E1 ) S1 else S2 @left@ expression E0, E1, E2; statement S1, S2; @@ + E1 = E2; if ( - (E1 = E2) + E1 == E0 ) S1 else S2 Signed-off-by:Chi Pham <fempsci@gmail.com> Signed-off-by:
-
Andreea-Cristina Bernat authored
This patch adds fallthrough comments for the cases not preceded by break or fallthrough comment Signed-off-by:
Andreea-Cristina Bernat <bernat.ada@gmail.com> Acked-by:
-
Himangi Saraogi authored
This patch fixes the following Sparse warnings in console.c : drivers/staging/lustre/lnet/selftest/console.c:65:1: warning: symbol 'lstcon_node_get' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:118:1: warning: symbol 'lstcon_node_put' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:348:1: warning: symbol 'lstcon_sesrpc_condition' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:377:1: warning: symbol 'lstcon_sesrpc_readent' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:834:1: warning: symbol 'lstcon_batch_find' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1002:1: warning: symbol 'lstcon_batrpc_condition' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1145:1: warning: symbol 'lstcon_testrpc_condition' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1374:1: warning: symbol 'lstcon_test_find' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1389:1: warning: symbol 'lstcon_tsbrpc_readent' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1468:1: warning: symbol 'lstcon_statrpc_readent' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1492:1: warning: symbol 'lstcon_ndlist_stat' was not declared. Should it be static? drivers/staging/lustre/lnet/selftest/console.c:1581:1: warning: symbol 'lstcon_debug_ndlist' was not declared. Should it be static? Signed-off-by:
Himangi Saraogi <himangi774@gmail.com> Acked-by:
-
Gulsah Kose authored
Return is not a method and doesn't need parentheses. Removed unnecaasary parentheses. Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following sparse warnings: drivers/staging/comedi/drivers/mite.c:343:44: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:343:44: expected unsigned int [unsigned] [usertype] count drivers/staging/comedi/drivers/mite.c:343:44: got restricted __le32 [usertype] <noident> drivers/staging/comedi/drivers/mite.c:344:43: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:344:43: expected unsigned int [unsigned] [usertype] addr drivers/staging/comedi/drivers/mite.c:344:43: got restricted __le32 [usertype] <noident> drivers/staging/comedi/drivers/mite.c:346:43: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:346:43: expected unsigned int [unsigned] [usertype] next drivers/staging/comedi/drivers/mite.c:346:43: got restricted __le32 [usertype] <noident> drivers/staging/comedi/drivers/mite.c:351:45: warning: incorrect type in assignment (different base types) drivers/staging/comedi/drivers/mite.c:351:45: expected unsigned int [unsigned] [usertype] next drivers/staging/comedi/drivers/mite.c:351:45: got restricted __le32 [usertype] <noident> Signed-off-by:
-
Gulsah Kose authored
Fix checkpatch.pl issues with line over 80 characters in rtsx.c Signed-off-by:
-
Andreea-Cristina Bernat authored
This patch solves the warning: "space prohibited before semicolon". Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following sparse warning: drivers/staging/rtl8187se/r8180_core.c:1328:40: warning: right shift by bigger than source value by adding parenthesis. Signed-off-by:
Bob Copeland <me@bobcopeland.com> Signed-off-by:
-
Andreea-Cristina Bernat authored
This patch solves the error of checkpatch.pl by deleting the extra paranthesis. Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the following sparse warnings: drivers/staging/rtl8187se/ieee80211/ieee80211_crypt_tkip.c:496:17: warning: cast to restricted __le16 drivers/staging/rtl8187se/ieee80211/ieee80211_crypt_tkip.c:540:12: warning: cast to restricted __le16 drivers/staging/rtl8187se/ieee80211/ieee80211_crypt_tkip.c:586:12: warning: cast to restricted __le16 Signed-off-by:
-
- 13 Mar, 2014 20 commits
-
-
Elena Oat authored
This patch fixes following warnings produced by sparse: drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3867:31: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3867:31: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3869:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3869:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3869:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3871:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3871:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3871:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3873:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3873:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3873:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3875:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3875:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3875:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3878:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3878:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3878:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3880:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3880:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3880:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3883:38: warning: incorrect type in argument 1 (different address spaces) drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3883:38: expected void const *<noident> drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3883:38: got void [noderef] <asn:1>*pointer drivers/staging/rtl8188eu/os_dep/ioctl_linux.c:3897:9: warning: cast removes address space of expression While at it, I have also fixed the warnings of lines over 80 characters. Signed-off-by:
Elena Oat <oat.elena@gmail.com> Signed-off-by:
-
Gulsah Kose authored
Remove unnecessary parentheses from rtl871x_cmd.c Signed-off-by:
-
Iulia Manda authored
Fix coccinelle warnings of better allocation by using kzalloc. In these cases, kzalloc is preferred, as kmalloc may fail if it does not find contiguous memory. Signed-off-by:
Luis R. Rodriguez <mcgrof@do-not-panic.com> Signed-off-by:
-
Iulia Manda authored
Fix the following coccinelle warnings in ion.c: drivers/staging/android/ion/ion.c:511:9-16: WARNING: ERR_CAST can be used with buffer drivers/staging/android/ion/ion.c:218:9-16: WARNING: ERR_CAST can be used with table drivers/staging/android/ion/ion.c:1150:9-16: WARNING: ERR_CAST can be used with dmabuf Signed-off-by:
-
Iulia Manda authored
It is preferred to use seq_puts instead of seq_printf here, as it suffices string printing. Signed-off-by:
-
Iulia Manda authored
Join strings from two separated lines, even if this makes line longer than 80 characters. Signed-off-by:
-
Kristina Martšenko authored
BA_para_set (of type __le16) is implicitly cast into an integer for bitwise operations, after which the result is converted back into a __le16. Make the initial cast explicit to make the code clearer. In addition, refactor the code for readability. This also removes five sparse warnings of the following type: drivers/staging/rtl8188eu//core/rtw_mlme_ext.c:6114:70: warning: restricted __le16 degrades to integer Signed-off-by:
Kristina Martšenko <kristina.martsenko@gmail.com> Signed-off-by:
-
Andreea-Cristina Bernat authored
This patch fixes the error "space required after that ';'" for the file /drivers/staging/media/solo6x10/solo6x10-tx28.c Signed-off-by:
-
Ashley Smith authored
This patch fixes the checkpatch warning "that open brace { should be on the previous line" for a non-function statement block. Signed-off-by:Ashley Smith <ashley@eclipso.ch> Acked-by:
-
Iulia Manda authored
Align test parts in if statement on separate lines. Signed-off-by:
-
Himangi Saraogi authored
The functions musycc_del_chan and musycc_del_chan_stats are not being used at the moment and may not be required. Remove them for now (with #if 0) till it is decided whether it is really required. Signed-off-by:
-
Elena Oat authored
This patch fixes the warnings produced by checkpatch "unchecked sscanf return value" in file ioctl_linux.c. Signed-off-by:
-
Ana Rey authored
This patch fixes braces errors as the codingStyle of the kernel recommends. Thus, this patch fixes these errors and warning messages found by checkpatch.pl: WARNING: braces {} are not necessary for single statement blocks ERROR: that open brace { should be on the previous line Signed-off-by: -
Ana Rey authored
This patch fixes whitespaces as the codingStyle of the kernel recommends. Thus, this patch fixes these errors found by checkpatch.pl: ERROR: space required before the open parenthesis '(' ERROR: space required before that '*' (ctx:OxV) ERROR: spaces required around that '=' (ctx:VxW) Signed-off-by: -
Ana Rey authored
After the next warning messages by sparse tools: drivers/staging/rtl8187se/r8180_rtl8225z2.c:609:6: warning: symbol 'rtl8225z2_rf_set_mode' was not declared. Should it be static? drivers/staging/rtl8187se/r8180_rtl8225z2.c:609:13: warning: ‘rtl8225z2_rf_set_mode’ defined but not used [-Wunused-function] static void rtl8225z2_rf_set_mode(struct net_device *dev) And the next search: ~/opw/dev/staging/drivers/staging/rtl8187se (staging-next) $ git grep rtl8225z2_rf_set_mode r8180_rtl8225z2.c:void rtl8225z2_rf_set_mode(struct net_device *dev) I decided to delete rtl8225z2_rf_set_mode function. Signed-off-by:
-
Ana Rey authored
This patch deletes all commented code in r8180.h that is not necessary. Signed-off-by:
-
Ana Rey authored
The Documentation/CodingStyle doesn't recommend the use of typedef, convert this into an enum. While at it, I have also renamed the variable names that were used in this typedef not to use Hungarian notation. Signed-off-by:
-
Ana Rey authored
The typedef _LED_CTL_MODE (and the enum constants) is not used in the driver. I delete it. Signed-off-by:
-
Himangi Saraogi authored
This patch fixes the sparse warning - drivers/staging/wlan-ng/p80211netdev.c:289:38: warning: cast to restricted __le16 by eliminating the variable fc as it is not required. Signed-off-by: -
Ana Rey authored
The Documentation/CodingStyle doesn't recommend the use of typedef, convert this to an enum. While at it, I have also renamed the variable names that were used in this typedef not to use Hungarian notation. Signed-off-by:
-
