- 13 Sep, 2021 40 commits
-
-
Michael Straube authored
Function ODM_IsWorkItemScheduled() is unused and empty, remove it. Acked-by:
Martin Kaiser <martin@kaiser.cx> Signed-off-by:
Michael Straube <straube.linux@gmail.com> Link: https://lore.kernel.org/r/20210911132635.30369-12-straube.linux@gmail.comSigned-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Michael Straube authored
Function ODM_ScheduleWorkItem() is unused and empty, remove it. Acked-by:
-
Michael Straube authored
Function ODM_FreeWorkItem() is unused and empty, remove it. Acked-by:
-
Michael Straube authored
Function ODM_StopWorkItem() is unused and empty, remove it. Acked-by:
-
Michael Straube authored
Function ODM_StartWorkItem() is unused and empty, remove it. Acked-by:
-
Michael Straube authored
Function ODM_InitializeWorkItem() is unused and empty, remove it. Acked-by:
-
Michael Straube authored
Function ODM_sleep_us() is unused, remove it. Acked-by:
-
Michael Straube authored
Function ODM_FillH2CCmd() is unused, remove it. Acked-by:
-
Michael Straube authored
Function ODM_ReleaseSpinLock() is unused and empty, remove it. Acked-by:
-
Michael Straube authored
Function ODM_AcquireSpinLock() is unused and empty, remove it. Acked-by:
-
Michael Straube authored
Function ODM_ReleaseTimer() is empty and its only caller is ODM_ReleaseAllTimers(). Remove both functions. Acked-by:
-
Saurav Girepunje authored
Remove unused variable ret and pwrpriv. Remove the condition with no effect (if == else) in usb_intf.c file. Remove rtw_resume_process() and move whole thing to rtw_resume(). Remove the padapter NULL check which couldn't NULL in rtw_resume function. Signed-off-by:
Saurav Girepunje <saurav.girepunje@gmail.com> Link: https://lore.kernel.org/r/YTshmPcZr/aVclHK@userSigned-off-by:
-
Martin Kaiser authored
Remove the last bunch of unused RT_usb_endpoint_... functions. Acked-by:
-
Martin Kaiser authored
RT_usb_endpoint_num is unused. Remove this function. Acked-by:
-
Martin Kaiser authored
Remove the RT_usb_endpoint_is_bulk_in function which is not used. RT_usb_endpoint_dir_in is no longer used either and can also be removed. Acked-by:
-
Martin Kaiser authored
The usb_endpoint_is_int is not used and can be removed. RT_usb_endpoint_xfer_int was used only by usb_endpoint_is_int. Remove this function as well. Acked-by:
-
Saurav Girepunje authored
Remove unused variable Adapter from rtw_IOL_cmd_buf_dump function. Signed-off-by:
-
Saurav Girepunje authored
Remove unused variable padapter from mp_init_xmit_attrib(). Reviewed-by:
Pavel Skripkin <paskripkin@gmail.com> Signed-off-by:
-
Martin Kaiser authored
bLCKInProgress in struct odm_rf_cal is never read. Remove it. Acked-by:
-
Michael Straube authored
Function rtw_IOL_append_LLT_cmd() just returns _SUCCESS and the only caller does not use the return value. Remove it. Signed-off-by:
-
Michael Straube authored
Function rtw_IOL_exec_cmds_sync() is just a wrapper around rtl8188e_IOL_exec_cmds_sync(). Remove it. Signed-off-by:
-
Michael Straube authored
Remove IOL_exec_cmds_sync() from struct hal_ops and its wrapper rtw_hal_iol_cmd(). Call rtl8188e_IOL_exec_cmds_sync() directly instead. Signed-off-by:
-
Michael Straube authored
These macros are not used in the driver, remove them. Found with GCC -Wunused-macros. Reviewed-by:
Hans de Goede <hdegoede@redhat.com> Signed-off-by:
-
Michael Straube authored
The macro ROUND is not used in the driver, remove it. Found with GCC -Wunused-macros. Acked-by:
-
Michael Straube authored
The macro READ_AND_CONFIG_TC is not used, remove it. Found with GCC -Wunused-macros. Acked-by:
-
Michael Straube authored
Function ODM_MacStatusQuery() is unused and empty, remove it. Signed-off-by:
-
Michael Straube authored
Function odm_Init_RSSIForDM() is unused and empty, remove it. Signed-off-by:
-
Michael Straube authored
Function rtl8192c_PHY_GetHWRegOriginalValue() is unused, remove it. Signed-off-by:
-
Michael Straube authored
Function PHY_UpdateTxPowerDbm8188E() is unused and it is the only user of function phy_DbmToTxPwrIdx(), remove both. Signed-off-by:
-
Michael Straube authored
Function PHY_ScanOperationBackup8188E() is unused and empty, remove it. Signed-off-by:
-
Michael Straube authored
Function PHY_GetTxPowerLevel8188E() is unused and it is the only user of function phy_TxPwrIdxToDbm(), remove both. Signed-off-by:
-
Krish Jain authored
This patch fixes the format of the code. It properly formats lines that should not end with a '(' as suggested by checkpath.pl. Signed-off-by:Krish Jain <krishjain02939@gmail.com> Link: https://lore.kernel.org/r/20210908205903.GA108157@pantherSigned-off-by:
-
Larry Finger authored
These rwo conditional compile symbols are explicitely defined. As a result, their included code is always compiled. They could be converted to Kconfig symbols; however, my experience with this driver in a GitHub repo is that many users of the device have it built into a SOC or SBC that they wish to use as an access point. As a result, CONFIG_88EU_APMODE would need to be selected. Thuse it should be built in. There have also been many users the wish to establish peer-to-peer networks with this device. For this reason, I am also proposing that CONFIG_88EU_P2P be automatically included. Signed-off-by:
Larry Finger <Larry.Finger@lwfinger.net> Link: https://lore.kernel.org/r/20210909165809.12634-1-Larry.Finger@lwfinger.netSigned-off-by:
-
Martin Kaiser authored
There's a funny "loop" in rtl8188eu_xmitframe_complete. It looks like do { ... some commands... break; } while (1); Remove it. Acked-by: -
Martin Kaiser authored
The RTW_SUSPEND_LOCK_NAME define is not used. Remove it. Acked-by:
-
Martin Kaiser authored
There's no rtw_write_scsi function in this driver. Remove its prototype. Acked-by:
-
Michael Straube authored
Remove unused function Hal_ProSetCrystalCap(). Acked-by:
Phillip Potter <phil@philpotter.co.uk> Signed-off-by:
-
Colin Ian King authored
Variable hoffset is being assigned values but it is never being used. The variable is redundant and can be removed. Acked-by:
Colin Ian King <colin.king@canonical.com> Addresses-Coverity: ("Unused value") Link: https://lore.kernel.org/r/20210907085302.10672-1-colin.king@canonical.comSigned-off-by:
-
Greg Kroah-Hartman authored
Using a struct class, a cdev, and another device just for a single minor device is total overkill. Just use a dynamic misc device instead, saving lots of logic and memory. Cc: Jacob Feder <jacobsfeder@gmail.com> Cc: Lee Jones <lee.jones@linaro.org> Cc: Muhammad Usama Anjum <musamaanjum@gmail.com> Link: https://lore.kernel.org/r/YTdPvBgKW6cKYrvb@kroah.comSigned-off-by:
-
Greg Kroah-Hartman authored
Using a struct class, a cdev, and another device just for a single minor device is total overkill. Just use a dynamic misc device instead, saving lots of logic and memory. Cc: Nicolas Saenz Julienne <nsaenz@kernel.org> Cc: Stefan Wahren <stefan.wahren@i2se.com> Cc: Arnd Bergmann <arnd@arndb.de> Cc: Dan Carpenter <dan.carpenter@oracle.com> Cc: Ojaswin Mujoo <ojaswin98@gmail.com> Cc: Phil Elwell <phil@raspberrypi.com> Cc: bcm-kernel-feedback-list@broadcom.com Cc: linux-rpi-kernel@lists.infradead.org Cc: linux-arm-kernel@lists.infradead.org Link: https://lore.kernel.org/r/20210907115045.2206083-1-gregkh@linuxfoundation.orgSigned-off-by:
-
