[PATCH] Staging: rts5139: fix parantheses coding style issue in ms.c

2013-12-13 Thread Preetam D'Souza
This patch fixes an unnecessary return statement parantheses error found in ms.c by the checkpatch.pl tool. Signed-off-by: Preetam D'Souza --- drivers/staging/rts5139/ms.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rts5139/ms.c b/drivers/staging/rts513

Re: [PATCH 1/8] staging:bcm:InterfaceDld: fix checkpatch error

2013-12-13 Thread Dan Carpenter
On Sat, Dec 14, 2013 at 01:05:29AM +0530, Gokulnath A wrote: > fixed all the errors found by checkpatch.pl > This is not very useful. The subjects of this patch series could be improved as well. > Signed-off-by: Gokulnath A > --- > drivers/staging/bcm/InterfaceDld.c | 2 +- > 1 file changed,

Re: [PATCH 2/8] staging:bcm:InterfaceIdleMode: fix checkpatch error

2013-12-13 Thread Dan Carpenter
On Sat, Dec 14, 2013 at 01:05:30AM +0530, Gokulnath A wrote: > if (0xbece3200 == (chip_id&~(0xF0))) > - chip_id = chip_id&~(0xF0); > + chip_id = chip_id&~(0xF01); > if (chip_id == Adapter->chip_id) >

RE: [PATCH net] netvsc: don't flush peers notifying work during setting mtu

2013-12-13 Thread Haiyang Zhang
> -Original Message- > From: Jason Wang [mailto:jasow...@redhat.com] > Sent: Friday, December 13, 2013 4:21 AM > To: KY Srinivasan; Haiyang Zhang; de...@linuxdriverproject.org; > net...@vger.kernel.org; linux-ker...@vger.kernel.org > Cc: Jason Wang > Subject: [PATCH net] netvsc: don't flu

[PATCH 3/8] staging:bcm:InterfaceInit: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceInit.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/drivers/staging/bcm/InterfaceInit.c b/drivers/staging/bcm/InterfaceInit.c index 3acdb58..eaec

[PATCH 7/8] staging:bcm:hostmibs: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/hostmibs.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/bcm/hostmibs.c b/drivers/staging/bcm/hostmibs.c index f55300d..38acfe3 100644 --- a/drivers/stagin

[PATCH 8/8] staging:bcm:Qos: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/Qos.c | 255 +- 1 file changed, 95 insertions(+), 160 deletions(-) diff --git a/drivers/staging/bcm/Qos.c b/drivers/staging/bcm/Qos.c index 1609a2b..417dedf

[PATCH 4/8] staging:bcm:InterfaceIsr: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceIsr.c | 107 - 1 file changed, 45 insertions(+), 62 deletions(-) diff --git a/drivers/staging/bcm/InterfaceIsr.c b/drivers/staging/bcm/InterfaceIsr.c inde

[PATCH 5/8] staging:bcm:InterfaceRx: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceRx.c | 154 +++--- 1 file changed, 62 insertions(+), 92 deletions(-) diff --git a/drivers/staging/bcm/InterfaceRx.c b/drivers/staging/bcm/InterfaceRx.c index

[PATCH 1/8] staging:bcm:InterfaceDld: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceDld.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/bcm/InterfaceDld.c b/drivers/staging/bcm/InterfaceDld.c index 463bdee..a8d0063 100644 --- a/drivers/s

[PATCH 6/8] staging:bcm:InterfaceTx: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceTx.c | 123 -- 1 file changed, 51 insertions(+), 72 deletions(-) diff --git a/drivers/staging/bcm/InterfaceTx.c b/drivers/staging/bcm/InterfaceTx.c index

[PATCH 2/8] staging:bcm:InterfaceIdleMode: fix checkpatch error

2013-12-13 Thread Gokulnath A
fixed all the errors found by checkpatch.pl Signed-off-by: Gokulnath A --- drivers/staging/bcm/InterfaceIdleMode.c | 19 --- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/drivers/staging/bcm/InterfaceIdleMode.c b/drivers/staging/bcm/InterfaceIdleMode.c index 595

[PATCH] staging: gdm72xx: Fix WARNING space prohibited before semicolon in gdm_qos.c

2013-12-13 Thread Masanari Iida
Fixed WARNING: space prohibited before semicolon found by checkpatch.pl in gdm_qos.c Signed-off-by: Masanari Iida --- drivers/staging/gdm72xx/gdm_qos.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/gdm72xx/gdm_qos.c b/drivers/staging/gdm72xx/gdm_qos.c index

[PATCH] staging: dgap: Fixed trailing white space from dgap_conf.h

2013-12-13 Thread Masanari Iida
This patch fixes "ERROR: trailing whitespace" found by checkpatch.pl. Signed-off-by: Masanari Iida --- drivers/staging/dgap/dgap_conf.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/dgap/dgap_conf.h b/drivers/staging/dgap/dgap_conf.h index 8809701..484

[PATCH 4/6] staging: lustre: Fix typo in lustre/lustre/osc

2013-12-13 Thread Masanari Iida
Correct spelling typo in lustre/lustre/osc Signed-off-by: Masanari Iida --- drivers/staging/lustre/lustre/osc/osc_cache.c | 6 +++--- drivers/staging/lustre/lustre/osc/osc_cl_internal.h | 2 +- drivers/staging/lustre/lustre/osc/osc_lock.c| 2 +- drivers/staging/lustre/lustre/osc/os

[PATCH 5/6] staging: lustre: Fix typo in lustre/lustre/fld and ldlm

2013-12-13 Thread Masanari Iida
Correct spelling typo in luster/lustre/fld and ldlm. Signed-off-by: Masanari Iida --- drivers/staging/lustre/lustre/fld/fld_cache.c | 2 +- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 8 drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 6 +++--- drivers/staging/lustre/

[PATCH 6/6] staging: lustre: Fix typo in lustre/lustre/libcfs

2013-12-13 Thread Masanari Iida
Correct spelling typo in lustre/lustre/libcfs Signed-off-by: Masanari Iida --- drivers/staging/lustre/lustre/libcfs/hash.c | 8 drivers/staging/lustre/lustre/libcfs/kernel_user_comm.c | 4 ++-- drivers/staging/lustre/lustre/libcfs/module.c | 2 +- 3 files changed,

[PATCH 3/6] staging: lustre: Fix typo in lustre/lustre/lmv

2013-12-13 Thread Masanari Iida
Fix spelling typo in lustre/lustre/lmv Signed-off-by: Masanari Iida --- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lustre/lmv/lmv_obd.c b/drivers/staging/lustre/lustre/lmv/lmv_obd.c index c286604..

[PATCH 1/6] staging: lustre: Fix typo in lustre/lnet/selftest

2013-12-13 Thread Masanari Iida
Correct spelling typo in lustre/lnet/selftest Signed-off-by: Masanari Iida --- drivers/staging/lustre/lnet/selftest/conctl.c| 8 drivers/staging/lustre/lnet/selftest/console.c | 2 +- drivers/staging/lustre/lnet/selftest/console.h | 2 +- drivers/staging/lustre/lnet/selftest/pin

[PATCH 2/6] staging: lustre: Fix typo in lustre/lustre/lov

2013-12-13 Thread Masanari Iida
Correct spelling typo in lustre/lustre/lov Signed-off-by: Masanari Iida --- drivers/staging/lustre/lustre/lov/lov_lock.c| 2 +- drivers/staging/lustre/lustre/lov/lov_merge.c | 2 +- drivers/staging/lustre/lustre/lov/lov_obd.c | 2 +- drivers/staging/lustre/lustre/lov/lov_pool.c|

RE: [PATCH] staging: comedi: 8255_pci: fix for newer PCI-DIO48H

2013-12-13 Thread Hartley Sweeten
On Friday, December 13, 2013 5:01 AM, Ian Abbott wrote: > > At some point, Measurement Computing / ComputerBoards redesigned the > PCI-DIO48H to use a PLX PCI interface chip instead of an AMCC chip. > This meant they had to put their hardware registers in the PCI BAR 2 > region instead of PCI BAR

[PATCH] Staging: bcm: DDRInit: fixed comment issues.

2013-12-13 Thread Gary Rookard
replaced C99 style with C89 standard style comments to follow the linux kernel coding style thus fixing checkpatch errors respectfully. also, in some instances changed comments from inline to side-bar style. Signed-off-by: Gary Alan Rookard --- On branch staging-next drivers/staging/bcm/DDRInit.

Re: [PATCH 2/2] staging: et131x: improve code consistency: access struct tx_ring

2013-12-13 Thread Mark Einon
On Sun, Dec 08, 2013 at 11:01:07AM +0800, ZHAO Gang wrote: > Let all the code use a pointer called tx_ring to access struct tx_ring > members. > > Signed-off-by: ZHAO Gang Acked-by: Mark Einon ___ devel mailing list de...@linuxdriverproject.org http:/

Re: [PATCH 1/2] staging: et131x: improve code consistency: access struct rx_ring

2013-12-13 Thread Mark Einon
On Sun, Dec 08, 2013 at 11:01:06AM +0800, ZHAO Gang wrote: > Let all the code use a pointer called rx_ring to access struct rx_ring > members. > > Signed-off-by: ZHAO Gang Acked-by: Mark Einon ___ devel mailing list de...@linuxdriverproject.org http:/

[PATCH] staging: comedi: 8255_pci: fix for newer PCI-DIO48H

2013-12-13 Thread Ian Abbott
At some point, Measurement Computing / ComputerBoards redesigned the PCI-DIO48H to use a PLX PCI interface chip instead of an AMCC chip. This meant they had to put their hardware registers in the PCI BAR 2 region instead of PCI BAR 1. Unfortunately, they kept the same PCI device ID for the new des

Re: [PATCH] Staging:bcm:DDRInit: fixed issues w/ indentation

2013-12-13 Thread Greg KH
Care to put a ' ' after your ':'? Look at how other commit messages look like, I really don't want to have to hand-edit commits, I just can't scale when doing that (remember, I get 1000 emails a day...) On Fri, Dec 13, 2013 at 01:00:21AM -0500, Gary Rookard wrote: > restructured the levels

[PATCH net] netvsc: don't flush peers notifying work during setting mtu

2013-12-13 Thread Jason Wang
There's a possible deadlock if we flush the peers notifying work during setting mtu: [ 22.991149] == [ 22.991173] [ INFO: possible circular locking dependency detected ] [ 22.991198] 3.10.0-54.0.1.el7.x86_64.debug #1 Not tainted [ 22.9912

Re: [PATCH v1 9/9] staging: android: binder: Add binder compat layer

2013-12-13 Thread Octavian Purdila
On Fri, Dec 13, 2013 at 7:14 AM, Arve Hjønnevåg wrote: > On Thu, Dec 12, 2013 at 12:45 AM, Octavian Purdila > wrote: >> On Thu, Dec 12, 2013 at 1:00 AM, Arve Hjønnevåg wrote: >>> On Wed, Dec 11, 2013 at 10:10 AM, Octavian Purdila >>> wrote: On Wed, Dec 11, 2013 at 5:21 AM, Arve Hjønnevåg