>>>>> "Colin" == Colin King <colin.king@xxxxxxxxxxxxx> writes: Colin, Colin> The check for ret being zero is redundant as a few statements Colin> earlier we break out of the while loop if ret is non-zero. Thus Colin> we can remove the zero check and also the dead-code non-zero case Colin> too. Applied to 4.11/scsi-fixes. -- Martin K. Petersen Oracle Linux Engineering