[PATCH] staging: fieldbus: anybuss: use devm_platform_ioremap_resource helper

2019-10-26 Thread Cristiane Naves
Use devm_platform_ioremap_resource helper which wraps platform_get_resource() and devm_ioremap_resource() together. Issue found by coccicheck. Signed-off-by: Cristiane Naves --- drivers/staging/fieldbus/anybuss/arcx-anybus.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --g

[PATCH] staging: media: allegro-dvt: remove bool comparison

2019-10-26 Thread Cristiane Naves
Bool tests don't need comparisons. Issue found by coccicheck. Signed-off-by: Cristiane Naves --- drivers/staging/media/allegro-dvt/nal-h264.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/allegro-dvt/nal-h264.c b/drivers/staging/media/allegro-dvt/nal-

[PATCH] staging: octeon: Remove unneeded variable

2019-10-26 Thread Cristiane Naves
Remove unneeded variable used to store return value. Issue found by coccicheck. Signed-off-by: Cristiane Naves --- drivers/staging/octeon/octeon-stubs.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/octeon/octeon-stubs.h b/drivers/staging/octeon/octeon-s

Re: [GIT PULL] Staging driver fix for 5.4-rc5

2019-10-26 Thread pr-tracker-bot
The pull request you sent on Sat, 26 Oct 2019 20:18:39 +0200: > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git > tags/staging-5.4-rc5 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/228bd6243447f3e5613457e6400112f197dbba7b Thank you! -- Deet-doot-d

Re: [Outreachy kernel] [PATCH v2 1/3] staging: rtl8712: Fix Alignment of open parenthesis

2019-10-26 Thread Julia Lawall
On Sat, 26 Oct 2019, Cristiane Naves wrote: > Fix alignment should match open parenthesis. Issue found by checkpatch. > > Signed-off-by: Cristiane Naves > --- > drivers/staging/rtl8712/rtl8712_recv.c | 30 +++--- > 1 file changed, 15 insertions(+), 15 deletions(-) > >

[PATCH v2 3/3] staging: rtl8712: Remove lines before a close brace

2019-10-26 Thread Cristiane Naves
Fix Blank lines aren't necessary before a close brace '}'. Issue found by checkpatch. Signed-off-by: Cristiane Naves --- drivers/staging/rtl8712/rtl8712_recv.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers/staging/rtl8712/rtl8712_recv.c in

[PATCH v2 2/3] staging: rtl8712: Remove unnecessary parentheses.

2019-10-26 Thread Cristiane Naves
Remove unnecessary parentheses. Issue suggested by Joe Perches . Signed-off-by: Cristiane Naves --- drivers/staging/rtl8712/rtl8712_recv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers/staging/rtl8712/rtl8712_recv.c index 12

[PATCH v2 1/3] staging: rtl8712: Fix Alignment of open parenthesis

2019-10-26 Thread Cristiane Naves
Fix alignment should match open parenthesis. Issue found by checkpatch. Signed-off-by: Cristiane Naves --- drivers/staging/rtl8712/rtl8712_recv.c | 30 +++--- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers

[PATCH v3] staging: gasket: Fix lines ending with a '('

2019-10-26 Thread Cristiane Naves
Fix lines ending with a '('. Issue found by checkpatch. Signed-off-by: Cristiane Naves --- Changes since v2: - Join the two similar patches staging: gasket: Fix lines ending with a '('. Changes since v1: - Arranging conflict between author name and signed-off-by. --- drivers/staging/gasket

Re: [Outreachy kernel] [PATCH v2 0/5] Remove typedef declarations in staging: octeon

2019-10-26 Thread Greg Kroah-Hartman
On Wed, Oct 23, 2019 at 08:43:04PM +0300, Aaro Koskinen wrote: > Hi, > > On Sat, Oct 12, 2019 at 08:35:19PM +0200, Julia Lawall wrote: > > On Sat, 12 Oct 2019, Wambui Karuga wrote: > > > This patchset removes the addition of new typedefs data types in octeon, > > > along with replacing the previou

[GIT PULL] Staging driver fix for 5.4-rc5

2019-10-26 Thread Greg KH
The following changes since commit 4f5cafb5cb8471e54afdc9054d973535614f7675: Linux 5.4-rc3 (2019-10-13 16:37:36 -0700) are available in the Git repository at: git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git tags/staging-5.4-rc5 for you to fetch changes up to 153c5d8191c261

Re: [RESEND PATCH 1/2] staging: rtl8712: Fix Alignment of open parenthesis

2019-10-26 Thread Greg Kroah-Hartman
On Fri, Oct 25, 2019 at 10:09:38PM -0300, Cristiane Naves wrote: > Fix alignment should match open parenthesis.Issue found by checkpatch. Space after a '.' between sentences, right? thanks greg k-h ___ devel mailing list de...@linuxdriverproject.org h

Re: [RESEND PATCH 2/2] staging: rtl8712: Remove lines before a close brace

2019-10-26 Thread Greg Kroah-Hartman
On Fri, Oct 25, 2019 at 10:10:04PM -0300, Cristiane Naves wrote: > Fix Blank lines aren't necessary before a close brace '}'. Issue found > by checkpatch. > > Signed-off-by: Cristiane Naves > --- > drivers/staging/rtl8712/rtl8712_recv.c | 2 -- > 1 file changed, 2 deletions(-) What changed from

Re: [RESEND PATCH] staging: gasket: Fix line ending with a '('

2019-10-26 Thread Greg Kroah-Hartman
On Fri, Oct 25, 2019 at 08:39:09PM -0300, Cristiane Naves wrote: > Fix line ending with a '(' > > Signed-off-by: Cristiane Naves > --- > drivers/staging/gasket/gasket_ioctl.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Why is this a "RESEND"? Did nothing change in it? Always v

[PATCH 4/4] media: cedrus: hevc: Add support for multiple slices

2019-10-26 Thread Jernej Skrabec
Now that segment address is available, support for multi-slice frames can be easily added. Signed-off-by: Jernej Skrabec --- .../staging/media/sunxi/cedrus/cedrus_h265.c | 21 +++ .../staging/media/sunxi/cedrus/cedrus_video.c | 1 + 2 files changed, 18 insertions(+), 4 deletion

[PATCH 2/4] media: cedrus: hevc: Add support for scaling matrix

2019-10-26 Thread Jernej Skrabec
HEVC frames may use scaling list feature. Add support for it. Signed-off-by: Jernej Skrabec --- drivers/staging/media/sunxi/cedrus/cedrus.c | 7 ++ drivers/staging/media/sunxi/cedrus/cedrus.h | 1 + .../staging/media/sunxi/cedrus/cedrus_dec.c | 2 + .../staging/media/sunxi/cedrus/cedrus

[PATCH 0/4] media: cedrus: hevc: Add support for scaling matrix and multi-slice frames

2019-10-26 Thread Jernej Skrabec
Currently there is no control for HEVC scaling matrix and segment address field is also missing. This series adds those and implements support for them in Cedrus driver. Following videos were using for testing: - scaling matrix: http://jernej.libreelec.tv/videos/h265/slist_test.mp4 - multi-slice:

[PATCH 3/4] media: uapi: hevc: Add segment address field

2019-10-26 Thread Jernej Skrabec
If HEVC frame consists of multiple slices, segment address has to be known in order to properly decode it. Add segment address field to slice parameters. Signed-off-by: Jernej Skrabec --- Documentation/media/uapi/v4l/ext-ctrls-codec.rst | 5 - include/media/hevc-ctrls.h

[PATCH 1/4] media: uapi: hevc: Add scaling matrix control

2019-10-26 Thread Jernej Skrabec
HEVC has a scaling matrix concept. Add support for it. Signed-off-by: Jernej Skrabec --- .../media/uapi/v4l/ext-ctrls-codec.rst| 41 +++ .../media/uapi/v4l/pixfmt-compressed.rst | 1 + drivers/media/v4l2-core/v4l2-ctrls.c | 10 + include/media/hevc-ctrl

Re: [PATCH 7/7] staging: rtl8188eu: reduce indentation level in rtw_alloc_stainfo

2019-10-26 Thread Joe Perches
On Sat, 2019-10-26 at 14:11 +0200, Michael Straube wrote: > Remove else-arm from if-else statement. Move the else code out of the > if-else and skip it by adding goto exit to the if block. The exit label > was directly after the else-arm, so there is no change in behaviour. > Reduces indentation le

[PATCH AUTOSEL 4.19 53/59] dpaa2-eth: add irq for the dpmac connect/disconnect event

2019-10-26 Thread Sasha Levin
From: Florin Chiculita [ Upstream commit 8398b375a9e3f5e4bba9bcdfed152a8a247dee01 ] Add IRQ for the DPNI endpoint change event, resolving the issue when a dynamically created DPNI gets a randomly generated hw address when the endpoint is a DPMAC object. Signed-off-by: Florin Chiculita Signed-o

[PATCH 3/7] staging: rtl8188eu: rename array bcast_addr

2019-10-26 Thread Michael Straube
Rename array bcast_addr to be more consistent in variable naming. In other places in this file buffers for broadcast addresses are named bc_addr as well. bcast_addr -> bc_addr Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 4 ++-- 1 file changed, 2 insertions(

[PATCH 5/7] staging: rtl8188eu: remove ternary operator

2019-10-26 Thread Michael Straube
Instead of using ternary operator to set variable res, use the value of variable match (or the negation) directly to simplify the code and improve readability. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-

[PATCH 6/7] staging: rtl8188eu: cleanup long lines in rtw_sta_mgt.c

2019-10-26 Thread Michael Straube
Cleanup lines over 80 characters in rtw_sta_mgt.c by adding line breaks where appropriate. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_sta_m

[PATCH 2/7] staging: rtl8188eu: convert unsigned char array to u8

2019-10-26 Thread Michael Straube
Convert array bcast_addr from unsigned char to u8. Clears a line over 80 characters checkpatch warning. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_sta_mgt.c

[PATCH 1/7] staging: rtl8188eu: cleanup comments in rtw_sta_mgt.c

2019-10-26 Thread Michael Straube
Cleanup comments in rtw_sta_mgt.c to use kernel block comment style and not exceed 80 characters line length. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/drivers/staging/rtl81

[PATCH 7/7] staging: rtl8188eu: reduce indentation level in rtw_alloc_stainfo

2019-10-26 Thread Michael Straube
Remove else-arm from if-else statement. Move the else code out of the if-else and skip it by adding goto exit to the if block. The exit label was directly after the else-arm, so there is no change in behaviour. Reduces indentation level and clears a line over 80 characters checkpatch warning. Sign

[PATCH 4/7] staging: rtl8188eu: convert rtw_access_ctrl to return bool

2019-10-26 Thread Michael Straube
Function rtw_access_ctrl returns boolean values, so change the return type to bool. Also convert the local variables that are used for the return value from u8 to bool. Signed-off-by: Michael Straube --- drivers/staging/rtl8188eu/core/rtw_sta_mgt.c | 6 +++--- drivers/staging/rtl8188eu/include/s

[PATCH 0/3] media: cedrus: Add support for 4k videos

2019-10-26 Thread Jernej Skrabec
While cedrus driver accepts videos up to 3840x2160, they are not decoded correctly. Driver doesn't correctly set mode register for widths greater than 2048 (patch 1). H264 engine also needs additional buffers which are not provided currently (patch 2). Finally, there are several different resolutio

[PATCH 2/3] media: cedrus: Fix H264 4k support

2019-10-26 Thread Jernej Skrabec
H264 decoder needs additional or bigger buffers in order to decode 4k videos. Signed-off-by: Jernej Skrabec --- drivers/staging/media/sunxi/cedrus/cedrus.h | 7 ++ .../staging/media/sunxi/cedrus/cedrus_h264.c | 83 +-- .../staging/media/sunxi/cedrus/cedrus_regs.h | 11 +++ 3

[PATCH 3/3] media: cedrus: Increase maximum supported size

2019-10-26 Thread Jernej Skrabec
There are few variations of 4k resolutions. The biggest one is 4096x2304 which is also supported by HW. It has also nice property that both width and size are divisible by maximum HEVC CTB size, which is 64. Signed-off-by: Jernej Skrabec --- drivers/staging/media/sunxi/cedrus/cedrus_video.c | 4

[PATCH 1/3] media: cedrus: Properly signal size in mode register

2019-10-26 Thread Jernej Skrabec
Mode register also holds information if video width is bigger than 2048 and if it is equal to 4096. Rework cedrus_engine_enable() to properly signal this properties. Signed-off-by: Jernej Skrabec --- drivers/staging/media/sunxi/cedrus/cedrus_h264.c | 2 +- drivers/staging/media/sunxi/cedrus/ce

[PATCH v3 0/2] media: cedrus: improvements

2019-10-26 Thread Jernej Skrabec
This is continuation of https://lkml.org/lkml/2019/5/30/1459 with several patches removed (2 merged, others needs redesign) and one added. Patch 1 fixes h264 playback issue which happens in rare cases. Patch 2 replaces direct accesses to capture queue from m2m contex with helpers which was design

[PATCH v3 2/2] media: cedrus: Use helpers to access capture queue

2019-10-26 Thread Jernej Skrabec
Accessing capture queue structue directly is not safe. Use helpers for that. Acked-by: Paul Kocialkowski Signed-off-by: Jernej Skrabec --- drivers/staging/media/sunxi/cedrus/cedrus.h | 8 ++-- drivers/staging/media/sunxi/cedrus/cedrus_h264.c | 8 ++-- 2 files changed, 12 insertions

[PATCH v3 1/2] media: cedrus: Fix decoding for some H264 videos

2019-10-26 Thread Jernej Skrabec
It seems that for some H264 videos at least one bitstream parsing trigger must be called in order to be decoded correctly. There is no explanation why this helps, but it was observed that two sample videos with this fix are now decoded correctly and there is no regression with others. Acked-by: Pa