On Tue, Apr 13, 2021 at 05:59:08PM +0200, Fabio M. De Francesco wrote: > Removed the led_blink_hdl() function (declaration, definition, and > caller code) because it's useless. It only seems to check whether or not a > given pointer is NULL. There are other (simpler) means for that purpose.
But you do not actually do that here. Why not? You just removed something, does it still work properly with that removal? > > Signed-off-by: Fabio M. De Francesco <[email protected]> Why the leading ":" in your subject line? > --- > drivers/staging/rtl8723bs/core/rtw_cmd.c | 1 - > drivers/staging/rtl8723bs/core/rtw_mlme_ext.c | 9 --------- > drivers/staging/rtl8723bs/include/rtw_mlme_ext.h | 1 - > 3 files changed, 11 deletions(-) Does this patch require some other patch to be applied first? thanks, greg k-h
