[PATCH v3 2/2] staging: lustre-libcfs: make static-variable constant

2015-08-10 Thread Patrick Boettcher
This static can be made constant as it is never modified. Found during sparse-cleanup. Signed-off-by: Patrick Boettcher --- drivers/staging/lustre/lustre/libcfs/module.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/libcfs/module.c b

[PATCH v3 1/2] staging: lustre-libcfs: fix sparse warning

2015-08-10 Thread Patrick Boettcher
Fix sparse warnings of the following type: warning: symbol '' was not declared. Should it be static? Signed-off-by: Patrick Boettcher --- drivers/staging/lustre/include/linux/libcfs/libcfs.h | 13 + drivers/staging/lustre/lustre/libcfs/linux/linux-module.c | 2 -

[PATCH v2] staging: lustre-libcfs: fix sparse warning

2015-08-08 Thread Patrick Boettcher
Fix sparse warnings of the following type: warning: symbol '' was not declared. Should it be static? Signed-off-by: Patrick Boettcher --- drivers/staging/lustre/include/linux/libcfs/libcfs.h | 13 + drivers/staging/lustre/lustre/libcfs/linux/linux-module.c | 2 -

[PATCH RESENT] staging: lustre-libcfs: fix sparse warning

2015-08-05 Thread Patrick Boettcher
>From b04a87fd78f5e0dcdfe83362df367e4c244a4e3e Mon Sep 17 00:00:00 2001 From: Patrick Boettcher Date: Wed, 5 Aug 2015 09:13:04 +0200 Subject: [PATCH] staging: lustre-libcfs: fix sparse warning Fix sparse warnings of the following type: warning: symbol '' was not declared. Should it be static? Sign

[PATCH] staging: lustre-libcfs: fix sparse warning

2015-08-05 Thread Patrick Boettcher
ick Boettcher Date: Wed, 5 Aug 2015 09:13:04 +0200 Subject: [PATCH] staging: lustre-libcfs: fix sparse warning Fix sparse warnings of the following type: warning: symbol '' was not declared. Should it be static? Signed-off-by: Patrick Boettcher --- drivers/staging/lustre/include

Re: [PATCH] staging: rtl8192u: r8192U_wx.h is now checkpatch-conform

2015-03-07 Thread Patrick Boettcher
On Fri, 6 Mar 2015 16:52:48 -0800 Greg Kroah-Hartman wrote: > > [..] > > Just remove the line please. Already done and integration one week ago. I won't ever make it. I'm too slow :) Thanks for you feedback though. -- Patrick. ___ devel mailing list

[PATCH] staging: rtl8192u: r8192U_wx.h is now checkpatch-conform

2015-02-16 Thread Patrick Boettcher
This patch fixes warnings/errors raised by checkpatch.pl relating to line-length and comments in r8192I_wx.h. Signed-off-by: Patrick Boettcher --- drivers/staging/rtl8192u/r8192U_wx.h | 32 1 file changed, 20 insertions(+), 12 deletions(-) diff --git a/drivers

Re: [PATCH] staging: android: fix coding style error (v2)

2015-01-14 Thread Patrick Boettcher
Hi Dan, Thanks for your valuable input. It's appreciated. On Wed, 14 Jan 2015 12:28:44 +0300 Dan Carpenter wrote: > > lots of Cc:'s [..] > > This block is not needed. This is a whitespace patch not something > controversial where we will need to look at who was CC'd so that we > can get annoye

[PATCH] staging: android: fix coding style error (v2)

2015-01-14 Thread Patrick Boettcher
Simple style fix - 80 char limit was exceeded. This is second version of the patch. Thanks Joe Perches. Context: eudyptula challenge (http://eudyptula-challenge.org/) Signed-off-by: Patrick Boettcher Cc: Greg Kroah-Hartman Cc: Arnd Bergmann Cc: John Stultz Cc: Peter Senna Tschudin Cc

[PATCH] staging: android: fix coding style error

2015-01-14 Thread Patrick Boettcher
Simple style fix - 80 char limit was exceeded. Context: eudyptula challenge (http://eudyptula-challenge.org/) Signed-off-by: Patrick Boettcher Cc: Greg Kroah-Hartman Cc: Arnd Bergmann Cc: John Stultz Cc: Peter Senna Tschudin Cc: Daniel Vetter Cc: Maarten Lankhorst Cc: Tapasweni Pathak