"Gustavo A. R. Silva" <gustavo@xxxxxxxxxxxxxx> wrote: > There is no need to compare *ps_state_enable* with < 0 because > such variable is of type u8 (8 bits, unsigned), making it > impossible to hold a negative value. > > Fix this by removing such comparison. > > Addresses-Coverity-ID: 1473921 ("Unsigned compared against 0") > Signed-off-by: Gustavo A. R. Silva <gustavo@xxxxxxxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. 7bfd82bff60e ath10k: remove unnecessary comparison of unsigned integer with < 0 -- https://patchwork.kernel.org/patch/10628679/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches