Re: [PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Greg KH
On Wed, Jul 29, 2015 at 07:36:59PM +0530, Chandra S Gorentla wrote: > Fixed - warning: ‘*((void *)+4).pu8Head’ may be used > uninitialized The warning is incorrect, look at the code, I don't see anything wrong with it. I suggest upgrading to a newer version of gcc that doesn't report incorrect

Re: [PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Chandra Gorentla
On Wed, Jul 29, 2015 at 05:20:37PM +0300, Dan Carpenter wrote: > On Wed, Jul 29, 2015 at 07:36:59PM +0530, Chandra S Gorentla wrote: > > Fixed - warning: ‘*((void *)+4).pu8Head’ may be used > > uninitialized > > > > I can't reproduce this warning and the original code looks fine to me. > >

Re: [PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Dan Carpenter
On Wed, Jul 29, 2015 at 07:36:59PM +0530, Chandra S Gorentla wrote: > Fixed - warning: ‘*((void *)+4).pu8Head’ may be used > uninitialized > I can't reproduce this warning and the original code looks fine to me. When you say the word "fix" it means the code is buggy. Otherwise we are just

[PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Chandra S Gorentla
Fixed - warning: ‘*((void *)+4).pu8Head’ may be used uninitialized Signed-off-by: Chandra S Gorentla --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c

Re: [PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Chandra Gorentla
On Wed, Jul 29, 2015 at 05:20:37PM +0300, Dan Carpenter wrote: On Wed, Jul 29, 2015 at 07:36:59PM +0530, Chandra S Gorentla wrote: Fixed - warning: ‘*((void *)strHostIFmsg+4).pu8Head’ may be used uninitialized I can't reproduce this warning and the original code looks fine to me.

Re: [PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Greg KH
On Wed, Jul 29, 2015 at 07:36:59PM +0530, Chandra S Gorentla wrote: Fixed - warning: ‘*((void *)strHostIFmsg+4).pu8Head’ may be used uninitialized The warning is incorrect, look at the code, I don't see anything wrong with it. I suggest upgrading to a newer version of gcc that doesn't report

[PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Chandra S Gorentla
Fixed - warning: ‘*((void *)strHostIFmsg+4).pu8Head’ may be used uninitialized Signed-off-by: Chandra S Gorentla csgoren...@gmail.com --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c

Re: [PATCH] staging: wilc1000: host_interface.c: Fix build warning

2015-07-29 Thread Dan Carpenter
On Wed, Jul 29, 2015 at 07:36:59PM +0530, Chandra S Gorentla wrote: Fixed - warning: ‘*((void *)strHostIFmsg+4).pu8Head’ may be used uninitialized I can't reproduce this warning and the original code looks fine to me. When you say the word fix it means the code is buggy. Otherwise we are