This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/media_tree.git tree:

Subject: [media] rc: img-ir: hw: Fix min/max bits setup
Author:  James Hogan <james.ho...@imgtec.com>
Date:    Thu Mar 13 07:29:22 2014 -0300

The calculated values for the minlen and maxlen fields, which were
rounded to multiples of 2 and clamped to a valid range, were left
unused. Use them in the calculation of the register value rather than
using the raw input minlen and maxlen.

This fixes the following warning with a W=1 build:
drivers/media/rc/img-ir/img-ir-hw.c In function ‘img_ir_free_timing’:
drivers/media/rc/img-ir/img-ir-hw.c +228 :23: warning: variable ‘maxlen’ set 
but not used [-Wunused-but-set-variable]
drivers/media/rc/img-ir/img-ir-hw.c +228 :15: warning: variable ‘minlen’ set 
but not used [-Wunused-but-set-variable]

Reported-by: Mauro Carvalho Chehab <m.che...@samsung.com>
Signed-off-by: James Hogan <james.ho...@imgtec.com>
Signed-off-by: Mauro Carvalho Chehab <m.che...@samsung.com>

 drivers/media/rc/img-ir/img-ir-hw.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

---

http://git.linuxtv.org/media_tree.git?a=commitdiff;h=b9e28d1f8301d6e393b8937282f325f13fb9cf6a

diff --git a/drivers/media/rc/img-ir/img-ir-hw.c 
b/drivers/media/rc/img-ir/img-ir-hw.c
index 2abf78a..579a52b 100644
--- a/drivers/media/rc/img-ir/img-ir-hw.c
+++ b/drivers/media/rc/img-ir/img-ir-hw.c
@@ -240,9 +240,9 @@ static u32 img_ir_free_timing(const struct 
img_ir_free_timing *timing,
        ft_min = (timing->ft_min*clock_hz + 999999) / 1000000;
        ft_min = (ft_min + 7) >> 3;
        /* construct register value */
-       return  (timing->maxlen << IMG_IR_MAXLEN_SHIFT) |
-               (timing->minlen << IMG_IR_MINLEN_SHIFT) |
-               (ft_min         << IMG_IR_FT_MIN_SHIFT);
+       return  (maxlen << IMG_IR_MAXLEN_SHIFT) |
+               (minlen << IMG_IR_MINLEN_SHIFT) |
+               (ft_min << IMG_IR_FT_MIN_SHIFT);
 }
 
 /**

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to