Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread DaeSeok Youn
OK. I send patches again. Thanks. Daeseok Youn. 2014-03-05 10:06 GMT+09:00 Greg KH : > On Wed, Mar 05, 2014 at 09:55:14AM +0900, DaeSeok Youn wrote: >> Hi, greg >> >> I already resend patch 4 and 5. :-) >> >> It had a bug which is noticed by Dan. >> >> I tried to fix assignment in if condition

Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread Greg KH
On Wed, Mar 05, 2014 at 09:55:14AM +0900, DaeSeok Youn wrote: > Hi, greg > > I already resend patch 4 and 5. :-) > > It had a bug which is noticed by Dan. > > I tried to fix assignment in if condition and missed curly brace in inner > loop. > So I fixed that bug and resend patch 4. And patch 5

Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread DaeSeok Youn
Hi, greg I already resend patch 4 and 5. :-) It had a bug which is noticed by Dan. I tried to fix assignment in if condition and missed curly brace in inner loop. So I fixed that bug and resend patch 4. And patch 5 is rebased after fixing patch 4. And I tested to apply these patch to

Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread Greg KH
On Tue, Mar 04, 2014 at 11:10:44AM +0900, Daeseok Youn wrote: > > clean up checkpatch.pl error in linux.c: > ERROR: that open brace { should be on the previous line > > Signed-off-by: Daeseok Youn > --- > drivers/staging/cxt1e1/linux.c | 67 --- > 1 files

Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread Greg KH
On Tue, Mar 04, 2014 at 11:10:44AM +0900, Daeseok Youn wrote: clean up checkpatch.pl error in linux.c: ERROR: that open brace { should be on the previous line Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/cxt1e1/linux.c | 67

Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread DaeSeok Youn
Hi, greg I already resend patch 4 and 5. :-) It had a bug which is noticed by Dan. I tried to fix assignment in if condition and missed curly brace in inner loop. So I fixed that bug and resend patch 4. And patch 5 is rebased after fixing patch 4. And I tested to apply these patch to

Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread Greg KH
On Wed, Mar 05, 2014 at 09:55:14AM +0900, DaeSeok Youn wrote: Hi, greg I already resend patch 4 and 5. :-) It had a bug which is noticed by Dan. I tried to fix assignment in if condition and missed curly brace in inner loop. So I fixed that bug and resend patch 4. And patch 5 is

Re: [PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-04 Thread DaeSeok Youn
OK. I send patches again. Thanks. Daeseok Youn. 2014-03-05 10:06 GMT+09:00 Greg KH gre...@linuxfoundation.org: On Wed, Mar 05, 2014 at 09:55:14AM +0900, DaeSeok Youn wrote: Hi, greg I already resend patch 4 and 5. :-) It had a bug which is noticed by Dan. I tried to fix assignment in if

[PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-03 Thread Daeseok Youn
clean up checkpatch.pl error in linux.c: ERROR: that open brace { should be on the previous line Signed-off-by: Daeseok Youn --- drivers/staging/cxt1e1/linux.c | 67 --- 1 files changed, 21 insertions(+), 46 deletions(-) diff --git

[PATCH 5/7] staging: cxt1e1: fix checkpatch errors with open brace '{'

2014-03-03 Thread Daeseok Youn
clean up checkpatch.pl error in linux.c: ERROR: that open brace { should be on the previous line Signed-off-by: Daeseok Youn daeseok.y...@gmail.com --- drivers/staging/cxt1e1/linux.c | 67 --- 1 files changed, 21 insertions(+), 46 deletions(-) diff --git