[PATCH 1/1] Staging: iio: Coding style correction

2015-02-07 Thread Tolga Ceylan
Line over 80 characters corrected Signed-off-by: Tolga Ceylan --- drivers/staging/iio/meter/ade7854-i2c.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/iio/meter/ade7854-i2c.c b/drivers/staging/iio/meter/ade7854-i2c.c index 5b33c7f..5d0671a 100644 --- a/d

[PATCH] staging: wlan-ng: Use setup_timer

2015-02-07 Thread Vaishali Thakkar
This patch introduces the use of function setup_timer. This is done using Coccinelle and semantic patch used is as follows: @@ expression x,y,z; @@ - init_timer (&x); + setup_timer (&x, y, z); - x.function = y; - x.data = z; Signed-off-by: Vaishali Thakkar --- drivers/staging/wlan-ng/hfa384x_

[PATCH] staging: comedi: Use setup_timer

2015-02-07 Thread Vaishali Thakkar
This patch introduces the use of function setup_timer. This is done using Coccinelle and semantic patch used is as follows: @@ expression x,y,z; @@ - init_timer (&x); + setup_timer (&x, y, z); - x.function = y; - x.data = z; Signed-off-by: Vaishali Thakkar --- drivers/staging/comedi/drivers/c

[PATCH Resend] staging: speakup: Fix warning of line over 80 characters.

2015-02-07 Thread Shirish Gajera
This patch fixes the checkpatch.pl warning: WARNING: line over 80 characters All line over 80 characters in driver/staging/speakup/* are fixed. Signed-off-by: Shirish Gajera --- drivers/staging/speakup/main.c | 12 drivers/staging/speakup/serialio.h | 3 ++- driver

Re: [PATCH 1/8] staging: ft1000: ft1000-pcmcia: removed unnecessary braces

2015-02-07 Thread Greg KH
On Fri, Feb 06, 2015 at 04:14:54PM +0530, Pushpendra Singh wrote: > Removed checkpatch.pl warnings > WARNING: braces {} are not necessary for single statement blocks > + if (tempword == 0xfefe) { > + break; > + } > > Signed-

Re: [PATCH 1/2] ft1000-pcmcia: ft1000_hw.c: fix style issues not requiring code refactoring

2015-02-07 Thread Greg Kroah-Hartman
On Wed, Feb 04, 2015 at 12:03:52AM +0100, Daniele Alessandrelli wrote: > Fix all the trivial style issues (as reported by checkpatch.pl) not requiring > code refactoring. A following patch is expected to fix the remaining issues by > performing some code refactoring. > > Signed-off-by: Daniele Ale

Re: [PATCH] Fixed spaced comma in speakup/i18n.h

2015-02-07 Thread Greg Kroah-Hartman
On Sat, Jan 31, 2015 at 07:31:54PM -0500, Derrick Greenspan wrote: > Fixed the checkpatch error: > > ERROR: space prohibited before that ',' (ctx:WxE) > 6: FILE: drivers/staging/speakup/i18n.h:6: > + MSG_FIRST_INDEX , > ^ > > Signed-off-by: Derrick Greenspan > --- >

Re: [PATCH 12/20] staging/lustre/obd: change type of cl_conn_count to size_t

2015-02-07 Thread Greg Kroah-Hartman
On Sun, Feb 01, 2015 at 09:52:11PM -0500, gr...@linuxhacker.ru wrote: > From: Dmitry Eremin > > Change type of cl_conn_count to size_t. Yes, that's what you did, but you don't say _why_ you did this. Please fix up and resend. thanks, greg k-h ___ de

Re: [PATCH 1/2] staging: unisys: remove unused variable

2015-02-07 Thread Greg Kroah-Hartman
On Fri, Feb 06, 2015 at 06:13:21PM +0530, Sudip Mukherjee wrote: > we were getting lots of warnings about _tempresult set but not used. > _tempresult was used in the macro ISSUE_IO_VMCALL_POSTCODE_SEVERITY > which was again using another macro ISSUE_IO_EXTENDED_VMCALL. > but the vallue assigned to

Re: [PATCH 2/2] staging: unisys: fix directory warning

2015-02-07 Thread Greg Kroah-Hartman
On Fri, Feb 06, 2015 at 06:13:22PM +0530, Sudip Mukherjee wrote: > we were getting three warnings about timskmod and sparstopdriver > and channels. These warnings were about no such file or directory. > These directory names were included in the Makefile, but the > directories were not existing. W

Re: [PATCH 2/2] staging: unisys: fix directory warning

2015-02-07 Thread Greg Kroah-Hartman
On Fri, Feb 06, 2015 at 06:13:22PM +0530, Sudip Mukherjee wrote: > we were getting three warnings about timskmod and sparstopdriver > and channels. These warnings were about no such file or directory. > These directory names were included in the Makefile, but the > directories were not existing. >

Re: [PATCH] staging: vt6656: removed erroneous else statement

2015-02-07 Thread Greg Kroah-Hartman
On Fri, Jan 30, 2015 at 10:43:19PM -0500, Derrick Greenspan wrote: > This patch fixes the checkpatch.pl warning: > > WARNING: else is not generally useful after a break or return > 559: FILE: drivers/staging/vt6656/rxtx.c:559: > return vnt_rxtx_datahead_g_fb(tx_context, &buf->data_head); > } else

Re: [PATCH] staging/fwserial: use correct vendor/version IDs

2015-02-07 Thread Greg Kroah-Hartman
On Wed, Jan 28, 2015 at 09:07:43PM +0100, Clemens Ladisch wrote: > The driver was using the vendor ID 0xd00d1e from the FireWire core. > However, this ID was not registered, and invalid. > > Instead, use the vendor/version IDs that now are officially assigned to > firewire-serial: > https://ieee13

Re: [PATCH v2] staging: speakup: Fix warning of line over 80 characters.

2015-02-07 Thread Greg KH
On Sat, Jan 31, 2015 at 02:19:43PM -0800, Shirish Gajera wrote: > This patch fixes the checkpatch.pl warning: > > WARNING: line over 80 characters > > All line over 80 characters in driver/staging/speakup/* are fixed. > Aditional changes suggest by mailing list are also fixed. > > Signed-off-by:

Re: [PATCH] staging: comedi: addi_apci_1500: fix array access out of bounds error

2015-02-07 Thread Greg KH
On Wed, Jan 28, 2015 at 09:58:56AM -0700, H Hartley Sweeten wrote: > The private data 'pm', 'pt', and 'pp' array members hold the trigger mode > parameters for ports A and B. Both ports are 8-bits and the arrays are > 16-bits. > Array index 0 defines the AND mode and index 1 the OR mode parameters

Re: [PATCH] staging: comedi: addi_apci_1500: fix array access out of bounds error

2015-02-07 Thread Greg KH
On Sat, Feb 07, 2015 at 05:00:06PM +0800, Greg KH wrote: > On Wed, Jan 28, 2015 at 09:58:56AM -0700, H Hartley Sweeten wrote: > > The private data 'pm', 'pt', and 'pp' array members hold the trigger mode > > parameters for ports A and B. Both ports are 8-bits and the arrays are > > 16-bits. > > Ar

RE: [PATCH 0/4] Drivers: hv: Further protection for the rescind path

2015-02-07 Thread KY Srinivasan
> -Original Message- > From: Dexuan Cui > Sent: Friday, February 6, 2015 6:54 AM > To: KY Srinivasan; Vitaly Kuznetsov > Cc: de...@linuxdriverproject.org; Haiyang Zhang; linux- > ker...@vger.kernel.org; Jason Wang > Subject: RE: [PATCH 0/4] Drivers: hv: Further protection for the rescind

[PATCH v2] staging: rtl8192u: r8192U_core: Fix driver_info dereference as a null pointer

2015-02-07 Thread Rickard Strandqvist
Fix possible use of use of driver_info as a null pointer in query_rxdesc_status() This could happen if stats->RxIs40MHzPacket still has the default value of zero. Signed-off-by: Rickard Strandqvist --- drivers/staging/rtl8192u/r8192U_core.c |9 +++-- 1 file changed, 3 insertions(+), 6 de

Re: [PATCH v2] staging: rtl8192u: r8192U_core: Fix for possible null pointer dereference

2015-02-07 Thread Rickard Strandqvist
2015-02-06 15:32 GMT+01:00 Sudip Mukherjee : > On Thu, Feb 05, 2015 at 06:39:13PM +0100, Rickard Strandqvist wrote: >> 2015-02-05 13:51 GMT+01:00 Dan Carpenter : >> > On Thu, Feb 05, 2015 at 06:13:22PM +0530, Sudip Mukherjee wrote: >> >> if (driver_info) { >> >> stats->RxIs40MHzPacket = drive

Re: [PATCH v3 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-07 Thread Robert Jarzmik
Krzysztof Kozlowski writes: > Use power_supply_put() to decrement the power supply's device reference > counter. > > Signed-off-by: Krzysztof Kozlowski > Reviewed-by: Bartlomiej Zolnierkiewicz > Reviewed-by: Sebastian Reichel Acked-by: Robert Jarzmik -- Robert __