Trivial cleanups for drivers/hv/connection.c

2013-12-31 Thread Pavel Machek
__u32 is only useful for kernel-user interface, u32 should be enough for kernel. Formatting was very confusing around __get_free_pages(). Signed-off-by: Pavel Machek diff --git a/drivers/hv/connection.c b/drivers/hv/connection.c index 936093e..1387208 100644 --- a/drivers/hv/connection.c +++ b/d

Re: [PATCH 1/7] vt6655: Remove unused macros in 80211hdr.h

2013-12-31 Thread Dan Carpenter
On Mon, Dec 30, 2013 at 03:52:31PM +0100, Michael Gunselmann wrote: > From: Martin Hofmann > > The file 80211hdr.h contained 4 macros (in both little and big endain fashion) > that were not used in the driver. Thus, this patch removes them. > Better to leave them in. They are useful as documen

Re: [PATCH 4/7] vt6655: Fixed most of the checkpatch warnings in wpa2

2013-12-31 Thread Dan Carpenter
On Mon, Dec 30, 2013 at 03:52:34PM +0100, Michael Gunselmann wrote: > wpa2.h: Checkpatch does no longer complain about anything > > wpa2.c: Checkpatch complains about some lines that are longer than > 80 characters. Breaking them would deteriorate the readability > so these lines were

[PATCH linux-next 1/2] staging: dgap: fix missing header inclusion

2013-12-31 Thread Vincent Stehlé
Include slab.h to fix the following compilation error: drivers/staging/dgap/dgap_fep5.c: In function ‘dgap_do_config_load’: drivers/staging/dgap/dgap_fep5.c:78:2: error: implicit declaration of function ‘kzalloc’ [-Werror=implicit-function-declaration] Signed-off-by: Vincent Stehlé Cc: Lidz

Re: [PATCH 2/7] vt6655: Fix most of checkpatch.pl errors in wroute

2013-12-31 Thread Dan Carpenter
On Mon, Dec 30, 2013 at 03:52:32PM +0100, Michael Gunselmann wrote: > wroute.h: Fixed all line-over-80-character errors. No errors remain. > wroute.c: Fixed line-over-80-character errors, bracing errors > and C99-comments. > Three warnings remain, fixing them would deteriorate reada

Re: [PATCH] Staging: bcm: Fixed excessive line lengths cleaned up some code

2013-12-31 Thread Dan Carpenter
On Mon, Dec 30, 2013 at 06:54:54AM -0500, Arthur Schwalbenberg wrote: > This is a patch to the Adapter.h file that fixes up excesssive > line length warnings found by the checkpatch.pl tool > > Signed off by: Arthur Schwalbenberg Don't indent the changelog like this. Also the

[PATCH linux-next 2/2] staging: dgap: fix checkpatch warnings

2013-12-31 Thread Vincent Stehlé
Remove a few spaces at beginning and end of line. Remove single statement braces {}. Remove two FSF boilerplate paragraphs. Signed-off-by: Vincent Stehlé Cc: Lidza Louina Cc: Greg Kroah-Hartman Cc: Scott H Kilau Cc: eng.li...@digi.com --- Hi, This can be seen with e.g. linux next-20131224 ar