[PATCH] staging: media: ipu3: Change 'unsigned long int' to 'unsigned long'

2020-01-24 Thread Jean-Baptiste Jouband
This patch fixes the checkpatch.pl warning: WARNING: Prefer 'unsigned long' over 'unsigned long int' as the int is unnecessary Signed-off-by: Jean-Baptiste Jouband --- drivers/staging/media/ipu3/ipu3-mmu.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/

[PATCH 18/22] staging: vc04_services: Get of even more suspend/resume states

2020-01-24 Thread Nicolas Saenz Julienne
After the first states cleanup pass some set_suspend/resume_state() calls disappeared which opened up even more opportunities for deletions. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 92 +-- .../interface/vchiq_arm/vchiq_arm.h

[PATCH 04/22] staging: vc04_services: get rid of blocked_blocker completion in struct vchiq_arm_state

2020-01-24 Thread Nicolas Saenz Julienne
Nobody is waiting on it, so delete it. Signed-off-by: Nicolas Saenz Julienne --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c| 5 - .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.h| 1 - 2 files changed, 6 deletions(-) diff --git a/drivers/staging/vc04_services

[PATCH 17/22] staging: vc04_services: Delete vchiq_platform_check_suspend()

2020-01-24 Thread Nicolas Saenz Julienne
The function does nothing. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_arm.c | 15 --- .../interface/vchiq_arm/vchiq_core.c | 3 --- .../interface/vchiq_arm/vchiq_core.h | 3 --- 3 files changed, 21 deletions(-) d

[PATCH 22/22] staging: vc04_services: vchiq_arm: Get rid of unused defines

2020-01-24 Thread Nicolas Saenz Julienne
They aren't used anywhere. Get rid of them. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_arm.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/drivers/staging/vc04_servic

[PATCH 05/22] staging: vc04_services: Delete blocked_count in struct vchiq_arm_state

2020-01-24 Thread Nicolas Saenz Julienne
The variable is always 0. So delete it an all the code conditional to it. Signed-off-by: Nicolas Saenz Julienne --- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 -- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h | 2 -- 2 files changed, 4 deletions(-) diff --

[PATCH 21/22] staging: vc04_services: Get rid of the rest of suspend/resume state handling

2020-01-24 Thread Nicolas Saenz Julienne
The only remaining state doesn't provide any information anymore nor the completion that depends on it, which is set to complete_all() before any user may wait on it. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 49 --- .../interface/v

[PATCH 06/22] staging: vc04_services: get rid of vchiq_platform_use_suspend_timer()

2020-01-24 Thread Nicolas Saenz Julienne
The function always returns 0, delete the function and all code conditional to it. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_2835_arm.c | 11 --- .../interface/vchiq_arm/vchiq_arm.c | 84 ++- .../interface/vchiq_arm/vchiq_arm.h

[PATCH 11/22] staging: vc04_services: Get rid of vchiq_on_remote_use_active()

2020-01-24 Thread Nicolas Saenz Julienne
Function does nothing. Signed-off-by: Nicolas Saenz Julienne --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 6 -- .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c | 1 - 2 files changed, 7 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_

[PATCH 15/22] staging: vc04_services: Get rid of unused suspend/resume states

2020-01-24 Thread Nicolas Saenz Julienne
It's impossible to get into them, so simply delete them. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 55 --- .../interface/vchiq_arm/vchiq_arm.h | 7 --- 2 files changed, 62 deletions(-) diff --git a/drivers/staging/vc04_s

[PATCH 13/22] staging: vc04_services: Get rid of vchiq_check_resume()

2020-01-24 Thread Nicolas Saenz Julienne
Nobody calls this function. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 23 --- .../interface/vchiq_arm/vchiq_arm.h | 3 --- 2 files changed, 26 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/v

[PATCH 19/22] staging: vc04_services: Get rid of vc_suspend_state in struct vchiq_arm_state

2020-01-24 Thread Nicolas Saenz Julienne
It's never used. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_arm.c | 11 ++- .../vc04_services/interface/vchiq_arm/vchiq_arm.h | 1 - 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/staging/vc04_services/interface

[PATCH 20/22] staging: vc04_services: Get rid of vchiq_arm_vcresume()'s signature

2020-01-24 Thread Nicolas Saenz Julienne
The function implementation does not exist. Signed-off-by: Nicolas Saenz Julienne --- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h b/drivers/staging/vc04_services

[PATCH 03/22] staging: vc04_services: Get rid of resume_blocker completion in struct vchiq_arm_state

2020-01-24 Thread Nicolas Saenz Julienne
Nobody is waiting on it, so delete all relevant code. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_arm.c | 11 --- .../vc04_services/interface/vchiq_arm/vchiq_arm.h | 4 2 files changed, 15 deletions(-) diff --git a/drivers/staging/

[PATCH 01/22] staging: vc04_services: Remove unused variables in struct vchiq_arm_state

2020-01-24 Thread Nicolas Saenz Julienne
There are not being used, so we're better off without them. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_arm.h | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h b/drivers/sta

[PATCH 00/22] staging: vc04_services: suspend/resume cleanup

2020-01-24 Thread Nicolas Saenz Julienne
While in the process of preparing vchiq to support the Raspberry Pi 4, I stumbled upon a bunch of dead code in it. Which this series deletes. Ultimately the idea is to clean up vchiq_bcm2835_arm.c up to a point where it'll be easy to join it with vchiq_arm.c, which IIRC the TODO file states, shoul

[PATCH 07/22] staging: vc04_services: Get rid of vchiq_platform_paused/resumed()

2020-01-24 Thread Nicolas Saenz Julienne
vchiq_platform_paused() and vchiq_platform_resumed() do nothing. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 10 -- .../vc04_services/interface/vchiq_arm/vchiq_core.c | 3 --- .../vc04_services/interface/vchiq_arm/vchiq_core.h

[PATCH 12/22] staging: vc04_services: Get rid of vchiq_arm_vcsuspend()

2020-01-24 Thread Nicolas Saenz Julienne
It's not used. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 51 --- .../interface/vchiq_arm/vchiq_arm.h | 3 -- 2 files changed, 54 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c b/

[PATCH 14/22] staging: vc04_services: Delete vc_suspend_complete completion

2020-01-24 Thread Nicolas Saenz Julienne
Nobody is waiting on it. Signed-off-by: Nicolas Saenz Julienne --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 7 --- .../staging/vc04_services/interface/vchiq_arm/vchiq_arm.h | 1 - 2 files changed, 8 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq

[PATCH 02/22] staging: vc04_services: Get rid of resume_blocked in struct vchiq_arm_state

2020-01-24 Thread Nicolas Saenz Julienne
The boolean value is never set to true, hence remove it and all the code that depends on it. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 51 +-- .../interface/vchiq_arm/vchiq_arm.h | 1 - 2 files changed, 2 insertions(+), 5

[PATCH 08/22] staging: vc04_services: Get rid of vchiq_platform_suspend/resume()

2020-01-24 Thread Nicolas Saenz Julienne
vchiq_platform_suspend() and vchiq_platform_resume() do nothing, get rid of them. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_2835_arm.c | 12 .../vc04_services/interface/vchiq_arm/vchiq_arm.c| 8 +--- .../vc04_services/interface/vchi

[PATCH 10/22] staging: vc04_services: Get rid of vchiq_platform_handle_timeout()

2020-01-24 Thread Nicolas Saenz Julienne
The function does nothing. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_2835_arm.c | 5 - .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c | 5 - .../staging/vc04_services/interface/vchiq_arm/vchiq_core.h | 3 --- 3 files chang

[PATCH 16/22] staging: vc04_services: Get rid of USE_TYPE_SERVICE_NO_RESUME

2020-01-24 Thread Nicolas Saenz Julienne
It's not used anywhere. Signed-off-by: Nicolas Saenz Julienne --- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 2 +- drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.h | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/vc04_services

[PATCH 09/22] staging: vc04_services: Get rid of vchiq_platform_videocore_wanted()

2020-01-24 Thread Nicolas Saenz Julienne
The function always returns true, which makes a whole bunch of code useless. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_2835_arm.c | 5 --- .../interface/vchiq_arm/vchiq_arm.c | 34 +++ .../interface/vchiq_arm/vchiq_arm.h |

[PATCH AUTOSEL 4.14 27/32] hv_netvsc: Fix memory leak when removing rndis device

2020-01-24 Thread Sasha Levin
From: Mohammed Gamal [ Upstream commit 536dc5df2808efbefc5acee334d3c4f701790ec0 ] kmemleak detects the following memory leak when hot removing a network device: unreferenced object 0x888083f63600 (size 256): comm "kworker/0:1", pid 12, jiffies 4294831717 (age 1113.676s) hex dump (first

[PATCH AUTOSEL 4.19 44/56] hv_netvsc: Fix memory leak when removing rndis device

2020-01-24 Thread Sasha Levin
From: Mohammed Gamal [ Upstream commit 536dc5df2808efbefc5acee334d3c4f701790ec0 ] kmemleak detects the following memory leak when hot removing a network device: unreferenced object 0x888083f63600 (size 256): comm "kworker/0:1", pid 12, jiffies 4294831717 (age 1113.676s) hex dump (first

[PATCH AUTOSEL 5.4 086/107] scsi: storvsc: Correctly set number of hardware queues for IDE disk

2020-01-24 Thread Sasha Levin
From: Long Li [ Upstream commit 7b571c19d4c0b78d27dd3bf1f3c42e4032390af6 ] Commit 0ed881027690 ("scsi: storvsc: setup 1:1 mapping between hardware queue and CPU queue") introduced a regression for disks attached to IDE. For these disks the host VSP only offers one VMBUS channel. Setting multiple

[PATCH AUTOSEL 5.4 081/107] hv_netvsc: Fix memory leak when removing rndis device

2020-01-24 Thread Sasha Levin
From: Mohammed Gamal [ Upstream commit 536dc5df2808efbefc5acee334d3c4f701790ec0 ] kmemleak detects the following memory leak when hot removing a network device: unreferenced object 0x888083f63600 (size 256): comm "kworker/0:1", pid 12, jiffies 4294831717 (age 1113.676s) hex dump (first

Re: [PATCH] media: allegro: fix spelling mistake "to" -> "too"

2020-01-24 Thread Michael Tretter
On Thu, 23 Jan 2020 01:06:43 +, Colin King wrote: > From: Colin Ian King > > There is a spelling mistake in a v4l2_err message. Fix it. > > Signed-off-by: Colin Ian King Reviewed-by: Michael Tretter > --- > drivers/staging/media/allegro-dvt/allegro-core.c | 2 +- > 1 file changed, 1 ins

Re: [PATCH v4 01/10] staging: most: remove device from interface structure

2020-01-24 Thread Greg KH
On Fri, Jan 24, 2020 at 08:56:56AM +, christian.gr...@microchip.com wrote: > On Thu, 2020-01-23 at 19:18 +0100, Greg KH wrote: > > EXTERNAL EMAIL: Do not click links or open attachments unless you > > know the content is safe > > > > On Thu, Jan 23, 2020 at 04:38:17PM +0100, Christian Gromm wr

Re: [PATCH v4 01/10] staging: most: remove device from interface structure

2020-01-24 Thread Christian.Gromm
On Thu, 2020-01-23 at 19:18 +0100, Greg KH wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you > know the content is safe > > On Thu, Jan 23, 2020 at 04:38:17PM +0100, Christian Gromm wrote: > > This patch makes the adapter drivers use their own device > > structures > > whe