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
--
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
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
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
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
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
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
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
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
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
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
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
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)
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
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
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
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
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
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
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 '=
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/
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
22 matches
Mail list logo