cron job: media_tree daily build: ERRORS

2016-11-04 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 Nov 5 05:00:17 CET 2016 media-tree git hash:bd676c0c04ec94bd830b9192e2c33f2c4532278d media_build

Re: [RFC] V4L2 unified low-level decoder API

2016-11-04 Thread Randy Li
On 11/04/2016 09:55 PM, Hugues FRUCHET wrote: Hi Randy, thanks for reply, some comments below: On 10/27/2016 03:08 AM, Randy Li wrote: On 10/26/2016 11:09 PM, Hugues FRUCHET wrote: Hi, This RFC aims to start discussions in order to define the codec specific controls structures to

Re: [PATCH v3 1/3] dt-bindings: mediatek: Add a binding for Mediatek JPEG Decoder

2016-11-04 Thread Laurent Pinchart
Hi Rick, Thank you for the patch. On Friday 04 Nov 2016 13:51:18 Rick Chang wrote: > Add a DT binding documentation for Mediatek JPEG Decoder of > MT2701 SoC. This version looks much better ! > Signed-off-by: Rick Chang > Signed-off-by: Minghsiu Tsai

[PATCH 2/4] v4l: vsp1: Move vsp1_video_pipeline_setup_partitions() function

2016-11-04 Thread Kieran Bingham
Separate the code change from the function move so that code changes can be clearly identified. This commit has no functional change. The partition algorithm functions will be changed, and vsp1_video_partition() will call vsp1_video_pipeline_setup_partitions(). To prepare for that, move the

[PATCH 1/4] v4l: vsp1: Implement partition algorithm restrictions

2016-11-04 Thread Kieran Bingham
The partition algorithm introduced to support scaling, and rotation on Gen3 hardware has some restrictions on pipeline configuration. The UDS must not be connected after the SRU in a pipeline, and whilst an SRU can be connected before the UDS, it can only do so in identity mode. On Gen3

[PATCH 4/4] v4l: vsp1: Remove redundant context variables

2016-11-04 Thread Kieran Bingham
The vsp1_pipe object context variables for div_size and current_partition allowed state to be maintained through processing the partitions during processing. Now that the partition tables are calculated during stream on, there is no requirement to store these variables in the pipe object.

[PATCH 0/4] vsp1 partition algorithm improvements

2016-11-04 Thread Kieran Bingham
Some updates and initial improvements for the VSP1 partition algorithm that remove redundant processing and variables, reducing the processing done in interrupt context slightly. Patch 1 brings in some protection against invalid pipeline configurations that are not supported by the partition

[PATCH 3/4] v4l: vsp1: Calculate partition sizes at stream start.

2016-11-04 Thread Kieran Bingham
Previously the active window and partition sizes for each partition is calculated for each partition every frame. This data is constant and only needs to be calculated once at the start of the stream. Extend the vsp1_pipe object to store the maximum number of partitions possible and pre-calculate

[PATCHv2 2/2] v4l: vsp1: Provide a writeback video device

2016-11-04 Thread Kieran Bingham
When the VSP1 is used in an active display pipeline, the output of the WPF can supply the LIF entity directly and simultaneously write to memory. Support this functionality in the VSP1 driver, by extending the WPF source pads, and establishing a V4L2 video device node connected to the new source.

[PATCHv2 1/2] Revert "[media] v4l: vsp1: Supply frames to the DU continuously"

2016-11-04 Thread Kieran Bingham
This reverts commit 3299ba5c0b21 ("[media] v4l: vsp1: Supply frames to the DU continuously") The DU output mode does not rely on frames being supplied on the WPF as its pipeline is supplied from DRM. For the upcoming WPF writeback functionality, we will choose to enable writeback mode if there is

[PATCHv2 0/2] vsp1 writeback prototype

2016-11-04 Thread Kieran Bingham
Resending this patch series to bring in dri-devel, and interested parties. Apologies for the resend to linux-media and linux-renesas-soc. This short series extends the VSP1 on the RCar platforms to allow creating a V4L2 video node on display pipelines where the hardware allows writing to memory

Re: [PATCH v2 1/3] dt-bindings: mediatek: Add a binding for Mediatek JPEG Decoder

2016-11-04 Thread Laurent Pinchart
Hi Rick, On Friday 04 Nov 2016 12:51:00 Rick Chang wrote: > On Thu, 2016-11-03 at 20:34 +0200, Laurent Pinchart wrote: > > On Thursday 03 Nov 2016 20:33:12 Laurent Pinchart wrote: > >> On Monday 31 Oct 2016 15:16:55 Rick Chang wrote: > >>> Add a DT binding documentation for Mediatek JPEG Decoder

Re: [PATCH v2] media: omap3isp: Use dma_request_chan_by_mask() to request the DMA channel

2016-11-04 Thread Laurent Pinchart
Hi Peter, Thank you for the patch. On Friday 04 Nov 2016 09:58:02 Peter Ujfalusi wrote: > When requesting the DMA channel it was mandatory that we do not have DMA > resource nor valid DMA channel via DT. In this case the > dma_request_slave_channel_compat() would fall back and request any

Re: [PATCH v1 4/9] [media] st-delta: STiH4xx multi-format video decoder v4l2 driver

2016-11-04 Thread Hugues FRUCHET
Many thanks Hans for your review, feedback below: On 11/03/2016 04:35 PM, Hans Verkuil wrote: > Hi Hugues, > > See some code comments below: > > On 20/09/16 16:33, Hugues Fruchet wrote: >> This V4L2 driver enables DELTA multi-format video decoder >> of STMicroelectronics STiH4xx SoC series. >> >>

Re: [PATCHv6] support for AD5820 camera auto-focus coil

2016-11-04 Thread Tony Lindgren
* Pavel Machek [161104 00:45]: > Hi! > > > > > I have to admit I mostly use an N9. > > > > > > Well, if you compare that to development on PC, I prefer PC. > > > > > > Even arm development boards are usually easier, as they don't need too > > > complex userspace, and do have

[GIT PULL FOR v4.10 v2] cec: fix remaining issues and move it from staging to drivers/media

2016-11-04 Thread Hans Verkuil
This resolves the remaining issues and get it out of staging. See the cover letter for more info: https://www.mail-archive.com/linux-media@vger.kernel.org/msg104311.html Three additional patches are added in this v2: cec: sanitize msg.flags cec.h/cec-funcs.h: don't use bool in

[PATCH] cec: an inner loop clobbered the outer loop variable

2016-11-04 Thread Hans Verkuil
An inner for-loop reused the outer loop variable. This was only noticeable with CEC adapters supporting more than one logical address. Signed-off-by: Hans Verkuil --- drivers/media/cec/cec-adap.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git

[PATCH] cec.h/cec-funcs.h: don't use bool in public headers

2016-11-04 Thread Hans Verkuil
Replace bool by int or __u8 (when used in a struct). Signed-off-by: Hans Verkuil --- include/uapi/linux/cec-funcs.h | 70 +- include/uapi/linux/cec.h | 37 +++--- 2 files changed, 54 insertions(+), 53

Re: [RFC] V4L2 unified low-level decoder API

2016-11-04 Thread Nicolas Dufresne
Le vendredi 04 novembre 2016 à 14:55 +0100, Hugues FRUCHET a écrit : > >> > >> I can help on H264 on a code review basis based on the functional H264 > >> setup I have in-house and codec knowledge, but I cannot provide > >> implementation in a reasonable timeframe, same for VP8. > >> > >> > >> >

Re: [RFC] V4L2 unified low-level decoder API

2016-11-04 Thread Hugues FRUCHET
Hi Randy, thanks for reply, some comments below: On 10/27/2016 03:08 AM, Randy Li wrote: > > > On 10/26/2016 11:09 PM, Hugues FRUCHET wrote: >> Hi, >> >> >> >> This RFC aims to start discussions in order to define the codec specific >> controls structures to fulfill the low-level decoder API

Re: [GIT PULL FOR v4.10] cec: fix remaining issues and move it from staging to drivers/media

2016-11-04 Thread Hans Verkuil
Cancelled this pull request. I found some issues with the public header (don't use bool/true/false) and a small bug in the framework causing memory not to be zeroed. I'll fix those and repost. Regards, Hans On 03/11/16 16:00, Hans Verkuil wrote: This resolves the remaining issues

[no subject]

2016-11-04 Thread Amir A. Khanmammadov
Thanks for your last email response to me. The information required should include the following-: Your full names Your address Telephone number Your private email Occupation Age This is to enable my further discussion with you in confidence. Best regards and wishes to you. Amir A. Khanmammadov

Re: [PATCH] media: s5p-mfc include buffer size in error message

2016-11-04 Thread Sylwester Nawrocki
On 10/18/2016 02:43 AM, Shuah Khan wrote: > Include buffer size in s5p_mfc_alloc_priv_buf() the error message when it > fails to allocate the buffer. Remove the debug message that does the same. > > Signed-off-by: Shuah Khan > --- >

[PATCH] cec: sanitize msg.flags

2016-11-04 Thread Hans Verkuil
The CEC_MSG_FL_REPLY_TO_FOLLOWERS message flag only makes sense for transmitted messages where you want to wait for the reply. Clear the flag in all other cases. Signed-off-by: Hans Verkuil --- diff --git a/drivers/media/cec/cec-adap.c b/drivers/media/cec/cec-adap.c

RE: [RFC 5/5] doc_rst: media: New SDR formats SC16, SC18 & SC20

2016-11-04 Thread Ramesh Shanmugasundaram
Hi Antti, Thanks for the response. > Subject: Re: [RFC 5/5] doc_rst: media: New SDR formats SC16, SC18 & SC20 > > Hello > > On 11/02/2016 10:58 PM, Laurent Pinchart wrote: > > Hi Ramesh, > > > > On Wednesday 02 Nov 2016 09:00:00 Ramesh Shanmugasundaram wrote: > >> Hi Laurent, > >> > >> Any

Re: [PATCH] staging: st-cec: add parentheses around complex macros

2016-11-04 Thread Benjamin Gaignard
Thanks, Acked-by: Benjamin Gaignard 2016-11-04 8:58 GMT+01:00 Maninder Singh : > This patch fixes the following checkpatch.pl error: > ERROR: Macros with complex values should be enclosed in parentheses > > Signed-off-by: Maninder Singh

Re: [PATCH 3/3] [media] au0828-video: Move two assignments in au0828_init_isoc()

2016-11-04 Thread Hans Verkuil
On 03/11/16 20:56, SF Markus Elfring wrote: From: Markus Elfring Date: Mon, 24 Oct 2016 22:44:02 +0200 Move the assignment for the data structure members "isoc_copy" and "num_bufs" behind the source code for memory allocations by this function. I don't see the

Re: [PATCH 10/34] [media] DaVinci-VPBE: Check return value of a setup_if_config() call in vpbe_set_output()

2016-11-04 Thread Hans Verkuil
On 03/11/16 21:54, SF Markus Elfring wrote: From: Markus Elfring Date: Wed, 12 Oct 2016 09:56:56 +0200 * A function was called over the pointer "setup_if_config" in the data structure "venc_platform_data". But the return value was not used so far. Thus assign

Re: [PATCH 17/34] [media] DaVinci-VPFE-Capture: Improve another size determination in vpfe_enum_input()

2016-11-04 Thread Hans Verkuil
On 03/11/16 22:05, SF Markus Elfring wrote: @@ -1091,7 +1091,7 @@ static int vpfe_enum_input(struct file *file, void *priv, return -EINVAL; } sdinfo = _dev->cfg->sub_devs[subdev]; -memcpy(inp, >inputs[index], sizeof(struct v4l2_input)); +memcpy(inp, >inputs[index],

Re: [PATCH RESEND] media: omap3isp: Use dma_request_chan() to requesting DMA channel

2016-11-04 Thread Peter Ujfalusi
Hi Laurent, On 11/03/2016 05:12 PM, Laurent Pinchart wrote: >> It is a bit misleading that it used dma_request_slave_channel_compat() >> for getting the channel. >> >> I think what would be correct is: >> dma_cap_mask_t mask; >> >> dma_cap_zero(mask); >> dma_cap_set(DMA_SLAVE, mask); >>

[PATCH] staging: st-cec: add parentheses around complex macros

2016-11-04 Thread Maninder Singh
This patch fixes the following checkpatch.pl error: ERROR: Macros with complex values should be enclosed in parentheses Signed-off-by: Maninder Singh --- drivers/staging/media/st-cec/stih-cec.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH v2] media: omap3isp: Use dma_request_chan_by_mask() to request the DMA channel

2016-11-04 Thread Peter Ujfalusi
When requesting the DMA channel it was mandatory that we do not have DMA resource nor valid DMA channel via DT. In this case the dma_request_slave_channel_compat() would fall back and request any channel with SW trigger. The same can be achieved with the dma_request_chan_by_mask() without the

Re: [PATCHv6] support for AD5820 camera auto-focus coil

2016-11-04 Thread Pavel Machek
Hi! > > > I have to admit I mostly use an N9. > > > > Well, if you compare that to development on PC, I prefer PC. > > > > Even arm development boards are usually easier, as they don't need too > > complex userspace, and do have working serial ports. > > > > But I do have a serial adapter for