Re: [PATCH][next] scsi: arcmsr: remove redundant check for secs < 0

From: Martin K. Petersen
Date: Mon Dec 11 2017 - 21:38:08 EST



Colin,

> The check for secs being less than zero is redundant for two reasons.
> Firstly, secs is unsigned so the check is always going to be false.
> Secondly, if secs was signed the proceeding calculation of secs is
> never going to be negative. Hence we can remove this redundant check
> and day and secs re-adjustment.

Applied to 4.16/scsi-queue, thanks!

--
Martin K. Petersen Oracle Linux Engineering