cron job: media_tree daily build: ERRORS

2015-12-02 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: Thu Dec 3 04:00:22 CET 2015 git branch: test git hash: 10897dacea26943dd80bd6629117f4620fc320ef gcc versio

Re: Sabrent (stk1160) / Easycap driver problem

2015-12-02 Thread Ezequiel Garcia
On 2 December 2015 at 14:43, Philippe Desrochers wrote: > I'm sending the email again (in plain text) since it seems it was blocked by > the server. > > EASYCAP CHINA CLONE (OK): > [ 8630.596236] usb 2-1: new high-speed USB device number 6 using ehci-pci > [ 8630.729074] usb 2-1: New USB device fo

Re: Sabrent (stk1160) / Easycap driver problem

2015-12-02 Thread Ezequiel Garcia
Hi Philippe, Ccing the linux-media ML. Please don't reply dropping the Cc! 2015-12-02 11:58 GMT-03:00 Philippe Desrochers : > Hello Ezequiel, > > I'm using your stk1160 driver with some EasyCap (china) clone and it is > working fine. > > However, a few days ago, I bought two Sabrent USB-AVCPT and

[PATCH 4/5] [media] coda: don't start streaming without queued buffers

2015-12-02 Thread Philipp Zabel
We could support start streaming with an empty output queue for the BIT decoders due to the bitstream buffer which could still contain data at this point, but there is really no reason for userspace to expect this to work. Simplify the code by disallowing it. Signed-off-by: Philipp Zabel --- dri

[PATCH 2/5] [media] coda: relax coda_jpeg_check_buffer for trailing bytes

2015-12-02 Thread Philipp Zabel
From: Philipp Zabel coda_jpeg_check_buffer only cares about the buffer length and contents, so change the parameter type back from v4l2_vb2_buffer to just the vb2_buffer. Instead of just checking the first and last bytes for the SOI and EOI markers, relax the EOI marker check a bit and allow up t

[PATCH 3/5] [media] coda: hook up vidioc_prepare_buf

2015-12-02 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-common.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/platform/coda/coda-common.c b/drivers/media/platform/coda/coda-common.c index 03b46de..007f458 100644 --- a/drivers/media/platform/coda/coda-common.c +++ b

[PATCH 5/5] [media] coda: enable MPEG-2 ES decoding

2015-12-02 Thread Philipp Zabel
Hook up the MPEG-2 ES decoder. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-common.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/media/platform/coda/coda-common.c b/drivers/media/platform/coda/coda-common.c index 2452f46..2197032 100644 --- a/drivers/med

[PATCH 1/5] [media] coda: make to_coda_video_device static

2015-12-02 Thread Philipp Zabel
This function is not used outside coda-common.c. Signed-off-by: Philipp Zabel --- drivers/media/platform/coda/coda-common.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/platform/coda/coda-common.c b/drivers/media/platform/coda/coda-common.c index 15516a6..

Re: [RESEND RFC/PATCH 6/8] media: platform: mtk-vcodec: Add Mediatek V4L2 Video Encoder Driver

2015-12-02 Thread Daniel Thompson
On 02/12/15 13:08, tiffany lin wrote: We need MTK_STATE_ABORT to inform encoder thread (mtk_venc_worker) that stop encodeing job from stopped ctx instance. When user space qbuf, we need to make sure everything is ready to sent buf to encode. Agree that you need a flag here. In fact currently yo

Re: [RFC PATCH 04/11] v4l2-ctrls: add config store support

2015-12-02 Thread Enric Balletbo Serra
2015-12-02 13:33 GMT+01:00 Hans Verkuil : > On 12/02/15 13:03, Enric Balletbo Serra wrote: >> Dear Hans, >> >> We've a driver that uses your confstore stuff and we'd like to push >> upstream. I'm wondering if there is any plan to upstream the confstore >> patches or if this was abandoned for some r

Re: [RESEND RFC/PATCH 6/8] media: platform: mtk-vcodec: Add Mediatek V4L2 Video Encoder Driver

2015-12-02 Thread tiffany lin
On Tue, 2015-12-01 at 15:42 +, Daniel Thompson wrote: > On 01/12/15 10:42, tiffany lin wrote: > > diff --git a/drivers/media/platform/mtk-vcodec/common/venc_drv_if.c > b/drivers/media/platform/mtk-vcodec/common/venc_drv_if.c > > new file mode 100644 > > index 000..

Re: [RFC PATCH 04/11] v4l2-ctrls: add config store support

2015-12-02 Thread Hans Verkuil
On 12/02/15 13:03, Enric Balletbo Serra wrote: > Dear Hans, > > We've a driver that uses your confstore stuff and we'd like to push > upstream. I'm wondering if there is any plan to upstream the confstore > patches or if this was abandoned for some reason. Thanks Ouch, that's really old code you'

Re: [RFC PATCH 04/11] v4l2-ctrls: add config store support

2015-12-02 Thread Enric Balletbo Serra
Dear Hans, 2014-11-17 9:46 GMT+01:00 Hans Verkuil : > On 11/14/2014 04:44 PM, Sakari Ailus wrote: >> Hi Hans, >> >> Some comments below. >> >> On Sun, Sep 21, 2014 at 04:48:22PM +0200, Hans Verkuil wrote: >>> From: Hans Verkuil >>> >>> Signed-off-by: Hans Verkuil >>> --- >>> drivers/media/v4l2-

[PATCH 2/6] s5p-mfc: make queue cleanup code common

2015-12-02 Thread Andrzej Hajda
Code for queue cleanup has nothing specific to hardware version. Signed-off-by: Andrzej Hajda --- drivers/media/platform/s5p-mfc/s5p_mfc.c| 26 + drivers/media/platform/s5p-mfc/s5p_mfc_common.h | 1 + drivers/media/platform/s5p-mfc/s5p_mfc_dec.c| 6 ++ d

[PATCH 0/6] MFC locking rework and code cleanup

2015-12-02 Thread Andrzej Hajda
Hi Kamil, This patchset contains one patch which should fix races during accessing context fields. As the patch significantly changes locking I have tested it in different different scenarios. Other patches just clean up the code. The patchset is based on the latest media-tree branch. Regards

[PATCH 1/6] s5p-mfc: use one implementation of s5p_mfc_get_new_ctx

2015-12-02 Thread Andrzej Hajda
Both version of MFC driver uses functions with the same body and name. The patch moves them to common location. It also simplifies it. Signed-off-by: Andrzej Hajda --- drivers/media/platform/s5p-mfc/s5p_mfc.c| 20 drivers/media/platform/s5p-mfc/s5p_mfc_common.h | 1

[PATCH 5/6] s5p-mfc: merge together s5p_mfc_hw_call and s5p_mfc_hw_call_void

2015-12-02 Thread Andrzej Hajda
Both macros can be merged into one. Signed-off-by: Andrzej Hajda --- drivers/media/platform/s5p-mfc/s5p_mfc.c| 38 - drivers/media/platform/s5p-mfc/s5p_mfc_common.h | 8 +- drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c | 16 +-- drivers/media/platfo

[PATCH 6/6] s5p-mfc: remove volatile attribute from MFC register addresses

2015-12-02 Thread Andrzej Hajda
MFC register addresses are used only by writel/readl macros which already takes care of proper register accessing. Signed-off-by: Andrzej Hajda --- drivers/media/platform/s5p-mfc/s5p_mfc_opr.h | 488 +-- 1 file changed, 244 insertions(+), 244 deletions(-) diff --git a/dr

[PATCH 3/6] s5p-mfc: remove unnecessary callbacks

2015-12-02 Thread Andrzej Hajda
Many version specific functions are not called by common code, so there is no need to use callbacks. Additionally some of them are not used at all, so they can be safely removed. Signed-off-by: Andrzej Hajda --- drivers/media/platform/s5p-mfc/s5p_mfc_opr.h| 17 - drivers/media/platfo

[PATCH 4/6] s5p-mfc: use spinlock to protect MFC context

2015-12-02 Thread Andrzej Hajda
MFC driver uses dev->irqlock spinlock to protect queues only, but many context fields requires protection also - they can be accessed concurrently from IOCTLs and IRQ handler. The patch increases protection range of irqlock to those fields also. Signed-off-by: Andrzej Hajda --- drivers/media/pla