- 25 Apr, 2014 40 commits
-
-
Masanari Iida authored
Fix spelling typo in comments within rtl8192e. Signed-off-by:
Masanari Iida <standby24x7@gmail.com> Reviewed-by:
James Cameron <quozl@laptop.org> Signed-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Masanari Iida authored
A word in printk is split into two lines. Change it into one line. Signed-off-by:
-
navin patidar authored
Signed-off-by:
navin patidar <navin.patidar@gmail.com> Signed-off-by:
-
navin patidar authored
Signed-off-by:
-
navin patidar authored
Signed-off-by:
-
navin patidar authored
struct led_priv contains two variables SwLed0 and SwLed1 but only SwLed0 is being used by SwLedControlModel1() function. SwLedControlModel1() function performs led operations. This patch removes SwLed1 and code which uses SwLed1. Signed-off-by:
-
navin patidar authored
Now driver has only one type of led blinking strategy, so we don't need enum LED_STRATEGY_871x variable to store led blinking strategy type. Signed-off-by:
-
navin patidar authored
driver has code to blink the led in 7 different ways. struct led_priv contains an enum LED_STRATEGY_871x variable which is initialized to SW_LED_MODE1 inside rtl8188eu_InitSwLeds() function. it means driver is hard coded to use only SW_LED_MODE1 blinking strategy. so we can remove the code related to other blinking strategies e.g. SW_LED_MODE[0|2|3|4|5|6]. Signed-off-by:
-
navin patidar authored
_InitHWLed() function does nothing. Signed-off-by:
-
navin patidar authored
_ReadLEDSetting() doesn't read led settings this function actually initialize member variables of struct led_priv, we should do that inside rtl8188eu_InitSwLeds(). Signed-off-by:
-
navin patidar authored
rtw_led_blink_cmd() isn't being used by driver. Signed-off-by:
-
navin patidar authored
drv_types_linux.h is empty, so remove the header file and its reference. Signed-off-by:
-
Konrad Zapalowicz authored
This commit fixes the following checkpatch errors: rtl8188eu/core/rtw_recv.c - 1874: ERROR: code indent should use tabs where possible Signed-off-by:Konrad Zapalowicz <bergo.torino@gmail.com> Signed-off-by:
-
Konrad Zapalowicz authored
This commit fixes the following checkpatch errors: rtl8188eu/core/rtw_p2p.c - 1508: ERROR: code indent should use tabs where possible - 1509: ERROR: code indent should use tabs where possible - 1510: ERROR: code indent should use tabs where possible Signed-off-by: -
Mark Einon authored
Fixes all checkpatch errors reporting that statics shouldn't be initialised to their defualt values. Signed-off-by:
Mark Einon <mark@einon.net> Signed-off-by:
-
Mark Einon authored
Fixes checkpatch error: ERROR: switch and case should be at the same indent + switch (coex_dm->cur_retrylimit_type) { + case 0: /* normal mode */ [...] + case 1: /* retry limit=8 */ [...] + default: Signed-off-by: -
Mark Einon authored
Fix checkpatch issues relating to using leading spaces instead of tabs. Whitespace changes only. Signed-off-by:
-
Mark Einon authored
Whitespace changes only to appease checkpatch warnings. Signed-off-by:
-
Mark Einon authored
We set btrssi to zero and then immediately assign it. Assign it on declaration instead. Signed-off-by:
-
Mark Einon authored
Sparse complains about halbtc8192e2ant_low_penalty_ra() not being used, and subsequently halbtc8192e2ant_set_sw_penalty_tx_rateadaptive() when this is removed. remove both functions. Signed-off-by:
-
Mark Einon authored
In halbtc8192e2ant.c, fix all instances of the sparse warning: warning: symbol 'xxx' was not declared. Should it be static? Signed-off-by:
-
Mark Einon authored
Remove functions not being called from anywhere: halbtc8192e2ant_set_adc_backoff() halbtc8192e2ant_adc_backoff() Signed-off-by:
-
Mark Einon authored
All code in this file is encapsulated in a useless #if 1 declaration, remove it. Signed-off-by:
-
Joel Pelaez Jorge authored
The two declarations with issues in rtl_wx.h: * function: struct iw_statistics *r8192_get_wireless_stats(); * variable: extern struct iw_handler_def r8192_wx_handlers_def; The symbol 'r8192_get_wireless_stats' is declared as 'extern' but it is only used in the local scope. The declaration is deleted of rtl_wx.h, fixing the conflict with the 'extern' and 'static' scope. The symbol 'r8192_wx_handlers_def' was fixed of some issues related to variable declaration marked as 'extern' implicitly, causing this sparse warning: drivers/staging/rtl8192e/rtl8192e/rtl_wx.c:1323:24: warning: symbol 'r8192_wx_handlers_def' was not declared. Should it be static? Resolved incluing the fixed header to rtl_wx.c Also, isn't declared as constant variable, so it declared as such. And delete unnecessary cast in rtl_core.c. Signed-off-by:
Joel Pelaez Jorge <joelpelaez@gmail.com> Signed-off-by:
-
Paul McQuade authored
ERROR: spaces required around that '=' (ctx:OxW) Signed-off-by:
Paul McQuade <paulmcquad@gmail.com> Signed-off-by:
-
Dan Carpenter authored
The ->SupportedRates[] array has NDIS_802_11_LENGTH_RATES_EX (16) elements. Since "ie_len" comes from then network and can go up to 255 then it means we should add a range check to prevent memory corruption. Fixes: d6846af6 ('staging: r8188eu: Add files for new driver - part 7') Signed-off-by:
Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by:
-
Loïc Pellegrino authored
Fix coding style problems (incorrect indentation). Signed-off-by:
Loïc Pellegrino <oort10@gmail.com> Signed-off-by:
-
huanqi chi authored
fix a coding style issue: Missing a blank line after declarations Signed-off-by:
huanqi chi <chihuanqi@gmail.com> Signed-off-by:
-
Daeseok Youn authored
The pci_get_drvdata() and checking NULL for dev are called twice in while loop in is_bypass_dev(). Signed-off-by:
Daeseok Youn <daeseok.youn@gmail.com> Signed-off-by:
-
Neil Armstrong authored
checkpatch script returns the following warning: ERROR: do not use assignment in if condition + if (((dev_num = get_dev_idx(dev->ifindex)) == -1) | This patch fixes the if condition by splitting it. Signed-off-by:Neil Armstrong <superna9999@gmail.com> Signed-off-by:
-
SeongJae Park authored
Some code is hide inside #ifdef BP_PROC_SUPPORT and it never defined anywhere. And, it made defined but not used function which calling code was hide inside #ifdef BP_PROC_SUPPORT and caused following build warning: drivers/staging/silicom/bpctl_mod.c:6786:12: warning: ‘bp_proc_create’ defined but not used [-Wunused-function] static int bp_proc_create(void) ^ Fix the warning and remove code complexity by remove whole code inside #ifdef BP_PROC_SUPPORT. Signed-off-by:SeongJae Park <sj38.park@gmail.com> Signed-off-by:
-
Ken Cox authored
Add x86_cpu_id struct and call to MODULE_DEVICE_TABLE so this module will be auto loaded on boot. Signed-off-by:
Ken Cox <jkc@redhat.com> Tested-by:
-
Benjamin Romer authored
This patch changes all of the various representations of the copyright symbol to (C). Signed-off-by:
Benjamin Romer <benjamin.romer@unisys.com> Signed-off-by:
-
Masanari Iida authored
Fix spelling typo in comments and printk within unisys. Signed-off-by:
-
Masanari Iida authored
Fix format string mismatch in info_proc_read() within virthba.c. Signed-off-by:
-
Rui Miguel Silva authored
fix sparse warning: drivers/staging/keucr/smilmain.c:163:5: warning: symbol 'Check_D_MediaFmt' was not declared. Should it be static? by declaring it in the right header. And remove duplication of definition of SM_INIT Signed-off-by:
Rui Miguel Silva <rmfrfs@gmail.com> Signed-off-by:
-
Jonas Hahnfeld authored
This patches fixes two warnings of checkpatch.pl, both of the type WARNING: Missing a blank line after declarations Signed-off-by:
Jonas Hahnfeld <hahnjo@hahnjo.de> Signed-off-by:
-
Gioh Kim authored
Because sg_table is cleared in sg_alloc_table via memset we don't need to use kzalloc to allocate sg_table. Signed-off-by:
Gioh Kim <gioh.kim@lge.com> Signed-off-by:
-
Bintian Wang authored
ram_console is replaced by pstore and pstore_ram drivers, and there is no code to use this head file, so remove it. Signed-off-by:
Bintian Wang <bintian.wang@huawei.com> Acked-by:
Colin Cross <ccross@android.com> Signed-off-by:
-
Fabio Falzoi authored
Fixed the following sparse complaints: drivers/staging/lustre/lustre/ptlrpc/../../lustre/ldlm/ldlm_pool.c:1039:61: warning: restricted gfp_t degrades to integer drivers/staging/lustre/lustre/ptlrpc/../../lustre/ldlm/ldlm_pool.c:1092:61: warning: restricted gfp_t degrades to integer drivers/staging/lustre/lustre/ptlrpc/../../lustre/ldlm/ldlm_pool.c:1136:58: warning: incorrect type in argument 2 (different base types) expected unsigned int [unsigned] gfp_mask got restricted gfp_t [usertype] gfp_mask drivers/staging/lustre/lustre/ptlrpc/../../lustre/ldlm/ldlm_pool.c:1142:34: warning: incorrect type in argument 3 (different base types) expected unsigned int [unsigned] gfp_mask got restricted gfp_t [usertype] gfp_mask drivers/staging/lustre/lustre/ptlrpc/../../lustre/ldlm/ldlm_pool.c:1147:58: warning: incorrect type in argument 2 (different base types) expected unsigned int [unsigned] gfp_mask got restricted gfp_t [usertype] gfp_mask drivers/staging/lustre/lustre/ptlrpc/../../lustre/ldlm/ldlm_pool.c:1153:34: warning: incorrect type in argument 3 (different base types) expected unsigned int [unsigned] gfp_mask got restricted gfp_t [usertype] gfp_mask Signed-off-by:
Fabio Falzoi <fabio.falzoi84@gmail.com> Signed-off-by:
-
