Re: [PATCH] Staging: rtl8723au: os_intfs: fixed case statement is variable issue

2016-08-13 Thread sunbing
On Aug 12, 2016, at 22:30, Jes Sorensen wrote: > sunbing writes: >> On Aug 11, 2016, at 23:25, Jes Sorensen wrote: >> >>> Bing Sun writes: >>>> Fixed sparse parse error: >>>> Expected constant expression in case statement. >>>>

Re: [PATCH] Staging: rtl8723au: os_intfs: fixed case statement is variable issue

2016-08-11 Thread sunbing
On Aug 11, 2016, at 23:25, Jes Sorensen wrote: > Bing Sun writes: >> Fixed sparse parse error: >> Expected constant expression in case statement. >> >> Signed-off-by: Bing Sun >> --- >> drivers/staging/rtl8723au/os_dep/os_intfs.c | 11 +-- >> 1 file changed, 5 insertions(+), 6 deletion

[PATCH] Staging: ks7010: michael_mic: fixed macros coding style issue

2016-07-17 Thread Sunbing
Fixed coding style issue: Enclose multiple statements macros definition in a do while loop. Use one space around binary operators. Signed-off-by: Sunbing --- drivers/staging/ks7010/michael_mic.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/drivers

Re: [PATCH] Enclose multiple statements macros in a do while loop

2016-07-17 Thread sunbing
On Jul 15, 2016, at 8:32, Greg KH wrote: > On Thu, Jul 14, 2016 at 05:01:51PM +0800, Sunbing wrote: >> The formatting of macros definetion in ks7010/michael_mic.c is not >> consistent with the general kernel coding style. >> >> Fix it by the result of scrip

[PATCH] Enclose multiple statements macros in a do while loop

2016-07-14 Thread Sunbing
The formatting of macros definetion in ks7010/michael_mic.c is not consistent with the general kernel coding style. Fix it by the result of scripts/checkpatch.pl. No functional changes. Signed-off-by: Sunbing --- drivers/staging/ks7010/michael_mic.c | 20 +--- 1 file changed