Colin King <colin.king@xxxxxxxxxxxxx> wrote: > The check on status not being zero is redundant as previous code > paths that set status to an error value break out of the while > loop and hence status is never non-zero at the check. Remove > this redundant code. > > Addresses-Coverity: ("Logically dead code") > Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx> > Reviewed-by: Mukesh Ojha <mojha@xxxxxxxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Patch applied to ath-next branch of ath.git, thanks. e643da21e19a ath6kl: remove redundant check of status != 0 -- https://patchwork.kernel.org/patch/10885625/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches