This small patch set cleans up the c2h_handler code in the HAL layer of the driver. In r8188eu, this field of struct hal_ops, is not even used, so dependent code has always returned _FAIL. For this reason, we should remove this function pointer field, and the wrapper function which checks it. This is done in stages by this set, and helps get the driver closer to the pointer where the HAL layer is deleted/integrated as necessary and no longer a separate entity. This V2 version contains a small change to the return type of c2h_evt_hdl, based on feedback from Pavel Skripkin. I've also folded currently received Acked-by tags into the commit messages. Phillip Potter (3): staging: r8188eu: remove c2h_handler field from struct hal_ops staging: r8188eu: simplify c2h_evt_hdl function staging: r8188eu: remove rtw_hal_c2h_handler function drivers/staging/r8188eu/core/rtw_cmd.c | 25 +++------------------- drivers/staging/r8188eu/hal/hal_intf.c | 9 -------- drivers/staging/r8188eu/include/hal_intf.h | 4 ---- 3 files changed, 3 insertions(+), 35 deletions(-) -- 2.31.1
