Re: [scsi] scsi: ufs: don't check unsigned type for a negative value

2017-03-15 Thread Subhash Jadavani
On 2017-03-15 10:26, James Bottomley wrote: On Mon, 2017-03-13 at 17:19 -0700, Subhash Jadavani wrote: On 2017-03-12 03:22, Tomas Winkler wrote: > Fix compilation warning > > drivers/scsi/ufs/ufshcd.c:7645:13: warning: comparison of unsigned > expression < 0 is always false [-Wtype-limits] > if

Re: [scsi] scsi: ufs: don't check unsigned type for a negative value

2017-03-15 Thread James Bottomley
On Mon, 2017-03-13 at 17:19 -0700, Subhash Jadavani wrote: > On 2017-03-12 03:22, Tomas Winkler wrote: > > Fix compilation warning > > > > drivers/scsi/ufs/ufshcd.c:7645:13: warning: comparison of unsigned > > expression < 0 is always false [-Wtype-limits] > > if ((value < UFS_PM_LVL_0) || (value

Re: [scsi] scsi: ufs: don't check unsigned type for a negative value

2017-03-13 Thread Martin K. Petersen
> "Tomas" == Tomas Winkler writes: Tomas> Fix compilation warning drivers/scsi/ufs/ufshcd.c:7645:13: Tomas> warning: comparison of unsigned expression < 0 is always false Tomas> [-Wtype-limits] if ((value < UFS_PM_LVL_0) || (value >= Tomas> UFS_PM_LVL_MAX)) Applied

Re: [scsi] scsi: ufs: don't check unsigned type for a negative value

2017-03-13 Thread Subhash Jadavani
On 2017-03-12 03:22, Tomas Winkler wrote: Fix compilation warning drivers/scsi/ufs/ufshcd.c:7645:13: warning: comparison of unsigned expression < 0 is always false [-Wtype-limits] if ((value < UFS_PM_LVL_0) || (value >= UFS_PM_LVL_MAX)) Signed-off-by: Tomas Winkler