- 13 Sep, 2021 40 commits
-
-
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:
Michael Straube <straube.linux@gmail.com> Signed-off-by:
Martin Kaiser <martin@kaiser.cx> Link: https://lore.kernel.org/r/20210910202151.4007-2-martin@kaiser.cxSigned-off-by:
Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-
Saurav Girepunje authored
Remove unused variable Adapter from rtw_IOL_cmd_buf_dump function. Signed-off-by:
Saurav Girepunje <saurav.girepunje@gmail.com> Link: https://lore.kernel.org/r/YTuh6iDyPMUC3wha@userSigned-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:
-
Michael Straube authored
Remove UpdateHalRAMask8188EUsb from hal_ops and call UpdateHalRAMask8188EUsb() directly instead. Signed-off-by:
-
Michael Straube authored
Remove SetBeaconRelatedRegistersHandler from hal_ops and remove its wrapper rtw_hal_bcn_related_reg_setting(). Call SetBeaconRelatedRegisters8188EUsb() directly instead. Signed-off-by:
-
Michael Straube authored
Remove unused function rtl8188e_clone_haldata(). Signed-off-by:
-
Michael Straube authored
Remove free_hal_data from hal_ops and remove its wrapper rtw_hal_free_data(). Call rtl8188e_free_hal_data() directly instead. Signed-off-by:
-
Michael Straube authored
Remove hal_notch_filter from hal_ops and remove its wrapper rtw_hal_notch_filter(). Call hal_notch_filter_8188e() directly instead. Signed-off-by:
-
Michael Straube authored
Remove empty function rtl8188e_stop_thread() and related code. Signed-off-by:
-
Michael Straube authored
Remove empty function rtl8188e_start_thread() and related code. Signed-off-by:
-
Michael Straube authored
Remove AntDivCompareHandler from hal_ops and remove its wrapper rtw_hal_antdiv_rssi_compared(). Call AntDivCompare8188E() directly instead. Signed-off-by:
-
Michael Straube authored
Remove AntDivBeforeLinkHandler from hal_ops and remove its wrapper rtw_hal_antdiv_before_linked(). Call AntDivBeforeLink8188E() directly instead. Signed-off-by:
-
Michael Straube authored
The local variable badworden is set to a new value immediately after an assignment. Remove the useless assignment and initialization. Signed-off-by:
-
Michael Straube authored
Remove Efuse_WordEnableDataWrite from hal_ops and remove its wrapper Efuse_WordEnableDataWrite(). Call rtl8188e_Efuse_WordEnableDataWrite() directly instead. Signed-off-by:
-
Michael Straube authored
Remove Efuse_PgPacketWrite from hal_ops and remove its wrapper Efuse_PgPacketWrite(). Call rtl8188e_Efuse_PgPacketWrite() directly instead. Signed-off-by:
-
Michael Straube authored
Remove Efuse_PgPacketRead from hal_ops and remove its wrapper Efuse_PgPacketRead(). Call rtl8188e_Efuse_PgPacketRead() directly instead. Signed-off-by:
-
Michael Straube authored
Remove empty comments. Signed-off-by:
-
Michael Straube authored
Remove EfuseGetCurrentSize from hal_ops and remove its wrapper Efuse_GetCurrentSize(). Call rtl8188e_EfuseGetCurrentSize() directly instead. Signed-off-by:
-
