- 27 Jul, 2021 22 commits
-
-
Martin Kaiser authored
The IntrMask array is set but never read. Remove it. Signed-off-by:
Martin Kaiser <martin@kaiser.cx> Link: https://lore.kernel.org/r/20210725155902.32433-12-martin@kaiser.cxSigned-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Martin Kaiser authored
RegTxPause and RegBcnCtrlVal from struct hal_data_8188e are set but never read. Remove them. Signed-off-by:
-
Martin Kaiser authored
pwrpriv->bFwCurrentInPSMode is initialized and updated but nobody reads its value. Remove this write-only component. Signed-off-by:
-
Martin Kaiser authored
This define can be removed. Signed-off-by:
-
Martin Kaiser authored
The hardware_type enum is not used. Remove it. Signed-off-by:
-
Martin Kaiser authored
Only RF_CHANGE_BY_PS is used by this driver, the other RF_CHANGE_BY_... defines can be removed. Signed-off-by:
-
Martin Kaiser authored
The rt_eeprom_type enum is not used. Remove it. Signed-off-by:
-
Martin Kaiser authored
The hal_intf_ps_func enum is not used. Remove it. Signed-off-by:
-
Martin Kaiser authored
The hwinfo and AutoLoadFail parameters can be obtained from struct adapter, there's no need to pass them as separate function parameters. Use memcpy instead of bytewise copy if we have to fall back to the hard-coded mac address. Signed-off-by:
-
Martin Kaiser authored
The "HAL layer variable" HW_VAR_TXPAUSE is not used in this driver. Remove its define and the code for setting and for reading a value. Signed-off-by:
-
Martin Kaiser authored
The "HAL layer variable" HW_VAR_MEDIA_STATUS1 is not used in this driver. Remove its define and the code for setting a value. Signed-off-by:
-
Martin Kaiser authored
Now that power cut mask is not used any more, we can also remove the defines for power cut masks. Signed-off-by:
-
Michael Straube authored
Remove braces from single line if blocks to clear checkpatch warnings. WARNING: braces {} are not necessary for single statement blocks Signed-off-by:Michael Straube <straube.linux@gmail.com> Link: https://lore.kernel.org/r/20210726092129.30334-1-straube.linux@gmail.comSigned-off-by:
-
Michael Straube authored
Remove unnecessary blank lines reported by checkpatch. Signed-off-by:
-
Fabio M. De Francesco authored
Remove the functions and variables from rtw_security.c that are no more necessary since the patch that replaces getcrc32() with crc32_le(). Signed-off-by:
Fabio M. De Francesco <fmdefrancesco@gmail.com> Link: https://lore.kernel.org/r/20210723192620.10669-3-fmdefrancesco@gmail.comSigned-off-by:
-
Fabio M. De Francesco authored
Use crc32_le() in place of the custom getcrc32(). Signed-off-by:
-
Len Baker authored
strcpy() performs no bounds checking on the destination buffer. This could result in linear overflows beyond the end of the buffer, leading to all kinds of misbehaviors. The safe replacement is strscpy(). Signed-off-by:
Len Baker <len.baker@gmx.com> Link: https://lore.kernel.org/r/20210717152032.10164-1-len.baker@gmx.comSigned-off-by:
-
Len Baker authored
strcpy() performs no bounds checking on the destination buffer. This could result in linear overflows beyond the end of the buffer, leading to all kinds of misbehaviors. The safe replacement is strscpy(). Signed-off-by:
-
Christophe JAILLET authored
'sleep_status' has 3 atomic_t members. Initialize the 3 of them instead of initializing only 2 of them and setting 0 twice to the same variable. Signed-off-by:
Christophe JAILLET <christophe.jaillet@wanadoo.fr> Link: https://lore.kernel.org/r/d2e52a33a9beab41879551d0ae2fdfc99970adab.1626856991.git.christophe.jaillet@wanadoo.frSigned-off-by:
-
Jignesh Patel authored
Break lines into multiple lines to respect 100 character width limit. Reported by checkpatch.pl Signed-off-by:
Jignesh Patel <jigs0101@gmail.com> Link: https://lore.kernel.org/r/20210723094524.8811-1-jigs0101@gmail.comSigned-off-by:
-
Benjamin Philip authored
This commit fixes some checkpatch checks for blank lines before and after braces in drivers/staging/rtl8188eu/core/rtw_ap.c Signed-off-by:
Benjamin Philip <benjamin.philip495@gmail.com> Link: https://lore.kernel.org/r/20210722064451.10331-1-benjamin.philip495@gmail.comSigned-off-by:
-
Fabio M. De Francesco authored
Remove unused iw_operation_mode array. Signed-off-by:
-
- 23 Jul, 2021 17 commits
-
-
Fabio Aiuto authored
fix camel case in struct ndis_802_11_wep all over the driver. Signed-off-by:
Fabio Aiuto <fabioaiuto83@gmail.com> Link: https://lore.kernel.org/r/d3a1407e1bcb07d423dfa0c9bd33021a59bc0b61.1626874164.git.fabioaiuto83@gmail.comSigned-off-by:
-
Fabio Aiuto authored
remove unused struct ndis_801_11_ai_resfi. Signed-off-by:
-
Fabio Aiuto authored
remove unused struct ndis_802_11_ai_reqfi. Signed-off-by:
-
Fabio Aiuto authored
fix camel case in IE structures all over the driver. Signed-off-by:
-
Fabio Aiuto authored
fix camel case in struct wlan_bcn_info all over the driver. Signed-off-by:
-
Fabio Aiuto authored
fix camel case in struct wlan_phy_info all over the driver. Signed-off-by:
-
Fabio Aiuto authored
fix camel case in struct ndis_802_11_ssid all over the driver. Signed-off-by:
-
Fabio Aiuto authored
all members of struct ndis_802_11_conf_fh fh_config in struct ndis_802_11_conf are set to zero and their values are never used. So remove struct ndis_802_11_conf_fh. Signed-off-by:
-
Fabio Aiuto authored
fix camel case in struct ndis_802_11_conf Signed-off-by:
-
Fabio Aiuto authored
fix camel case in struct wlan_bssid_ex. Signed-off-by:
-
Fabio Aiuto authored
remove commented out condition checking channel > 14. Signed-off-by:
-
Fabio Aiuto authored
remove 5 Ghz code blocks, related to networks working over channel numbers above 14. Signed-off-by:
-
Fabio Aiuto authored
fix the following post-commit hook checkpatch warning: CHECK: spaces preferred around that '+' (ctx:VxV) 29: FILE: drivers/staging/rtl8723bs/hal/rtl8723b_hal_init.c:2238: + hal_get_chnl_group_8723b(ch+1, &group); Signed-off-by:
-
Fabio Aiuto authored
convert Hal_GetChnlGroup8723B() function name to snake case. Signed-off-by:
-
Fabio Aiuto authored
fix camel case inside function Hal_GetChnlGroup8723B() Signed-off-by:
-
Fabio Aiuto authored
simplify function Hal_GetChnlGroup8723B(). It returns an unused and unnecessary bool value telling which band the device works on. Since we work only on 2.4Ghz band and the useful return value is the second function argument we convert the return type to void. remove 5Ghz dead code either (for channel > 14). Signed-off-by:
-
Mauro Carvalho Chehab authored
This driver is ready for mainstream. So, move it out of staging. Acked-by:
Lee Jones <lee.jones@linaro.org> Signed-off-by:
Mauro Carvalho Chehab <mchehab+huawei@kernel.org> Link: https://lore.kernel.org/r/dd150f3ffa19c2dda0171f7dbe1dd63cce2a7af5.1627025657.git.mchehab+huawei@kernel.orgSigned-off-by:
-
- 21 Jul, 2021 1 commit
-
-
Mauro Carvalho Chehab authored
The regex at the patternProperties is wrong, although this was not reported as the DT schema was not enforcing properties. Fix it. Signed-off-by:
-
