[PATCH] staging: lustre: lustre: libcfs: define symbols as static

2015-02-05 Thread Chen Weixiang
This patch fixes the following warning using sparse - warning: symbol 'libcfs_debug_mb' was not declared. Should it be static? - warning: symbol 'portal_enter_debugger' was not declared. Should it be static? Signed-off-by: Chen Weixiang --- drivers/staging/lustre/lustr

[PATCH] staging: lustre: lustre: libcfs: define symbols as static

2015-02-05 Thread Chen Weixiang
-- Best regards, Chen Weixiang (Alex) >From 8de8c7f65f4529e3febb384402344d062f64230b Mon Sep 17 00:00:00 2001 From: Chen Weixiang Date: Thu, 5 Feb 2015 20:07:02 +0800 Subject: [PATCH] staging: lustre: lustre: libcfs: define symbols as static This patch fixes the following warning using spa

Re: [PATCH v2 1/6] staging: ft1000: spaces required around that '=', '<' and '=='

2014-11-09 Thread Chen Weixiang
On Mon, Nov 10, 2014 at 08:16:16AM +0800, Chen Weixiang wrote: > On Mon, Nov 03, 2014 at 04:05:43PM -0800, Greg Kroah-Hartman wrote: > > > > This patch doesn't apply against my staging.git tree on git.kernel.org > > Please refresh the series against the staging-testing a

Re: [PATCH v2 1/6] staging: ft1000: spaces required around that '=', '<' and '=='

2014-11-09 Thread Chen Weixiang
On Mon, Nov 03, 2014 at 04:05:43PM -0800, Greg Kroah-Hartman wrote: > On Sat, Nov 01, 2014 at 10:47:22AM +0800, Chen Weixiang wrote: > > Remove following code style errors from ft1000/ft1000-usb/ft1000_debug.c > > and ft1000/ft1000-pcmcia/ft1000_hw.c: > > ERROR: spaces

Re: [PATCH v2 1/6] staging: ft1000: spaces required around that '=', '<' and '=='

2014-11-01 Thread Chen Weixiang
t that I'm not fetching latest tags on linux-next before checking the code style errors. Please ignore this patch, sorry. -- Best regards, Chen Weixiang (Alex) ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.or

Re: [PATCH v2 1/6] staging: ft1000: spaces required around that '=', '<' and '=='

2014-11-01 Thread Chen Weixiang
arnings. But those errors/warnings are not related to spaces around '=', '<' and '==' anymore. I thought it's better to fix one code style error/warning in one patch, and put the code style error/warning to Subject line. It's much cle

[PATCH v2 5/6] staging: ft1000: else should follow close brace '}'

2014-10-31 Thread Chen Weixiang
Remove code style error from ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: else should follow close brace '}' Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/stag

[PATCH v2 6/6] staging: ft1000: space required after that ','

2014-10-31 Thread Chen Weixiang
Remove following code style error from ft1000/ft1000-usb/ft1000_debug.c: ERROR: space required after that ',' (ctx:VxV) Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) di

[PATCH v2 4/6] staging: ft1000: "foo * bar" should be "foo *bar"

2014-10-31 Thread Chen Weixiang
Remove following code style error from ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: "foo * bar" should be "foo *bar" Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/s

[PATCH v2 2/6] staging: ft1000: space prohibited after '(', '&' and before ')'

2014-10-31 Thread Chen Weixiang
Remove following code style errors from ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: space prohibited after that open parenthesis '(' ERROR: space prohibited before that close parenthesis ')' ERROR: space prohibited after that '&' (ctx:WxW) Signed-off-by: Chen Wei

[PATCH v2 3/6] staging: ft1000: "(foo*)" should be "(foo *)"

2014-10-31 Thread Chen Weixiang
Remove following code style error from ft1000/ft1000-usb/ft1000_debug.c: ERROR: "(foo*)" should be "(foo *)" Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/s

[PATCH v2 1/6] staging: ft1000: spaces required around that '=', '<' and '=='

2014-10-31 Thread Chen Weixiang
Remove following code style errors from ft1000/ft1000-usb/ft1000_debug.c and ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: spaces required around that '=' (ctx:VxV) ERROR: spaces required around that '<' (ctx:VxV) ERROR: spaces required around that '==' (ctx:V

[PATCH v2 0/6] Remove checkpatch.pl errors from drivers/staging/ft1000

2014-10-31 Thread Chen Weixiang
esis ')' ERROR: space prohibited after that '&' (ctx:WxW) ERROR: "(foo*)" should be "(foo *)" ERROR: "foo * bar" should be "foo *bar" ERROR: else should follow close brace '}' ERROR: space required after that ',' (ctx:VxV)

Re: [PATCH 3/7] staging: ft1000: space required after that ','

2014-10-20 Thread Chen Weixiang
On Sun, Oct 19, 2014 at 03:27:00PM +0200, Konrad Zapalowicz wrote: > On 10/19, Chen Weixiang wrote: > > -u8 ConnectionMsg[] = > > {0x00,0x44,0x10,0x20,0x80,0x00,0x00,0x00,0x00,0x00,0x03,0x00,0x00

[PATCH 2/7] staging: ft1000: spaces required around that '=', '<' and '=='

2014-10-19 Thread Chen Weixiang
Remove code style errors from ft1000/ft1000-usb/ft1000_debug.c and ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: spaces required around that '=' (ctx:VxV) ERROR: spaces required around that '<' (ctx:VxV) ERROR: spaces required around that '==' (ctx:VxV) Signed-of

[PATCH 1/7] staging: ft1000: do not initialise statics to 0 or NULL

2014-10-19 Thread Chen Weixiang
Remove following checkpatch.pl error from ft1000/ft1000-usb/ft1000_debug.c ERROR: do not initialise statics to 0 or NULL Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/ft1000

[PATCH 5/7] staging: ft1000: space prohibited after '(' and '&', before ')'

2014-10-19 Thread Chen Weixiang
Remove code style error from ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: space prohibited after that open parenthesis '(' ERROR: space prohibited before that close parenthesis ')' ERROR: space prohibited after that '&' (ctx:WxW) Signed-off-by: Chen Weixiang --- dr

[PATCH 3/7] staging: ft1000: space required after that ','

2014-10-19 Thread Chen Weixiang
Remove code style error from ft1000/ft1000-usb/ft1000_debug.c: ERROR: space required after that ',' (ctx:VxV) Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drive

[PATCH 4/7] staging: ft1000: else should follow close brace '}'

2014-10-19 Thread Chen Weixiang
Remove code style error from ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: else should follow close brace '}' Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 15 +-- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/drivers/stag

[PATCH 0/7] Remove checkpatch.pl errors from drivers/staging/ft1000

2014-10-19 Thread Chen Weixiang
7; (ctx:WxW) ERROR: "(foo*)" should be "(foo *)" ERROR: "foo * bar" should be "foo *bar" Chen Weixiang (7): staging: ft1000: do not initialise statics to 0 or NULL staging: ft1000: spaces required around that '=', '<' and '=

[PATCH 7/7] staging: ft1000: "foo * bar" should be "foo *bar"

2014-10-19 Thread Chen Weixiang
Remove code style error form ft1000/ft1000-pcmcia/ft1000_hw.c: ERROR: "foo * bar" should be "foo *bar" Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/

[PATCH 6/7] staging: ft1000: "(foo*)" should be "(foo *)"

2014-10-19 Thread Chen Weixiang
Remove code style error from ft1000/ft1000-usb/ft1000_debug.c: ERROR: "(foo*)" should be "(foo *)" Signed-off-by: Chen Weixiang --- drivers/staging/ft1000/ft1000-usb/ft1000_debug.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/stagi