Re: [PATCH v3] vcodec: mediatek: Add g/s_selection support for V4L2 Encoder

2016-08-04 Thread Tiffany Lin
Hi Hans, On Thu, 2016-08-04 at 15:00 +0200, Hans Verkuil wrote: > > On 08/04/2016 12:08 PM, Tiffany Lin wrote: > > This patch add g/s_selection support for MT8173 v4l2 encoder > > > > Signed-off-by: Tiffany Lin > > --- > > v3: > > - add v4l2_s_selection to check constraint flags > > - remove vi

cron job: media_tree daily build: OK

2016-08-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: Fri Aug 5 04:00:24 CEST 2016 git branch: test git hash: 292eaf50c7df4ae2ae8aaa9e1ce3f1240a353ee8 gcc versi

Re: [PATCH] DocBook: use DOCBOOKS="" to ignore DocBooks instead of IGNORE_DOCBOOKS=1

2016-08-04 Thread Jonathan Corbet
On Thu, 4 Aug 2016 11:48:26 +0300 Jani Nikula wrote: > Instead of a separate ignore flag, use the obvious DOCBOOKS="" to ignore > all DocBook files. Makes sense, applied. Thanks, jon -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord..

[PATCH] media: tw686x: Rework initial hardware configuration

2016-08-04 Thread Ezequiel Garcia
Currently, the hardware is not given a complete initial configuration. In order to fix this, this rather large commit reworks standard, frame format and input configuration. While at it, we introduce proper functions to configure each parameter, and as a result the code is a bit cleaner. Reported

Evromedia USB Full Hybrid Full HD works!!!

2016-08-04 Thread Oleh Kravchenko
Hello Antti, Olli! I tried media-video/kaffeine and now I have all 32 channels which translated in Kyiv, Ukraine! https://owncloud.kaa.org.ua/index.php/s/SvyIiTry3u7zZqj What I should do with `si2157_ops` structure? si2147 and si2157 have 42-870 MHz frequency range, but si2158 have 42-1002 MHz fr

Re: pwc over musb: 100% frame drop (lost) on high resolution stream

2016-08-04 Thread Matwey V. Kornilov
When DMA is not used, I see the same behavior: lots of zero-length packages received. Can It be related to some kind of USB overflow due to long input data processing with disabled IRQ? When HCD_BC is used then part of processing is postponed and this can explain greater throughput due to better l

Re: pwc over musb: 100% frame drop (lost) on high resolution stream

2016-08-04 Thread Matwey V. Kornilov
I've just found that in such cases, when DMA actual length is zero, both cppi41_channel->prog_len and txstate.residue equal 960 at musb_cppi41 line 225: http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/drivers/usb/musb/musb_cppi41.c#n225 2016-08-04 22:08 GMT+03:00 Matwey V. Ko

Re: pwc over musb: 100% frame drop (lost) on high resolution stream

2016-08-04 Thread Matwey V. Kornilov
I've just found that dma->actual_len equals to zero in most cases at musb_host.c line 1946. And this produces zero-length packages. http://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/tree/drivers/usb/musb/musb_host.c#n1946 Any ideas why? 2016-08-04 19:57 GMT+03:00 Matwey V. Kornilov

[PATCH v5 0/2] Add GS1662 driver

2016-08-04 Thread Charles-Antoine Couret
These patches add a driver for GS1662 component, a video serializer which supports CEA and SDI timings. To perform that, we need to determine SDI definition and some flags. The associated documentation will be into another patchset to be Sphinx comaptible. This patchset add: * redefine SM

Re: pwc over musb: 100% frame drop (lost) on high resolution stream

2016-08-04 Thread Matwey V. Kornilov
I've just found that many packages in URBs have zero actual_length (It is a question why). Then the following end of frame criteria leads to `frame underflow' message: if (flen < pdev->vlast_packet_size) { /* Shorter packet... end of frame */

Re: [PATCH] V4L2: Add documentation for SDI timings and related flags

2016-08-04 Thread Charles-Antoine Couret
Le 04/08/2016 à 12:11, Hans Verkuil a écrit : >> --- a/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst >> +++ b/Documentation/media/uapi/v4l/vidioc-g-dv-timings.rst >> @@ -339,6 +339,13 @@ EBUSY >> >> - The timings follow the VESA Generalized Timings Formula standard >> >> +-

Re: [PATCH v2] V4L2: Add documentation for SDI timings and related flags

2016-08-04 Thread Mauro Carvalho Chehab
Em Thu, 4 Aug 2016 18:04:50 +0200 Markus Heiser escreveu: > Am 04.08.2016 um 17:39 schrieb Charles-Antoine Couret > : > > > Signed-off-by: Charles-Antoine Couret > > --- > > Documentation/media/uapi/v4l/vidioc-enuminput.rst | 31 > > +- > > .../media/uapi/v4l/vidioc-g-dv-t

[PATCH v5 2/2] Add GS1662 driver, a video serializer

2016-08-04 Thread Charles-Antoine Couret
You can read datasheet here: http://www.c-dis.net/media/871/GS1662_Datasheet.pdf It's a component which supports HD and SD CEA or SDI formats to SDI output. It's configured through SPI bus. GS1662 driver is implemented as v4l2 subdev. Signed-off-by: Charles-Antoine Couret --- drivers/media/Kco

[PATCH v5 1/2] SDI: add flag for SDI formats and SMPTE 125M definition

2016-08-04 Thread Charles-Antoine Couret
Adding others generic flags, which could be used by many components like GS1662. Signed-off-by: Charles-Antoine Couret --- drivers/media/v4l2-core/v4l2-dv-timings.c | 11 +++ include/uapi/linux/v4l2-dv-timings.h | 12 include/uapi/linux/videodev2.h| 8 +

Re: [PATCH v2] V4L2: Add documentation for SDI timings and related flags

2016-08-04 Thread Markus Heiser
Am 04.08.2016 um 17:39 schrieb Charles-Antoine Couret : > Signed-off-by: Charles-Antoine Couret > --- > Documentation/media/uapi/v4l/vidioc-enuminput.rst | 31 +- > .../media/uapi/v4l/vidioc-g-dv-timings.rst | 16 +++ > 2 files changed, 40 insertions(+), 7 del

[PATCH v2] V4L2: Add documentation for SDI timings and related flags

2016-08-04 Thread Charles-Antoine Couret
Signed-off-by: Charles-Antoine Couret --- Documentation/media/uapi/v4l/vidioc-enuminput.rst | 31 +- .../media/uapi/v4l/vidioc-g-dv-timings.rst | 16 +++ 2 files changed, 40 insertions(+), 7 deletions(-) diff --git a/Documentation/media/uapi/v4l/vidioc-enumin

Re: [PATCHv2] v4l2-common: add s_selection helper function

2016-08-04 Thread Sakari Ailus
On Thu, Aug 04, 2016 at 04:47:46PM +0200, Hans Verkuil wrote: > > > On 08/04/2016 04:38 PM, Sakari Ailus wrote: > > Hi Hans, > > > > On Thu, Aug 04, 2016 at 04:27:27PM +0200, Hans Verkuil wrote: > >> > >> > >> On 08/04/2016 04:17 PM, Sakari Ailus wrote: > >>> On Thu, Aug 04, 2016 at 04:11:55PM +

Re: [PATCHv2] v4l2-common: add s_selection helper function

2016-08-04 Thread Hans Verkuil
On 08/04/2016 04:38 PM, Sakari Ailus wrote: > Hi Hans, > > On Thu, Aug 04, 2016 at 04:27:27PM +0200, Hans Verkuil wrote: >> >> >> On 08/04/2016 04:17 PM, Sakari Ailus wrote: >>> On Thu, Aug 04, 2016 at 04:11:55PM +0200, Hans Verkuil wrote: On 08/04/2016 04:03 PM, Sakari Ailus wrot

Re: [PATCHv2] v4l2-common: add s_selection helper function

2016-08-04 Thread Sakari Ailus
Hi Hans, On Thu, Aug 04, 2016 at 04:27:27PM +0200, Hans Verkuil wrote: > > > On 08/04/2016 04:17 PM, Sakari Ailus wrote: > > On Thu, Aug 04, 2016 at 04:11:55PM +0200, Hans Verkuil wrote: > >> > >> > >> On 08/04/2016 04:03 PM, Sakari Ailus wrote: > >>> Hi Hans, > >>> > >>> On Mon, Aug 01, 2016 at

Re: [PATCHv2] v4l2-common: add s_selection helper function

2016-08-04 Thread Hans Verkuil
On 08/04/2016 04:17 PM, Sakari Ailus wrote: > On Thu, Aug 04, 2016 at 04:11:55PM +0200, Hans Verkuil wrote: >> >> >> On 08/04/2016 04:03 PM, Sakari Ailus wrote: >>> Hi Hans, >>> >>> On Mon, Aug 01, 2016 at 12:33:39PM +0200, Hans Verkuil wrote: Checking the selection constraint flags is often

Re: [PATCHv2] v4l2-common: add s_selection helper function

2016-08-04 Thread Sakari Ailus
On Thu, Aug 04, 2016 at 04:11:55PM +0200, Hans Verkuil wrote: > > > On 08/04/2016 04:03 PM, Sakari Ailus wrote: > > Hi Hans, > > > > On Mon, Aug 01, 2016 at 12:33:39PM +0200, Hans Verkuil wrote: > >> Checking the selection constraint flags is often forgotten by drivers, > >> especially > >> if

Re: [PATCH] [media] si2157: Improve support Si2158-A20 tuner

2016-08-04 Thread Oleh Kravchenko
By the way, may be "C0 00 0C" enable RF receiver? On 04.08.16 15:03, Oleh Kravchenko wrote: > I got this from tracing I2C bus of tuner. > > Windows driver send this to tuner before set frequency, bandwidth, etc.: > wC0 | 15 | C0 00 00 00 00 01 01 01 01 01 01 02 00 00 01 | ... > > ; I c

Re: [PATCHv2] v4l2-common: add s_selection helper function

2016-08-04 Thread Hans Verkuil
On 08/04/2016 04:03 PM, Sakari Ailus wrote: > Hi Hans, > > On Mon, Aug 01, 2016 at 12:33:39PM +0200, Hans Verkuil wrote: >> Checking the selection constraint flags is often forgotten by drivers, >> especially >> if the selection code just clamps the rectangle to the minimum and maximum >> allo

Re: [PATCHv2] v4l2-common: add s_selection helper function

2016-08-04 Thread Sakari Ailus
Hi Hans, On Mon, Aug 01, 2016 at 12:33:39PM +0200, Hans Verkuil wrote: > Checking the selection constraint flags is often forgotten by drivers, > especially > if the selection code just clamps the rectangle to the minimum and maximum > allowed > rectangles. > > This patch adds a simple helper f

I need your urgent response.

2016-08-04 Thread JOJO AKPE
I am Barr, Jojo Akpe, esq, a personal attorney to your late relative. I want to contact you to work with me in securing the transfer of fund(US$10,000,000.00 M) legacy by your late relative. I solicit your consent to enable me produce you as the Next of Kin to my deceased client since both of you

Re: [PATCH] v4l: ioctl: Clear the v4l2_pix_format_mplane reserved field

2016-08-04 Thread Laurent Pinchart
Hi Hans, On Friday 01 Jul 2016 12:52:31 Hans Verkuil wrote: > On 06/28/2016 03:20 PM, Laurent Pinchart wrote: > > The S_FMT and TRY_FMT handlers in multiplane mode attempt at clearing > > the reserved fields of the v4l2_format structure after the pix_mp > > member. However, the reserved fields are

Re: [PATCH] v4l: platform: Add Renesas R-Car FDP1 Driver

2016-08-04 Thread Hans Verkuil
On 08/04/2016 03:01 PM, Laurent Pinchart wrote: > Hi Hans, > > Thank you for the review. > > I'll temporarily take over work on the FDP driver until the end of this month > as Kieran is away. > > On Tuesday 19 Jul 2016 14:28:28 Hans Verkuil wrote: >> Hi Kieran, >> >> Hmm, I don't think I ever

Re: [PATCH] v4l: platform: Add Renesas R-Car FDP1 Driver

2016-08-04 Thread Laurent Pinchart
Hi Hans, Thank you for the review. I'll temporarily take over work on the FDP driver until the end of this month as Kieran is away. On Tuesday 19 Jul 2016 14:28:28 Hans Verkuil wrote: > Hi Kieran, > > Hmm, I don't think I ever reviewed this one. > > So here is my quick review: > > General no

Re: [PATCH v3] vcodec: mediatek: Add g/s_selection support for V4L2 Encoder

2016-08-04 Thread Hans Verkuil
On 08/04/2016 12:08 PM, Tiffany Lin wrote: > This patch add g/s_selection support for MT8173 v4l2 encoder > > Signed-off-by: Tiffany Lin > --- > v3: > - add v4l2_s_selection to check constraint flags > - remove visible_height modification in s_fmt_out Is this selection-related, or is this an u

Re: [PATCH v3] vcodec: mediatek: Add g/s_selection support for V4L2 Encoder

2016-08-04 Thread kbuild test robot
Hi Tiffany, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on next-20160804] [cannot apply to v4.7] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Tiffany-Lin/vcodec

Re: [PATCH v2] media-ctl: Initialize ioctl arguments to 0

2016-08-04 Thread Sakari Ailus
On Thu, Aug 04, 2016 at 02:38:06PM +0300, Laurent Pinchart wrote: > This ensures that the reserved fields are properly set to 0 as required > by the API. > > Signed-off-by: Laurent Pinchart Thanks!!! Acked-by: Sakari Ailus -- Sakari Ailus e-mail: sakari.ai...@iki.fi XMPP: sai...@retiisi.

Re: [PATCH] [media] si2157: Improve support Si2158-A20 tuner

2016-08-04 Thread Oleh Kravchenko
I got this from tracing I2C bus of tuner. Windows driver send this to tuner before set frequency, bandwidth, etc.: wC0 | 15 | C0 00 00 00 00 01 01 01 01 01 01 02 00 00 01 | ... ; I cut firmware from here, by the way Windows driver use firmware v2.1.8 wC0

[PATCH v2] media-ctl: Initialize ioctl arguments to 0

2016-08-04 Thread Laurent Pinchart
This ensures that the reserved fields are properly set to 0 as required by the API. Signed-off-by: Laurent Pinchart --- utils/media-ctl/libmediactl.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) Changes since v1: - Use struct initializers instead of memset diff --git a/utils/me

Re: [PATCH] v4l: Extend FCP compatible list to support the FDP

2016-08-04 Thread Laurent Pinchart
Hi Kieran, Thank you for the patch. On Thursday 09 Jun 2016 18:06:43 Kieran Bingham wrote: > The FCP must be powered up for the FDP1 to function, even when the FDP1 > does not make use of the FCNL features. Extend the compatible list > to allow us to use the power domain and runtime-pm support. >

[GIT PULL for v4.8-rc1] removal of media DocBook and some fixups

2016-08-04 Thread Mauro Carvalho Chehab
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/v4.8-5 For: - Removal of the media DocBook; - videobuf2: Fix an allocation regression; - A few fixes related to the CEC drivers. You may expect a trivial conflict at Documentation/Do

Re: [PATCH] [media] si2157: Improve support Si2158-A20 tuner

2016-08-04 Thread Oleh Kravchenko
Hi Olli, Looks like I miss something, what parameter do you mean? On 04.08.16 13:08, Olli Salonen wrote: > Hi Oleh, > > Do you have any idea of what this parameter change does? I have a gut > feeling that this is an option that's not Si2158-specific, but a > parameter that's valid for the other t

Re: [PATCH] DocBook: use DOCBOOKS="" to ignore DocBooks instead of IGNORE_DOCBOOKS=1

2016-08-04 Thread Mauro Carvalho Chehab
Em Thu, 4 Aug 2016 11:48:26 +0300 Jani Nikula escreveu: > Instead of a separate ignore flag, use the obvious DOCBOOKS="" to ignore > all DocBook files. This is also in line with the Sphinx build being > ignored if a non-empty DOCBOOKS make variable is specified on the make > command line. > > T

Re: [PATCH] V4L2: Add documentation for SDI timings and related flags

2016-08-04 Thread Hans Verkuil
On 07/21/2016 05:04 PM, Charles-Antoine Couret wrote: > Signed-off-by: Charles-Antoine Couret > --- > Documentation/media/uapi/v4l/vidioc-enuminput.rst | 31 > +- > .../media/uapi/v4l/vidioc-g-dv-timings.rst | 15 +++ > 2 files changed, 39 insertions(+), 7

[PATCH v3] vcodec: mediatek: Add g/s_selection support for V4L2 Encoder

2016-08-04 Thread Tiffany Lin
This patch add g/s_selection support for MT8173 v4l2 encoder Signed-off-by: Tiffany Lin --- v3: - add v4l2_s_selection to check constraint flags - remove visible_height modification in s_fmt_out --- drivers/media/platform/mtk-vcodec/mtk_vcodec_enc.c | 81 +--- 1 file changed, 7

[GIT PULL FOR v4.8] cec: fix typo and locking improvements

2016-08-04 Thread Hans Verkuil
These three patches improve low-level locking which was a bit messy, and they fix a silly typo in cec-funcs.h. I'd like to have this fixed in 4.8. Regards, Hans The following changes since commit 292eaf50c7df4ae2ae8aaa9e1ce3f1240a353ee8: [media] cec: fix off-by-one memset (2016-07-28

[PATCH -next] [media] adv7511: fix error return code in adv7511_probe()

2016-08-04 Thread Wei Yongjun
Fix to return error code -ENOMEM from the i2c client register error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/media/i2c/adv7511.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/i2c/adv7511.c b/drivers/media/i2c/adv75

[PATCH 1/7] videodev2.h: fix sYCC/AdobeYCC default quantization range

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil The default quantization range of the Y'CbCr encodings of sRGB and AdobeRGB are full range instead of limited range according to the corresponding standards. Fix this in the V4L2_MAP_QUANTIZATION_DEFAULT macro. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/v4l/pi

[PATCH 5/7] pixfmt.rst: drop V4L2_YCBCR_ENC_SYCC from the documentation

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil The V4L2_YCBCR_ENC_SYCC encoding is identical to V4L2_YCBCR_ENC_601. Remove V4L2_YCBCR_ENC_SYCC from the documentation since it should not be used. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/v4l/pixfmt-006.rst | 10 ++ Documentation/media/uapi/v4l/pixfm

[PATCH 3/7] v4l2-tpg-core: drop SYCC, use higher precision 601 conversion matrix

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil The SYCC Y'CbCr encoding is identical to the 601 encoding. Since the SYCC define is about to be removed for use in the kernel we need to drop it in the TPG code as well. This patch also adds a 4th decimal to the 601 conversion matrix. That was specified by the sYCC spec and it

[PATCH 7/7] pixfmt-007.rst: fix copy-and-paste error in SMPTE-240M doc

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil Instead of referring to Y', Cb and Cr, it referred to Yc', Cbc and Crc, which were accidentally copied from the BT.2020 constant luminance text. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/v4l/pixfmt-007.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 2/7] vivid: don't mention the obsolete sYCC Y'CbCr encoding

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil This encoding is identical to the 601 encoding. The old duplicate SYCC define is about to be removed for use in the kernel, so remove its use in vivid first. Signed-off-by: Hans Verkuil --- drivers/media/platform/vivid/vivid-ctrls.c | 3 ++- 1 file changed, 2 insertions(+),

[PATCH 4/7] videodev2.h: put V4L2_YCBCR_ENC_SYCC under #ifndef __KERNEL__

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil This define is a duplicate of V4L2_YCBCR_ENC_601. So mark it as 'should not be used' and put it under #ifndef __KERNEL__ to prevent drivers from referring to it. Signed-off-by: Hans Verkuil --- include/uapi/linux/videodev2.h | 14 +- 1 file changed, 9 insertions(

[PATCH 6/7] pixfmt-007.rst: fix a messed up note in the DCI-P3 doc

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil The text of the note included text that shouldn't have been part of the note. Move that out of the note into the proper place. Signed-off-by: Hans Verkuil --- Documentation/media/uapi/v4l/pixfmt-007.rst | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff -

[PATCH 0/7] Colorspace fixes

2016-08-04 Thread Hans Verkuil
From: Hans Verkuil This patch series fixes a number of bugs/documentation mistakes w.r.t. colorspaces. The main one is that the V4L2_YCBCR_ENC_SYCC was mistakenly added. It turns out it is identical to the BT.601 Y'CbCr encoding. So the use of the old SYCC define has been removed from the kernel

[PATCH] DocBook: use DOCBOOKS="" to ignore DocBooks instead of IGNORE_DOCBOOKS=1

2016-08-04 Thread Jani Nikula
Instead of a separate ignore flag, use the obvious DOCBOOKS="" to ignore all DocBook files. This is also in line with the Sphinx build being ignored if a non-empty DOCBOOKS make variable is specified on the make command line. This replaces the IGNORE_DOCBOOKS introduced in commit 547218864afb2745

Re: [PATCHv2] serio: add hangup support

2016-08-04 Thread Hans Verkuil
On 08/03/2016 05:26 PM, Dmitry Torokhov wrote: > On Wed, Aug 03, 2016 at 01:00:44PM +0200, Hans Verkuil wrote: >> The Pulse-Eight USB CEC adapter is a usb device that shows up as a ttyACM0 >> device. >> It requires that you run inputattach in order to communicate with it via >> serio. >> >> Thi

[PATCH] [media] si2157: Improve support Si2158-A20 tuner

2016-08-04 Thread Oleh Kravchenko
Signed-off-by: Oleh Kravchenko --- drivers/media/tuners/si2157.c | 34 +++--- drivers/media/tuners/si2157_priv.h | 1 + 2 files changed, 28 insertions(+), 7 deletions(-) diff --git a/drivers/media/tuners/si2157.c b/drivers/media/tuners/si2157.c index 57b2508..d7