This series removes all remaining pointers from struct hal_ops that are never set anywhere else in the driver. As a part of this, it also takes out the wrapper functions and their callers, given that they mostly never end up calling/returning the function pointer value as they check for NULL. There are two exceptions to this however, and therefore this series also fixes two potentially triggerable NULL pointer dereference bugs. Phillip Potter (14): staging: r8188eu: remove rtw_hal_reset_security_engine function staging: r8188eu: remove hal_reset_security_engine from struct hal_ops staging: r8188eu: remove rtw_hal_enable_interrupt function staging: r8188eu: remove enable_interrupt from struct hal_ops staging: r8188eu: remove rtw_hal_disable_interrupt function staging: r8188eu: remove disable_interrupt from struct hal_ops staging: r8188eu: remove rtw_hal_interrupt_handler function staging: r8188eu: remove interrupt_handler from struct hal_ops staging: r8188eu: remove rtw_hal_xmitframe_enqueue function staging: r8188eu: remove hal_xmitframe_enqueue from struct hal_ops staging: r8188eu: remove Efuse_PgPacketWrite_BT function staging: r8188eu: remove Efuse_PgPacketWrite_BT from struct hal_ops staging: r8188eu: remove rtw_hal_c2h_id_filter_ccx function staging: r8188eu: remove c2h_id_filter_ccx from struct hal_ops drivers/staging/r8188eu/core/rtw_cmd.c | 9 +---- drivers/staging/r8188eu/core/rtw_efuse.c | 13 +------ drivers/staging/r8188eu/core/rtw_recv.c | 2 - drivers/staging/r8188eu/hal/hal_intf.c | 44 ---------------------- drivers/staging/r8188eu/include/hal_intf.h | 19 ---------- 5 files changed, 3 insertions(+), 84 deletions(-) -- 2.31.1
