[PATCH] staging: csr: Fix coding style

2012-10-25 Thread SeongJae Park
Fix coding style for csr_log.h Signed-off-by: SeongJae Park --- drivers/staging/csr/csr_log.h | 165 - 1 file changed, 80 insertions(+), 85 deletions(-) diff --git a/drivers/staging/csr/csr_log.h b/drivers/staging/csr/csr_log.h index b280856..26657a5

[PATCH] staging: csr: Fix coding style

2012-10-25 Thread SeongJae Park
Fix coding style for csr_log.h Signed-off-by: SeongJae Park sj38.p...@gmail.com --- drivers/staging/csr/csr_log.h | 165 - 1 file changed, 80 insertions(+), 85 deletions(-) diff --git a/drivers/staging/csr/csr_log.h b/drivers/staging/csr/csr_log.h index

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread SeongJae Park
Thank you for fast reply, Greg. I was very impressed by your session at Linux Forum, sent linux kernel patch, made a document about your session(write and submit your first Linux kernel patch) in Korean, and my patch merged, have discussion with Linux kernel guys... These whole thing about my

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread Greg KH
On Tue, Oct 23, 2012 at 11:41:06AM +0900, SeongJae Park wrote: > Hello, > > > I think it was my mistake. > > CsrThread is enclosed by macro #ifdef __KERNEL__ > and it used by csr_framework_ext.h as CsrThreadHandle type(typedef > used at csr_framework_ext_types.h). > > I think that's why I

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread SeongJae Park
Hello, I think it was my mistake. CsrThread is enclosed by macro #ifdef __KERNEL__ and it used by csr_framework_ext.h as CsrThreadHandle type(typedef used at csr_framework_ext_types.h). I think that's why I didn't see build failure. So, I think I should not remove CsrThread. If there is

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread SeongJae Park
Hello, Thank you very much for your kindness, Greg. I tried to reply to email-list when I received Lauri's email, but I coudln't because my gmail service has some problems. I'm still newbie, sorry. I will send new patch that removes whole struct, as soon as possible(maybe 4~5 hours later).

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread Greg KH
On Tue, Oct 16, 2012 at 10:13:02AM +0300, Lauri Hintsala wrote: > Hi, > > On 10/15/2012 10:55 AM, SeongJae Park wrote: > >Fix coding style of csr_framework_ext_types.h > > > >Signed-off-by: SeongJae Park > >--- > > drivers/staging/csr/csr_framework_ext_types.h | 33 > >

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread Greg KH
On Tue, Oct 16, 2012 at 10:13:02AM +0300, Lauri Hintsala wrote: Hi, On 10/15/2012 10:55 AM, SeongJae Park wrote: Fix coding style of csr_framework_ext_types.h Signed-off-by: SeongJae Park sj38.p...@gmail.com --- drivers/staging/csr/csr_framework_ext_types.h | 33

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread SeongJae Park
Hello, Thank you very much for your kindness, Greg. I tried to reply to email-list when I received Lauri's email, but I coudln't because my gmail service has some problems. I'm still newbie, sorry. I will send new patch that removes whole struct, as soon as possible(maybe 4~5 hours later).

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread SeongJae Park
Hello, I think it was my mistake. CsrThread is enclosed by macro #ifdef __KERNEL__ and it used by csr_framework_ext.h as CsrThreadHandle type(typedef used at csr_framework_ext_types.h). I think that's why I didn't see build failure. So, I think I should not remove CsrThread. If there is

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread Greg KH
On Tue, Oct 23, 2012 at 11:41:06AM +0900, SeongJae Park wrote: Hello, I think it was my mistake. CsrThread is enclosed by macro #ifdef __KERNEL__ and it used by csr_framework_ext.h as CsrThreadHandle type(typedef used at csr_framework_ext_types.h). I think that's why I didn't see

Re: [PATCH] staging: csr: fix coding style

2012-10-22 Thread SeongJae Park
Thank you for fast reply, Greg. I was very impressed by your session at Linux Forum, sent linux kernel patch, made a document about your session(write and submit your first Linux kernel patch) in Korean, and my patch merged, have discussion with Linux kernel guys... These whole thing about my

Re: [PATCH] staging: csr: fix coding style

2012-10-16 Thread Lauri Hintsala
Hi, On 10/15/2012 10:55 AM, SeongJae Park wrote: Fix coding style of csr_framework_ext_types.h Signed-off-by: SeongJae Park --- drivers/staging/csr/csr_framework_ext_types.h | 33 +++-- 1 file changed, 15 insertions(+), 18 deletions(-) diff --git

Re: [PATCH] staging: csr: fix coding style

2012-10-16 Thread Lauri Hintsala
Hi, On 10/15/2012 10:55 AM, SeongJae Park wrote: Fix coding style of csr_framework_ext_types.h Signed-off-by: SeongJae Park sj38.p...@gmail.com --- drivers/staging/csr/csr_framework_ext_types.h | 33 +++-- 1 file changed, 15 insertions(+), 18 deletions(-) diff --git

[PATCH] staging: csr: fix coding style

2012-10-15 Thread SeongJae Park
Fix coding style of csr_framework_ext_types.h Signed-off-by: SeongJae Park --- drivers/staging/csr/csr_framework_ext_types.h | 33 +++-- 1 file changed, 15 insertions(+), 18 deletions(-) diff --git a/drivers/staging/csr/csr_framework_ext_types.h

[PATCH] staging: csr: fix coding style

2012-10-15 Thread SeongJae Park
Fix coding style of csr_framework_ext_types.h Signed-off-by: SeongJae Park sj38.p...@gmail.com --- drivers/staging/csr/csr_framework_ext_types.h | 33 +++-- 1 file changed, 15 insertions(+), 18 deletions(-) diff --git a/drivers/staging/csr/csr_framework_ext_types.h