[PATCH] storvsc: do not assume SG list is continuous when doing bounce buffers (for 4.1 stable only)

2017-08-21 Thread Long Li
From: Long Li This patch is for linux-stable 4.1 branch only. storvsc checks the SG list for gaps before passing them to Hyper-v device. If there are gaps, data is copied to a bounce buffer and a continuous data buffer is passed to Hyper-V. The check on gaps assumes SG list is continuous, and n

Re: [PATCH 4/4] staging: iio: tsl2x7x: constify i2c_device_id

2017-08-21 Thread Arvind Yadav
Hi On Tuesday 22 August 2017 01:50 AM, Dan Carpenter wrote: Don't say "[PATCH 4/4]". It's not a patchset or a part of an email thread. Yes, It's part of these patchset. [PATCH 1/4] misc: apds9802als: constify i2c_device_id [PATCH 2/4] misc: hmc6352: constify i2c_device_id [PATCH 3/4] mi

Re: [PATCH] mtd: nand: Rename nand.h into rawnand.h

2017-08-21 Thread Boris Brezillon
Le Fri, 4 Aug 2017 17:29:09 +0200, Boris Brezillon a écrit : > We are planning to share more code between different NAND based > devices (SPI NAND, OneNAND and raw NANDs), but before doing that > we need to move the existing include/linux/mtd/nand.h file into > include/linux/mtd/rawnand.h so we

Re: [PATCH 4/4] staging: iio: tsl2x7x: constify i2c_device_id

2017-08-21 Thread Dan Carpenter
Don't say "[PATCH 4/4]". It's not a patchset or a part of an email thread. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 06/15] mtd: make device_type const

2017-08-21 Thread Boris Brezillon
Le Sat, 19 Aug 2017 13:52:17 +0530, Bhumika Goyal a écrit : > Make this const as it is only stored in the type field of a device > structure, which is const. > Done using Coccinelle. > Applied to l2-mtd/master. Thanks, Boris > Signed-off-by: Bhumika Goyal > --- > drivers/mtd/mtdcore.c | 2

Re: [PATCH 3/3] ANDROID: binder: fix proc->tsk check.

2017-08-21 Thread John Stultz
On Tue, Aug 8, 2017 at 11:08 AM, Greg KH wrote: > On Tue, Aug 08, 2017 at 10:34:47AM -0700, John Stultz wrote: >> On Fri, Jul 28, 2017 at 4:56 AM, Martijn Coenen wrote: >> > Commit c4ea41ba195d ("binder: use group leader instead of open thread")' >> > was incomplete and didn't update a check in b

[PATCH 4/4] staging: iio: tsl2x7x: constify i2c_device_id

2017-08-21 Thread Arvind Yadav
i2c_device_id are not supposed to change at runtime. All functions working with i2c_device_id provided by work with const i2c_device_id. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/staging/iio/light/tsl2x7x.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [PATCH] staging: greybus: audio: constify snd_soc_dai_ops structures

2017-08-21 Thread Mark Greer
On Mon, Aug 21, 2017 at 10:28:24AM +0530, Arvind Yadav wrote: > snd_soc_dai_ops are not supposed to change at runtime. All functions > working with snd_soc_dai_ops provided by work with > const snd_soc_dai_ops. So mark the non-const structs as const. > > Signed-off-by: Arvind Yadav > --- > driv

Re: [PATCH 1/1 v2] binder: free memory on error

2017-08-21 Thread Dan Carpenter
On Mon, Aug 21, 2017 at 04:13:28PM +0200, Christian Brauner wrote: > On binder_init() the devices string is duplicated and smashed into individual > device names which are passed along. However, the original duplicated string > wasn't freed in case binder_init() failed. Let's free it on error. > >

[PATCH 0/1 v2] binder: free memory on error

2017-08-21 Thread Christian Brauner
On binder_init() the devices string is duplicated and smashed into individual device names which are passed along. However, if I'm not mistaken the original duplicated string wasn't freed in case binder_init() failed. Christian Changelog v2 2017-08-21 (Thanks Dan!) - use more descriptive commit m

[PATCH 1/1 v2] binder: free memory on error

2017-08-21 Thread Christian Brauner
On binder_init() the devices string is duplicated and smashed into individual device names which are passed along. However, the original duplicated string wasn't freed in case binder_init() failed. Let's free it on error. Signed-off-by: Christian Brauner --- Changelog v2 2017-08-21: - use more de

Re: [PATCH 1/1] binder: free memory on error

2017-08-21 Thread Dan Carpenter
You need a commit message. On Mon, Aug 21, 2017 at 03:36:16PM +0200, Christian Brauner wrote: > Signed-off-by: Christian Brauner > --- > drivers/android/binder.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/android/binder.c b/drivers/android/binder.c >

[PATCH 1/1] binder: free memory on error

2017-08-21 Thread Christian Brauner
Signed-off-by: Christian Brauner --- drivers/android/binder.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/android/binder.c b/drivers/android/binder.c index f7665c31feca..41a11661eed7 100644 --- a/drivers/android/binder.c +++ b/drivers/android/binder.c @@ -421

[PATCH 0/1] binder: free memory on error

2017-08-21 Thread Christian Brauner
On binder_init() the devices string is duplicated and smashed into individual device names which are passed along. However, if I'm not mistaken the original duplicated string wasn't freed in case binder_init() failed. Christian Christian Brauner (1): binder: free memory on error drivers/andro

[PATCH] staging: rtl8192e: Change fixed function names with "%s:", __func__

2017-08-21 Thread Shurong Zhang
Replace hard-coded function names in strings with "%s", __func__ in the dot11d.c file. Issue found by checkpatch.pl. Signed-off-by: Shurong Zhang --- drivers/staging/rtl8192e/dot11d.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rtl8192e/dot11d.c b/dr

Re: [PATCH] staging: lustre: fix structure size for ARM OABI

2017-08-21 Thread Cihangir Akturk
On Sun, Aug 20, 2017 at 03:52:26AM +0100, James Simmons wrote: > > > On Aug 17, 2017, at 10:26, Greg KH wrote: > > > > > > On Wed, Aug 16, 2017 at 05:44:15PM +0300, Cihangir Akturk wrote: > > >> When building the kernel for the ARM architecture without setting > > >> CONFIG_AEABI, size of struct

Re: [PATCH] staging: vboxvideo: constify drm_fb_helper_funcs

2017-08-21 Thread Hans de Goede
Hi, On 21-08-17 12:06, Arvind Yadav wrote: drm_fb_helper_funcs are not supposed to change at runtime. All functions working with drm_fb_helper_funcs provided by work with const drm_fb_helper_funcs. So mark the non-const structs as const. Signed-off-by: Arvind Yadav Thanks, patch looks good

Re: [PATCH] Staging: bcm2048: fix bare use of 'unsigned' in radio-bcm2048.c

2017-08-21 Thread Hans Verkuil
On 08/04/2017 04:58 PM, Branislav Radocaj wrote: > This is a patch to the radio-bcm2048.c file that fixes up > a warning found by the checkpatch.pl tool. > > Signed-off-by: Branislav Radocaj > --- > drivers/staging/media/bcm2048/radio-bcm2048.c | 50 > +-- > 1 file chang

[PATCH] iio staging: tsl2x7x: clean up limit checks

2017-08-21 Thread Dan Carpenter
The second part of this patch is probably the most interesting. We use "TSL2X7X_MAX_LUX_TABLE_SIZE * 3" as the limit instead of just "TSL2X7X_MAX_LUX_TABLE_SIZE". It creates a static checker warning that we are going of of bounds, but in real life we always hit the break statement on the last ele

[PATCH] staging: vboxvideo: constify drm_fb_helper_funcs

2017-08-21 Thread Arvind Yadav
drm_fb_helper_funcs are not supposed to change at runtime. All functions working with drm_fb_helper_funcs provided by work with const drm_fb_helper_funcs. So mark the non-const structs as const. Signed-off-by: Arvind Yadav --- drivers/staging/vboxvideo/vbox_fb.c | 2 +- 1 file changed, 1 insert

[PATCH 3/3] staging:rtl8188eu:core Fix coding style issues

2017-08-21 Thread Janani S
This patch solves the following warning shown by checkpatch script WARNING: Comparisons should place the constants on the right side of the test Signed-off-by: Janani S --- drivers/staging/rtl8188eu/core/rtw_ioctl_set.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/dr

driverdev-devel@linuxdriverproject.org

2017-08-21 Thread Janani S
This patch is created to solve the coding style issues reported by checkpatch script. Signed-off-by: Janani S --- drivers/staging/rtl8188eu/core/rtw_ap.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_ap.c b/drivers/staging/