Re: [PATCH] scsi: aacraid: remove redundant zero check on ret

2017-02-27 Thread Martin K. Petersen
> "Colin" == Colin King 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.

RE: [PATCH] scsi: aacraid: remove redundant zero check on ret

2017-02-26 Thread Dave Carroll
> -Original Message- > From: Colin King [mailto:colin.k...@canonical.com] > Sent: Friday, February 24, 2017 7:44 AM > To: dl-esc-Aacraid Linux Driver; James E . J . Bottomley; Martin K . > Petersen; linux-scsi@vger.kernel.org > Cc: kernel-janit...@vger.kernel.org;