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 >> +++ b/drivers/staging/

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. > > drivers/staging/rtl8192u/

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 a/drivers/sta