[PATCH 05/14] cxd2820r_core: fix sparse warnings

2013-10-04 Thread Hans Verkuil 
From: Hans Verkuil hans.verk...@cisco.com

drivers/media/dvb-frontends/cxd2820r_core.c:34:32: error: cannot size expression
drivers/media/dvb-frontends/cxd2820r_core.c:68:32: error: cannot size expression

Signed-off-by: Hans Verkuil hans.verk...@cisco.com
Cc: Antti Palosaari cr...@iki.fi
---
 drivers/media/dvb-frontends/cxd2820r_core.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/dvb-frontends/cxd2820r_core.c 
b/drivers/media/dvb-frontends/cxd2820r_core.c
index 7ca5c69..d9eeeb1 100644
--- a/drivers/media/dvb-frontends/cxd2820r_core.c
+++ b/drivers/media/dvb-frontends/cxd2820r_core.c
@@ -31,7 +31,7 @@ static int cxd2820r_wr_regs_i2c(struct cxd2820r_priv *priv, 
u8 i2c, u8 reg,
{
.addr = i2c,
.flags = 0,
-   .len = sizeof(buf),
+   .len = len + 1,
.buf = buf,
}
};
@@ -65,7 +65,7 @@ static int cxd2820r_rd_regs_i2c(struct cxd2820r_priv *priv, 
u8 i2c, u8 reg,
}, {
.addr = i2c,
.flags = I2C_M_RD,
-   .len = sizeof(buf),
+   .len = len,
.buf = buf,
}
};
-- 
1.8.3.2

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 05/14] cxd2820r_core: fix sparse warnings

2013-10-04 Thread Antti Palosaari 

On 04.10.2013 17:01, Hans Verkuil wrote:

From: Hans Verkuil hans.verk...@cisco.com

drivers/media/dvb-frontends/cxd2820r_core.c:34:32: error: cannot size expression
drivers/media/dvb-frontends/cxd2820r_core.c:68:32: error: cannot size expression

Signed-off-by: Hans Verkuil hans.verk...@cisco.com


Acked-by: Antti Palosaari cr...@iki.fi
Reviewed-by: Antti Palosaari cr...@iki.fi



Cc: Antti Palosaari cr...@iki.fi
---
  drivers/media/dvb-frontends/cxd2820r_core.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/media/dvb-frontends/cxd2820r_core.c 
b/drivers/media/dvb-frontends/cxd2820r_core.c
index 7ca5c69..d9eeeb1 100644
--- a/drivers/media/dvb-frontends/cxd2820r_core.c
+++ b/drivers/media/dvb-frontends/cxd2820r_core.c
@@ -31,7 +31,7 @@ static int cxd2820r_wr_regs_i2c(struct cxd2820r_priv *priv, 
u8 i2c, u8 reg,
{
.addr = i2c,
.flags = 0,
-   .len = sizeof(buf),
+   .len = len + 1,
.buf = buf,
}
};
@@ -65,7 +65,7 @@ static int cxd2820r_rd_regs_i2c(struct cxd2820r_priv *priv, 
u8 i2c, u8 reg,
}, {
.addr = i2c,
.flags = I2C_M_RD,
-   .len = sizeof(buf),
+   .len = len,
.buf = buf,
}
};




--
http://palosaari.fi/
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 05/14] cxd2820r_core: fix sparse warnings

2013-10-04 Thread Michael Krufky 
On Fri, Oct 4, 2013 at 10:36 AM, Antti Palosaari cr...@iki.fi wrote:
 On 04.10.2013 17:01, Hans Verkuil wrote:

 From: Hans Verkuil hans.verk...@cisco.com

 drivers/media/dvb-frontends/cxd2820r_core.c:34:32: error: cannot size
 expression
 drivers/media/dvb-frontends/cxd2820r_core.c:68:32: error: cannot size
 expression

 Signed-off-by: Hans Verkuil hans.verk...@cisco.com


 Acked-by: Antti Palosaari cr...@iki.fi
 Reviewed-by: Antti Palosaari cr...@iki.fi



 Cc: Antti Palosaari cr...@iki.fi
 ---
   drivers/media/dvb-frontends/cxd2820r_core.c | 4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)

 diff --git a/drivers/media/dvb-frontends/cxd2820r_core.c
 b/drivers/media/dvb-frontends/cxd2820r_core.c
 index 7ca5c69..d9eeeb1 100644
 --- a/drivers/media/dvb-frontends/cxd2820r_core.c
 +++ b/drivers/media/dvb-frontends/cxd2820r_core.c
 @@ -31,7 +31,7 @@ static int cxd2820r_wr_regs_i2c(struct cxd2820r_priv
 *priv, u8 i2c, u8 reg,
 {
 .addr = i2c,
 .flags = 0,
 -   .len = sizeof(buf),
 +   .len = len + 1,
 .buf = buf,
 }
 };
 @@ -65,7 +65,7 @@ static int cxd2820r_rd_regs_i2c(struct cxd2820r_priv
 *priv, u8 i2c, u8 reg,
 }, {
 .addr = i2c,
 .flags = I2C_M_RD,
 -   .len = sizeof(buf),
 +   .len = len,
 .buf = buf,
 }
 };



 --
 http://palosaari.fi/

 --
 To unsubscribe from this list: send the line unsubscribe linux-media in
 the body of a message to majord...@vger.kernel.org
 More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reviewed-by: Michael Krufky mkru...@linuxtv.org
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html