Re: [PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-10 Thread Christopher Li
On Tue, May 6, 2014 at 1:47 AM, Dan Carpenter wrote: >> diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c >> b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c >> index 426f223..c96dbab 100644 >> --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c >> +++

Re: [PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-10 Thread Christopher Li
On Tue, May 6, 2014 at 1:47 AM, Dan Carpenter dan.carpen...@oracle.com wrote: diff --git a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c b/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c index 426f223..c96dbab 100644 --- a/drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c +++

Re: [PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-06 Thread Christopher Li
On Tue, May 6, 2014 at 1:47 AM, Dan Carpenter wrote: >> - boolsearch_dir[4] = {0, 0, 0, 0}; >> + boolsearch_dir[4] = {0}; > > That's weird. The original code is valid but it generates a sparse > warning. > >

Re: [PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-06 Thread Dan Carpenter
On Tue, May 06, 2014 at 10:22:08AM +0200, Martin Kepplinger wrote: > Don't list elements to initialize. Remaining elements of a partly > initialized array are set to zero. Sparse complained here. > > Signed-off-by: Martin Kepplinger > --- > drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c |

[PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-06 Thread Martin Kepplinger
Don't list elements to initialize. Remaining elements of a partly initialized array are set to zero. Sparse complained here. Signed-off-by: Martin Kepplinger --- drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-06 Thread Martin Kepplinger
Don't list elements to initialize. Remaining elements of a partly initialized array are set to zero. Sparse complained here. Signed-off-by: Martin Kepplinger mart...@posteo.de --- drivers/staging/rtl8192u/ieee80211/rtl819x_TSProc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-06 Thread Dan Carpenter
On Tue, May 06, 2014 at 10:22:08AM +0200, Martin Kepplinger wrote: Don't list elements to initialize. Remaining elements of a partly initialized array are set to zero. Sparse complained here. Signed-off-by: Martin Kepplinger mart...@posteo.de ---

Re: [PATCH] staging: rtl8192u: initialize array in C compliant way

2014-05-06 Thread Christopher Li
On Tue, May 6, 2014 at 1:47 AM, Dan Carpenter dan.carpen...@oracle.com wrote: - boolsearch_dir[4] = {0, 0, 0, 0}; + boolsearch_dir[4] = {0}; That's weird. The original code is valid but it generates a sparse warning.