* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() @ 2011-11-24 7:52 Shawn Guo 2011-11-24 8:00 ` Wolfram Sang 2011-11-24 8:29 ` Uwe Kleine-König 0 siblings, 2 replies; 12+ messages in thread From: Shawn Guo @ 2011-11-24 7:52 UTC (permalink / raw) To: linux-arm-kernel CC drivers/i2c/busses/i2c-mxs.o drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used uninitialized in this function Signed-off-by: Shawn Guo <shawn.guo@linaro.org> --- drivers/i2c/busses/i2c-mxs.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c index 7e78f7c..00f098f 100644 --- a/drivers/i2c/busses/i2c-mxs.c +++ b/drivers/i2c/busses/i2c-mxs.c @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct mxs_i2c_dev *i2c) static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int len) { - u32 data; + u32 data = 0; int i; for (i = 0; i < len; i++) { -- 1.7.4.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 7:52 [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() Shawn Guo @ 2011-11-24 8:00 ` Wolfram Sang 2011-11-24 8:24 ` Shawn Guo 2011-11-24 8:29 ` Uwe Kleine-König 1 sibling, 1 reply; 12+ messages in thread From: Wolfram Sang @ 2011-11-24 8:00 UTC (permalink / raw) To: linux-arm-kernel On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > CC drivers/i2c/busses/i2c-mxs.o > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used uninitialized in this function > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> Which compiler? What flags? I don't see that here. -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20111124/780636bf/attachment.sig> ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 8:00 ` Wolfram Sang @ 2011-11-24 8:24 ` Shawn Guo 0 siblings, 0 replies; 12+ messages in thread From: Shawn Guo @ 2011-11-24 8:24 UTC (permalink / raw) To: linux-arm-kernel On Thu, Nov 24, 2011 at 09:00:04AM +0100, Wolfram Sang wrote: > On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > > CC drivers/i2c/busses/i2c-mxs.o > > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used uninitialized in this function > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > Which compiler? What flags? I don't see that here. > gcc version 4.5.2 (Ubuntu/Linaro 4.5.2-8ubuntu3) $ make ARCH=arm CROSS_COMPILE=arm-linux-gnueabi- -j 2 zImage -- Regards, Shawn ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 7:52 [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() Shawn Guo 2011-11-24 8:00 ` Wolfram Sang @ 2011-11-24 8:29 ` Uwe Kleine-König 2011-11-24 8:37 ` Wolfram Sang 1 sibling, 1 reply; 12+ messages in thread From: Uwe Kleine-König @ 2011-11-24 8:29 UTC (permalink / raw) To: linux-arm-kernel Hello Shawn, On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > CC drivers/i2c/busses/i2c-mxs.o > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used uninitialized in this function > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > --- > drivers/i2c/busses/i2c-mxs.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c > index 7e78f7c..00f098f 100644 > --- a/drivers/i2c/busses/i2c-mxs.c > +++ b/drivers/i2c/busses/i2c-mxs.c > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct mxs_i2c_dev *i2c) > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int len) > { > - u32 data; > + u32 data = 0; > int i; > > for (i = 0; i < len; i++) { Looks like a stupid compiler. Then better use u32 uninitialized_var(data); Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-K?nig | Industrial Linux Solutions | http://www.pengutronix.de/ | ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 8:29 ` Uwe Kleine-König @ 2011-11-24 8:37 ` Wolfram Sang 2011-11-24 9:02 ` Dong Aisheng-B29396 2011-11-25 1:15 ` Shawn Guo 0 siblings, 2 replies; 12+ messages in thread From: Wolfram Sang @ 2011-11-24 8:37 UTC (permalink / raw) To: linux-arm-kernel On Thu, Nov 24, 2011 at 09:29:03AM +0100, Uwe Kleine-K?nig wrote: > Hello Shawn, > > On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > > CC drivers/i2c/busses/i2c-mxs.o > > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used uninitialized in this function > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > --- > > drivers/i2c/busses/i2c-mxs.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c > > index 7e78f7c..00f098f 100644 > > --- a/drivers/i2c/busses/i2c-mxs.c > > +++ b/drivers/i2c/busses/i2c-mxs.c > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct mxs_i2c_dev *i2c) > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int len) > > { > > - u32 data; > > + u32 data = 0; > > int i; > > > > for (i = 0; i < len; i++) { > Looks like a stupid compiler. Then better use > > u32 uninitialized_var(data); Leave it as it is and fix the compiler? Will do another test later to make sure. -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20111124/5712290e/attachment.sig> ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 8:37 ` Wolfram Sang @ 2011-11-24 9:02 ` Dong Aisheng-B29396 2011-11-24 9:06 ` Uwe Kleine-König 2011-11-24 9:11 ` Lothar Waßmann 2011-11-25 1:15 ` Shawn Guo 1 sibling, 2 replies; 12+ messages in thread From: Dong Aisheng-B29396 @ 2011-11-24 9:02 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: linux-arm-kernel-bounces at lists.infradead.org [mailto:linux-arm- > kernel-bounces at lists.infradead.org] On Behalf Of Wolfram Sang > Sent: Thursday, November 24, 2011 4:38 PM > To: Uwe Kleine-K?nig > Cc: Shawn Guo; linux-arm-kernel at lists.infradead.org; Ben Dooks; linux- > i2c at vger.kernel.org > Subject: Re: [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() > > On Thu, Nov 24, 2011 at 09:29:03AM +0100, Uwe Kleine-K?nig wrote: > > Hello Shawn, > > > > On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > > > CC drivers/i2c/busses/i2c-mxs.o > > > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > > > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used > > > uninitialized in this function > > > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > > --- > > > drivers/i2c/busses/i2c-mxs.c | 2 +- > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c > > > b/drivers/i2c/busses/i2c-mxs.c index 7e78f7c..00f098f 100644 > > > --- a/drivers/i2c/busses/i2c-mxs.c > > > +++ b/drivers/i2c/busses/i2c-mxs.c > > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct > > > mxs_i2c_dev *i2c) > > > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, > > > int len) { > > > - u32 data; > > > + u32 data = 0; > > > int i; > > > > > > for (i = 0; i < len; i++) { > > Looks like a stupid compiler. Then better use > > > > u32 uninitialized_var(data); > > Leave it as it is and fix the compiler? Will do another test later to > make sure. > Compiler is wrong? Just from the code that it seems to be right the 'data' might be used uninitialized. Something I missed? static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int len) { u32 data; int i; for (i = 0; i < len; i++) { if ((i & 3) == 0) { if (mxs_i2c_wait_for_data(i2c)) return -ETIMEDOUT; data = readl(i2c->regs + MXS_I2C_QUEUEDATA); } buf[i] = data & 0xff; data >>= 8; } return 0; } Regards Dong Aisheng ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 9:02 ` Dong Aisheng-B29396 @ 2011-11-24 9:06 ` Uwe Kleine-König 2011-11-24 9:09 ` Dong Aisheng-B29396 2011-11-24 9:11 ` Lothar Waßmann 1 sibling, 1 reply; 12+ messages in thread From: Uwe Kleine-König @ 2011-11-24 9:06 UTC (permalink / raw) To: linux-arm-kernel Hello, On Thu, Nov 24, 2011 at 09:02:21AM +0000, Dong Aisheng-B29396 wrote: > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c > > > > b/drivers/i2c/busses/i2c-mxs.c index 7e78f7c..00f098f 100644 > > > > --- a/drivers/i2c/busses/i2c-mxs.c > > > > +++ b/drivers/i2c/busses/i2c-mxs.c > > > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct > > > > mxs_i2c_dev *i2c) > > > > > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, > > > > int len) { > > > > - u32 data; > > > > + u32 data = 0; > > > > int i; > > > > > > > > for (i = 0; i < len; i++) { > > > Looks like a stupid compiler. Then better use > > > > > > u32 uninitialized_var(data); > > > > Leave it as it is and fix the compiler? Will do another test later to > > make sure. > > > > Compiler is wrong? > Just from the code that it seems to be right the 'data' might be used > uninitialized. > > Something I missed? > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int len) > { > u32 data; > int i; > > for (i = 0; i < len; i++) { > if ((i & 3) == 0) { > if (mxs_i2c_wait_for_data(i2c)) > return -ETIMEDOUT; > data = readl(i2c->regs + MXS_I2C_QUEUEDATA); > } > buf[i] = data & 0xff; > data >>= 8; > } When data is used the first time, i is 0, so it went through the body of the if above before which assigned data. So no, data is initialized when used to assign buf[i]. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-K?nig | Industrial Linux Solutions | http://www.pengutronix.de/ | ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 9:06 ` Uwe Kleine-König @ 2011-11-24 9:09 ` Dong Aisheng-B29396 0 siblings, 0 replies; 12+ messages in thread From: Dong Aisheng-B29396 @ 2011-11-24 9:09 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: Uwe Kleine-K?nig [mailto:u.kleine-koenig at pengutronix.de] > Sent: Thursday, November 24, 2011 5:07 PM > To: Dong Aisheng-B29396 > Cc: Wolfram Sang; Shawn Guo; linux-arm-kernel at lists.infradead.org; Ben > Dooks; linux-i2c at vger.kernel.org > Subject: Re: [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() > > Hello, > > On Thu, Nov 24, 2011 at 09:02:21AM +0000, Dong Aisheng-B29396 wrote: > > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c > > > > > b/drivers/i2c/busses/i2c-mxs.c index 7e78f7c..00f098f 100644 > > > > > --- a/drivers/i2c/busses/i2c-mxs.c > > > > > +++ b/drivers/i2c/busses/i2c-mxs.c > > > > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct > > > > > mxs_i2c_dev *i2c) > > > > > > > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 > > > > > *buf, int len) { > > > > > - u32 data; > > > > > + u32 data = 0; > > > > > int i; > > > > > > > > > > for (i = 0; i < len; i++) { > > > > Looks like a stupid compiler. Then better use > > > > > > > > u32 uninitialized_var(data); > > > > > > Leave it as it is and fix the compiler? Will do another test later > > > to make sure. > > > > > > > Compiler is wrong? > > Just from the code that it seems to be right the 'data' might be used > > uninitialized. > > > > Something I missed? > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int > > len) { > > u32 data; > > int i; > > > > for (i = 0; i < len; i++) { > > if ((i & 3) == 0) { > > if (mxs_i2c_wait_for_data(i2c)) > > return -ETIMEDOUT; > > data = readl(i2c->regs + MXS_I2C_QUEUEDATA); > > } > > buf[i] = data & 0xff; > > data >>= 8; > > } > When data is used the first time, i is 0, so it went through the body of > the if above before which assigned data. So no, data is initialized when > used to assign buf[i]. > Ok, Thanks for the info. So the compiler should do intelligent checking. Regards Dong Aisheng ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 9:02 ` Dong Aisheng-B29396 2011-11-24 9:06 ` Uwe Kleine-König @ 2011-11-24 9:11 ` Lothar Waßmann 1 sibling, 0 replies; 12+ messages in thread From: Lothar Waßmann @ 2011-11-24 9:11 UTC (permalink / raw) To: linux-arm-kernel Hi, Dong Aisheng-B29396 writes: > > -----Original Message----- > > From: linux-arm-kernel-bounces at lists.infradead.org [mailto:linux-arm- > > kernel-bounces at lists.infradead.org] On Behalf Of Wolfram Sang > > Sent: Thursday, November 24, 2011 4:38 PM > > To: Uwe Kleine-K?nig > > Cc: Shawn Guo; linux-arm-kernel at lists.infradead.org; Ben Dooks; linux- > > i2c at vger.kernel.org > > Subject: Re: [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() > > > > On Thu, Nov 24, 2011 at 09:29:03AM +0100, Uwe Kleine-K?nig wrote: > > > Hello Shawn, > > > > > > On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > > > > CC drivers/i2c/busses/i2c-mxs.o > > > > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > > > > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used > > > > uninitialized in this function > > > > > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > > > --- > > > > drivers/i2c/busses/i2c-mxs.c | 2 +- > > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c > > > > b/drivers/i2c/busses/i2c-mxs.c index 7e78f7c..00f098f 100644 > > > > --- a/drivers/i2c/busses/i2c-mxs.c > > > > +++ b/drivers/i2c/busses/i2c-mxs.c > > > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct > > > > mxs_i2c_dev *i2c) > > > > > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, > > > > int len) { > > > > - u32 data; > > > > + u32 data = 0; > > > > int i; > > > > > > > > for (i = 0; i < len; i++) { > > > Looks like a stupid compiler. Then better use > > > > > > u32 uninitialized_var(data); > > > > Leave it as it is and fix the compiler? Will do another test later to > > make sure. > > > > Compiler is wrong? > Just from the code that it seems to be right the 'data' might be used > uninitialized. > Only iff you assume that (0 & 3) could be != 0... Lothar Wa?mann -- ___________________________________________________________ Ka-Ro electronics GmbH | Pascalstra?e 22 | D - 52076 Aachen Phone: +49 2408 1402-0 | Fax: +49 2408 1402-10 Gesch?ftsf?hrer: Matthias Kaussen Handelsregistereintrag: Amtsgericht Aachen, HRB 4996 www.karo-electronics.de | info at karo-electronics.de ___________________________________________________________ ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-24 8:37 ` Wolfram Sang 2011-11-24 9:02 ` Dong Aisheng-B29396 @ 2011-11-25 1:15 ` Shawn Guo 2011-11-25 9:08 ` Wolfram Sang 1 sibling, 1 reply; 12+ messages in thread From: Shawn Guo @ 2011-11-25 1:15 UTC (permalink / raw) To: linux-arm-kernel On Thu, Nov 24, 2011 at 09:37:47AM +0100, Wolfram Sang wrote: > On Thu, Nov 24, 2011 at 09:29:03AM +0100, Uwe Kleine-K?nig wrote: > > Hello Shawn, > > > > On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > > > CC drivers/i2c/busses/i2c-mxs.o > > > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > > > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used uninitialized in this function > > > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > > --- > > > drivers/i2c/busses/i2c-mxs.c | 2 +- > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c > > > index 7e78f7c..00f098f 100644 > > > --- a/drivers/i2c/busses/i2c-mxs.c > > > +++ b/drivers/i2c/busses/i2c-mxs.c > > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct mxs_i2c_dev *i2c) > > > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int len) > > > { > > > - u32 data; > > > + u32 data = 0; > > > int i; > > > > > > for (i = 0; i < len; i++) { > > Looks like a stupid compiler. Then better use > > > > u32 uninitialized_var(data); > > Leave it as it is and fix the compiler? Will do another test later to make > sure. > Ben, do you have an opinion on this, leave it there or fix it? -- Regards, Shawn ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-25 1:15 ` Shawn Guo @ 2011-11-25 9:08 ` Wolfram Sang 2011-12-02 19:42 ` Marek Vasut 0 siblings, 1 reply; 12+ messages in thread From: Wolfram Sang @ 2011-11-25 9:08 UTC (permalink / raw) To: linux-arm-kernel On Fri, Nov 25, 2011 at 09:15:20AM +0800, Shawn Guo wrote: > On Thu, Nov 24, 2011 at 09:37:47AM +0100, Wolfram Sang wrote: > > On Thu, Nov 24, 2011 at 09:29:03AM +0100, Uwe Kleine-K?nig wrote: > > > Hello Shawn, > > > > > > On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > > > > CC drivers/i2c/busses/i2c-mxs.o > > > > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > > > > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used uninitialized in this function > > > > > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > > > --- > > > > drivers/i2c/busses/i2c-mxs.c | 2 +- > > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c > > > > index 7e78f7c..00f098f 100644 > > > > --- a/drivers/i2c/busses/i2c-mxs.c > > > > +++ b/drivers/i2c/busses/i2c-mxs.c > > > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct mxs_i2c_dev *i2c) > > > > > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, int len) > > > > { > > > > - u32 data; > > > > + u32 data = 0; > > > > int i; > > > > > > > > for (i = 0; i < len; i++) { > > > Looks like a stupid compiler. Then better use > > > > > > u32 uninitialized_var(data); > > > > Leave it as it is and fix the compiler? Will do another test later to make > > sure. > > > Ben, do you have an opinion on this, leave it there or fix it? Sorry, forgot the tests yesterday. I don't like random annotations because of faulty compilers. Please fix that one. NACK from me. -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 198 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20111125/d660798d/attachment.sig> ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() 2011-11-25 9:08 ` Wolfram Sang @ 2011-12-02 19:42 ` Marek Vasut 0 siblings, 0 replies; 12+ messages in thread From: Marek Vasut @ 2011-12-02 19:42 UTC (permalink / raw) To: linux-arm-kernel > On Fri, Nov 25, 2011 at 09:15:20AM +0800, Shawn Guo wrote: > > On Thu, Nov 24, 2011 at 09:37:47AM +0100, Wolfram Sang wrote: > > > On Thu, Nov 24, 2011 at 09:29:03AM +0100, Uwe Kleine-K?nig wrote: > > > > Hello Shawn, > > > > > > > > On Thu, Nov 24, 2011 at 03:52:20PM +0800, Shawn Guo wrote: > > > > > CC drivers/i2c/busses/i2c-mxs.o > > > > > > > > > > drivers/i2c/busses/i2c-mxs.c: In function ?mxs_i2c_xfer?: > > > > > drivers/i2c/busses/i2c-mxs.c:196:6: warning: ?data? may be used > > > > > uninitialized in this function > > > > > > > > > > Signed-off-by: Shawn Guo <shawn.guo@linaro.org> > > > > > --- > > > > > > > > > > drivers/i2c/busses/i2c-mxs.c | 2 +- > > > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > > > > > diff --git a/drivers/i2c/busses/i2c-mxs.c > > > > > b/drivers/i2c/busses/i2c-mxs.c index 7e78f7c..00f098f 100644 > > > > > --- a/drivers/i2c/busses/i2c-mxs.c > > > > > +++ b/drivers/i2c/busses/i2c-mxs.c > > > > > @@ -193,7 +193,7 @@ static int mxs_i2c_wait_for_data(struct > > > > > mxs_i2c_dev *i2c) > > > > > > > > > > static int mxs_i2c_finish_read(struct mxs_i2c_dev *i2c, u8 *buf, > > > > > int len) { > > > > > > > > > > - u32 data; > > > > > + u32 data = 0; > > > > > > > > > > int i; > > > > > > > > > > for (i = 0; i < len; i++) { > > > > > > > > Looks like a stupid compiler. Then better use > > > > > > > > u32 uninitialized_var(data); > > > > > > Leave it as it is and fix the compiler? Will do another test later to > > > make sure. > > > > Ben, do you have an opinion on this, leave it there or fix it? > > Sorry, forgot the tests yesterday. I don't like random annotations > because of faulty compilers. Please fix that one. NACK from me. Hi, I hit this with emdebian gcc 4.6.1 ... I'd be for merging this u32 data = 0; stuff, so on my side it's ACK. M ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2011-12-02 19:42 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-11-24 7:52 [PATCH] i2c-mxs: fix compile warning in mxs_i2c_xfer() Shawn Guo 2011-11-24 8:00 ` Wolfram Sang 2011-11-24 8:24 ` Shawn Guo 2011-11-24 8:29 ` Uwe Kleine-König 2011-11-24 8:37 ` Wolfram Sang 2011-11-24 9:02 ` Dong Aisheng-B29396 2011-11-24 9:06 ` Uwe Kleine-König 2011-11-24 9:09 ` Dong Aisheng-B29396 2011-11-24 9:11 ` Lothar Waßmann 2011-11-25 1:15 ` Shawn Guo 2011-11-25 9:08 ` Wolfram Sang 2011-12-02 19:42 ` Marek Vasut
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).