cron job: media_tree daily build: WARNINGS

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 Sep 3 05:00:14 CEST 2017 media-tree git hash:fce4b371fe5c99a9c05db8493d72f0d1a474ab26 media_build gi

[PATCH 04/12] media: v4l uAPI docs: adjust some tables for PDF output

On tests with Spinx 1.4, some tables are still writing text outside cells. Adjust those tables. PS.: As this was revisited several times, I suspect that this will only be fully fixed if we add tabularcolumns to all tables at the V4L2 part of the book. Signed-off-by: Mauro Carvalho Chehab --- Do

[PATCH 06/12] media: rc-sysfs-nodes.rst: better use literals

A literal box provides a better visual when pdf and html output is generated for things like the output of a sysfs devnode. It alsod matches other conventions used within the media book. So, use it. While here, use literals for protocol names. Signed-off-by: Mauro Carvalho Chehab --- Documenta

[PATCH 03/12] media: vidioc-g-tuner.rst: Fix table number of cols

The Tuner Audio Matrix table is broken: the first row has 7 columns instead of 6, causing it to be parsed wrong and displayed very badly on PDF output. Fix it and adjust the table to look nice at PDF output Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/v4l/vidioc-g-tuner.rst

[PATCH 02/12] media: vidioc-querycap: use a more realistic value for KERNEL_VERSION

In the past, V4L2 versions were 0.x.y, but that changed years ago. Since Kernel 3.1, however, the numbering schema was changed to match the Kernel version. However, the presented example still uses the old numerating schema, with is a misleading information. So, update it to the new schema. Sign

[PATCH 09/12] media: docs: don't show ToC for each part on PDF output

The "Table of Contents" of a PDF file is generated only once, at the beginning fo the output. It doesn't produce it on each part. So, don't output this text on each part of the document. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/cec-drivers/index.rst | 4 +++- Do

[PATCH 10/12] media: v4l2-event.rst: adjust table to fit on PDF output

The tables there don't quite fit on PDF output. Adjust it by adding a tabularcolumns macro. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/kapi/v4l2-event.rst | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/media/kapi/v4l2-event.rst b/Documentation/media/kapi/

[PATCH 01/12] media: v4l uAPI: add descriptions for arguments to all ioctls

Several ioctls are missing descriptions for the third argument of the ioctl() command. They should have a description, as otherwise the output won't be ok, and will sound like something is missing. So, add them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/v4l/vidioc-create

[PATCH 00/12] media documentation ReST fixes

This series contain media documentation ReST fixes, mostly related to table parsing and the corresponding PDF output (tested with Sphinx 1.4). It also contains a few documentation trivial improvements, from a few minor issues I noticed when reading the PDF output text on some places. Probably the

[PATCH 05/12] media: docs: fix PDF build with Sphinx 1.4

changeset 70b074df4ed1 ("media: fix pdf build with Spinx 1.6") caused a regression at Sphinx 1.4 PDF build: although it produces a full document in batch mode, it returns errors on interactive mode: [63] Runaway argument? {\relax ! Paragraph ended before \multicolum

[PATCH 11/12] media: em28xx-cardlist.rst: update to reflect last changes

There were some new board additions. Update the cardlist accordingly. Signed-off-by: Mauro Carvalho Chehab --- .../media/v4l-drivers/em28xx-cardlist.rst | 210 +++-- 1 file changed, 106 insertions(+), 104 deletions(-) diff --git a/Documentation/media/v4l-drivers/em28xx-

[PATCH 07/12] media: mc uapi: adjust some table sizes for PDF output

Some cells are too small to fit the text written to it. Increase it. No text changes. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/mediactl/media-ioc-enum-entities.rst | 2 +- Documentation/media/uapi/mediactl/media-ioc-g-topology.rst| 2 +- Documentation/media/uapi/med

[PATCH 12/12] media: docs-rst: cardlists: change their format to flat-tables

Most tables there don't fit on 80 columns. Some are very big. While those tables are actually generated via scripts, every time a new board is added, the entire table could be reformatted. >From the diffstat PoV, that's bad, as it is hard to see what happened. One such example is at changeset 486

[PATCH 08/12] media: cec uapi: Adjust table sizes for PDF output

Several tables at this media book chapter have issues when PDF is produced. Adjust them. Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/uapi/cec/cec-ioc-adap-g-log-addrs.rst | 7 +-- Documentation/media/uapi/cec/cec-ioc-dqevent.rst | 9 + Documentation/media/ua

[PATCH 7/7] [media] Mantis: Delete an unnecessary variable initialisation in mantis_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 22:05:22 +0200 The variable "err" will eventually be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/mantis_cards.c | 2 +- 1 file changed, 1 i

[PATCH 6/7] [media] Mantis: Improve a size determination in mantis_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:56:07 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 5/7] [media] Mantis: Delete an error message for a failed memory allocation in mantis_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:55:05 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/mantis_cards.c | 4 +--- 1 file changed, 1 ins

[PATCH 4/7] [media] Hopper: Delete an unnecessary variable initialisation in hopper_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:50:55 +0200 The variable "err" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/hopper_cards.c | 2 +- 1 file changed, 1 insertion(+)

[PATCH 3/7] [media] Hopper: Adjust a null pointer check in two functions

From: Markus Elfring Date: Sat, 2 Sep 2017 21:25:50 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus fix the affected source code pla

[PATCH 2/7] [media] Hopper: Improve a size determination in hopper_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:21:22 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 1/7] [media] Hopper: Delete an error message for a failed memory allocation in hopper_pci_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 21:19:05 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/pci/mantis/hopper_cards.c | 1 - 1 file changed, 1 deleti

[PATCH 0/7] [media] Mantis: Adjustments for three function implementations

From: Markus Elfring Date: Sat, 2 Sep 2017 22:18:22 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (7): Delete an error message for a failed memory allocation in hopper_pci_probe() Improve a size determination in hopper_pci_probe() A

[PATCH 3/3] [media] cx18: Adjust ten checks for null pointers

From: Markus Elfring Date: Sat, 2 Sep 2017 19:49:23 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written … Thus fix the affected source code plac

[PATCH 2/3] [media] cx18: Improve a size determination in cx18_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 19:42:12 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 1/3] [media] cx18: Delete an error message for a failed memory allocation in cx18_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 19:39:56 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/pci/cx18/cx18-driver.c | 6 ++ 1 file changed, 2 inse

[PATCH 0/3] [media] cx18: Adjustments for seven function implementations

From: Markus Elfring Date: Sat, 2 Sep 2017 20:03:45 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete an error message for a failed memory allocation Improve a size determination Adjust ten checks for null pointers drivers

[PATCH 7/7] [media] ov9740: Improve a size determination in ov9740_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:46:03 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 6/7] [media] ov9740: Delete an error message for a failed memory allocation in ov9740_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:44:36 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/soc_camera/ov9740.c | 4 +--- 1 file changed, 1 inser

[PATCH 5/7] [media] ov9640: Improve a size determination in ov9640_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:37:03 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 4/7] [media] ov9640: Delete an error message for a failed memory allocation in ov9640_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:34:27 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/soc_camera/ov9640.c | 5 + 1 file changed, 1 inse

[PATCH 3/7] [media] ov6650: Delete an error message for a failed memory allocation in ov6650_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:16:39 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/ov6650.c | 5 + 1 file changed, 1 insertion(+), 4

[PATCH 2/7] [media] ov2640: Improve a size determination in ov2640_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:09:35 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 1/7] [media] ov2640: Delete an error message for a failed memory allocation in ov2640_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 16:07:31 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/ov2640.c | 5 + 1 file changed, 1 insertion(+), 4

[PATCH 0/7] [media] OmniVision: Adjustments for four function implementations

From: Markus Elfring Date: Sat, 2 Sep 2017 17:01:23 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (7): Delete an error message for a failed memory allocation in ov2640_probe() Improve a size determination in ov2640_probe() Delete an

Re: [PATCH v4l-utils] configure.ac: drop --disable-libv4l, disable plugin support instead

Hello, On Wed, 23 Aug 2017 13:06:13 +0200, Hans Verkuil wrote: > On 08/21/17 23:02, Thomas Petazzoni wrote: > > In commit 2e604dfbcd09b93f0808cedb2a0b324c5569a599 ("configure.ac: add > > --disable-libv4l option"), an option --disable-libv4l was added. As > > part of this, libv4l is no longer built

[PATCH 4/4] [media] adv7842: Improve a size determination in adv7842_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 12:53:15 +0200 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. This issue was dete

[PATCH 3/4] [media] adv7842: Delete an error message for a failed memory allocation in adv7842_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 12:50:19 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/adv7842.c | 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH 2/4] [media] adv7604: Adjust a null pointer check in three functions

From: Markus Elfring Date: Sat, 2 Sep 2017 11:43:00 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script “checkpatch.pl” pointed information out like the following. Comparison to NULL could be written !… Thus fix the affected source code pla

[PATCH 1/4] [media] adv7604: Delete an error message for a failed memory allocation in adv76xx_probe()

From: Markus Elfring Date: Sat, 2 Sep 2017 11:28:55 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drivers/media/i2c/adv7604.c | 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH 0/4] [media] ADV7...: Adjustments for four function implementations

From: Markus Elfring Date: Sat, 2 Sep 2017 14:49:43 +0200 A few update suggestions were taken into account from static source code analysis. Markus Elfring (4): Delete an error message for a failed memory allocation in adv76xx_probe() Adjust a null pointer check in three functions Delete a

[PATCH 4/7] media: rc: ensure that protocols are enabled for scancode drivers

rc scancode drivers without change_protocol should have all protocols enabled at all time. This was only true for cec and ir-kbd-i2c. Signed-off-by: Sean Young --- drivers/media/cec/cec-core.c | 1 - drivers/media/i2c/ir-kbd-i2c.c | 1 - drivers/media/rc/rc-main.c | 3 +++ 3 files changed,

[PATCH 2/2] keytable: if the protocols cannot be changed, don't try

Note that if an invalid protocol was selected, we already get the error "Invalid protocols selected". Signed-off-by: Sean Young --- utils/keytable/keytable.c | 5 + 1 file changed, 5 insertions(+) diff --git a/utils/keytable/keytable.c b/utils/keytable/keytable.c index 7b0d1acb..1fd545b0 10

[PATCH 1/2] keytable: use DEV_NAME if available

The result from EVIOCGNAME can be truncated. Signed-off-by: Sean Young --- utils/keytable/keytable.c | 22 +++--- 1 file changed, 19 insertions(+), 3 deletions(-) diff --git a/utils/keytable/keytable.c b/utils/keytable/keytable.c index 5d12ec31..7b0d1acb 100644 --- a/utils/keyta

[PATCH 1/7] media: dvb: i2c transfers over usb cannot be done from stack

Since commit 29d2fef8be11 ("usb: catch attempts to submit urbs with a vmalloc'd transfer buffer"), the AverMedia AverTV DVB-T USB 2.0 (a800) fails to probe. Cc: sta...@vger.kernel.org Signed-off-by: Sean Young --- drivers/media/dvb-frontends/dib3000mc.c | 50 ++-- drivers

[PATCH 7/7] media: rc: include device name in rc udev event

This name is also stored in the input's device name, but that is not available in TX only hardware (no input device). Signed-off-by: Sean Young --- drivers/media/rc/rc-main.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/media/rc/rc-main.c b/drivers/media/rc/rc-main.c index cf0c4

[PATCH 6/7] media: rc: if protocols can't be changed, don't be writable

If the protocols of an rc device cannot be changed, ensure the sysfs file is not writable. This makes it possible to detect this from userspace, so ir-keytable can deal with case without giving an error. Signed-off-by: Sean Young --- drivers/media/rc/rc-main.c | 28 +---

[PATCH 5/7] media: rc: dvb: use dvb device name for rc device

"IR-receiver inside an USB DVB receiver" is not descriptive and we have a proper name available. Signed-off-by: Sean Young --- drivers/media/usb/dvb-usb/dvb-usb-remote.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/usb/dvb-usb/dvb-usb-remote.c b/drivers/medi

[PATCH 2/7] media: dvb: a800: port to rc-core

This receiver only accepts nec16 messages, I've tried many other protocols and they're all dropped. Signed-off-by: Sean Young --- drivers/media/usb/dvb-usb/a800.c | 65 +--- 1 file changed, 14 insertions(+), 51 deletions(-) diff --git a/drivers/media/usb/dvb-

[PATCH 3/7] media: rc: avermedia keymap for a800

The keymap is missing one key, and correct another. Signed-off-by: Sean Young --- drivers/media/rc/keymaps/rc-avermedia-m135a.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/media/rc/keymaps/rc-avermedia-m135a.c b/drivers/media/rc/keymaps/rc-avermedia-m135a.c ind

[PATCH 0/7] a800 dvb rc changes

Various changes to make the a800 dvb adapter work again and to improve its ir receiver. Sean Young (7): media: dvb: i2c transfers over usb cannot be done from stack media: dvb: a800: port to rc-core media: rc: avermedia keymap for a800 media: rc: ensure that protocols are enabled for scanc

Re: [PATCH V2 0/1] build: gpio-ir-tx for 3.13

Hello Hans! > I moved gpio-ir-tx to 3.17. Is there a really urgent need to compile > it for older kernels? No, 3.17 is fine also. BR, Jasmin

Re: [PATCH v6 5/5] v4l: fwnode: Support generic parsing of graph endpoints in a single port

Hi Sakari, On Saturday, 2 September 2017 01:57:48 EEST Sakari Ailus wrote: > On Fri, Sep 01, 2017 at 01:28:40PM +0200, Hans Verkuil wrote: > >> diff --git a/include/media/v4l2-fwnode.h b/include/media/v4l2-fwnode.h > >> index d063ab4ff67b..dd13604178b4 100644 > >> --- a/include/media/v4l2-fwnode.h

Re: [PATCH v6.1 4/5] v4l: fwnode: Support generic parsing of graph endpoints in a device

Hi Hans, On Friday, 1 September 2017 14:18:55 EEST Hans Verkuil wrote: > On 30/08/17 14:45, Sakari Ailus wrote: > > The current practice is that drivers iterate over their endpoints and > > parse each endpoint separately. This is very similar in a number of > > drivers, implement a generic functio

Re: [PATCH v2 14/14] [media] v4l: Document explicit synchronization behaviour

On 09/01/2017 08:21 PM, Gustavo Padovan wrote: > Hi Hans, > > 2017-09-01 Hans Verkuil : > >> Hi Gustavo, >> >> I think I concentrate on this last patch first. Once I fully understand this >> I can review the code. Remember, it's been a while for me since I last looked >> at fences, so forgive me