Re: [PATCH] wimax/i2400m: remove redundant variables ack_status, bcf and protocol

From: David Miller
Date: Thu Jul 12 2018 - 01:54:56 EST


From: Colin King <colin.king@xxxxxxxxxxxxx>
Date: Mon, 9 Jul 2018 13:23:13 +0100

> From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> Variables ack_status, bcf and protocol are being assigned but are
> never used hence they are redundant and can be removed.
>
> Also declare ack_type as unsigned int rather than unsigned to clean
> up a checkpatch warning.
>
> Cleans up clang warnings:
> warning: variable 'ack_status' set but not used [-Wunused-but-set-variable]
> warning: variable 'bcf' set but not used [-Wunused-but-set-variable]
> warning: variable 'protocol' set but not used [-Wunused-but-set-variable]
>
> Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>

This is simple enough that I'll just apply this directly, thanks
Colin.