[PATCH 3/5] staging: wilc1000: remove the warnings on line over 80 characters.

2015-06-16 Thread Sunghoon Cho
This removes the warnings reported by checkpatch.pl for line over 80 characters. Signed-off-by: Sunghoon Cho --- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.h b/drivers/stagi

[PATCH 4/5] staging: wilc1000: remove prohibited spaces.

2015-06-16 Thread Sunghoon Cho
This patch removes the warnings on the prohibited spaces between function name and open parenthesis. Signed-off-by: Sunghoon Cho --- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.h

[PATCH 1/5] staging: wilc1000: remove the warnings on the multiple blank lines.

2015-06-16 Thread Sunghoon Cho
This patch removes the warnings reported by checkpatch.pl regarding on the multiple blank line uses. Signed-off-by: Sunghoon Cho --- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.h b/

[PATCH 5/5] staging: wilc1000: add a blank line

2015-06-16 Thread Sunghoon Cho
This adds a blank line after struct declaration, WILC_WFI_mon_priv. Signed-off-by: Sunghoon Cho --- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.h b/drivers/staging/wilc1000/wilc_wfi_netdevice.h ind

[PATCH 2/5] staging: wilc1000: remove unnecessary blank line.

2015-06-16 Thread Sunghoon Cho
This patch removes blank line which is not necesssary after an open brace. Signed-off-by: Sunghoon Cho --- drivers/staging/wilc1000/wilc_wfi_netdevice.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_netdevice.h b/drivers/staging/wilc1000/wilc_wfi_netdevice

[PATCH v2] Staging: wilc1000: Remove casting the values returned by kmalloc()

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes casting the values returned by memory allocation functions. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_mon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/staging/w

Re: [PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

2015-06-16 Thread Abdul, Hussain (H.)
On Wednesday 17 June 2015 07:48 AM, gre...@linuxfoundation.org wrote: > On Tue, Jun 16, 2015 at 07:09:56AM +, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch removes casting the values returned by memory allocation >> functions. >> >> Signed-off-by: Abdul Hussain >> --- >>

Re: [PATCH 2/2] staging: wilc1000: modify uninitialized warning

2015-06-16 Thread Greg KH
On Tue, Jun 16, 2015 at 04:11:25PM +0900, Dean Lee wrote: > initialize it. > > Signed-off-by: Dean Lee > --- > drivers/staging/wilc1000/host_interface.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/staging/wilc1000/host_interface.c > b/drivers/staging/wilc

Re: [PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

2015-06-16 Thread gre...@linuxfoundation.org
On Tue, Jun 16, 2015 at 07:09:56AM +, Abdul, Hussain (H.) wrote: > From: Abdul Hussain > > This patch removes casting the values returned by memory allocation functions. > > Signed-off-by: Abdul Hussain > --- > drivers/staging/wilc1000/linux_mon.c | 4 ++-- > 1 file changed, 2 insertions(+

Re: [PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

2015-06-16 Thread gre...@linuxfoundation.org
On Tue, Jun 16, 2015 at 07:10:28AM +, Abdul, Hussain (H.) wrote: > From: Abdul Hussain > > This patch removes casting the values returned by memory allocation functions. > > Signed-off-by: Abdul Hussain > --- > drivers/staging/wilc1000/linux_wlan.c | 6 +++--- > 1 file changed, 3 insertion

RE: [PATCH 5/5] staging: unisys: add error messages to visornic

2015-06-16 Thread Kershner, David A
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, June 16, 2015 10:09 PM > To: Sell, Timothy C > Cc: driverdev-devel@linuxdriverproject.org; jes.soren...@redhat.com; *S- > Par-Maintainer; Romer, Benjamin M > Subject: Re: [PATCH 5/5] staging: unisys: a

Re: [PATCH 5/5] staging: unisys: add error messages to visornic

2015-06-16 Thread Greg KH
On Wed, Jun 17, 2015 at 12:08:11AM +, Sell, Timothy C wrote: > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Tuesday, June 16, 2015 5:36 PM > > To: Romer, Benjamin M > > Cc: driverdev-devel@linuxdriverproject.org; jes.soren...@redhat.com; *S- > > P

[PATCH v3 09/18] staging/lirc_serial: Remove TSC-based timing

2015-06-16 Thread Andy Lutomirski
It wasn't compiled in by default. I suspect that the driver was and still is broken, though -- it's calling udelay with a parameter that's derived from loops_per_jiffy. Cc: Jarod Wilson Cc: de...@driverdev.osuosl.org Cc: Greg Kroah-Hartman Signed-off-by: Andy Lutomirski --- drivers/staging/me

RE: [PATCH 5/5] staging: unisys: add error messages to visornic

2015-06-16 Thread Sell, Timothy C
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, June 16, 2015 5:36 PM > To: Romer, Benjamin M > Cc: driverdev-devel@linuxdriverproject.org; jes.soren...@redhat.com; *S- > Par-Maintainer; Sell, Timothy C > Subject: Re: [PATCH 5/5] staging: unisys: ad

Re: [PATCH 5/5] staging: unisys: add error messages to visornic

2015-06-16 Thread Greg KH
On Mon, Jun 15, 2015 at 11:32:00PM -0400, Benjamin Romer wrote: > From: Tim Sell > > Add error message to genuine rare error paths, and debug messages > to enable relatively non-verbose tracing of code paths > > You can enable debug messages by including this on the kernel command line: > >

Re: [PATCH 3/5] staging: unisys: define structures outside of iochannel

2015-06-16 Thread Greg KH
On Mon, Jun 15, 2015 at 11:31:58PM -0400, Benjamin Romer wrote: > From: David Kershner > > During testing with visornic the offset of num_rcv_bufs > was being reported at 188 instead of 186. The vnic structure > starts at 180 and the macaddr is only 6 bytes long. > > When I defined and packed th

Re: [3.13.y-ckt][PATCH 0/1][Hyper-V] Add support for VMBus panic notifier handler

2015-06-16 Thread Joseph Salisbury
On 06/16/2015 04:41 PM, Greg KH wrote: > On Tue, Jun 16, 2015 at 04:10:46PM -0400, Joseph Salisbury wrote: >> Hello, >> >> Please consider including upstream commits >> 96c1d0581d00f7abe033350edb021a9d947d8d81 and >> 5ef5b6927f14f29cacd78fa1fb861661a5367f13 in the next v3.13.y-ckt release. >> Th

Re: [PATCH 7/8] staging: sm7xxfb: move mode table

2015-06-16 Thread Joe Perches
On Tue, 2015-06-16 at 19:12 +0530, Sudip Mukherjee wrote: > move vgamode table from the header file to the C file and mark it > as const. The slightly odd indentation could be fixed here too. > +static const struct ModeInit vgamode[] = { > + { > + /* mode#0: 640 x 480 16Bpp 60Hz */ >

Re: [3.13.y-ckt][PATCH 0/1][Hyper-V] Add support for VMBus panic notifier handler

2015-06-16 Thread Greg KH
On Tue, Jun 16, 2015 at 04:10:46PM -0400, Joseph Salisbury wrote: > Hello, > > Please consider including upstream commits > 96c1d0581d00f7abe033350edb021a9d947d8d81 and > 5ef5b6927f14f29cacd78fa1fb861661a5367f13 in the next v3.13.y-ckt release. > They were > included mainline as of v4.1-rc1. T

Re: [v3.14.y][PATCH 1/1] Drivers: hv: vmbus: Add support for VMBus panic notifier handler

2015-06-16 Thread Greg KH
On Tue, Jun 16, 2015 at 04:31:38PM -0400, Joseph Salisbury wrote: > From: Nick Meier > > > Hyper-V allows a guest to notify the Hyper-V host that a panic > condition occured. This notification can include up to five 64 > bit values. These 64 bit values are written into crash MSRs. > Once the d

[v3.14.y][PATCH 1/1] Drivers: hv: vmbus: Add support for VMBus panic notifier handler

2015-06-16 Thread Joseph Salisbury
From: Nick Meier Hyper-V allows a guest to notify the Hyper-V host that a panic condition occured. This notification can include up to five 64 bit values. These 64 bit values are written into crash MSRs. Once the data has been written into the crash MSRs, the host is then notified by writing i

[v3.16.y-ckt][v3.18.y][v3.19.y-ckt][v4.0.y][PATCH 1/1]Drivers: hv: vmbus: Add support for VMBus panic notifier handler

2015-06-16 Thread Joseph Salisbury
From: Nick Meier BugLink: http://bugs.launchpad.net/bugs/1463584 Hyper-V allows a guest to notify the Hyper-V host that a panic condition occured. This notification can include up to five 64 bit values. These 64 bit values are written into crash MSRs. Once the data has been written into the cr

[3.13.y-ckt][PATCH 0/1][Hyper-V] Add support for VMBus panic notifier handler

2015-06-16 Thread Joseph Salisbury
Hello, Please consider including upstream commits 96c1d0581d00f7abe033350edb021a9d947d8d81 and 5ef5b6927f14f29cacd78fa1fb861661a5367f13 in the next v3.13.y-ckt release. They were included mainline as of v4.1-rc1. They have been tested and confirmed to resolve http://bugs.launchpad.net/bugs/14

[3.13.y-ckt][PATCH 1/1] Drivers: hv: vmbus: Add support for VMBus panic notifier handler

2015-06-16 Thread Joseph Salisbury
From: Nick Meier BugLink: http://bugs.launchpad.net/bugs/1463584 Hyper-V allows a guest to notify the Hyper-V host that a panic condition occured. This notification can include up to five 64 bit values. These 64 bit values are written into crash MSRs. Once the data has been written into the cr

Hadiah Bonus

2015-06-16 Thread Nestle (Malaysia) Berhad
NestlE (Malaysia) Berhad 22-1, 22nd floor, Menara Surian No. 1, Jalan PJU 7/3 Mutiara Damansara 47810 Petaling Jaya Selangor, Malaysia. HADIAH BONUS KHAS TAHUN BARU. Kami berasa sukacita untuk memaklumkan kepada anda nantikan siaran keputusan Hadiah Bonus NestlE (Malaysia) Berhad khas yang diad

Re: [PATCH] staging: comedi: use BIT macro for bit shift operation

2015-06-16 Thread Ian Abbott
On 16/06/15 15:15, Gujulan Elango, Hari Prasath (H.) wrote: From: Hari Prasath Gujulan Elango This patch silences the Checkpatch.pl warning 'Prefer using the BIT macro' Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/comedi/comedidev.h | 12 ++-- 1 file changed, 6 in

Re: [PATCH] staging: comedi: fix warning to use BIT Macro

2015-06-16 Thread Ian Abbott
On 16/06/15 15:22, Gujulan Elango, Hari Prasath (H.) wrote: From: Hari Prasath Gujulan Elango This patch fixes the checkpatch warning that suggests to use the BIT macro for the shifting operation. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/comedi/comedi.h | 10 +-

[PATCH] staging: nvec: remove duplicated const

2015-06-16 Thread Peng Fan
Sparse checking warning: "drivers/staging/nvec/nvec_ps2.c:172:14: warning: duplicate const". Remove the duplicated const to fix the warning. Signed-off-by: Peng Fan --- drivers/staging/nvec/nvec_ps2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/nvec/nv

[PATCH] staging: comedi: fix warning to use BIT Macro

2015-06-16 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch fixes the checkpatch warning that suggests to use the BIT macro for the shifting operation. Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/comedi/comedi.h | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a

[PATCH] staging: comedi: use BIT macro for bit shift operation

2015-06-16 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch silences the Checkpatch.pl warning 'Prefer using the BIT macro' Signed-off-by: Hari Prasath Gujulan Elango --- drivers/staging/comedi/comedidev.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/comedi

Re: [PATCH] Staging: comedi: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
On Tuesday 16 June 2015 06:42 PM, Ian Abbott wrote: > On 16/06/15 08:58, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch simplify a trivial if-return sequence. Possibly combine with >> a preceding function call. >> >> Signed-off-by: Abdul Hussain >> --- >> drivers/staging/com

[PATCH 2/6] Staging: comedi: fl512: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/fl512.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 5/6] Staging: comedi: ni_daq_dio24: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/ni_daq_dio24.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/

[PATCH 4/6] Staging: comedi: dac02: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers

[PATCH 6/6] Staging: comedi: s626: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/s626.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drivers/

[PATCH 0/6] Staging: comedi: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Abdul Hussain (6): staging: comedi: dmm32at: Simplify a trivial if-return sequence staging: comedi: fl512: Simplify a trivial if-return sequence staging: comedi: daqboard2000: Simplify a

[PATCH 3/6] Staging: comedi: daqboard2000: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/daqboard2000.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/comedi

[PATCH 1/6] Staging: comedi: dmm32at: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dmm32at.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/staging/comedi/drive

Re: [PATCH] staging: comedi: amplc_pci230: rename 'todo' variable

2015-06-16 Thread Ian Abbott
On 16/06/15 14:43, Gujulan Elango, Hari Prasath (H.) wrote: From: Hari Prasath Gujulan Elango This patch renames the very generic variable name 'todo' to nsamples. Signed-off-by: Hari Prasath Gujulan Elango --- Subject line modified and variable name kept short as per suggestion from

[PATCH 7/8] staging: sm7xxfb: move mode table

2015-06-16 Thread Sudip Mukherjee
move vgamode table from the header file to the C file and mark it as const. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xx.h | 660 -- drivers/staging/sm7xxfb/sm7xxfb.c | 660 ++ 2 files changed, 660 inse

[PATCH 5/8] staging: sm7xxfb: fix alignment

2015-06-16 Thread Sudip Mukherjee
Fix the alignment. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xxfb.c | 17 +++-- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c b/drivers/staging/sm7xxfb/sm7xxfb.c index c832399..6bdcd5b 100644 --- a/drivers/stagin

[PATCH 2/8] staging: sm7xxfb: remove numvgamodes

2015-06-16 Thread Sudip Mukherjee
numvgamodes was only used in one place, so remove the #define and use its defined value. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xx.h | 2 -- drivers/staging/sm7xxfb/sm7xxfb.c | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/sm7xxfb/sm

[PATCH] staging: ft1000: ft1000-usb: ft1000_hw.c: fix long lines

2015-06-16 Thread Michał Kępień
Fix checkpatch.pl warnings about lines longer than 80 characters. Signed-off-by: Michał Kępień --- drivers/staging/ft1000/ft1000-usb/ft1000_hw.c | 58 - 1 file changed, 29 insertions(+), 29 deletions(-) diff --git a/drivers/staging/ft1000/ft1000-usb/ft1000_hw.c b/driv

[PATCH 6/8] staging: sm7xxfb: reduce indention

2015-06-16 Thread Sudip Mukherjee
reduce code indention keeping the logic same. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xxfb.c | 173 ++ 1 file changed, 82 insertions(+), 91 deletions(-) diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c b/drivers/staging/sm7xxfb/sm7xxfb.c

[PATCH] staging: comedi: amplc_pci230: rename 'todo' variable

2015-06-16 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch renames the very generic variable name 'todo' to nsamples. Signed-off-by: Hari Prasath Gujulan Elango --- Subject line modified and variable name kept short as per suggestion from Ian Abbott. --- drivers/staging/comedi/drivers/amplc_pci230.c

[PATCH 8/8] staging: sm7xxfb: fix camelcase

2015-06-16 Thread Sudip Mukherjee
Fix the checkpatch warning about CamelCase. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xx.h | 2 +- drivers/staging/sm7xxfb/sm7xxfb.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/sm7xxfb/sm7xx.h b/drivers/staging/sm7xxfb/sm7xx.h inde

[PATCH 4/8] staging: sm7xxfb: no space after cast

2015-06-16 Thread Sudip Mukherjee
fixes the checkpatch warning about using space after typecast. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xxfb.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c b/drivers/staging/sm7xxfb/sm7xxfb.c index f18fef9..c832

[PATCH 1/8] staging: sm7xxfb: declare struct as const

2015-06-16 Thread Sudip Mukherjee
The vesa_mode_table is not to be modified, so declare it as const. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xxfb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c b/drivers/staging/sm7xxfb/sm7xxfb.c index 5db26f1..4d9b69

[PATCH] staging:lustre:mdc: Fix return statement reported by coccinelle

2015-06-16 Thread Mutharaju, Prasanna (P.)
From: Prasanna Karthik Modified return statement and removed local declaration no longer needed. No Compiler warnings. Signed-off-by: Prasanna Karthik --- drivers/staging/lustre/lustre/mdc/mdc_request.c |5 + 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/drivers/stagin

[PATCH 3/8] staging: sm7xxfb: add newline

2015-06-16 Thread Sudip Mukherjee
new line was missing in some of the dev_* macros while printing the debug and error messages. Signed-off-by: Sudip Mukherjee --- drivers/staging/sm7xxfb/sm7xxfb.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/staging/sm7xxfb/sm7xxfb.c b/drivers/staging

Re: [PATCH] staging: comedi: use meaningful variable name

2015-06-16 Thread Gujulan Elango, Hari Prasath (H.)
On Tue, Jun 16, 2015 at 02:04:21PM +0100, Ian Abbott wrote: > On 16/06/15 06:34, Gujulan Elango, Hari Prasath (H.) wrote: > >On Mon, Jun 15, 2015 at 03:42:07PM +0100, Ian Abbott wrote: > >>On 15/06/15 15:01, Gujulan Elango, Hari Prasath (H.) wrote: > >>>From: Hari Prasath Gujulan Elango > >>> > >>

Re: [PATCH] staging: comedi: use meaningful variable name

2015-06-16 Thread Gujulan Elango, Hari Prasath (H.)
On Tue, Jun 16, 2015 at 02:04:21PM +0100, Ian Abbott wrote: > On 16/06/15 06:34, Gujulan Elango, Hari Prasath (H.) wrote: > >On Mon, Jun 15, 2015 at 03:42:07PM +0100, Ian Abbott wrote: > >>On 15/06/15 15:01, Gujulan Elango, Hari Prasath (H.) wrote: > >>>From: Hari Prasath Gujulan Elango > >>> > >>

[PATCH] staging: unisys: visorchannel_write() fix potential memory corruption

2015-06-16 Thread Benjamin Romer
From: Jes Sorensen This fixes the memory corruption case, if nbytes is less than offset and sizeof(struct channel_header) Reported-by: Dan Carpenter Signed-off-by: Jes Sorensen Signed-off-by: Benjamin Romer --- drivers/staging/unisys/visorbus/visorchannel.c | 2 +- 1 file changed, 1 insertio

Re: [PATCH] Staging: comedi: Simplify a trivial if-return sequence

2015-06-16 Thread Ian Abbott
On 16/06/15 08:58, Abdul, Hussain (H.) wrote: From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c| 6 +- drivers/staging/comedi/drivers/daq

Re: [PATCH 1/2] staging: wilc1000: modify type casting warning

2015-06-16 Thread Greg KH
On Tue, Jun 16, 2015 at 04:11:24PM +0900, Dean Lee wrote: > modify message "warning: cast from pointer to integer of different size". > add type cast 'uintptr_t'. > > Signed-off-by: Dean Lee > --- > Change Log: > add Signed-off-by line > > drivers/staging/wilc1000/host_interface.c | 12

Re: [PATCH] staging: comedi: use meaningful variable name

2015-06-16 Thread Ian Abbott
On 16/06/15 06:34, Gujulan Elango, Hari Prasath (H.) wrote: On Mon, Jun 15, 2015 at 03:42:07PM +0100, Ian Abbott wrote: On 15/06/15 15:01, Gujulan Elango, Hari Prasath (H.) wrote: From: Hari Prasath Gujulan Elango This patch renames the very generic variable name 'todo' to num_samples_to_read

Re: [PATCH] Staging: wilc1000: Boolean tests don't need comparisons

2015-06-16 Thread Abdul, Hussain (H.)
On Tuesday 16 June 2015 04:41 PM, Sudip Mukherjee wrote: > On Tue, Jun 16, 2015 at 08:55:13AM +, Abdul, Hussain (H.) wrote: >> On Tuesday 16 June 2015 01:40 PM, Sudip Mukherjee wrote: >>> On Tue, Jun 16, 2015 at 07:33:42AM +, Abdul, Hussain (H.) wrote: From: Abdul Hussain >>> regards

Re: [PATCH] Staging: wilc1000: Boolean tests don't need comparisons

2015-06-16 Thread Sudip Mukherjee
On Tue, Jun 16, 2015 at 08:55:13AM +, Abdul, Hussain (H.) wrote: > On Tuesday 16 June 2015 01:40 PM, Sudip Mukherjee wrote: > > On Tue, Jun 16, 2015 at 07:33:42AM +, Abdul, Hussain (H.) wrote: > >> From: Abdul Hussain > > > > regards > > sudip > > > Sudip, > > Yes that is an assignment to

[PATCH] Staging: wilc1000: Assign proper boolean value

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch assign proper boolean value to boolean variable. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/host_interface.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/h

[PATCH v2] Staging: wilc1000: Boolean tests don't need comparisons

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes unwanted true and false from boolean tests. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/coreconfigurator.c | 8 drivers/staging/wilc1000/linux_mon.c | 2 +- drivers/staging/wilc1000/linux_wlan.c | 2 +-

[PATCH] iio/accel: fixed 1 coding style problem

2015-06-16 Thread Tan Nguyen
Break line to fix "line over 80 characters" style problem. Signed-off-by: Tan Nguyen --- drivers/staging/iio/accel/adis16220_core.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/accel/adis16220_core.c b/drivers/staging/iio/accel/adis16220_core.c index

Re: [PATCH] Staging: wilc1000: Boolean tests don't need comparisons

2015-06-16 Thread Abdul, Hussain (H.)
On Tuesday 16 June 2015 01:40 PM, Sudip Mukherjee wrote: > On Tue, Jun 16, 2015 at 07:33:42AM +, Abdul, Hussain (H.) wrote: >> From: Abdul Hussain >> >> This patch removes unwanted true and false from boolean tests. >> >> Signed-off-by: Abdul Hussain >> --- > >> diff --git a/drivers/staging/

Re: [PATCH] Staging: wilc1000: Boolean tests don't need comparisons

2015-06-16 Thread Sudip Mukherjee
On Tue, Jun 16, 2015 at 07:33:42AM +, Abdul, Hussain (H.) wrote: > From: Abdul Hussain > > This patch removes unwanted true and false from boolean tests. > > Signed-off-by: Abdul Hussain > --- > diff --git a/drivers/staging/wilc1000/host_interface.c > b/drivers/staging/wilc1000/host_inter

[PATCH] Staging: comedi: Simplify a trivial if-return sequence

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch simplify a trivial if-return sequence. Possibly combine with a preceding function call. Signed-off-by: Abdul Hussain --- drivers/staging/comedi/drivers/dac02.c| 6 +- drivers/staging/comedi/drivers/daqboard2000.c | 6 +- drivers/staging/comedi/dr

[PATCHv2] staging: wilc1000: remove unwanted code

2015-06-16 Thread Gujulan Elango, Hari Prasath (H.)
From: Hari Prasath Gujulan Elango This patch removes SIOCDEVPRIVATE + 1 ioctl. It currently is just a stub which does some useless printks and returns. In the original code, if the user passes priv_cmd.total_len == 0 then it will Oops. Also it leaks memory every time it's called. In the futur

[PATCH] Staging: wilc1000: Boolean tests don't need comparisons

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes unwanted true and false from boolean tests. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/coreconfigurator.c | 8 drivers/staging/wilc1000/host_interface.c | 2 +- drivers/staging/wilc1000/linux_mon.c | 2 +-

[PATCH v3] Staging: rtl8192e: Timer setup using macro rather assignment

2015-06-16 Thread DHANAPAL, GNANACHANDRAN (G.)
From: Gnanachandran Dhanapal This patch shall replaces user defined timer setup function with standard timer setup macro. Also removes init_timer, because timer can be initialized in setup_timer macro as well. Signed-off-by: Gnanachandran Dhanapal --- v3: From name and Signed-Off-by

Re: [PATCH] staging: wilc1000: modify type casting warning

2015-06-16 Thread Dean . lee
On 2015년 06월 16일 16:09, Sudip Mukherjee wrote: On Tue, Jun 16, 2015 at 03:46:04PM +0900, Dean Lee wrote: modify message "warning: cast from pointer to integer of different size". add type cast 'uintptr_t'. --- [] @@ -620,7 +621,7 @@ static s32 Handle_SetChannel(void *drvHandler, tstrHostIFS

[PATCH v2] Staging: lusture: Remove an open coded simple_open() function

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes an open coded simple_open() function and replace file operations references to the function with simple_open() instead Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 9 + 1 file changed, 1 insertion(+), 8 deletion

[PATCH 1/2] staging: wilc1000: modify type casting warning

2015-06-16 Thread Dean Lee
modify message "warning: cast from pointer to integer of different size". add type cast 'uintptr_t'. Signed-off-by: Dean Lee --- Change Log: add Signed-off-by line drivers/staging/wilc1000/host_interface.c | 129 +++--- drivers/staging/wilc1000/linux_wlan.c

[PATCH 2/2] staging: wilc1000: modify uninitialized warning

2015-06-16 Thread Dean Lee
initialize it. Signed-off-by: Dean Lee --- drivers/staging/wilc1000/host_interface.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index 358283f..03e6bc5 100644 --- a/drivers/staging/wi

[PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes casting the values returned by memory allocation functions. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_wlan.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/s

[PATCH] Staging: wilc1000: Remove casting the values returned by kmalloc()

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes casting the values returned by memory allocation functions. Signed-off-by: Abdul Hussain --- drivers/staging/wilc1000/linux_mon.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/stagi

Re: [PATCH] staging: wilc1000: modify type casting warning

2015-06-16 Thread Sudip Mukherjee
On Tue, Jun 16, 2015 at 03:46:04PM +0900, Dean Lee wrote: > modify message "warning: cast from pointer to integer of different size". > add type cast 'uintptr_t'. > --- [] > @@ -620,7 +621,7 @@ static s32 Handle_SetChannel(void *drvHandler, > tstrHostIFSetChan *pstrHostIFSetC > > PRINT_D(H

Re: [PATCH] Staging: lusture: Remove an open coded simple_open() function

2015-06-16 Thread Drokin, Oleg
On Jun 16, 2015, at 3:03 AM, Abdul, Hussain (H.) wrote: > From: Abdul Hussain > > This patch removes an open coded simple_open() function and replace file > operations references to the function with simple_open() instead > > Signed-off-by: Abdul Hussain > --- > drivers/staging/lustre/lustre/

Re: [PATCH] staging: wilc1000: modify type casting warning

2015-06-16 Thread Dean . lee
On 2015년 06월 16일 15:46, Dean Lee wrote: modify message "warning: cast from pointer to integer of different size". add type cast 'uintptr_t'. --- drivers/staging/wilc1000/host_interface.c | 129 +++--- drivers/staging/wilc1000/linux_wlan.c | 8 +- driver

[PATCH] Staging: lustre: Use memdup_user rather than duplicating its implementation

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch uses memdup_user rather than duplicating its implementation Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/llite/file.c | 22 ++ 1 file changed, 6 insertions(+), 16 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite

[PATCH] Staging: lustre: Use memdup_user rather than duplicating its implementation

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch uses memdup_user rather than duplicating its implementation Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/llite/dir.c | 32 +-- 1 file changed, 9 insertions(+), 23 deletions(-) diff --git a/drivers/staging/lustre/lust

[PATCH] Staging: lusture: Remove an open coded simple_open() function

2015-06-16 Thread Abdul, Hussain (H.)
From: Abdul Hussain This patch removes an open coded simple_open() function and replace file operations references to the function with simple_open() instead Signed-off-by: Abdul Hussain --- drivers/staging/lustre/lustre/fld/lproc_fld.c | 9 + 1 file changed, 1 insertion(+), 8 deletion