[PATCH] staging: wilc1000: fix NULL dereference inside wilc_scan()

2018-12-14 Thread Ajay.Kathat
From: Ajay Singh Added NULL check before accessing 'hidden_net' pointer inside wilc_scan() to fix the issue found by static code checker. Fixes: 8f1a0ac1eba7 ("staging: wilc1000: handle scan operation callback from cfg80211 context") Reported-by: Dan Carpenter Signed-off-by: Ajay Singh --- d

[PATCH v3] binder: fix use-after-free due to ksys_close() during fdget()

2018-12-14 Thread Todd Kjos
44d8047f1d8 ("binder: use standard functions to allocate fds") exposed a pre-existing issue in the binder driver. fdget() is used in ksys_ioctl() as a performance optimization. One of the rules associated with fdget() is that ksys_close() must not be called between the fdget() and the fdput(). The

[PATCH v6] gpu: ipu-csi: Swap fields according to input/output field types

2018-12-14 Thread Steve Longerbeam
The function ipu_csi_init_interface() was inverting the F-bit for NTSC case, in the CCIR_CODE_1/2 registers. The result being that for NTSC bottom-top field order, the CSI would swap fields and capture in top-bottom order. Instead, base field swap on the field order of the input to the CSI, and th

Re: [PATCH v5 02/12] gpu: ipu-csi: Swap fields according to input/output field types

2018-12-14 Thread Steve Longerbeam
On 12/13/18 4:59 AM, Philipp Zabel wrote: Hi Steve, On Tue, 2018-10-16 at 17:00 -0700, Steve Longerbeam wrote: The function ipu_csi_init_interface() was inverting the F-bit for NTSC case, in the CCIR_CODE_1/2 registers. The result being that for NTSC bottom-top field order, the CSI would swa

Re: [PATCH v2] binder: fix use-after-free due to ksys_close() during fdget()

2018-12-14 Thread Al Viro
On Fri, Dec 14, 2018 at 12:38:15PM -0800, Todd Kjos wrote: > 44d8047f1d8 ("binder: use standard functions to allocate fds") > exposed a pre-existing issue in the binder driver. > > fdget() is used in ksys_ioctl() as a performance optimization. > One of the rules associated with fdget() is that ksy

[PATCH 3/7] staging: olpc_dcon: add a missing dependency

2018-12-14 Thread Arnd Bergmann
From: Lubomir Rintel Commit 33f49571d75024b1044cd02689ad2bdb4924cc80 upstream. WARNING: unmet direct dependencies detected for BACKLIGHT_CLASS_DEVICE Depends on [n]: HAS_IOMEM [=y] && BACKLIGHT_LCD_SUPPORT [=n] Selected by [y]: - FB_OLPC_DCON [=y] && STAGING [=y] && X86 [=y] && OLP

Re: [PATCH 04/11] staging: iio: adt7316: fix handling of dac high resolution option

2018-12-14 Thread Jeremy Fertic
On Fri, Dec 14, 2018 at 09:26:18AM +0300, Dan Carpenter wrote: > On Thu, Dec 13, 2018 at 03:01:46PM -0700, Jeremy Fertic wrote: > > On Wed, Dec 12, 2018 at 11:23:16AM +0300, Dan Carpenter wrote: > > > On Tue, Dec 11, 2018 at 05:54:56PM -0700, Jeremy Fertic wrote: > > > > @@ -651,10 +649,12 @@ stati

[PATCH v2] binder: fix use-after-free due to ksys_close() during fdget()

2018-12-14 Thread Todd Kjos
44d8047f1d8 ("binder: use standard functions to allocate fds") exposed a pre-existing issue in the binder driver. fdget() is used in ksys_ioctl() as a performance optimization. One of the rules associated with fdget() is that ksys_close() must not be called between the fdget() and the fdput(). The

[PATCH] staging: most: sound: fix noderef.cocci warnings

2018-12-14 Thread kbuild test robot
From: kbuild test robot drivers/staging/most/sound/sound.c:673:21-27: ERROR: application of sizeof to pointer sizeof when applied to a pointer typed expression gives the size of the pointer Generated by: scripts/coccinelle/misc/noderef.cocci Fixes: 58bce1efd0e8 ("staging: most: sound: renam

Re: [PATCH 3/6] staging: most: sound: rename variable

2018-12-14 Thread kbuild test robot
Hi Christian, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.20-rc6 next-20181214] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https

[PATCH v3] binder: implement binderfs

2018-12-14 Thread Christian Brauner
As discussed at Linux Plumbers Conference 2018 in Vancouver [1] this is the implementation of binderfs. /* Abstract */ binderfs is a backwards-compatible filesystem for Android's binder ipc mechanism. Each ipc namespace will mount a new binderfs instance. Mounting binderfs multiple times at differ

[PATCH v2 12/33] staging: vc04_services: Remove VCHIQ_BULK_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_2835_arm.c| 5 +++-- .../vc04_services/interface/vchiq_arm/vchiq_arm.c | 4 ++-- .../vc04_services/interface/vchiq_arm/vchiq_core.c |

[PATCH v2 29/33] staging: vc04_services: Remove VCHIQ_SLOT_INFO_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../vc04_services/interface/vchiq_arm/vchiq_core.c| 11 ++- .../vc04_services/interface/vchiq_arm/vchiq_core.h| 10 +- 2 files changed, 11 insertions(+), 10

[PATCH v2 27/33] staging: vc04_services: Remove VCHIQ_SET_SERVICE_OPTION_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 +- .../staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 2 files changed, 4 insertions(+), 4 dele

[PATCH v2 24/33] staging: vc04_services: Remove VCHIQ_SERVICE_PARAMS_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 6 +++--- .../vc04_services/interface/vchiq_arm/vchiq_core.c| 2 +- .../vc04_services/interface/vchiq_arm/vchiq_

[PATCH v2 25/33] staging: vc04_services: Remove VCHIQ_SERVICE_QUOTA_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_core.c | 16 .../interface/vchiq_arm/vchiq_core.h | 6 +++--- 2 files changed, 11 insertions(+), 11 deletion

[PATCH v2 22/33] staging: vc04_services: Remove VCHIQ_QUEUE_MESSAGE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 4 ++-- .../staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 2 files changed, 5 insertions(+), 5 de

[PATCH v2 23/33] staging: vc04_services: Remove VCHIQ_SERVICE_BASE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_core.h| 2 +- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_if.h | 4 ++-- 2 files changed, 3 insertions(+), 3 de

[PATCH v2 18/33] staging: vc04_services: Remove VCHIQ_DUMP_MEM_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_

[PATCH v2 30/33] staging: vc04_services: Remove VCHIQ_SLOT_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c | 2 +- .../staging/vc04_services/interface/vchiq_arm/vchiq_core.h | 6 +++--- 2 files changed, 4 insertions(+), 4 dele

[PATCH v2 13/33] staging: vc04_services: Remove VCHIQ_COMPLETION_DATA_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_arm.c | 21 ++- .../interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 2 files changed, 14 insertions(+), 13 deletions(-

[PATCH v2 09/33] staging: vc04_services: Remove VCHIQ_ARM_STATE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_2835_arm.c | 4 +- .../interface/vchiq_arm/vchiq_arm.c | 55 ++- .../interface/vchiq_arm/vchiq_arm.h | 18 ++

[PATCH v2 32/33] staging: vc04_services: Remove VCHIQ_STATE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_2835_arm.c | 22 ++--- .../interface/vchiq_arm/vchiq_arm.c | 64 +++-- .../interface/vchiq_arm/vchiq_arm.h | 39 -

[PATCH v2 26/33] staging: vc04_services: Remove VCHIQ_SERVICE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_arm.c | 51 +++ .../interface/vchiq_arm/vchiq_arm.h | 8 +- .../interface/vchiq_arm/vchiq_core.c | 137 +--

[PATCH v2 33/33] staging: vc04_services: Remove VCHIU_QUEUE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_shim.c | 2 +- .../interface/vchiq_arm/vchiq_util.c | 14 +++--- .../interface/vchiq_arm/vchiq_util.h | 19 +++

[PATCH v2 31/33] staging: vc04_services: Remove VCHIQ_SLOT_ZERO_T

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_2835_arm.c | 2 +- .../vc04_services/interface/vchiq_arm/vchiq_core.c | 14 +++--- .../vc04_services/interface/vchiq_arm/vchiq_core

[PATCH v2 21/33] staging: vc04_services: Remove VCHIQ_QUEUE_BULK_TRANSFER_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 6 +++--- .../vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 8 2 files changed, 7 insertions(+), 7 de

[PATCH v2 16/33] staging: vc04_services: Remove VCHIQ_DEBUGFS_NODE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 4 ++-- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h | 2 +- .../staging/vc04_services/interface/vc

[PATCH v2 20/33] staging: vc04_services: Remove VCHIQ_HEADER_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_arm.c | 20 +++--- .../interface/vchiq_arm/vchiq_core.c | 66 ++- .../interface/vchiq_arm/vchiq_if.h| 1

[PATCH v2 19/33] staging: vc04_services: Remove VCHIQ_GET_CONFIG_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 4 ++-- .../staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 2 files changed, 5 insertions(+), 5 de

[PATCH v2 17/33] staging: vc04_services: Remove VCHIQ_DEQUEUE_MESSAGE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 4 ++-- .../staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 2 files changed, 5 insertions(+), 5 de

[PATCH v2 14/33] staging: vc04_services: Remove VCHIQ_CONFIG_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 +- .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c | 2 +- .../staging/vc04_services/interface/vchiq_ar

[PATCH v2 28/33] staging: vc04_services: Remove VCHIQ_SHARED_STATE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_core.c | 24 +-- .../interface/vchiq_arm/vchiq_core.h | 12 +- 2 files changed, 18 insertions(+), 18 deletio

[PATCH v2 10/33] staging: vc04_services: Remove VCHIQ_AWAIT_COMPLETION_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 8 .../vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 2 files changed, 7 insertions(+), 7 de

[PATCH v2 15/33] staging: vc04_services: Remove VCHIQ_CREATE_SERVICE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 6 +++--- .../staging/vc04_services/interface/vchiq_arm/vchiq_ioctl.h | 6 +++--- 2 files changed, 6 insertions(+), 6

[PATCH v2 11/33] staging: vc04_services: Remove VCHIQ_BULK_QUEUE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../vc04_services/interface/vchiq_arm/vchiq_core.c | 13 +++-- .../vc04_services/interface/vchiq_arm/vchiq_core.h | 8 2 files changed, 11 insertions(+), 10 del

[PATCH v2 08/33] staging: vc04_services: Remove VCHI_MSG_VECTOR_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchi/vchi_common.h| 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchi/v

[PATCH v2 06/33] staging: vc04_services: Remove VCHI_HELD_MSG_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../vc04_services/bcm2835-camera/mmal-vchiq.c | 28 +-- .../vc04_services/interface/vchi/vchi.h | 20 ++--- .../interface/vchiq_arm/vchiq_shim.c

[PATCH v2 05/33] staging: vc04_services: Remove USER_SERVICE_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_arm.c | 44 ++- 1 file changed, 23 insertions(+), 21 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchi

[PATCH v2 07/33] staging: vc04_services: Remove VCHI_MSG_ITER_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- drivers/staging/vc04_services/interface/vchi/vchi.h | 12 ++-- .../vc04_services/interface/vchi/vchi_common.h | 4 ++-- 2 files changed, 8 insertions(+), 8 deletion

[PATCH v2 02/33] staging: vc04_services: Remove PAGELIST_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 10 +- .../vc04_services/interface/vchiq_arm/vchiq_pagelist.h | 4 ++-- 2 files changed, 7 insertions(+), 7 deleti

[PATCH v2 03/33] staging: vc04_services: Remove REMOTE_EVENT_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../interface/vchiq_arm/vchiq_2835_arm.c | 2 +- .../vc04_services/interface/vchiq_arm/vchiq_core.c | 8 .../vc04_services/interface/vchiq_arm/vchiq_core.h | 1

[PATCH v2 04/33] staging: vc04_services: Remove SERVICE_CREATION_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- drivers/staging/vc04_services/bcm2835-audio/bcm2835-vchiq.c | 2 +- drivers/staging/vc04_services/bcm2835-camera/mmal-vchiq.c | 2 +- drivers/staging/vc04_services/interface/vchi

[PATCH v2 00/33] staging: vc04_services: Remove most struct typedefs

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Dominic Braun (33): staging: vc04_services: Remove DUMP_CONTEXT_T typedef staging: vc04_services: Remove PAGELIST_T typedef staging: vc04_services: Remove REMOTE_EVENT_T typedef staging: vc04_services: Remove SERVICE_CREATION_T typedef

[PATCH v2 01/33] staging: vc04_services: Remove DUMP_CONTEXT_T typedef

2018-12-14 Thread Dominic Braun
Typedefing structs is not encouraged in the kernel. Signed-off-by: Dominic Braun Signed-off-by: Tobias Büttner --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_

Re: [PATCH v2] binder: implement binderfs

2018-12-14 Thread Christian Brauner
On Fri, Dec 14, 2018 at 10:11:23AM +0300, Dan Carpenter wrote: > On Thu, Dec 13, 2018 at 10:59:11PM +0100, Christian Brauner wrote: > > +/** > > + * binderfs_new_inode - allocate inode from super block of a binderfs mount > > + * @ref_inode: inode from wich the super block will be taken > > + * @us

Re: [PATCH 6/6] staging: most: Documentation: add information to driver_usage file

2018-12-14 Thread Greg KH
On Fri, Dec 14, 2018 at 09:33:50AM +, christian.gr...@microchip.com wrote: > On Fr, 2018-12-14 at 12:10 +0300, Dan Carpenter wrote: > > On Fri, Dec 14, 2018 at 09:06:01AM +, Christian.Gromm@microchip.c > > om wrote: > > > > > > Message received. Anyway, I would like to resent the current >

Re: [PATCH 6/6] staging: most: Documentation: add information to driver_usage file

2018-12-14 Thread Dan Carpenter
On Fri, Dec 14, 2018 at 09:33:50AM +, christian.gr...@microchip.com wrote: > On Fr, 2018-12-14 at 12:10 +0300, Dan Carpenter wrote: > > On Fri, Dec 14, 2018 at 09:06:01AM +, Christian.Gromm@microchip.c > > om wrote: > > > > > > Message received. Anyway, I would like to resent the current >

Re: [PATCH 6/6] staging: most: Documentation: add information to driver_usage file

2018-12-14 Thread Christian.Gromm
On Fr, 2018-12-14 at 12:10 +0300, Dan Carpenter wrote: > On Fri, Dec 14, 2018 at 09:06:01AM +, Christian.Gromm@microchip.c > om wrote: > > > > Message received. Anyway, I would like to resent the current > > patch set with the things Dan found fixed. And then looking > > into Configfs. Would t

Re: [PATCH 6/6] staging: most: Documentation: add information to driver_usage file

2018-12-14 Thread Dan Carpenter
On Fri, Dec 14, 2018 at 09:06:01AM +, christian.gr...@microchip.com wrote: > Message received. Anyway, I would like to resent the current > patch set with the things Dan found fixed. And then looking > into Configfs. Would this be ok for you guys? > Sure. regards, dan carpenter

Re: [PATCH 6/6] staging: most: Documentation: add information to driver_usage file

2018-12-14 Thread Christian.Gromm
On Do, 2018-12-13 at 18:55 +0100, Greg KH wrote: > On Thu, Dec 13, 2018 at 04:32:25PM +, Christian.Gromm@microchip.c > om wrote: > > > > On Do, 2018-12-13 at 13:32 +0100, Greg KH wrote: > > > > > > On Thu, Dec 13, 2018 at 02:58:00PM +0300, Dan Carpenter wrote: > > > > > > > > > > > > On Wed