* [PATCH] staging: rtl8723bs: remove unused function declarations
@ 2026-03-21 22:33 Oskar Ray-Frayssinet
2026-03-23 4:47 ` Ethan Tidmore
0 siblings, 1 reply; 3+ messages in thread
From: Oskar Ray-Frayssinet @ 2026-03-21 22:33 UTC (permalink / raw)
To: gregkh; +Cc: straube.linux, linux-staging, linux-kernel, Oskar Ray-Frayssinet
Remove unused declarations of MRateToHwRate8723B() and
HwRateToMRate8723B() from rtl8723b_hal.h as they have no
implementation and are never called.
Signed-off-by: Oskar Ray-Frayssinet <rayfraytech@gmail.com>
---
drivers/staging/rtl8723bs/include/rtl8723b_hal.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/staging/rtl8723bs/include/rtl8723b_hal.h b/drivers/staging/rtl8723bs/include/rtl8723b_hal.h
index 7ec84304a19e..ffd03927841c 100644
--- a/drivers/staging/rtl8723bs/include/rtl8723b_hal.h
+++ b/drivers/staging/rtl8723bs/include/rtl8723b_hal.h
@@ -236,8 +236,6 @@ int FirmwareDownloadBT(struct adapter *adapter, struct rt_firmware *firmware);
void CCX_FwC2HTxRpt_8723b(struct adapter *padapter, u8 *pdata, u8 len);
s32 c2h_id_filter_ccx_8723b(u8 *buf);
s32 c2h_handler_8723b(struct adapter *padapter, u8 *pC2hEvent);
-u8 MRateToHwRate8723B(u8 rate);
-u8 HwRateToMRate8723B(u8 rate);
void Hal_ReadRFGainOffset(struct adapter *padapter, u8 *hwinfo,
bool AutoLoadFail);
--
2.43.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] staging: rtl8723bs: remove unused function declarations
2026-03-21 22:33 Oskar Ray-Frayssinet
@ 2026-03-23 4:47 ` Ethan Tidmore
0 siblings, 0 replies; 3+ messages in thread
From: Ethan Tidmore @ 2026-03-23 4:47 UTC (permalink / raw)
To: Oskar Ray-Frayssinet, gregkh; +Cc: straube.linux, linux-staging, linux-kernel
On Sat Mar 21, 2026 at 5:33 PM CDT, Oskar Ray-Frayssinet wrote:
> Remove unused declarations of MRateToHwRate8723B() and
> HwRateToMRate8723B() from rtl8723b_hal.h as they have no
> implementation and are never called.
>
> Signed-off-by: Oskar Ray-Frayssinet <rayfraytech@gmail.com>
> ---
LGTM.
Reviewed-by: Ethan Tidmore <ethantidmore06@gmail.com>
Thanks,
ET
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] staging: rtl8723bs: remove unused function declarations
@ 2026-06-22 5:25 Aditya Chari
0 siblings, 0 replies; 3+ messages in thread
From: Aditya Chari @ 2026-06-22 5:25 UTC (permalink / raw)
To: gregkh; +Cc: linux-staging, linux-kernel, Aditya Chari
rtl8192c_translate_rx_signal_stuff() and rtl8192c_query_rx_desc_status()
are declared in rtl8192c_recv.h but have no implementation and no
caller anywhere in the kernel tree. They are leftover from when this
driver was derived from Realtek's shared vendor codebase covering
multiple chip families.
Verified via grep across the full tree that neither symbol is
referenced outside this declaration, and confirmed via a full build
of the module before and after removal that no warnings or errors
are introduced.
This addresses the "find and remove any code for other chips that is
left over" item in the driver's TODO file.
Signed-off-by: Aditya Chari <adi25charis@gmail.com>
---
drivers/staging/rtl8723bs/include/rtl8192c_recv.h | 4 ----
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/rtl8723bs/include/rtl8192c_recv.h b/drivers/staging/rtl8723bs/include/rtl8192c_recv.h
index e2e9aa03f..b63625ab4 100644
--- a/drivers/staging/rtl8723bs/include/rtl8192c_recv.h
+++ b/drivers/staging/rtl8723bs/include/rtl8192c_recv.h
@@ -30,8 +30,4 @@ struct phy_stat {
/* Rx smooth factor */
#define Rx_Smooth_Factor (20)
-
-void rtl8192c_translate_rx_signal_stuff(union recv_frame *precvframe, struct phy_stat *pphy_status);
-void rtl8192c_query_rx_desc_status(union recv_frame *precvframe, struct recv_stat *pdesc);
-
#endif
--
2.53.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2026-06-22 18:37 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-22 5:25 [PATCH] staging: rtl8723bs: remove unused function declarations Aditya Chari
-- strict thread matches above, loose matches on Subject: below --
2026-03-21 22:33 Oskar Ray-Frayssinet
2026-03-23 4:47 ` Ethan Tidmore
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox