cron job: media_tree daily build: WARNINGS

2019-03-29 Thread Hans Verkuil
This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. Results of the daily build of media_tree: date: Sat Mar 30 05:00:12 CET 2019 media-tree git hash:94fa831e3b359f24c01b9936de33b84c38caacbb media_build git

[PATCH v2] media: pdf docs: adjust several tables for better display

2019-03-29 Thread Mauro Carvalho Chehab
Avoid cell overlapping by changing some sizes, and changing the font sizes when needed. Tested with Sphinx 1.7.8. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/v4l/buffer.rst | 19 ++-- .../media/uapi/v4l/colorspaces-defs.rst | 4 +- Documentation/media/uapi/v4l

[PATCH v2] media: platform: fix a kernel warning on clk control

2019-03-29 Thread Jae Hyun Yoo
video_on and video_off functions in the Aspeed video engine driver are being called from multiple contexts without any protection so video clocks can be disabled twice and eventually it causes a kernel warning with stack dump printing out like below: [ 120.034729] WARNING: CPU: 0 PID: 1334 at dri

Re: [PATCH] media: platform: Fix a kernel warning on clk control

2019-03-29 Thread Jae Hyun Yoo
Hi Eddie, On 3/29/2019 1:08 PM, Eddie James wrote: On 3/28/19 4:25 PM, Jae Hyun Yoo wrote: To prevent this issue, this commit adds spinlock protection and clock status checking logic into the Aspeed video engine driver. Thanks Jae. Do you have a reliable way to reproduce this? Haven't seen i

Re: [PATCH] media: staging: use strscpy() instead of strlcpy()

2019-03-29 Thread Mukesh Ojha
On 3/29/2019 8:59 PM, Mauro Carvalho Chehab wrote: There are a few left overs at staging with were still using the deprecated strlcpy() function. Signed-off-by: Mauro Carvalho Chehab s/with/which Reviewed-by: Mukesh Ojha Cheers, -Mukesh --- drivers/staging/media/imx/imx-media-dev-co

Re: [PATCH] media: platform: Fix a kernel warning on clk control

2019-03-29 Thread Eddie James
On 3/28/19 4:25 PM, Jae Hyun Yoo wrote: Video engine clock control functions in the Aspeed video engine driver are being called from multiple context without any protection so video clocks can be disabled twice and eventually it causes a kernel warning with stack dump printing out like below:

[PATCH] media: pixfmt-packed-yuv.rst: better adjust cell sizes

2019-03-29 Thread Mauro Carvalho Chehab
Avoid cell overlapping by changing some sizes. Tested with Sphinx 1.7.8. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/v4l/pixfmt-packed-yuv.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/media/uapi/v4l/pixfmt-packed-yuv.rst b/Document

Re: [PATCH] media: pixfmt-packed-yuv.rst: fix PDF output

2019-03-29 Thread Hans Verkuil
On 3/29/19 5:04 PM, Mauro Carvalho Chehab wrote: > Some lines inside this table has more than 34 columns, with is what > it is expected, according with .. tabularcolumns:: definition. You probably mean: Some lines inside this table has more than 34 columns, which isn't what is expected, according

[PATCH] media: pixfmt-packed-yuv.rst: fix PDF output

2019-03-29 Thread Mauro Carvalho Chehab
Some lines inside this table has more than 34 columns, with is what it is expected, according with .. tabularcolumns:: definition. That causes the PDF output to fail. This patch ensures that all columns will have exactly the same size, merging the unused ones using :cpan: Signed-off-by: Mauro Ca

Re: [PATCH] media: staging: use strscpy() instead of strlcpy()

2019-03-29 Thread Hans Verkuil
On 3/29/19 4:29 PM, Mauro Carvalho Chehab wrote: > There are a few left overs at staging with were still using the > deprecated strlcpy() function. > > Signed-off-by: Mauro Carvalho Chehab Reviewed-by: Hans Verkuil Thanks! Hans > --- > drivers/staging/media/imx/imx-media-dev-common.

[PATCH] media: staging: use strscpy() instead of strlcpy()

2019-03-29 Thread Mauro Carvalho Chehab
There are a few left overs at staging with were still using the deprecated strlcpy() function. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/imx/imx-media-dev-common.c | 4 ++-- drivers/staging/media/rockchip/vpu/rockchip_vpu_drv.c | 2 +- 2 files changed, 3 insertions(+),

[RFC PATCH 1/5] vicodec: always return a valid format.

2019-03-29 Thread hverkuil-cisco
From: Hans Verkuil Rather than returning width/height values of 0, just default to a format. Formats in V4L2 are always supposed to be valid, there is no concept of an invalid format. Signed-off-by: Hans Verkuil --- drivers/media/platform/vicodec/vicodec-core.c | 26 ++- 1 file

[RFC PATCH 2/5] vicodec: pass on enc output format to capture side

2019-03-29 Thread hverkuil-cisco
From: Hans Verkuil Setting the encoder output format to e.g. 1920x1080 will set the crop rectangle to 1920x1080, the coded resolution to 1920x1088 and the capture coded resolution and sizeimage to 1920x1088 as well. Signed-off-by: Hans Verkuil --- drivers/media/platform/vicodec/vicodec-core.c

[RFC PATCH 4/5] videobuf2-v4l2: set last_buffer_dequeued in dqbuf

2019-03-29 Thread hverkuil-cisco
From: Hans Verkuil last_buffer_dequeued was set to true in __fill_v4l2_buffer, but this is called for qbuf as well. Move it to vb2_dqbuf. Signed-off-by: Hans Verkuil --- drivers/media/common/videobuf2/videobuf2-v4l2.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git

[RFC PATCH 0/5] vicodec: sync with stateful codec specs

2019-03-29 Thread hverkuil-cisco
From: Hans Verkuil This RFC series fixes vicodec to conform to the upcoming version of the codec specs. This is RFC since the decoder side isn't done yet (missing compliance tests) and the last patch isn't fully verified. I pushed most of the compliance tests for the stateful encoder, but some

[RFC PATCH 5/5] vicodec: improve handling of ENC_CMD_STOP/START

2019-03-29 Thread hverkuil-cisco
From: Hans Verkuil Correctly handle stopping and restarting the encoder, keeping track of the stop and drain states. Signed-off-by: Hans Verkuil --- drivers/media/platform/vicodec/vicodec-core.c | 121 +++--- 1 file changed, 100 insertions(+), 21 deletions(-) diff --git a/drivers/

[RFC PATCH 3/5] vicodec: add V4L2_CID_MIN_BUFFERS_FOR_OUTPUT

2019-03-29 Thread hverkuil-cisco
From: Hans Verkuil The stateful encoder requires the presence of this control. Since a single buffer is sufficient for vicodec, we just set this control to 1. Signed-off-by: Hans Verkuil --- drivers/media/platform/vicodec/vicodec-core.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-)

[PATCH] media: bt8xx: avoid a needless u8 to char conversion

2019-03-29 Thread Mauro Carvalho Chehab
Instead of doing the cast, just change the type to char. Suggested-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab --- drivers/media/pci/bt8xx/dst.c| 4 ++-- drivers/media/pci/bt8xx/dst_common.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/media/p

Re: [PATCH] media: vim2m: Fix RGB 565 BE/LE support

2019-03-29 Thread Mauro Carvalho Chehab
Em Fri, 29 Mar 2019 15:01:23 +0100 Hans Verkuil escreveu: > On 3/29/19 2:44 PM, Mauro Carvalho Chehab wrote: > > The support for those two formats are archtecture-dependent. > > Use the endianness to CPU macros to do it right. > > > > Signed-off-by: Mauro Carvalho Chehab > > --- > > drivers/me

Re: [PATCH v2] media: replace strncpy() by strscpy()

2019-03-29 Thread Hans Verkuil
On 3/29/19 3:02 PM, Mauro Carvalho Chehab wrote: > The strncpy() function is being deprecated upstream. Replace > it by the safer strscpy(). > > While here, replace a few occurences of strlcpy() that were > recently added to also use strscpy(). > > Signed-off-by: Mauro Carvalho Chehab > --- > d

[PATCH v2] media: replace strncpy() by strscpy()

2019-03-29 Thread Mauro Carvalho Chehab
The strncpy() function is being deprecated upstream. Replace it by the safer strscpy(). While here, replace a few occurences of strlcpy() that were recently added to also use strscpy(). Signed-off-by: Mauro Carvalho Chehab --- drivers/media/dvb-frontends/as102_fe.c | 2 +- drivers/me

Re: [PATCH] media: vim2m: Fix RGB 565 BE/LE support

2019-03-29 Thread Hans Verkuil
On 3/29/19 2:44 PM, Mauro Carvalho Chehab wrote: > The support for those two formats are archtecture-dependent. > Use the endianness to CPU macros to do it right. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/platform/vim2m.c | 30 +- > 1 file changed

[PATCH] media: vim2m: Fix RGB 565 BE/LE support

2019-03-29 Thread Mauro Carvalho Chehab
The support for those two formats are archtecture-dependent. Use the endianness to CPU macros to do it right. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/platform/vim2m.c | 30 +- 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/drivers/media

[GIT PULL for 5.2] Sensor driver patch and SoC camera cleanups

2019-03-29 Thread Sakari Ailus
Hi Mauro, Here's a single sensor driver patch plus a few SoC camera cleanups. Please pull. The following changes since commit 122d0e8dd050cc5dc3fb9e9b5f2dee3c5276ce35: media: dvbdev: remove double-unlock (2019-03-28 14:36:14 -0400) are available in the git repository at: ssh://linuxtv.or

Мотивиран персонал

2019-03-29 Thread Radoslav Dobrev
Здравейте! Ваучерите за храна се нареждат сред любимите социални придобивки на работещите хора и сред най-предпочитаните начини за стимулиране от работодателите. Те подобряват ефективността и производителността на Вашите служители. Благодарение на ваучерите за храна реализирате спестявания – ст

Re: [PATCH v2 07/11] rockchip/vpu: Open-code media controller register

2019-03-29 Thread Tomasz Figa
On Fri, Mar 29, 2019 at 5:05 AM Ezequiel Garcia wrote: > > On Thu, 2019-03-28 at 16:11 +0900, Tomasz Figa wrote: > > On Tue, Mar 5, 2019 at 4:27 AM Ezequiel Garcia > > wrote: [snip] > > > + * @source_pad:&struct media_pad with the source pad. > > > + * Used on

Re: [PATCH v2 09/11] rockchip/vpu: Add decoder boilerplate

2019-03-29 Thread Tomasz Figa
On Fri, Mar 29, 2019 at 4:23 AM Ezequiel Garcia wrote: > > Hey Tomasz, > > Thanks for taking the time to review this carefully! > > On Thu, 2019-03-28 at 18:57 +0900, Tomasz Figa wrote: > > On Tue, Mar 5, 2019 at 4:27 AM Ezequiel Garcia > > wrote: [snip] > > > +} > > > + > > > +static int > > >