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] cx231xxx: fix typo in saddr_len check
Author:  Dan Carpenter <[email protected]>
Date:    Thu Dec 23 16:38:53 2010 -0300

The original code compared "saddr_len" with zero twice in a nonsensical
way.  I asked the list, and Andy Walls and Sri Deevi say that the second
check should be if "saddr_len == 1".

Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: Srinivasa Deevi <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/video/cx231xx/cx231xx-core.c |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

---

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

diff --git a/drivers/media/video/cx231xx/cx231xx-core.c 
b/drivers/media/video/cx231xx/cx231xx-core.c
index 44d124c..7d62d58 100644
--- a/drivers/media/video/cx231xx/cx231xx-core.c
+++ b/drivers/media/video/cx231xx/cx231xx-core.c
@@ -1515,7 +1515,7 @@ int cx231xx_read_i2c_master(struct cx231xx *dev, u8 
dev_addr, u16 saddr,
 
        if (saddr_len == 0)
                saddr = 0;
-       else if (saddr_len == 0)
+       else if (saddr_len == 1)
                saddr &= 0xff;
 
        /* prepare xfer_data struct */
@@ -1566,7 +1566,7 @@ int cx231xx_write_i2c_master(struct cx231xx *dev, u8 
dev_addr, u16 saddr,
 
        if (saddr_len == 0)
                saddr = 0;
-       else if (saddr_len == 0)
+       else if (saddr_len == 1)
                saddr &= 0xff;
 
        /* prepare xfer_data struct */
@@ -1600,7 +1600,7 @@ int cx231xx_read_i2c_data(struct cx231xx *dev, u8 
dev_addr, u16 saddr,
 
        if (saddr_len == 0)
                saddr = 0;
-       else if (saddr_len == 0)
+       else if (saddr_len == 1)
                saddr &= 0xff;
 
        /* prepare xfer_data struct */
@@ -1641,7 +1641,7 @@ int cx231xx_write_i2c_data(struct cx231xx *dev, u8 
dev_addr, u16 saddr,
 
        if (saddr_len == 0)
                saddr = 0;
-       else if (saddr_len == 0)
+       else if (saddr_len == 1)
                saddr &= 0xff;
 
        /* prepare xfer_data struct */

_______________________________________________
linuxtv-commits mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to