Re: [PATCH 3/3] IDE: Coding Style fixes to drivers/ide/pci/cy82c693.c

2008-02-25 Thread Paolo Ciarrocchi
On Mon, Feb 25, 2008 at 8:31 AM, Ingo Molnar <[EMAIL PROTECTED]> wrote: > > * Paolo Ciarrocchi <[EMAIL PROTECTED]> wrote: > > > Before: > > total: 34 errors, 14 warnings, 456 lines checked > > > > After: > > total: 0 errors, 8 warnings, 45

Re: [PATCH 3/3] IDE: Coding Style fixes to drivers/ide/pci/cy82c693.c

2008-02-25 Thread Paolo Ciarrocchi
On Mon, Feb 25, 2008 at 8:31 AM, Ingo Molnar [EMAIL PROTECTED] wrote: * Paolo Ciarrocchi [EMAIL PROTECTED] wrote: Before: total: 34 errors, 14 warnings, 456 lines checked After: total: 0 errors, 8 warnings, 456 lines checked sidenote: please also indicate to maintainers

Re: [PATCH] Change a WARN message in checkpatch

2008-02-24 Thread Paolo Ciarrocchi
On 2/25/08, Andy Whitcroft > As we want the messages to be as short as possible, I am leaning towards > standardising on: > > spaces prohibited > spaces required > in that case i would prefer: space not required space required ciao, -- Paolo http://paolo.ciarrocchi.googlepages.com/ -- To

Re: [PATCH 2/3] IDE: Replace __FUNCTION__ with __func__

2008-02-24 Thread Paolo Ciarrocchi
On 2/25/08, Joe Perches <[EMAIL PROTECTED]> wrote: > On Sun, 2008-02-24 at 22:11 +0100, Paolo Ciarrocchi wrote: > > - "driver.\n", __FUNCTION__, drive->name); > > + "driver.\n", __func__, drive->name); > > Is there consensus on this style co

[PATCH 0/2] x86: 2 Coding Style patches

2008-02-24 Thread Paolo Ciarrocchi
This email didn't reach lkml... Hi Ingo, two more coding style patches. Compile tested, against x86/testing 4a6af5bbbd74dd812224900d3468b008d117da29 Paolo Ciarrocchi (2): x86: Coding Style fixes to arch/x86/power/cpu_32.c x86: Coding Style fixes to arch/x86/kernel/cpu/common.c

[PATCH 3/3] IDE: Coding Style fixes to drivers/ide/pci/cy82c693.c

2008-02-24 Thread Paolo Ciarrocchi
Before: total: 34 errors, 14 warnings, 456 lines checked After: total: 0 errors, 8 warnings, 456 lines checked Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/cy82c693.c | 74 ++-- 1 files changed, 37 insertions(

[PATCH 2/3] IDE: Replace __FUNCTION__ with __func__

2008-02-24 Thread Paolo Ciarrocchi
Applies on top of the previous patch. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/ide-cd.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/ide/ide-cd.c b/drivers/ide/ide-cd.c index 3181e71..76317ab 100644 --- a/drive

[PATCH 1/3] IDE: Coding Style fixes to drivers/ide/ide-cd.c

2008-02-24 Thread Paolo Ciarrocchi
Before: total: 43 errors, 66 warnings, 2183 lines checked After: total: 0 errors, 36 warnings, 2192 lines checked I didn't (and I don't plan to) fix the warnings: WARNING: line over 80 characters Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/ide-cd.c

[PATCH 0/3] Various IDE coding style fixes

2008-02-24 Thread Paolo Ciarrocchi
Hi Bart, three more patches, all are compile tested and against yesterday linux-next "IDE: Replace __FUNCTION__ with __func__" depends on "IDE: Coding Style fixes to drivers/ide/ide-cd.c" Paolo Ciarrocchi (3): IDE: Coding Style fixes to drivers/ide/ide-cd.

[PATCH 4/6] IDE: Coding Style fixes to drivers/ide/pci/it8213.c

2008-02-24 Thread Paolo Ciarrocchi
File is now error free, only a few WARNING: line over 80 characters are left. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/it8213.c |9 - 1 files changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/ide/pci/it8213.c b/drive

Re: [PATCH] Change a WARN message in checkpatch

2008-02-24 Thread Paolo Ciarrocchi
On Sun, Feb 24, 2008 at 4:18 PM, Benny Halevy <[EMAIL PROTECTED]> wrote: [...] > How about: > - WARN("no space between function name and > open parenthesis '('\n" . $herecurr); > + WARN("there should be no space between > function

Re: [PATCH 0/6] IDE: new series of Coding Style Fixes

2008-02-24 Thread Paolo Ciarrocchi
On Sun, Feb 24, 2008 at 5:38 PM, Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]> wrote: > > On Saturday 23 February 2008, Paolo Ciarrocchi wrote: > > Bart, > > here is a new series of patches that remove some errors > > and warnings reported by checkpatch.pl from th

[PATCH 2/2] x86: Coding Style fixes to arch/x86/kernel/cpu/common.c

2008-02-24 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/common.c | 105 ++--- 1 files changed, 56 insertions(+), 49 deletions(-) diff --git a/arch/x86/kernel/cpu/common.c b/arch/x86/kern

[PATCH 1/2] x86: Coding Style fixes to arch/x86/power/cpu_32.c

2008-02-24 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Patch is against x86#testing Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/power/cpu_32.c | 41 +++-- 1 files changed, 23 insertions(+), 18 deletions(-) diff --git a/arch/x86/power/cpu_32.c

[PATCH 1/2] x86: Coding Style fixes to arch/x86/power/cpu_32.c

2008-02-24 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Patch is against x86#testing Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/power/cpu_32.c | 41 +++-- 1 files changed, 23 insertions(+), 18 deletions(-) diff --git a/arch/x86/power/cpu_32.c b/arch

[PATCH 2/2] x86: Coding Style fixes to arch/x86/kernel/cpu/common.c

2008-02-24 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/cpu/common.c | 105 ++--- 1 files changed, 56 insertions(+), 49 deletions(-) diff --git a/arch/x86/kernel/cpu/common.c b/arch/x86/kernel/cpu

Re: [PATCH 0/6] IDE: new series of Coding Style Fixes

2008-02-24 Thread Paolo Ciarrocchi
On Sun, Feb 24, 2008 at 5:38 PM, Bartlomiej Zolnierkiewicz [EMAIL PROTECTED] wrote: On Saturday 23 February 2008, Paolo Ciarrocchi wrote: Bart, here is a new series of patches that remove some errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches

Re: [PATCH] Change a WARN message in checkpatch

2008-02-24 Thread Paolo Ciarrocchi
On Sun, Feb 24, 2008 at 4:18 PM, Benny Halevy [EMAIL PROTECTED] wrote: [...] How about: - WARN(no space between function name and open parenthesis '('\n . $herecurr); + WARN(there should be no space between function name and

[PATCH 4/6] IDE: Coding Style fixes to drivers/ide/pci/it8213.c

2008-02-24 Thread Paolo Ciarrocchi
File is now error free, only a few WARNING: line over 80 characters are left. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/it8213.c |9 - 1 files changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/ide/pci/it8213.c b/drivers/ide

[PATCH 0/3] Various IDE coding style fixes

2008-02-24 Thread Paolo Ciarrocchi
Hi Bart, three more patches, all are compile tested and against yesterday linux-next IDE: Replace __FUNCTION__ with __func__ depends on IDE: Coding Style fixes to drivers/ide/ide-cd.c Paolo Ciarrocchi (3): IDE: Coding Style fixes to drivers/ide/ide-cd.c IDE: Replace __FUNCTION__

[PATCH 1/3] IDE: Coding Style fixes to drivers/ide/ide-cd.c

2008-02-24 Thread Paolo Ciarrocchi
Before: total: 43 errors, 66 warnings, 2183 lines checked After: total: 0 errors, 36 warnings, 2192 lines checked I didn't (and I don't plan to) fix the warnings: WARNING: line over 80 characters Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/ide-cd.c | 245

[PATCH 2/3] IDE: Replace __FUNCTION__ with __func__

2008-02-24 Thread Paolo Ciarrocchi
Applies on top of the previous patch. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/ide-cd.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/ide/ide-cd.c b/drivers/ide/ide-cd.c index 3181e71..76317ab 100644 --- a/drivers/ide/ide

[PATCH 3/3] IDE: Coding Style fixes to drivers/ide/pci/cy82c693.c

2008-02-24 Thread Paolo Ciarrocchi
Before: total: 34 errors, 14 warnings, 456 lines checked After: total: 0 errors, 8 warnings, 456 lines checked Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/cy82c693.c | 74 ++-- 1 files changed, 37 insertions(+), 37 deletions

[PATCH 0/2] x86: 2 Coding Style patches

2008-02-24 Thread Paolo Ciarrocchi
This email didn't reach lkml... Hi Ingo, two more coding style patches. Compile tested, against x86/testing 4a6af5bbbd74dd812224900d3468b008d117da29 Paolo Ciarrocchi (2): x86: Coding Style fixes to arch/x86/power/cpu_32.c x86: Coding Style fixes to arch/x86/kernel/cpu/common.c

Re: [PATCH 2/3] IDE: Replace __FUNCTION__ with __func__

2008-02-24 Thread Paolo Ciarrocchi
On 2/25/08, Joe Perches [EMAIL PROTECTED] wrote: On Sun, 2008-02-24 at 22:11 +0100, Paolo Ciarrocchi wrote: - driver.\n, __FUNCTION__, drive-name); + driver.\n, __func__, drive-name); Is there consensus on this style conversion? I did that in a separate patch because i'm not sure about

Re: [PATCH] Change a WARN message in checkpatch

2008-02-24 Thread Paolo Ciarrocchi
On 2/25/08, Andy Whitcroft As we want the messages to be as short as possible, I am leaning towards standardising on: spaces prohibited where spaces required where in that case i would prefer: space not required where space required where ciao, -- Paolo

[PATCH 6/6] IDE: Coding Style fixes to drivers/ide/pci/cmd640.c

2008-02-23 Thread Paolo Ciarrocchi
Fix all the errors and a few warnings. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/cmd640.c | 90 +++--- 1 files changed, 45 insertions(+), 45 deletions(-) diff --git a/drivers/ide/pci/cmd640.c b/drive

[PATCH 3/6] IDE: Coding Style fixes to drivers/ide/legacy/ali14xx.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free, 2 warnings left. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/legacy/ali14xx.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/ide/legacy/ali14xx.c b/drivers/ide/legacy/ali14xx.c

[PATCH 5/6] IDE: Coding Style fixes to drivers/ide/ide-floppy.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/ide-floppy.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 5f133df..cea77e0 100644 --- a/d

[PATCH 2/6] IDE: Coding Style fixes to drivers/ide/legacy/hd.c

2008-02-23 Thread Paolo Ciarrocchi
Fix a lot of errors and warnings. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/legacy/hd.c | 79 --- 1 files changed, 40 insertions(+), 39 deletions(-) diff --git a/drivers/ide/legacy/hd.c b/drivers/ide/

[PATCH 1/6] IDE: Coding Style fixes to drivers/ide/pci/opti621.c

2008-02-23 Thread Paolo Ciarrocchi
Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/opti621.c | 50 ++-- 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/ide/pci/opti621.c b/drivers/ide/pci/opti621.c index cfd46ae..3

[PATCH 0/6] IDE: new series of Coding Style Fixes

2008-02-23 Thread Paolo Ciarrocchi
Bart, here is a new series of patches that remove some errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches have been compile tested and are against this morning (CET time :-) linux-next Paolo Ciarrocchi (6): IDE: Coding Style fixes to drivers/ide/pci

Re: [PATCH 08/10] IDE: Coding Style fixes to drivers/ide/pci/generic.c

2008-02-23 Thread Paolo Ciarrocchi
> | -/** > | +/* > ^^^ > > Hi Paolo, it seems you just broke kdoc structure thank you for your comment, i didn't know that. Won't happen in future patches. Bart, can you fix that or do you want me to re-create the series? Ciao, -- Paolo http://paolo.ciarrocchi.googlepages.com/ -- To

[PATCH 10/10] IDE: Coding Style fixes to drivers/ide/pci/tc86c001.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/tc86c001.c | 28 ++-- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/ide/pci/tc86c001.c b/drivers/ide/pci/tc86

Re: [PATCH 00/10] IDE: Coding Style Fixes

2008-02-23 Thread Paolo Ciarrocchi
On Sat, Feb 23, 2008 at 3:04 PM, Paolo Ciarrocchi <[EMAIL PROTECTED]> wrote: > Hi Bartlomiej, > The following 10 patches remove a huge number of errors and warnings > reported by checkpatch.pl from the IDE subsystem. > > All the patches have been compile tested.

[PATCH 09/10] IDE: Coding Style fixes to drivers/ide/pci/jmicron.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/jmicron.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/ide/pci/jmicron.c b/drivers/ide/pci/jmicron.c index 440266f..673f7dc 100644 --- a/d

[PATCH 07/10] IDE: Coding Style fixes to drivers/ide/pci/slc90e66.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free, only 1 warning left. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/slc90e66.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/ide/pci/slc90e66.c b/drivers/ide/pci/slc90e66.c index 2

[PATCH 08/10] IDE: Coding Style fixes to drivers/ide/pci/generic.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/generic.c | 50 ++-- 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/ide/pci/generic.c b/drivers/ide/pci/gen

[PATCH 05/10] IDE: Coding Style fixes to drivers/ide/ide-proc.c

2008-02-23 Thread Paolo Ciarrocchi
Lot of errors and warnings removed. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/ide-proc.c | 175 +++ 1 files changed, 86 insertions(+), 89 deletions(-) diff --git a/drivers/ide/ide-proc.c b/drivers/i

[PATCH 06/10] IDE: Coding Style fixes to drivers/ide/legacy/ide-4drives.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/legacy/ide-4drives.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/ide/legacy/ide-4drives.c b/drivers/ide/legacy/ide-4drives.c

[PATCH 03/10] IDE: Coding Style fixes to drivers/ide/ide-pnp.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/ide-pnp.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/ide/ide-pnp.c b/drivers/ide/ide-pnp.c index c125b05..479663d 100644 --- a/drive

[PATCH 04/10] IDE: Coding Style fixes to drivers/ide/setup-pci.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/setup-pci.c | 45 - 1 files changed, 20 insertions(+), 25 deletions(-) diff --git a/drivers/ide/setup-pci.c b/drivers/ide/setup-pci.c

[PATCH 02/10] IDE: Coding Style fixes to drivers/ide/legacy/umc8672.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/legacy/umc8672.c | 53 - 1 files changed, 26 insertions(+), 27 deletions(-) diff --git a/drivers/ide/legacy/umc8672.c b/drivers/ide/

[PATCH 01/10] IDE: Coding Style fixes to drivers/ide/pci/sis5513.c

2008-02-23 Thread Paolo Ciarrocchi
About 300 errors and warnings fixed. File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- drivers/ide/pci/sis5513.c | 190 ++--- 1 files changed, 93 insertions(+), 97 deletions(-) diff --git a/drivers/i

[PATCH 00/10] IDE: Coding Style Fixes

2008-02-23 Thread Paolo Ciarrocchi
Hi Bartlomiej, The following 10 patches remove a huge number of errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches have been compile tested. Paolo Ciarrocchi (10): IDE: Coding Style fixes to drivers/ide/pci/sis5513.c IDE: Coding Style fixes

Re: [PATCH] Change a WARN message in checkpatch

2008-02-23 Thread Paolo Ciarrocchi
On Mon, Jan 28, 2008 at 4:13 PM, Paolo Ciarrocchi <[EMAIL PROTECTED]> wrote: > > On Jan 28, 2008 3:56 PM, Andy Whitcroft <[EMAIL PROTECTED]> wrote: > > On Mon, Jan 14, 2008 at 11:29:13PM +0100, Paolo Ciarrocchi wrote: > > > Hi Andy, > > > When

Re: [PATCH] Change a WARN message in checkpatch

2008-02-23 Thread Paolo Ciarrocchi
On Mon, Jan 28, 2008 at 4:13 PM, Paolo Ciarrocchi [EMAIL PROTECTED] wrote: On Jan 28, 2008 3:56 PM, Andy Whitcroft [EMAIL PROTECTED] wrote: On Mon, Jan 14, 2008 at 11:29:13PM +0100, Paolo Ciarrocchi wrote: Hi Andy, When I started using checkpatch I was confused by the following WARN

[PATCH 00/10] IDE: Coding Style Fixes

2008-02-23 Thread Paolo Ciarrocchi
Hi Bartlomiej, The following 10 patches remove a huge number of errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches have been compile tested. Paolo Ciarrocchi (10): IDE: Coding Style fixes to drivers/ide/pci/sis5513.c IDE: Coding Style fixes

[PATCH 01/10] IDE: Coding Style fixes to drivers/ide/pci/sis5513.c

2008-02-23 Thread Paolo Ciarrocchi
About 300 errors and warnings fixed. File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/sis5513.c | 190 ++--- 1 files changed, 93 insertions(+), 97 deletions(-) diff --git a/drivers/ide/pci

[PATCH 02/10] IDE: Coding Style fixes to drivers/ide/legacy/umc8672.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/legacy/umc8672.c | 53 - 1 files changed, 26 insertions(+), 27 deletions(-) diff --git a/drivers/ide/legacy/umc8672.c b/drivers/ide/legacy/umc8672

[PATCH 03/10] IDE: Coding Style fixes to drivers/ide/ide-pnp.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/ide-pnp.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/ide/ide-pnp.c b/drivers/ide/ide-pnp.c index c125b05..479663d 100644 --- a/drivers/ide/ide

[PATCH 04/10] IDE: Coding Style fixes to drivers/ide/setup-pci.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/setup-pci.c | 45 - 1 files changed, 20 insertions(+), 25 deletions(-) diff --git a/drivers/ide/setup-pci.c b/drivers/ide/setup-pci.c index

[PATCH 05/10] IDE: Coding Style fixes to drivers/ide/ide-proc.c

2008-02-23 Thread Paolo Ciarrocchi
Lot of errors and warnings removed. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/ide-proc.c | 175 +++ 1 files changed, 86 insertions(+), 89 deletions(-) diff --git a/drivers/ide/ide-proc.c b/drivers/ide/ide

[PATCH 06/10] IDE: Coding Style fixes to drivers/ide/legacy/ide-4drives.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/legacy/ide-4drives.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/ide/legacy/ide-4drives.c b/drivers/ide/legacy/ide-4drives.c index d7bc94f

[PATCH 07/10] IDE: Coding Style fixes to drivers/ide/pci/slc90e66.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free, only 1 warning left. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/slc90e66.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/ide/pci/slc90e66.c b/drivers/ide/pci/slc90e66.c index 2ce384a

[PATCH 08/10] IDE: Coding Style fixes to drivers/ide/pci/generic.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/generic.c | 50 ++-- 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/ide/pci/generic.c b/drivers/ide/pci/generic.c

[PATCH 09/10] IDE: Coding Style fixes to drivers/ide/pci/jmicron.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/jmicron.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/drivers/ide/pci/jmicron.c b/drivers/ide/pci/jmicron.c index 440266f..673f7dc 100644 --- a/drivers

[PATCH 10/10] IDE: Coding Style fixes to drivers/ide/pci/tc86c001.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/tc86c001.c | 28 ++-- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/ide/pci/tc86c001.c b/drivers/ide/pci/tc86c001.c

Re: [PATCH 00/10] IDE: Coding Style Fixes

2008-02-23 Thread Paolo Ciarrocchi
On Sat, Feb 23, 2008 at 3:04 PM, Paolo Ciarrocchi [EMAIL PROTECTED] wrote: Hi Bartlomiej, The following 10 patches remove a huge number of errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches have been compile tested. I forgot to mention that the series

Re: [PATCH 08/10] IDE: Coding Style fixes to drivers/ide/pci/generic.c

2008-02-23 Thread Paolo Ciarrocchi
| -/** | +/* ^^^ Hi Paolo, it seems you just broke kdoc structure thank you for your comment, i didn't know that. Won't happen in future patches. Bart, can you fix that or do you want me to re-create the series? Ciao, -- Paolo http://paolo.ciarrocchi.googlepages.com/ -- To unsubscribe

[PATCH 0/6] IDE: new series of Coding Style Fixes

2008-02-23 Thread Paolo Ciarrocchi
Bart, here is a new series of patches that remove some errors and warnings reported by checkpatch.pl from the IDE subsystem. All the patches have been compile tested and are against this morning (CET time :-) linux-next Paolo Ciarrocchi (6): IDE: Coding Style fixes to drivers/ide/pci

[PATCH 1/6] IDE: Coding Style fixes to drivers/ide/pci/opti621.c

2008-02-23 Thread Paolo Ciarrocchi
Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/opti621.c | 50 ++-- 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/drivers/ide/pci/opti621.c b/drivers/ide/pci/opti621.c index cfd46ae..3df74b7

[PATCH 2/6] IDE: Coding Style fixes to drivers/ide/legacy/hd.c

2008-02-23 Thread Paolo Ciarrocchi
Fix a lot of errors and warnings. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/legacy/hd.c | 79 --- 1 files changed, 40 insertions(+), 39 deletions(-) diff --git a/drivers/ide/legacy/hd.c b/drivers/ide/legacy

[PATCH 3/6] IDE: Coding Style fixes to drivers/ide/legacy/ali14xx.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free, 2 warnings left. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/legacy/ali14xx.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/ide/legacy/ali14xx.c b/drivers/ide/legacy/ali14xx.c index

[PATCH 5/6] IDE: Coding Style fixes to drivers/ide/ide-floppy.c

2008-02-23 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/ide-floppy.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/ide/ide-floppy.c b/drivers/ide/ide-floppy.c index 5f133df..cea77e0 100644 --- a/drivers/ide

[PATCH 6/6] IDE: Coding Style fixes to drivers/ide/pci/cmd640.c

2008-02-23 Thread Paolo Ciarrocchi
Fix all the errors and a few warnings. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- drivers/ide/pci/cmd640.c | 90 +++--- 1 files changed, 45 insertions(+), 45 deletions(-) diff --git a/drivers/ide/pci/cmd640.c b/drivers/ide

[PATCH 17/20] x86: Coding Style fixes to arch/x86/oprofile/nmi_timer_int.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/oprofile/nmi_timer_int.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/x86/oprofile/nmi_timer_int.c b/arch/x86/oprofile/nmi_timer

[PATCH 20/20] x86: Coding Style fixes to arch/x86/xen/multicalls.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/xen/multicalls.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/xen/multicalls.c b/arch/x86/xen/multicalls.c index 5e6f36f..5791eb2 100644 ---

[PATCH 19/20] x86: Coding Style fixes to arch/x86/kernel/msr.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/msr.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/kernel/msr.c b/arch/x86/kernel/msr.c index af51ea8..4dfb405 100644 ---

[PATCH 18/20] x86: Coding Style fixes to arch/x86/kernel/acpi/sleep.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/acpi/sleep.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/kernel/acpi/sleep.c b/arch/x86/kernel/acpi/sleep.c index 6bc815c..dd78326

[PATCH 16/20] x86: Coding Style fixes to arch/x86/kernel/cpu/cyrix.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/cyrix.c | 115 +-- 1 files changed, 57 insertions(+), 58 deletions(-) diff --git a/arch/x86/kernel/cpu/cyrix.c b/arch/x86/kern

[PATCH 14/20] x86: Coding Style fixes to arch/x86/kernel/cpu/mcheck/mce_32.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/mcheck/mce_32.c | 50 +- 1 files changed, 25 insertions(+), 25 deletions(-) diff --git a/arch/x86/kernel/cpu/mcheck/mce_32.c b/arch/x86/

[PATCH 15/20] x86: Coding Style fixes to arch/x86/boot/cpucheck.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/boot/cpucheck.c | 20 ++-- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/x86/boot/cpucheck.c b/arch/x86/boot/cpucheck.c index 7

[PATCH 13/20] x86: Coding Style fixes to arch/x86/kernel/cpu/mcheck/winchip.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/mcheck/winchip.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/x86/kernel/cpu/mcheck/winchip.c b/arch/x86/kernel/cpu/

[PATCH 11/20] x86: Coding Style fixes to arch/x86/kernel/vm86_32.c

2008-02-22 Thread Paolo Ciarrocchi
Fix plenty of errors and warnings. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/vm86_32.c | 174 +++-- 1 files changed, 89 insertions(+), 85 deletions(-) diff --git a/arch/x86/kernel/vm86_32.c b/arch/x86/

[PATCH 12/20] x86: Coding Style fixes to arch/x86/kernel/cpu/mcheck/non-fatal.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/mcheck/non-fatal.c | 21 - 1 files changed, 12 insertions(+), 9 deletions(-) diff --git a/arch/x86/kernel/cpu/mcheck/non-fatal.c b/ar

[PATCH 09/20] x86: Coding Style fixes to arch/x86/kernel/cpu/transmeta.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/transmeta.c | 22 -- 1 files changed, 12 insertions(+), 10 deletions(-) diff --git a/arch/x86/kernel/cpu/transmeta.c b/arch/x86/kernel/cpu/transmeta.c

[PATCH 10/20] x86: Coding Style fixes to arch/x86/kernel/cpu/amd.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/amd.c | 94 +++-- 1 files changed, 48 insertions(+), 46 deletions(-) diff --git a/arch/x86/kernel/cpu/amd.c b/arch/x86/kernel/cpu

[PATCH 08/20] x86: Coding Style fixes to arch/x86/lib/memcpy_32.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/lib/memcpy_32.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/lib/memcpy_32.c b/arch/x86/lib/memcpy_32.c index 37756b6..5415a9d

[PATCH 07/20] x86: Coding Style fixes to arch/x86/kernel/cpu/mtrr/state.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/mtrr/state.c | 14 -- 1 files changed, 8 insertions(+), 6 deletions(-) diff --git a/arch/x86/kernel/cpu/mtrr/state.c b/arch/x86/kernel/cpu/mtrr/state.c

[PATCH 06/20] x86: Coding Style fixes to arch/x86/kernel/mca_32.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/mca_32.c | 96 + 1 files changed, 53 insertions(+), 43 deletions(-) diff --git a/arch/x86/kernel/mca_32.c b/arch/x86/kernel/mc

[PATCH 04/20] x86: Coding Style fixes to arch/x86/oprofile/init.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/oprofile/init.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/x86/oprofile/init.c b/arch/x86/oprofile/init.c index 5341d48..cdfe4c5

[PATCH 05/20] x86: Coding Style fixes to arch/x86/lib/strstr_3

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/lib/strstr_32.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/lib/strstr_32.c b/arch/x86/lib/strstr_32.c index a3dafbf..42e8a50 100644 --- a/ar

[PATCH 03/20] x86: Coding Style fixes to arch/x86/kernel/cpu/intel.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/cpu/intel.c | 83 ++ 1 files changed, 43 insertions(+), 40 deletions(-) diff --git a/arch/x86/kernel/cpu/intel.c b/arch/x86/kern

[PATCH 01/20] x86: Coding Style fix to arch/x86/boot/pm.c

2008-02-22 Thread Paolo Ciarrocchi
Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/boot/pm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/boot/pm.c b/arch/x86/boot/pm.c index 1a0f936..a93cb8b 100644 --- a/arch/x86/boot/pm.c +++ b/arch/x86/boot/pm.c @@ -100,7 +100,7 @@

[PATCH 02/20] x86: Coding Style fixes to arch/x86/kernel/summit_32.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Signed-off-by: Paolo Ciarrocchi <[EMAIL PROTECTED]> --- arch/x86/kernel/summit_32.c | 43 +++ 1 files changed, 23 insertions(+), 20 deletions(-) diff --git a/arch/x86/kernel/summit_32.c b/arch/x86/kernel/summit_32.c

[PATCH 00/20] Various Coding Style fixes

2008-02-22 Thread Paolo Ciarrocchi
Various coding style fixes. Paolo Ciarrocchi (20): x86: Coding Style fix to arch/x86/boot/pm.c x86: Coding Style fixes to arch/x86/kernel/summit_32.c x86: Coding Style fixes to arch/x86/kernel/cpu/intel.c x86: Coding Style fixes to arch/x86/oprofile/init.c x86

[PATCH 00/20] Various Coding Style fixes

2008-02-22 Thread Paolo Ciarrocchi
Various coding style fixes. Paolo Ciarrocchi (20): x86: Coding Style fix to arch/x86/boot/pm.c x86: Coding Style fixes to arch/x86/kernel/summit_32.c x86: Coding Style fixes to arch/x86/kernel/cpu/intel.c x86: Coding Style fixes to arch/x86/oprofile/init.c x86

[PATCH 01/20] x86: Coding Style fix to arch/x86/boot/pm.c

2008-02-22 Thread Paolo Ciarrocchi
Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/boot/pm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/boot/pm.c b/arch/x86/boot/pm.c index 1a0f936..a93cb8b 100644 --- a/arch/x86/boot/pm.c +++ b/arch/x86/boot/pm.c @@ -100,7 +100,7 @@ static

[PATCH 02/20] x86: Coding Style fixes to arch/x86/kernel/summit_32.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/summit_32.c | 43 +++ 1 files changed, 23 insertions(+), 20 deletions(-) diff --git a/arch/x86/kernel/summit_32.c b/arch/x86/kernel/summit_32.c index 72f4634

[PATCH 03/20] x86: Coding Style fixes to arch/x86/kernel/cpu/intel.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/cpu/intel.c | 83 ++ 1 files changed, 43 insertions(+), 40 deletions(-) diff --git a/arch/x86/kernel/cpu/intel.c b/arch/x86/kernel/cpu/intel.c

[PATCH 04/20] x86: Coding Style fixes to arch/x86/oprofile/init.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/oprofile/init.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/x86/oprofile/init.c b/arch/x86/oprofile/init.c index 5341d48..cdfe4c5 100644

[PATCH 05/20] x86: Coding Style fixes to arch/x86/lib/strstr_3

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/lib/strstr_32.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/lib/strstr_32.c b/arch/x86/lib/strstr_32.c index a3dafbf..42e8a50 100644 --- a/arch/x86/lib

[PATCH 06/20] x86: Coding Style fixes to arch/x86/kernel/mca_32.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/mca_32.c | 96 + 1 files changed, 53 insertions(+), 43 deletions(-) diff --git a/arch/x86/kernel/mca_32.c b/arch/x86/kernel/mca_32.c index

[PATCH 07/20] x86: Coding Style fixes to arch/x86/kernel/cpu/mtrr/state.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/cpu/mtrr/state.c | 14 -- 1 files changed, 8 insertions(+), 6 deletions(-) diff --git a/arch/x86/kernel/cpu/mtrr/state.c b/arch/x86/kernel/cpu/mtrr/state.c index 9f8ba92

[PATCH 08/20] x86: Coding Style fixes to arch/x86/lib/memcpy_32.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/lib/memcpy_32.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/lib/memcpy_32.c b/arch/x86/lib/memcpy_32.c index 37756b6..5415a9d 100644

[PATCH 09/20] x86: Coding Style fixes to arch/x86/kernel/cpu/transmeta.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/cpu/transmeta.c | 22 -- 1 files changed, 12 insertions(+), 10 deletions(-) diff --git a/arch/x86/kernel/cpu/transmeta.c b/arch/x86/kernel/cpu/transmeta.c index

[PATCH 10/20] x86: Coding Style fixes to arch/x86/kernel/cpu/amd.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/cpu/amd.c | 94 +++-- 1 files changed, 48 insertions(+), 46 deletions(-) diff --git a/arch/x86/kernel/cpu/amd.c b/arch/x86/kernel/cpu/amd.c

[PATCH 11/20] x86: Coding Style fixes to arch/x86/kernel/vm86_32.c

2008-02-22 Thread Paolo Ciarrocchi
Fix plenty of errors and warnings. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/vm86_32.c | 174 +++-- 1 files changed, 89 insertions(+), 85 deletions(-) diff --git a/arch/x86/kernel/vm86_32.c b/arch/x86/kernel

[PATCH 12/20] x86: Coding Style fixes to arch/x86/kernel/cpu/mcheck/non-fatal.c

2008-02-22 Thread Paolo Ciarrocchi
File is now error and warning free. Compile tested. Signed-off-by: Paolo Ciarrocchi [EMAIL PROTECTED] --- arch/x86/kernel/cpu/mcheck/non-fatal.c | 21 - 1 files changed, 12 insertions(+), 9 deletions(-) diff --git a/arch/x86/kernel/cpu/mcheck/non-fatal.c b/arch/x86

  1   2   3   4   >