[PATCH 0/4] staging:rtl8192u: Style changes r819xU_firmware.h

2018-09-08 Thread John Whitmore
This short series of patches just clears simple checkpatch issues with the header file r819xU_firmware.h. John Whitmore (4): staging:rtl8192u: Remove unused RTL8190_CPU_START_OFFSET - Style staging:rtl8192u: Refactor GET_COMMAND_PACKET_FRAG_THRESHOLD - Style staging:rtl8192u: Remove typedef

[PATCH 4/4] staging:rtl8192u: Remove typedef from enum opt_rst_type_e - Style

2018-09-08 Thread John Whitmore
Remove the typedef directive from enumerated type opt_rst_type_e, this change clears the checkpatch issue with defining new types in the code. This is a coding style change which should not impact runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_firmware.

[PATCH 3/4] staging:rtl8192u: Remove typedef from firmware_init_step_e - Style

2018-09-08 Thread John Whitmore
Remove the typedef directive from enumerated type firmware_init_step_e this clears the checkpatch issue with adding new types to the code. This is a coding style change which should have no impact on runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_firmwa

[PATCH 2/4] staging:rtl8192u: Refactor GET_COMMAND_PACKET_FRAG_THRESHOLD - Style

2018-09-08 Thread John Whitmore
The MACRO GET_COMMAND_PACKET_FRAG_THRESHOLD causes a number of checkpatch issues so has been refactored to use braces around the parameter 'v' to avoid precedence issues, and to add spaces around operators. These changes are coding style changes which should have no impact on runtime code executio

[PATCH 1/4] staging:rtl8192u: Remove unused RTL8190_CPU_START_OFFSET - Style

2018-09-08 Thread John Whitmore
The defined constant RTL8190_CPU_START_OFFSET is not used in the code so has been removed. This is a coding style change which should have no impact on runtime code execution. Signed-off-by: John Whitmore --- drivers/staging/rtl8192u/r819xU_firmware.h | 1 - 1 file changed, 1 deletion(-) diff -

Re: [PATCH 0/2] Follow-up patches for Cedrus v9

2018-09-08 Thread Hans Verkuil
On 09/07/2018 06:33 PM, Paul Kocialkowski wrote: > This brings the requested modifications on top of version 9 of the > Cedrus VPU driver, that implements stateless video decoding using the > Request API. > > Paul Kocialkowski (2): > media: cedrus: Fix error reporting in request validation > m

Re: [PATCH 0/2] Follow-up patches for Cedrus v9

2018-09-08 Thread Chen-Yu Tsai
On Sat, Sep 8, 2018 at 6:06 PM Hans Verkuil wrote: > > On 09/07/2018 06:33 PM, Paul Kocialkowski wrote: > > This brings the requested modifications on top of version 9 of the > > Cedrus VPU driver, that implements stateless video decoding using the > > Request API. > > > > Paul Kocialkowski (2): >

Re: [PATCH 0/2] Follow-up patches for Cedrus v9

2018-09-08 Thread Hans Verkuil
On 09/08/2018 12:22 PM, Chen-Yu Tsai wrote: > On Sat, Sep 8, 2018 at 6:06 PM Hans Verkuil wrote: >> >> On 09/07/2018 06:33 PM, Paul Kocialkowski wrote: >>> This brings the requested modifications on top of version 9 of the >>> Cedrus VPU driver, that implements stateless video decoding using the >

[PATCH 2/3] staging: rtl8188eu: remove unnecessary parentheses in rtw_led.c

2018-09-08 Thread Michael Straube
Remove unnecessary parentheses from conditionals. Also clears 'Alignment should match open parenthesis' checkpatch issue. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_led.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rtl818

[PATCH 1/3] staging: rtl8188eu: refactor SwLedControlMode1()

2018-09-08 Thread Michael Straube
Refactor switch cases in SwLedControlMode1() to reduce indentation level. Also clears line over 80 characters checkpatch warnings. Suggested-by: Joe Perches Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_led.c | 223 --- 1 file changed, 113 insertions(

[PATCH 3/3] staging: rtl8188eu: fix lines over 80 characters in rtw_led.c

2018-09-08 Thread Michael Straube
Wrap lines over 80 characters where appropriate to clear checkpatch warnings. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_led.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_led.c b/drivers/staging/rtl81

[PATCH] staging: android: ion: Fix compilation warning

2018-09-08 Thread Alexey Skidanov
Label unlock is defined but not used Signed-off-by: Alexey Skidanov --- drivers/staging/android/ion/ion.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c index 5fa5363..080ff1c 100644 --- a/drivers/sta

vadgknrwzbfiosw

2018-09-08 Thread Oehl
ixbs___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH] staging: android: ion: Fix compilation warning

2018-09-08 Thread Fabio Estevam
On Sat, Sep 8, 2018 at 1:33 PM, Alexey Skidanov wrote: > Label unlock is defined but not used You are also removing the variable 'ret', which is a different change. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.

Re: [PATCH] staging: android: ion: Fix compilation warning

2018-09-08 Thread Dan Carpenter
On Sat, Sep 08, 2018 at 02:04:06PM -0300, Fabio Estevam wrote: > On Sat, Sep 8, 2018 at 1:33 PM, Alexey Skidanov > wrote: > > Label unlock is defined but not used > > You are also removing the variable 'ret', which is a different change. I feel like it's fine. It doesn't make the patch harder t

Re: [PATCH] staging: android: ion: Fix compilation warning

2018-09-08 Thread Dan Carpenter
On Sat, Sep 08, 2018 at 07:33:40PM +0300, Alexey Skidanov wrote: > Label unlock is defined but not used > > Signed-off-by: Alexey Skidanov There is no Fixes tag. I don't understand how the compile warning was added in the first place? The code in linux-next from Friday doesn't match. What the

Re: [PATCH] staging: android: ion: Fix compilation warning

2018-09-08 Thread Alexey Skidanov
On 09/08/2018 09:42 PM, Dan Carpenter wrote: > On Sat, Sep 08, 2018 at 07:33:40PM +0300, Alexey Skidanov wrote: >> Label unlock is defined but not used >> >> Signed-off-by: Alexey Skidanov > > There is no Fixes tag. I don't understand how the compile warning was > added in the first place? T

Re: [PATCH 0/2] Follow-up patches for Cedrus v9

2018-09-08 Thread Paul Kocialkowski
Hi, Le samedi 08 septembre 2018 à 13:24 +0200, Hans Verkuil a écrit : > On 09/08/2018 12:22 PM, Chen-Yu Tsai wrote: > > On Sat, Sep 8, 2018 at 6:06 PM Hans Verkuil wrote: > > > > > > On 09/07/2018 06:33 PM, Paul Kocialkowski wrote: > > > > This brings the requested modifications on top of versio

Collaboration

2018-09-08 Thread Elliot Matare
Hello there, I hope this message reaches you in good spirit. My name is Elliot Matare. I got your contact from an online business directory as a reputable industrialist with high integrity. I have a proposition that I think will be of great interest to you. I am the brother in law of a major

[PATCH] staging: android: ion: Add per-heap counters

2018-09-08 Thread Alexey Skidanov
The heap statistics have been removed and currently even basics statistics are missing. This patch creates per heap debugfs directory /sys/kernel/debug/ and adds two counters - the number of allocated buffers and number of allocated bytes. Signed-off-by: Alexey Skidanov --- drivers/staging/andr