Kenneth Lu <kuohsianglu@xxxxxxxxx> wrote: > Variable buf_len and num_vdev_stats are bging assigned but never read. > These are redundant and can be remove. > > Signed-off-by: Kenneth Lu <kuohsianglu@xxxxxxxxx> Fails to compile: drivers/net/wireless/ath/ath10k/wmi.c: In function 'ath10k_wmi_10_4_op_pull_fw_stats': drivers/net/wireless/ath/ath10k/wmi.c:3067:18: error: 'num_vdev_stats' undeclared (first use in this function); did you mean 'num_pdev_stats'? for (i = 0; i < num_vdev_stats; i++) { ^~~~~~~~~~~~~~ num_pdev_stats drivers/net/wireless/ath/ath10k/wmi.c:3067:18: note: each undeclared identifier is reported only once for each function it appears in make[5]: *** [drivers/net/wireless/ath/ath10k/wmi.o] Error 1 make[4]: *** [drivers/net/wireless/ath/ath10k] Error 2 make[3]: *** [drivers/net/wireless/ath] Error 2 make[2]: *** [drivers/net/wireless] Error 2 make[1]: *** [drivers/net] Error 2 make[1]: *** Waiting for unfinished jobs.... make: *** [drivers] Error 2 Patch set to Changes Requested. -- https://patchwork.kernel.org/patch/10291063/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches