With the help of Coccinelle, I find some place to use redundant variable to store the return value, It is unnecessary. Just remove it and make the funtion to be void. zhong jiang (3): brcmsmac: Remove unneeded variable and make function to be void wlegacy: Remove unneeded variable and make function to be void libertas: Remove unneeded variable and make function to be void drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c | 5 +---- drivers/net/wireless/intel/iwlegacy/4965-mac.c | 8 ++------ drivers/net/wireless/marvell/libertas/cmd.h | 2 +- drivers/net/wireless/marvell/libertas/cmdresp.c | 5 +---- 4 files changed, 5 insertions(+), 15 deletions(-) -- 1.7.12.4