From: Sachin Kamat <sachin.kamat@samsung.com>
Date: Fri, 7 Mar 2014 12:56:00 +0000 (+0000)
Subject: staging: iio: ad9832: Remove redundant check
X-Git-Tag: v3.17-rc1~123^2~779^2~13
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=260974704ea4c01fb21166182e8b0cf266cbd77a;p=thirdparty%2Fkernel%2Flinux.git

staging: iio: ad9832: Remove redundant check

'val' is unsigned and cannot be negative.

Signed-off-by: Sachin Kamat <sachin.kamat@samsung.com>
Cc: Michael Hennerich <hennerich@blackfin.uclinux.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
---

diff --git a/drivers/staging/iio/frequency/ad9832.c b/drivers/staging/iio/frequency/ad9832.c
index b7c8351ce2ffc..cf68159a58481 100644
--- a/drivers/staging/iio/frequency/ad9832.c
+++ b/drivers/staging/iio/frequency/ad9832.c
@@ -120,7 +120,7 @@ static ssize_t ad9832_write(struct device *dev, struct device_attribute *attr,
 		ret = spi_sync(st->spi, &st->msg);
 		break;
 	case AD9832_PHASE_SYM:
-		if (val < 0 || val > 3) {
+		if (val > 3) {
 			ret = -EINVAL;
 			break;
 		}