From: Sean Young <sean@mess.org>
Date: Mon, 20 Jan 2014 22:10:39 +0000 (-0300)
Subject: upstream: [media] iguanair: simplify tx loop
X-Git-Tag: submit/tizen/20141121.110247~1473
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=900759140520d671d79e83e51383e223fd7f0667;p=platform%2Fkernel%2Flinux-3.10.git

upstream: [media] iguanair: simplify tx loop

Make the code simpler.

Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
---

diff --git a/drivers/media/rc/iguanair.c b/drivers/media/rc/iguanair.c
index 84d2448..fb31d90 100644
--- a/drivers/media/rc/iguanair.c
+++ b/drivers/media/rc/iguanair.c
@@ -364,20 +364,14 @@ static int iguanair_tx(struct rc_dev *dev, unsigned *txbuf, unsigned count)
 			rc = -EINVAL;
 			goto out;
 		}
-		while (periods > 127) {
-			ir->packet->payload[size++] = 127 | space;
-			periods -= 127;
+		while (periods) {
+			unsigned p = min(periods, 127u);
+			ir->packet->payload[size++] = p | space;
+			periods -= p;
 		}
-
-		ir->packet->payload[size++] = periods | space;
 		space ^= 0x80;
 	}
 
-	if (count == 0) {
-		rc = -EINVAL;
-		goto out;
-	}
-
 	ir->packet->header.start = 0;
 	ir->packet->header.direction = DIR_OUT;
 	ir->packet->header.cmd = CMD_SEND;