Re: [GIT PULL FOR v4.8] Various fixes/improvements

2016-07-04 Thread Florian Echtler
Hello Hans, On 01.07.2016 16:45, Hans Verkuil wrote: > Florian Echtler (3): > sur40: properly report a single frame rate of 60 FPS > sur40: lower poll interval to fix occasional FPS drops to ~56 FPS > sur40: fix occasional oopses on device close Thanks for merging this, AFAICS

cron job: media_tree daily build: ERRORS

2016-07-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: Tue Jul 5 04:00:25 CEST 2016 git branch: test git hash: d81295d1bed850335f9f4ccb6b1aa4f6a123d4f0 gcc

ERROR: "bad_dma_ops" [sound/core/snd-pcm.ko] undefined!

2016-07-04 Thread kbuild test robot
Hi, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: a99cde438de0c4c0cecc1d1af1a55a75b10bfdef commit: 420520766a796d3607639ba1e4fb1aadeadd [media] media: Kconfig: add dependency of HAS_DMA date: 5 months ago

[PATCH 27/41] Documentation: userp.rst: Add ioctl cross references

2016-07-04 Thread Mauro Carvalho Chehab
There are lots of ioctls mentioned there that aren't cross-referenced. Convert the const to cross references. That makes it visually better, and improves navigation along the document. While here, remove bad whitespaces. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 10/41] Documentation: FE_READ_SIGNAL_STRENGTH: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 01/41] Documentation: linux_tv: Fix some occurences of :sub:

2016-07-04 Thread Mauro Carvalho Chehab
The right way to use it seems to do suscript is to use this pattern: "\ :sub:" Make sure all places of the media document will fit, by using this script: $n=0; while (<>) { $n++; $t = $_; @matches = $t =~ m/(..\:sub\:)/g; foreach my $m (@matches) {

[PATCH 08/41] Documentation: FE_READ_UNCORRECTED_BLOCKS: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 15/41] Documentation: linux_tv: fix remaining lack of escapes

2016-07-04 Thread Mauro Carvalho Chehab
Add escape before asterisk to fix those warnings: Documentation/linux_tv/media/dvb/FE_GET_EVENT.rst:47: WARNING: Inline emphasis start-string without end-string. Documentation/linux_tv/media/v4l/media-ioc-enum-links.rst:78: WARNING: Inline emphasis start-string without end-string.

[PATCH 03/41] Documentation: dmx_fcalls.rst: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. cpp:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho

[PATCH 14/41] Documentation: FE_DISHNETWORK_SEND_LEGACY_CMD: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 02/41] Documentation: pixfmt-nv16.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. It also added an extra line on one of the texts, breaking the original paragraph into two ones. Remove them. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 12/41] Documentation: FE_GET_FRONTEND.rst: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 30/41] Documentation: dmabuf.rst: Add ioctl cross references

2016-07-04 Thread Mauro Carvalho Chehab
There are lots of ioctls mentioned there that aren't cross-referenced. Convert the const to cross references. That makes it visually better, and improves navigation along the document. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 11/41] Documentation: FE_READ_BER.rst: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 37/41] Documentation: dev-raw-vbi.rst fix conversion issues

2016-07-04 Thread Mauro Carvalho Chehab
There are several things that didn't convert well. Fix them, in order to improve the layout of the formatted document. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/dev-raw-vbi.rst | 29 1 file changed, 10

[PATCH 05/41] Documentation: ca_function_calls: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. cpp:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho

[PATCH 39/41] Documentation: dev-subdev.rst: fix some format issues

2016-07-04 Thread Mauro Carvalho Chehab
The conversion from DocBook made somethings look ugly. Improve them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/dev-subdev.rst | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git

[PATCH 31/41] Documentation: buffer.rst: numerate tables

2016-07-04 Thread Mauro Carvalho Chehab
Sphinx actually doesn't numerate tables. So, let's add a subtitle before each table. That makes them numerated. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/buffer.rst | 38 +++-- 1 file changed, 31 insertions(+), 7

[PATCH 13/41] Documentation: FE_GET_EVENT.rst: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 26/41] Documentation: userp.rst: re-add the missing captions

2016-07-04 Thread Mauro Carvalho Chehab
The conversion from DocBook removed them. Re-add. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/userp.rst | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/linux_tv/media/v4l/userp.rst

[PATCH 16/41] Documentation: linux_tv: avoid using c:func::

2016-07-04 Thread Mauro Carvalho Chehab
Using auto-generated links is dangerous, as there are multiple definitions for syscalls (at least one on each book part). So, reference them by their explicit reference. I used this small script to help writing this patch: for i in $(git grep -l "c:func:"); do perl -ne

[PATCH 18/41] Documentation: linux_tv: move MC stuff to a separate dir

2016-07-04 Thread Mauro Carvalho Chehab
When we wrote the media controller's section, we re-used the V4L, just because we were lazy to create a brand new DocBook. Yet, it is a little ackward to have it mixed with V4L. So, move it to its own directory, in order to have it better organized. No functional changes. Signed-off-by: Mauro

[PATCH 41/41] Documentation: libv4l-introduction.rst: improve format

2016-07-04 Thread Mauro Carvalho Chehab
Fix some cross-references and improve the layout of this page. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/libv4l-introduction.rst | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH 17/41] Documentation: linux_tv: move RC stuff to a separate dir

2016-07-04 Thread Mauro Carvalho Chehab
When we wrote the remote controller's section, we re-used the V4L, just because we were lazy to create a brand new DocBook. Yet, it is a little ackward to have it mixed with V4L. So, move it to its own directory, in order to have it better organized. No functional changes. Signed-off-by: Mauro

[PATCH 25/41] Documentation: mmap.rst: Add ioctl cross references

2016-07-04 Thread Mauro Carvalho Chehab
There are lots of ioctls mentioned there that aren't cross-referenced. Convert the const to cross references. That makes it visually better, and improves navigation along the document. While here, remove bad whitespaces. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 35/41] Documentation: dev-codec.rst: Fix a reference for _STREAMON

2016-07-04 Thread Mauro Carvalho Chehab
The referenced ioctl there is only VIDIOC_STREAMON, so we should override the name, to avoid it to also show _STREAMOFF. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/dev-codec.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

[PATCH 20/41] Documentation: linux_tv: Fix a warning at lirc_dev_intro.rst

2016-07-04 Thread Mauro Carvalho Chehab
Documentation/linux_tv/media/rc/lirc_dev_intro.rst:17: WARNING: Inline substitution_reference start-string without end-string. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/rc/lirc_dev_intro.rst | 10 +- 1 file changed, 5 insertions(+),

[PATCH 21/41] Documentation: pixfmt-nv16m.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. It also added an extra line on one of the texts, breaking the original paragraph into two ones. Remove them. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 36/41] Documentation: linux_tv: Use references for read()/write()

2016-07-04 Thread Mauro Carvalho Chehab
Use cross-references for read()/write() on a few places where they weren't used. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/dev-capture.rst | 4 ++-- Documentation/linux_tv/media/v4l/dev-raw-vbi.rst | 4 ++--

[PATCH 23/41] Documentation: pixfmt-nv16m.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. Remove it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-m420.rst | 7 --- 1 file changed, 7 deletions(-) diff --git

[PATCH 33/41] Documentation: dev-overlay.rst: Fix conversion issues

2016-07-04 Thread Mauro Carvalho Chehab
There were several conversion issues on this file, causing it to be badly formatted. Fix them, in order to match the design used on DocBook. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/dev-overlay.rst | 45 +++- 1 file

[PATCH 22/41] Documentation: v4l/pixfmt: re-join a broken paragraph

2016-07-04 Thread Mauro Carvalho Chehab
Originally, at the DocBook, the "Byte Order" were a single paragraph with the string that follows it. The conversion broke it, and, sometimes, it added an extra dot. Fix them altogheter at pixfmt-*.rst. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 34/41] Documentation: dev-osd.rst: Fix some issues due to conversion

2016-07-04 Thread Mauro Carvalho Chehab
The conversion to ReST broke a minor things. Fix them. While here, also make EBUSY constant, just like on other places. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/dev-osd.rst | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)

[PATCH 24/41] Documentation: mmap.rst: re-add the missing captions

2016-07-04 Thread Mauro Carvalho Chehab
The conversion from DocBook removed them. Re-add. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/mmap.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/linux_tv/media/v4l/mmap.rst

[PATCH 19/41] Documentation: linux_tv: promote generic documents to the parent dir

2016-07-04 Thread Mauro Carvalho Chehab
The generic error codes and licensing sections are general to the entire media book. They should not be inside the v4l dir. So, promote them to the parent directory. No functional changes. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/index.rst

[PATCH 07/41] Documentation: FE_SET_FRONTEND.rst: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 06/41] Documentation: audio_function_calls: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. cpp:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho

[PATCH 29/41] Documentation: dmabuf.rst: re-add the missing captions

2016-07-04 Thread Mauro Carvalho Chehab
The conversion from DocBook removed them. Re-add. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/dmabuf.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Documentation/linux_tv/media/v4l/dmabuf.rst

[PATCH 09/41] Documentation: FE_READ_SNR: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. c:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho Chehab

[PATCH 04/41] Documentation: video_function_calls: improve man-like format

2016-07-04 Thread Mauro Carvalho Chehab
Parsing this file were causing lots of warnings with sphinx, due to the c function prototypes. Fix that by prepending them with .. cpp:function:: While here, use the same way we document man-like pages, at the V4L side of the book and add escapes to asterisks. Signed-off-by: Mauro Carvalho

[PATCH 38/41] Documentation: dev-sliced-vbi.rst: convert table captions into headers

2016-07-04 Thread Mauro Carvalho Chehab
Sphinx doesn't format nice table captions, nor auto-numberate them. So, convert tables into chapters. Signed-off-by: Mauro Carvalho Chehab --- .../linux_tv/media/v4l/dev-sliced-vbi.rst | 45 +- 1 file changed, 36 insertions(+), 9

[PATCH 40/41] Documentation: subdev-formats.rst: Improve figure caption

2016-07-04 Thread Mauro Carvalho Chehab
Add a numbering for the figure. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/subdev-formats.rst | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/Documentation/linux_tv/media/v4l/subdev-formats.rst

[PATCH 32/41] Documentation: buffer.rst: numerate tables and figures

2016-07-04 Thread Mauro Carvalho Chehab
Sphinx actually doesn't numerate tables nor figures. So, let's add a subtitle before each table. That makes them "numerated". While here, fix the git binary that got corrupted. Let's hope this will work, as the reason why we had to encode them were to prevent some issues on commiting those gif

Re: [GIT PULL FOR v4.8] Various dvb/rc fixes/improvements

2016-07-04 Thread Hans Verkuil
On 07/04/2016 10:19 PM, Sean Young wrote: > On Mon, Jul 04, 2016 at 01:54:56PM +0200, Hans Verkuil wrote: >> Mauro, >> >> As requested, I'm helping out with reducing the backlog. >> >> Regards, >> >> Hans >> >> The following changes since commit ab46f6d24bf57ddac0f5abe2f546a78af57b476c: >> >>

[GIT PULL FOR v4.8] Various dvb/rc fixes/improvements

2016-07-04 Thread Hans Verkuil
Mauro, As requested, I'm helping out with reducing the backlog. This is the second pull request. The only difference with the superseded first pull request is that I've dropped this patch https://patchwork.linuxtv.org/patch/34339/ since there are some concerns about it. Regards, Hans

Re: IR remote stopped working in kernels 4.5 and 4.6

2016-07-04 Thread James Bottomley
On Mon, 2016-07-04 at 23:08 +0200, Heiner Kallweit wrote: > Am 04.07.2016 um 22:36 schrieb James Bottomley: > > This looks to be a problem with the rc subsystem. The IR > > controller in question is part of a cx8800 atsc card. In the 4.4 > > kernel, where it works, this is what ir-keytable

Re: IR remote stopped working in kernels 4.5 and 4.6

2016-07-04 Thread Heiner Kallweit
Am 04.07.2016 um 22:36 schrieb James Bottomley: > This looks to be a problem with the rc subsystem. The IR controller in > question is part of a cx8800 atsc card. In the 4.4 kernel, where it > works, this is what ir-keytable says: > > Found /sys/class/rc/rc0/ (/dev/input/event12) with: >

Re: [PATCH] media: rc: nuvoton: decrease size of raw event fifo

2016-07-04 Thread Sean Young
On Mon, Jul 04, 2016 at 10:51:50PM +0200, Heiner Kallweit wrote: > Am 04.07.2016 um 22:13 schrieb Sean Young: > > On Wed, May 18, 2016 at 10:29:41PM +0200, Heiner Kallweit wrote: > >> This chip has a 32 byte HW FIFO only. Therefore the default fifo size > >> of 512 raw events is not needed and can

Re: [PATCH] media: rc: nuvoton: decrease size of raw event fifo

2016-07-04 Thread Heiner Kallweit
Am 04.07.2016 um 22:13 schrieb Sean Young: > On Wed, May 18, 2016 at 10:29:41PM +0200, Heiner Kallweit wrote: >> This chip has a 32 byte HW FIFO only. Therefore the default fifo size >> of 512 raw events is not needed and can be significantly decreased. >> >> Signed-off-by: Heiner Kallweit

IR remote stopped working in kernels 4.5 and 4.6

2016-07-04 Thread James Bottomley
This looks to be a problem with the rc subsystem. The IR controller in question is part of a cx8800 atsc card. In the 4.4 kernel, where it works, this is what ir-keytable says: Found /sys/class/rc/rc0/ (/dev/input/event12) with: Driver cx88xx, table rc-hauppauge Supported

Re: [GIT PULL FOR v4.8] Various dvb/rc fixes/improvements

2016-07-04 Thread Sean Young
On Mon, Jul 04, 2016 at 01:54:56PM +0200, Hans Verkuil wrote: > Mauro, > > As requested, I'm helping out with reducing the backlog. > > Regards, > > Hans > > The following changes since commit ab46f6d24bf57ddac0f5abe2f546a78af57b476c: > > [media] videodev2.h: Fix V4L2_PIX_FMT_YUV411P

Re: [PATCH] media: rc: nuvoton: decrease size of raw event fifo

2016-07-04 Thread Sean Young
On Wed, May 18, 2016 at 10:29:41PM +0200, Heiner Kallweit wrote: > This chip has a 32 byte HW FIFO only. Therefore the default fifo size > of 512 raw events is not needed and can be significantly decreased. > > Signed-off-by: Heiner Kallweit The 32 byte hardware queue is

Re: si2157: new revision?

2016-07-04 Thread Antti Palosaari
I am not interested at all about analog support. Analog transmissions are ran down many, many years ago here. regards Antti On 07/04/2016 09:56 PM, Oleh Kravchenko wrote: Thank you for your reply! What about Analog TV support? Do you plan to implement it? On 04.07.16 21:47, Antti Palosaari

Re: si2157: new revision?

2016-07-04 Thread Oleh Kravchenko
Thank you for your reply! What about Analog TV support? Do you plan to implement it? On 04.07.16 21:47, Antti Palosaari wrote: > Hello > On 07/04/2016 09:38 PM, Oleh Kravchenko wrote: >> Hello Antti! >> >> I started reverse-engineering of my new TV tuner "Evromedia USB Full >> Hybrid Full HD"

Re: si2157: new revision?

2016-07-04 Thread Antti Palosaari
Hello On 07/04/2016 09:38 PM, Oleh Kravchenko wrote: Hello Antti! I started reverse-engineering of my new TV tuner "Evromedia USB Full Hybrid Full HD" and discovered that start sequence is different from si2157.c: i2c_read_C1 1 \xFE i2c_write_C0 15

Re: [PATCH RFC v2 0/2] pxa_camera transition to v4l2 standalone device

2016-07-04 Thread Robert Jarzmik
Hans Verkuil writes: > Hi Robert, > > On 04/02/2016 04:26 PM, Robert Jarzmik wrote: >> Hi Hans and Guennadi, >> >> This is the second opus of this RFC. The goal is still to see how close our >> ports are to see if there are things we could either reuse of change. >> >> From

Re: [PATCH 2/2] tw686x: Support VIDIOC_{S,G}_PARM ioctls

2016-07-04 Thread Ezequiel Garcia
On 01 Jul 12:46 PM, Hans Verkuil wrote: > On 06/29/2016 04:17 AM, Ezequiel Garcia wrote: > > Now that the frame rate can be properly set, this commit adds support > > for S_PARM and G_PARM. > > As mentioned in our irc discussion you also need to add > enum_framesize/intervals. > Otherwise

Re: [PATCH] tw686x: make const structs static

2016-07-04 Thread Ezequiel Garcia
On 01 Jul 04:39 PM, Hans Verkuil wrote: > Fix sparse warnings: > > tw686x-video.c:148:29: warning: symbol 'memcpy_dma_ops' was not declared. > Should it be static? > tw686x-video.c:195:29: warning: symbol 'contig_dma_ops' was not declared. > Should it be static? > tw686x-video.c:361:29:

[PATCH 5/7] adv7604/adv7842: drop unused op_656_range and alt_data_sat fields.

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil These platform_data fields are no longer needed, drop them. Signed-off-by: Hans Verkuil --- include/media/i2c/adv7604.h | 2 -- include/media/i2c/adv7842.h | 2 -- 2 files changed, 4 deletions(-) diff --git

[PATCH 7/7] v4l2-ioctl: zero the v4l2_bt_timings reserved field

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil This field should be zeroed. Both for s_dv_timings and for enum_dv_timings. In fact, for the latter anything after the pad field can be zeroed. Signed-off-by: Hans Verkuil --- drivers/media/v4l2-core/v4l2-ioctl.c | 4 ++-- 1

[PATCH 3/7] adv7604/adv7842: fix quantization range handling

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil Fix a number of bugs that appeared when support for mediabus formats was added: - Support for V4L2_DV_RGB_RANGE_FULL/LIMITED should only be enabled for HDMI RGB formats, not for YCbCr formats. Since, as the name says, this setting is for RGB only.

[PATCH 2/7] adv7511: fix quantization range handling

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil Commit 1fb69bfd29e4b2e5e93922105326dd6cbd5ef6eb (adv7511: improve colorspace handling) introduced a number of bugs, specifically with regards to YCbCr output and quantization range handling: - if the output is not RGB, then disable the full-to-limited

[PATCH 4/7] ezkit/cobalt: drop unused op_656_range setting

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil The adv7604/adv7842 drivers now handle that register setting themselves and need no input from platform data anymore. This was a left-over from the time that the pixelport output format was decided by the platform data. Signed-off-by: Hans Verkuil

[PATCH 6/7] DocBook media: document the v4l2_bt_timings reserved field

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil This field was never documented, and neither was it mentioned that it should be zeroed by the application. Signed-off-by: Hans Verkuil --- Documentation/DocBook/media/v4l/vidioc-g-dv-timings.xml | 7 +++ 1 file changed, 7

[PATCH 1/7] adv7511: drop adv7511_set_IT_content_AVI_InfoFrame

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil The IT Content bit has nothing to do with CE vs IT timings. Delete this code. This will also fix a bug where this could override the 'content type' control, which is actually the correct place to set/clear the ITC bit. Signed-off-by: Hans Verkuil

[PATCH 0/7] adv quantization fixes, doc and zero reserved field

2016-07-04 Thread Hans Verkuil
From: Hans Verkuil This patch series fixes a number of quantization range problems in the adv drivers. These problems surfaced because of work done to support different mediabus formats. In the past this was hardcoded via the platform data, but today this is set by the

[GIT PULL FOR v4.8] Update MAINTAINERS and two gspca fixes

2016-07-04 Thread Hans Verkuil
The following changes since commit 80aa26593e3eb48f16c4222aa27ff40806f57c45: [media] media: rcar-vin: add DV timings support (2016-06-28 09:06:40 -0300) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v4.8g for you to fetch changes up to

Re: [PATCH] MAINTAINERS: change maintainer for gscpa/pwc/radio-shark

2016-07-04 Thread Hans de Goede
Hi, On 04-07-16 15:20, Hans Verkuil wrote: Hans de Goede has no more time to work on those, so I'll take over. For gspca/pwc I'll do 'Odd Fixes', for radio-shark I'll be a full maintainer. Signed-off-by: Hans Verkuil Thanks! Acked-by: Hans de Goede

Re: Stepping down as gspca and pwc maintainer

2016-07-04 Thread Hans Verkuil
On 07/04/2016 03:00 PM, Hans de Goede wrote: > Hi, > > On 04-07-16 09:36, Hans Verkuil wrote: >> Hi Hans, >> >> On 07/03/2016 11:31 PM, Hans de Goede wrote: >>> Hi All, >>> >>> Admittedly I've not been all that active as gspca and pwc >>> maintainer lately, but officially I'm still the maintainer

[PATCH alternative] [media] gspca: avoid unused variable warnings

2016-07-04 Thread Arnd Bergmann
When CONFIG_INPUT is disabled, multiple gspca backend drivers print compile-time warnings about unused variables: media/usb/gspca/cpia1.c: In function 'sd_stopN': media/usb/gspca/cpia1.c:1627:13: error: unused variable 'sd' [-Werror=unused-variable] media/usb/gspca/konica.c: In function

[PATCH] MAINTAINERS: change maintainer for gscpa/pwc/radio-shark

2016-07-04 Thread Hans Verkuil
Hans de Goede has no more time to work on those, so I'll take over. For gspca/pwc I'll do 'Odd Fixes', for radio-shark I'll be a full maintainer. Signed-off-by: Hans Verkuil --- MAINTAINERS | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff

[PATCH] [media] gspca: avoid unused variable warnings

2016-07-04 Thread Arnd Bergmann
When CONFIG_INPUT is disabled, multiple gspca backend drivers print compile-time warnings about unused variables: media/usb/gspca/cpia1.c: In function 'sd_stopN': media/usb/gspca/cpia1.c:1627:13: error: unused variable 'sd' [-Werror=unused-variable] media/usb/gspca/konica.c: In function

Re: Stepping down as gspca and pwc maintainer

2016-07-04 Thread Hans de Goede
Hi, On 04-07-16 09:36, Hans Verkuil wrote: Hi Hans, On 07/03/2016 11:31 PM, Hans de Goede wrote: Hi All, Admittedly I've not been all that active as gspca and pwc maintainer lately, but officially I'm still the maintainer for both. Between my $dayjob, other foss projects and last but not

[PATCH 2/2] dvb_frontend: eliminate blocking wait in dvb_unregister_frontend()

2016-07-04 Thread Max Kellermann
The wait_event() call in dvb_unregister_frontend() waits synchronously for other tasks to free a file descriptor, but it does that while holding several mutexes. That alone is a bad idea, but if one user process happens to keep a (defunct) file descriptor open indefinitely, the kernel will

[PATCH 1/2] drivers/media/dvb-core/en50221: use kref to manage struct dvb_ca_private

2016-07-04 Thread Max Kellermann
Don't free the object until the file handle has been closed. Fixes use-after-free bug which occurs when I disconnect my DVB-S received while VDR is running. This is a crash dump of such a use-after-free: general protection fault: [#1] SMP CPU: 0 PID: 2541 Comm: CI adapter on d Not

[GIT PULL FOR v4.8] Various dvb/rc fixes/improvements

2016-07-04 Thread Hans Verkuil
Mauro, As requested, I'm helping out with reducing the backlog. Regards, Hans The following changes since commit ab46f6d24bf57ddac0f5abe2f546a78af57b476c: [media] videodev2.h: Fix V4L2_PIX_FMT_YUV411P description (2016-06-28 11:54:52 -0300) are available in the git repository at:

[GIT PULL FOR v4.8] Various v4l2/gspca/pwc fixes/improvements

2016-07-04 Thread Hans Verkuil
The following changes since commit ab46f6d24bf57ddac0f5abe2f546a78af57b476c: [media] videodev2.h: Fix V4L2_PIX_FMT_YUV411P description (2016-06-28 11:54:52 -0300) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v4.8e for you to fetch changes up to

[PATCH 08/51] Documentation: v4l2.rst: numerate the V4L2 chapters

2016-07-04 Thread Mauro Carvalho Chehab
The Video4Linux documentation is big. We want to numerate the items, as it makes easier to reference an item while discussing the document on meetings. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/v4l2.rst | 2 ++ 1 file changed, 2

[PATCH 02/51] Documentation: some fixups at linux_tv/index.rst

2016-07-04 Thread Mauro Carvalho Chehab
Fix some issues from the conversion and improve the documentation a little bit, by adding two relevent keywords (SDR and DTMB). Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/index.rst | 21 +++-- 1 file changed, 7 insertions(+), 14

[PATCH 03/51] Documentation: v4l2.rst: Fix authors and revisions lists

2016-07-04 Thread Mauro Carvalho Chehab
The conversion of the authors and revision history lists didn't end too well. Manually fix them. The revision history still needs some care in the future. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/v4l2.rst | 70

[PATCH 01/51] Documentation: linuxt_tv: update the documentation year

2016-07-04 Thread Mauro Carvalho Chehab
The RST version was produced in 2016. So, update it where it was still pointing to the wrong year. While here, added the missing (copyright) symbols. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/conf.py | 2 +-

[PATCH 31/51] Documentation: pixfmt-007.rst: Fix formula parsing

2016-07-04 Thread Mauro Carvalho Chehab
There are lots of warnings there: /devel/v4l/patchwork/Documentation/linux_tv/media/v4l/pixfmt-007.rst:74: ERROR: Unexpected indentation. /devel/v4l/patchwork/Documentation/linux_tv/media/v4l/pixfmt-007.rst:89: ERROR: Unexpected indentation.

[PATCH 18/51] Documentation: control.rst: read the example captions

2016-07-04 Thread Mauro Carvalho Chehab
Those got removed by the doc conversion. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/control.rst | 4 1 file changed, 4 insertions(+) diff --git a/Documentation/linux_tv/media/v4l/control.rst

[PATCH 17/51] Documentation: linux_tv: remove controls.rst

2016-07-04 Thread Mauro Carvalho Chehab
This auto-generated file is bogus: it just adds a toc for two other files. Instead, just move the controls and extended-controls directly to the common.rst file. this avoids a blank page, and better organize the document. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 50/51] Documentation: pixfmt-nv12.rst: remove empty columns

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added empty columns (probably, it was used on DocBook just to increase spacing. Remove them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-nv12.rst | 13 ++--- 1 file changed, 2 insertions(+), 11 deletions(-)

[PATCH 48/51] Documentation: pixfmt-yuv422p.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. Remove it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-yuv422p.rst | 5 - 1 file changed, 5 deletions(-) diff --git

[PATCH 45/51] Documentation: pixfmt-41p.rst: remove empty columns

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added empty columns (probably, it was used on DocBook just to increase spacing. Remove them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-y41p.rst | 27 1 file changed, 27 deletions(-) diff

[PATCH 49/51] Documentation: pixfmt-yuv411p.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. It also added an extra line on one of the texts, breaking the original paragraph into two ones. Remove them. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 23/51] Documentation: planar-apis.rst: fix some conversion troubles

2016-07-04 Thread Mauro Carvalho Chehab
There is a missing escape caracter, causing troubles at the format of one of the paragraphs. Also, the ioctl description was producing some warnings about wrong identation. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/planar-apis.rst | 14

[PATCH 20/51] Documentation: extended-controls.rst: use reference for VIDIOC_S_CTRL

2016-07-04 Thread Mauro Carvalho Chehab
Instead of using a constant, use references, just like the other references for ioctl's. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/extended-controls.rst | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[PATCH 46/51] Documentation: pixfmt-yuv422m.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. Remove it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-yuv422m.rst | 5 - 1 file changed, 5 deletions(-) diff --git

[PATCH 47/51] Documentation: pixfmt-yuv444m.rst: remove empty columns

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added empty columns (probably, it was used on DocBook just to increase spacing. Remove them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-yuv444m.rst | 17 - 1 file changed, 17 deletions(-) diff --git

[PATCH 19/51] Documentation: control.rst: Fix missing reference for example 8

2016-07-04 Thread Mauro Carvalho Chehab
The conversion broke references and captions for examples. Fix the missing reference for control enumeration. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/control.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 42/51] Documentation: pixfmt-uyvy.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. Remove it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-uyvy.rst | 7 --- 1 file changed, 7 deletions(-) diff --git

[PATCH 43/51] Documentation: pixfmt-yvyu.rst: remove an empty column

2016-07-04 Thread Mauro Carvalho Chehab
The conversion added an empty column (probably, it was used on DocBook just to increase spacing. Remove it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-yvyu.rst | 9 - 1 file changed, 9 deletions(-) diff --git

[PATCH 12/51] Documentation: linux_tv: simplify references

2016-07-04 Thread Mauro Carvalho Chehab
There are lots of internal references in the form: :ref:`foo ` Simplify them to be just: :ref:`foo`. Patch generated via this small script: for j in $(find . -name '*.rst'); do echo $j; perl -ne 'if (m/\`(\S+)\s*\<(\S+)\>\`/) { if (!($1=~'http') && $1 eq $2) { s,\s*\<(\S+)\>,,; } }

[PATCH 41/51] Documentation: pixfmt-y12i.rst: correct format conversion

2016-07-04 Thread Mauro Carvalho Chehab
The format conversion broke one paragraph into two. Fix it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/linux_tv/media/v4l/pixfmt-y12i.rst | 4 1 file changed, 4 deletions(-) diff --git a/Documentation/linux_tv/media/v4l/pixfmt-y12i.rst

[PATCH 34/51] Documentation: fe_property_parameters.rst: improve descriptions

2016-07-04 Thread Mauro Carvalho Chehab
The asterisks cause parsing warnings with Sphinx: Documentation/linux_tv/media/dvb/fe_property_parameters.rst:954: WARNING: Inline substitution_reference start-string without end-string. /devel/v4l/patchwork/Documentation/linux_tv/media/dvb/fe_property_parameters.rst:993:

[PATCH 32/51] Documentation: fdl-appendix: Fix formatting issues

2016-07-04 Thread Mauro Carvalho Chehab
The conversion didn't add blank lines where needed, but it added were it weren't ;) Fix it, to make it to parse correctly by Sphinx. This also fixes a bunch or warnings: /devel/v4l/patchwork/Documentation/linux_tv/media/v4l/fdl-appendix.rst:44: WARNING: Explicit markup ends without a blank line;

[PATCH 21/51] Documentation: vidioc-queryctl.rst: change the title of this chapter

2016-07-04 Thread Mauro Carvalho Chehab
This chapter is referenced on several parts of the extended controls. Change the title to make it nicer where it is referenced. Ok, we might, instead, just change the name for the references to only mention the oldest ioctl, but IMHO, it is better to keep the name of all ioctls where it is

  1   2   >