cron job: media_tree daily build: ERRORS

2016-07-16 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: Sun Jul 17 04:00:10 CEST 2016 git branch: test git hash: e05b1872f29a85532c2b34e3a4974a27158f1463 gcc versi

Re: [PATCH 2/4] dt-bindings: Add a binding for Mediatek MDP

2016-07-16 Thread Rob Herring
On Thu, Jul 14, 2016 at 08:17:59PM +0800, Minghsiu Tsai wrote: > Add a DT binding documentation of MDP for the MT8173 SoC > from Mediatek > > Signed-off-by: Minghsiu Tsai > --- > .../devicetree/bindings/media/mediatek-mdp.txt | 92 > > 1 file changed, 92 insertions(+)

Re: [PATCH] af9035: fix dual tuner detection with PCTV 79e

2016-07-16 Thread Stefan Pöschel
Am 15.07.2016 um 08:21 schrieb Antti Palosaari: > Applied and PULL requested for 4.7. Great, thanks! > Anyhow, it does not apply for 4.6. You must backport that patch to 4.6 > stable also! I have never done backporting before, so I need some advice I think: Am I right that I have to create the p

Re: [PATCH] [media] dw2102: Add support for Terratec Cinergy S2 USB BOX

2016-07-16 Thread Mauro Carvalho Chehab
Em Sat, 16 Jul 2016 18:26:35 +0200 Olli Salonen escreveu: > Hi guys, > > It seems Philipp added the support for this device in dw2102 driver > and Benjamin did that for the dvbsky driver a bit earlier. > > # grep -i 0ccdp0105 /lib/modules/$(uname -r)/modules.alias > alias usb:v0CCDp0105d*dc*dsc

Re: [PATCH] [media] dw2102: Add support for Terratec Cinergy S2 USB BOX

2016-07-16 Thread Olli Salonen
Hi guys, It seems Philipp added the support for this device in dw2102 driver and Benjamin did that for the dvbsky driver a bit earlier. # grep -i 0ccdp0105 /lib/modules/$(uname -r)/modules.alias alias usb:v0CCDp0105d*dc*dsc*dp*ic*isc*ip*in* dvb_usb_dvbsky alias usb:v0CCDp0105d*dc*dsc*dp*ic*isc*ip

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Hans Verkuil
On 07/16/2016 05:57 PM, Ricardo Ribalda Delgado wrote: > Hi Hans > > On Sat, Jul 16, 2016 at 5:28 PM, Hans Verkuil wrote: > >> >>> + >>> +enum v4l2_rgb_encoding { >>> + V4L2_RGB_ENC_FULL = 32, >>> + V4L2_HSV_ENC_16_235 = 33, >>> +}; >> >> No. > > I was trying to fit also L

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
Hi Hans On Sat, Jul 16, 2016 at 5:28 PM, Hans Verkuil wrote: > >> + >> +enum v4l2_rgb_encoding { >> + V4L2_RGB_ENC_FULL = 32, >> + V4L2_HSV_ENC_16_235 = 33, >> +}; > > No. I was trying to fit also Laurent special 16-235 RGB format. I will remove it on future versions. Ca

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Hans Verkuil
On 07/16/2016 04:32 PM, Ricardo Ribalda Delgado wrote: > Hi > > On Sat, Jul 16, 2016 at 4:12 PM, Laurent Pinchart > wrote: > >> I'd still like to know about it for my personal information :-) > > Maybe it is just a very cheap gamma. > >> >>> Anyway, I am inclined to use ycbcr_enc as well. >> >

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Hans Verkuil
On 07/16/2016 04:12 PM, Laurent Pinchart wrote: >> Limited vs full range quantization is handled by the quantization field. >> It's there already. > > Right. I wonder how we'll deal with that when someone will come up with more > than one limited range quantizations, have you thought about it ?

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
Hi On Sat, Jul 16, 2016 at 4:12 PM, Laurent Pinchart wrote: > I'd still like to know about it for my personal information :-) Maybe it is just a very cheap gamma. > >> Anyway, I am inclined to use ycbcr_enc as well. > > I'm glad we agree. > Are you thinking about something like this: diff -

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Laurent Pinchart
Hi Hans, On Saturday 16 Jul 2016 15:59:08 Hans Verkuil wrote: > On 07/16/2016 02:38 PM, Laurent Pinchart wrote: >> On Saturday 16 Jul 2016 10:19:29 Hans Verkuil wrote: >>> On 07/15/2016 08:11 PM, Laurent Pinchart wrote: On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: > Descr

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Hans Verkuil
On 07/16/2016 02:38 PM, Laurent Pinchart wrote: > Hi Hans, > > On Saturday 16 Jul 2016 10:19:29 Hans Verkuil wrote: >> On 07/15/2016 08:11 PM, Laurent Pinchart wrote: >>> On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: Describe the HSV formats Signed-off-by: Ricardo Ri

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Laurent Pinchart
Hi Hans, On Saturday 16 Jul 2016 10:19:29 Hans Verkuil wrote: > On 07/15/2016 08:11 PM, Laurent Pinchart wrote: > > On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: > >> Describe the HSV formats > >> > >> Signed-off-by: Ricardo Ribalda Delgado > >> --- > >> > >> Documentation/medi

Re: [PATCH] [media] vb2: map dmabuf for planes on driver queue instead of vidioc_qbuf

2016-07-16 Thread Luis de Bethencourt
On 15/07/16 17:26, Javier Martinez Canillas wrote: > The buffer planes' dma-buf are currently mapped when buffers are queued > from userspace but it's more appropriate to do the mapping when buffers > are queued in the driver since that's when the actual DMA operation are > going to happen. > > Su

[PATCH v2] [media] ad9389b: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue work_queue is involved in EDID (Extended Display Identification Data) handling. It has a single work item(&state->edid_handler) and hence doesn't require ordering. It is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of sys

Re: [PATCH v3 9/9] [media] vivid: Local optimization

2016-07-16 Thread Ricardo Ribalda Delgado
Hi On Sat, Jul 16, 2016 at 12:41 PM, Ricardo Ribalda Delgado wrote: > - cr = clamp(cr, 16 << 4, 240 << 4); > + y = clamp(y >> 4, 16, 235); > + cb = clamp(cb >> 4, 16, 240); > + cr = clamp(cr > 4, 16, 240); Th

Re: [PATCH] [media] ad9389b: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread kbuild test robot
-deprecated-create_singlethread_workqueue/20160716-174501 base: git://linuxtv.org/media_tree.git master config: sparc64-allmodconfig (attached as .config) compiler: sparc64-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp

[PATCH v3 8/9] [media] vivid: Fix YUV555 and YUV565 handling

2016-07-16 Thread Ricardo Ribalda Delgado
precalculate_color() had a optimization that avoided duplicated conversion for YUV formats. This optimization did not take into consideration YUV444, YUV555, YUV565 or limited range quantization. This patch keeps the optimization, but fixes the wrong handling. Signed-off-by: Ricardo Ribalda Delga

[PATCH v3 6/9] [media] vivid: Rename variable

2016-07-16 Thread Ricardo Ribalda Delgado
r_y and g_u now also contain the H and V components on the HSV formats. Rename the variables to reflect this. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 209 +- 1 file changed, 105 insertions(+), 104 deletions(-) diff --git

[PATCH v3 2/9] [media] Documentation: Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
Describe the HSV formats Signed-off-by: Ricardo Ribalda Delgado --- Documentation/media/uapi/v4l/hsv-formats.rst | 19 +++ Documentation/media/uapi/v4l/pixfmt-packed-hsv.rst | 158 + Documentation/media/uapi/v4l/pixfmt.rst| 1 + Documentation/media/uapi/v

[PATCH v3 9/9] [media] vivid: Local optimization

2016-07-16 Thread Ricardo Ribalda Delgado
Avoid duplicated data shifts when possible. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/media/common/v4l2-tpg/v4l2-tpg-core.c b/drivers/media/common/v4l2-tpg/v4

[PATCH v3 3/9] [media] Documentation: Add Ricardo Ribalda

2016-07-16 Thread Ricardo Ribalda Delgado
My initials were on the Changelog, but there was no link to my name. Signed-off-by: Ricardo Ribalda Delgado --- Documentation/media/uapi/v4l/v4l2.rst | 4 1 file changed, 4 insertions(+) diff --git a/Documentation/media/uapi/v4l/v4l2.rst b/Documentation/media/uapi/v4l/v4l2.rst index 6d23b

[PATCH v3 0/9] Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
HSV formats are extremely useful for image segmentation. This set of patches makes v4l2 aware of this kind of formats. Vivid changes have been divided to ease the reviewing process. We are working on patches for Gstreamer and OpenCV that will make use of these formats. We still need to decide if

[PATCH v3 7/9] [media] vivid: Introduce TPG_COLOR_ENC_LUMA

2016-07-16 Thread Ricardo Ribalda Delgado
Simplifies handling of Gray formats. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 26 +++-- drivers/media/platform/vivid/vivid-vid-common.c | 6 +++--- include/media/v4l2-tpg.h| 1 + 3 files changed, 24

[PATCH v3 1/9] [media] videodev2.h Add HSV formats

2016-07-16 Thread Ricardo Ribalda Delgado
These formats store the color information of the image in a geometrical representation. The colors are mapped into a cylinder, where the angle is the HUE, the height is the VALUE and the distance to the center is the SATURATION. This is a very useful format for image segmentation algorithms. Signe

[PATCH v3 4/9] [media] vivid: code refactor for color encoding

2016-07-16 Thread Ricardo Ribalda Delgado
Replace is_yuv with color_enc Which can be used by other color encodings such us HSV. This change should ease the review of the following patches. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 49 +++ drivers/media/platform/vivi

[PATCH v3 5/9] [media] vivid: Add support for HSV formats

2016-07-16 Thread Ricardo Ribalda Delgado
This patch adds support for V4L2_PIX_FMT_HSV24 and V4L2_PIX_FMT_HSV32. Signed-off-by: Ricardo Ribalda Delgado --- drivers/media/common/v4l2-tpg/v4l2-tpg-core.c | 93 +++-- drivers/media/platform/vivid/vivid-vid-common.c | 14 include/media/v4l2-tpg.h

[PATCH] cec: clear fields before transmit and set sequence only if timeout !=0

2016-07-16 Thread Hans Verkuil
Clear all status-related fields before transmitting the message. Also set the sequence counter only for messages with a non-zero timeout (== they wait for a reply) and make sure the sequence counter is never 0. Signed-off-by: Hans Verkuil diff --git a/drivers/staging/media/cec/cec-adap.c b/dr

[PATCH] [media] ad9389b: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue work_queue is involved in EDID (Extended Display Identification Data) handling. It has a single work item(&state->edid_handler) and hence doesn't require ordering. It is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of sys

[PATCH 2/2] [media] cx25821: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "_irq_audio_queues" runs the audio upstream handler. It has a single work item(&dev->_audio_work_entry) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System wor

[PATCH 1/2] [media] cx25821: Drop Freeing of Workqueue

2016-07-16 Thread Bhaktipriya Shridhar
Workqueues shouldn't be freed. destroy_workqueue should be used instead. destroy_workqueue safely destroys a workqueue and ensures that all pending work items are done before destroying the workqueue. Signed-off-by: Bhaktipriya Shridhar --- drivers/media/pci/cx25821/cx25821-audio-upstream.c | 2

[PATCH 0/2] Remove improper workqueue usage

2016-07-16 Thread Bhaktipriya Shridhar
This patch set fixes the improper usage of the workqueue API. This includes dropping the freeing of workqueue and removing the deprecated create_singlethread_workqueue instance. Bhaktipriya Shridhar (2): [media] cx25821: Drop Freeing of Workqueue [media] cx25821: Remove deprecated create_singl

[PATCH] [media] gspca: finepix: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in streaming the camera data. It has a single work item(&dev->work_struct) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System workqu

[PATCH] [media] gspca: jl2005bcd: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in streaming the camera data. It has a single work item(&sd->work_struct) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System workque

[PATCH] [media] gspca: vicam: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in streaming the camera data. It has a single work item(&sd->work_struct) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. System workque

[PATCH] [media] gspca: sonixj: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "work_thread" is involved in updating the JPEG quality of the gspca_dev. It has a single work item(&sd->work) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. Syst

[PATCH] [media] pvrusb2: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
The workqueue "workqueue" is involved in polling the pvrusb2 hardware (pvr2_hdw). It has a single work item(&hdw->workpoll) and hence doesn't require ordering. Also, it is not being used on a memory reclaim path. Hence, the singlethreaded workqueue has been replaced with the use of system_wq. Sys

[PATCH] [media] s5p-mfc: Remove deprecated create_singlethread_workqueue

2016-07-16 Thread Bhaktipriya Shridhar
alloc_workqueue replaces deprecated create_singlethread_workqueue(). The MFC device driver is a v4l2 driver which can encode/decode video raw/elementary streams and has support for all popular video codecs. The driver's watchdog_workqueue has been replaced with system_wq since it queues a single

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Ricardo Ribalda Delgado
Hi Laurent It is actually a very good comment. :) In our case we have implemented the format ourselves in the FPGA and we support both 0-255 and 0-179 Hue ranges. After some weeks of use, only the 0-179 range is used in userpace. The reasons for this is mainly that it is the format used by OpenCV

Re: [PATCH v2 2/6] [media] Documentation: Add HSV format

2016-07-16 Thread Hans Verkuil
On 07/15/2016 08:11 PM, Laurent Pinchart wrote: > Hi Ricardo, > > Thank you for the patch. > > On Friday 15 Jul 2016 18:13:15 Ricardo Ribalda Delgado wrote: >> Describe the HSV formats >> >> Signed-off-by: Ricardo Ribalda Delgado >> --- >> Documentation/media/uapi/v4l/hsv-formats.rst | 1