[PATCH] staging/vme: explicitly use `unsigned int` instead of `unsigned`

2016-05-20 Thread Daniel Lockyer
Kernel code style is to prefer the former so add `int` to the method parameters. Signed-off-by: Daniel Lockyer --- drivers/staging/vme/devices/vme_pio2_gpio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/vme/devices/vme_pio2_gpio.c b/drivers/staging

[PATCH 4/4] SM750FB: Replace C99-style comments

2015-06-09 Thread Daniel Lockyer
This patch replaces the C99-style comments (//) with /* */. Signed-off-by: Daniel Lockyer --- drivers/staging/sm750fb/ddk750_dvi.c | 9 +++-- drivers/staging/sm750fb/ddk750_help.c | 4 ++-- drivers/staging/sm750fb/sm750_cursor.c | 5 +++-- 3 files changed, 8 insertions(+), 10 deletions

[PATCH 3/4] SM750FB: Fix indenting issues from checkpatch.pl

2015-06-09 Thread Daniel Lockyer
This patch fixes the indenting in several files. Most changes were removing spaces or tabbing into place. Signed-off-by: Daniel Lockyer --- drivers/staging/sm750fb/ddk750_dvi.c | 69 +- drivers/staging/sm750fb/ddk750_power.c | 34 - 2 files

[PATCH 2/4] SM750FB: Fix spacing issues from checkpatch.pl

2015-06-09 Thread Daniel Lockyer
This patch fixes spacing issues reported from checkpatch.pl. These include: * between a C keyword and the next character ( "if(" => "if (" ) * between a comma and the next character * between a * and the next character Signed-off-by: Daniel Lockyer --- drivers/staging/s

[PATCH 1/4] SM750FB: Fix bracket placement issue from checkpatch.pl

2015-06-09 Thread Daniel Lockyer
Fixed the issues with curly braces either needing to be on the line above or below in several files. Signed-off-by: Daniel Lockyer --- drivers/staging/sm750fb/ddk750_display.c | 66 ++-- drivers/staging/sm750fb/ddk750_dvi.c | 6 +-- drivers/staging/sm750fb

[PATCH 0/4] Code style fixes from checkpatch.pl

2015-06-09 Thread Daniel Lockyer
Went through a few files and fixed some code style issues reported by checkpatch.pl. Signed-off-by: Daniel Lockyer Daniel Lockyer (4): SM750FB: Fix bracket placement issue from checkpatch.pl SM750FB: Fix spacing issues from checkpatch.pl SM750FB: Fix indenting issues from checkpatch.pl

Re: [PATCH] SM750FB: Checkpatch.pl cleanup and code tidying

2015-06-09 Thread Daniel Lockyer
Understood, working on it now! :) Regards, Daniel On Tue, Jun 09, 2015 at 10:47:34AM -0700, Greg KH wrote: > On Tue, Jun 09, 2015 at 06:14:37PM +0100, Daniel Lockyer wrote: > > Fixed loads of checkpatch.pl issues. Nothing particularly notable. > > But you need to note it. > &

[PATCH] SM750FB: Checkpatch.pl cleanup and code tidying

2015-06-09 Thread Daniel Lockyer
Fixed loads of checkpatch.pl issues. Nothing particularly notable. Mainly indenting, comments or removing dead code. Signed-off-by: Daniel Lockyer --- drivers/staging/sm750fb/ddk750_display.c | 124 +-- drivers/staging/sm750fb/ddk750_dvi.c | 86

Re: [PATCH] Staging: wlan-ng: hfa384x: Added a new line to fix coding style issue

2015-02-24 Thread Daniel Lockyer
On Tue, Feb 24, 2015 at 11:21:37AM -0800, Greg KH wrote: > On Tue, Feb 24, 2015 at 07:15:11PM +0000, Daniel Lockyer wrote: > > On Tue, Feb 24, 2015 at 09:56:39AM -0800, Greg KH wrote: > > > On Tue, Feb 24, 2015 at 05:53:55PM +0000, Daniel Lockyer wrote: > > > &

Re: [PATCH] Staging: wlan-ng: hfa384x: Added a new line to fix coding style issue

2015-02-24 Thread Daniel Lockyer
On Tue, Feb 24, 2015 at 09:56:39AM -0800, Greg KH wrote: > On Tue, Feb 24, 2015 at 05:53:55PM +0000, Daniel Lockyer wrote: > > Fixed a coding style issue. > > > > Signed-off-by: Daniel Lockyer > > --- > > drivers/staging/wlan-ng/hfa384x.h | 1 + > > 1 fil

[PATCH] Staging: wlan-ng: hfa384x: Added a new line to fix coding style issue

2015-02-24 Thread Daniel Lockyer
Fixed a coding style issue. Signed-off-by: Daniel Lockyer --- drivers/staging/wlan-ng/hfa384x.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/wlan-ng/hfa384x.h b/drivers/staging/wlan-ng/hfa384x.h index 8f20910..3bb8c1b 100644 --- a/drivers/staging/wlan-ng/hfa384x.h +++ b