cron job: media_tree daily build: ERRORS

2017-05-01 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 May 2 05:00:19 CEST 2017 media-tree git hash:3622d3e77ecef090b5111e3c5423313f11711dfa media_build

Re: [PATCH 1/2] em28xx: allow setting the eeprom bus at cards struct

2017-05-01 Thread Mauro Carvalho Chehab
Hi Frank, Em Mon, 1 May 2017 16:11:51 +0200 Frank Schäfer escreveu: > Am 01.05.2017 um 13:38 schrieb Mauro Carvalho Chehab: > > Right now, all devices use bus 0 for eeprom. However, newer > > versions of Terratec H6 use a different buffer for eeprom. > > > > So,

Re: [PATCH 2/6] rc-core: cleanup rc_register_device

2017-05-01 Thread David Härdeman
On Mon, May 01, 2017 at 05:49:53PM +0100, Sean Young wrote: >On Thu, Apr 27, 2017 at 10:34:03PM +0200, David Härdeman wrote: >> The device core infrastructure is based on the presumption that >> once a driver calls device_add(), it must be ready to accept >> userspace interaction. >> >> This

Re: [PATCH v2 3/3] libv4l-codecparsers: add GStreamer mpeg2 parser

2017-05-01 Thread Nicolas Dufresne
Le vendredi 28 avril 2017 à 17:02 +0200, Hugues Fruchet a écrit : > Add the mpeg2 codecparser backend glue which will > call the GStreamer parsing functions. > > Signed-off-by: Hugues Fruchet > --- >  configure.ac|  21 ++ >  

Re: [PATCH] [RFC] rc-core: report protocol information to userspace

2017-05-01 Thread Sean Young
On Mon, May 01, 2017 at 02:49:57PM +0200, David Härdeman wrote: > On Mon, May 01, 2017 at 11:38:30AM +0100, Sean Young wrote: > >On Sat, Apr 29, 2017 at 12:52:12PM +0200, David Härdeman wrote: > >> Whether we decide to go for any new keytable ioctl():s or not in rc-core, > >> we > >> should

Re: [PATCH 2/6] rc-core: cleanup rc_register_device

2017-05-01 Thread Sean Young
On Thu, Apr 27, 2017 at 10:34:03PM +0200, David Härdeman wrote: > The device core infrastructure is based on the presumption that > once a driver calls device_add(), it must be ready to accept > userspace interaction. > > This requires splitting rc_setup_rx_device() into two functions > and

Re: [PATCH] ir-lirc-codec: let lirc_dev handle the lirc_buffer (v3)

2017-05-01 Thread Sean Young
On Mon, May 01, 2017 at 03:32:34PM +0200, David Härdeman wrote: > ir_lirc_register() currently creates its own lirc_buffer before > passing the lirc_driver to lirc_register_driver(). > > When a module is later unloaded, ir_lirc_unregister() gets called > which performs a call to

[PATCH 6/7] rc-core: cx231xx - leave the internals of rc_dev alone

2017-05-01 Thread David Härdeman
Just some debug statements to change. Signed-off-by: David Härdeman --- drivers/media/usb/cx231xx/cx231xx-input.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/media/usb/cx231xx/cx231xx-input.c b/drivers/media/usb/cx231xx/cx231xx-input.c

[PATCH 7/7] rc-core: tm6000 - leave the internals of rc_dev alone

2017-05-01 Thread David Härdeman
Not sure what the driver is trying to do, however, IR handling seems incomplete ATM so deleting the offending parts shouldn't affect functionality Signed-off-by: David Härdeman --- drivers/media/usb/tm6000/tm6000-input.c |4 1 file changed, 4 deletions(-) diff --git

[PATCH 3/7] rc-core: img-nec-decoder - leave the internals of rc_dev alone

2017-05-01 Thread David Härdeman
Obvious fix, leave repeat handling to rc-core Signed-off-by: David Härdeman --- drivers/media/rc/ir-nec-decoder.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/media/rc/ir-nec-decoder.c b/drivers/media/rc/ir-nec-decoder.c index

[PATCH 5/7] rc-core: ir-raw - leave the internals of rc_dev alone

2017-05-01 Thread David Härdeman
Replace the REP_DELAY value with a static value, which makes more sense. Automatic repeat handling in the input layer has no relevance for the drivers idea of "a long time". Signed-off-by: David Härdeman --- drivers/media/rc/rc-ir-raw.c |4 +--- 1 file changed, 1

[PATCH 2/7] rc-core: img-ir - leave the internals of rc_dev alone

2017-05-01 Thread David Härdeman
Changing the protocol does not imply that the keymap changes. Signed-off-by: David Härdeman --- drivers/media/rc/img-ir/img-ir-hw.c |4 1 file changed, 4 deletions(-) diff --git a/drivers/media/rc/img-ir/img-ir-hw.c b/drivers/media/rc/img-ir/img-ir-hw.c index

[PATCH 0/7] rc: don't poke around in rc_dev internals

2017-05-01 Thread David Härdeman
The following patch series fixes up various drivers which go poking around in struct rc_dev for no good reason. --- David Härdeman (7): rc-core: ati_remote - leave the internals of rc_dev alone rc-core: img-ir - leave the internals of rc_dev alone rc-core: img-nec-decoder -

[PATCH 1/7] rc-core: ati_remote - leave the internals of rc_dev alone

2017-05-01 Thread David Härdeman
The REP_DELAY setting on the input device is independent of hardware. This change should not change how to driver works (as it does a keydown/keyup and has no real repeat handling). Signed-off-by: David Härdeman --- drivers/media/rc/ati_remote.c |3 --- 1 file changed, 3

[PATCH 4/7] rc-core: sanyo - leave the internals of rc_dev alone

2017-05-01 Thread David Härdeman
Leave repeat handling to rc-core. Signed-off-by: David Härdeman --- drivers/media/rc/ir-sanyo-decoder.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/media/rc/ir-sanyo-decoder.c b/drivers/media/rc/ir-sanyo-decoder.c index

[PATCH 12/16] lirc_dev: return POLLHUP and POLLERR when device is gone

2017-05-01 Thread David Härdeman
Most drivers return both values when the device is gone. Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index

[PATCH 14/16] lirc_dev: cleanup includes

2017-05-01 Thread David Härdeman
Remove superfluous includes and defines. Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index 7db7d4c57991..4ba6c7e2d41b

[PATCH 16/16] lirc_dev: cleanup header

2017-05-01 Thread David Härdeman
Remove some stuff from lirc_dev.h which is not used anywhere. Signed-off-by: David Härdeman --- include/media/lirc_dev.h | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/include/media/lirc_dev.h b/include/media/lirc_dev.h index

[PATCH 15/16] lirc_dev: remove name from struct lirc_driver

2017-05-01 Thread David Härdeman
The name is only used for a few debug messages and the name of the parent device as well as the name of the lirc device (e.g. "lirc0") are sufficient anyway. Signed-off-by: David Härdeman --- drivers/media/rc/ir-lirc-codec.c|2 -- drivers/media/rc/lirc_dev.c

[PATCH 13/16] lirc_dev: use an ida instead of a hand-rolled array to keep track of minors

2017-05-01 Thread David Härdeman
Using the kernel ida facilities, we can avoid a lot of unnecessary code and at the same time get rid of lirc_dev_lock in favour of per-device locks (the irctls array was used throughout lirc_dev so this patch necessarily touches a lot of code). Signed-off-by: David Härdeman

[PATCH 11/16] lirc_dev: remove unused module parameter

2017-05-01 Thread David Härdeman
The "debug" parameter isn't actually used anywhere. Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c |5 - 1 file changed, 5 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index 34bd3f8bf30d..57d21201ff93 100644 ---

[PATCH 10/16] lirc_dev: remove superfluous get/put_device() calls

2017-05-01 Thread David Härdeman
device_add() and friends alredy manage the references to the parent device so these calls aren't necessary. Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c |4 1 file changed, 4 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c

[PATCH 08/16] lirc_zilog: remove module parameter minor

2017-05-01 Thread David Härdeman
Remove the "minor" module parameter in preparation for the next patch. Signed-off-by: David Härdeman --- drivers/staging/media/lirc/lirc_zilog.c | 16 ++-- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/drivers/staging/media/lirc/lirc_zilog.c

[PATCH 09/16] lirc_dev: remove lirc_irctl_init() and lirc_cdev_add()

2017-05-01 Thread David Härdeman
These two functions only make the logic in lirc_register_driver() harder to follow. (Note that almost no other driver calls kobject_set_name() on their cdev so I simply removed that part). Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c | 44

[PATCH 07/16] lirc_dev: merge lirc_register_driver() and lirc_allocate_driver()

2017-05-01 Thread David Härdeman
Merging the two means that lirc_allocate_buffer() is called before device_add() and cdev_add() which makes more sense. This also simplifies the locking slightly because lirc_allocate_buffer() will always be called with lirc_dev_lock held. Signed-off-by: David Härdeman ---

[PATCH 06/16] lirc_dev: make fops mandatory

2017-05-01 Thread David Härdeman
Every caller of lirc_register_driver() passes their own fops and there are no users of lirc_dev_fop_write() in the kernel tree. Thus we can make fops mandatory and remove lirc_dev_fop_write(). Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c | 41

[PATCH 05/16] lirc_dev: clarify error handling

2017-05-01 Thread David Härdeman
out_sysfs is misleading, sysfs only comes into play after device_add(). Also, calling device_init() before the rest of struct dev is filled out is clearer. Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c |6 +++--- 1 file changed, 3 insertions(+), 3

[PATCH 04/16] lirc_dev: remove sampling kthread

2017-05-01 Thread David Härdeman
There are no drivers which use this functionality. Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c | 94 +-- drivers/staging/media/lirc/lirc_zilog.c |1 include/media/lirc_dev.h| 16 - 3 files

[PATCH 02/16] lirc_dev: remove unused set_use_inc/set_use_dec

2017-05-01 Thread David Härdeman
Since there are no users of this functionality, it can be removed altogether. Signed-off-by: David Härdeman --- drivers/media/rc/ir-lirc-codec.c |2 -- drivers/media/rc/lirc_dev.c | 24 ++-- include/media/lirc_dev.h |6 -- 3

[PATCH 01/16] lirc_dev: remove pointless functions

2017-05-01 Thread David Härdeman
drv->set_use_inc and drv->set_use_dec are already optional so we can remove all dummy functions. Signed-off-by: David Härdeman --- drivers/media/rc/ir-lirc-codec.c| 14 ++ drivers/staging/media/lirc/lirc_zilog.c | 11 --- 2 files changed, 2

[PATCH 00/16] lirc_dev spring cleaning

2017-05-01 Thread David Härdeman
lirc_dev has lots of functionality which is unused and the code isn't exactly up-to-date with current kernel practices. This patchset removes the unused bits and also simplifies the locking by moving lirc_dev over to only use per-device mutexes rather than a big lirc lock in addition to per-device

[PATCH 03/16] lirc_dev: correct error handling

2017-05-01 Thread David Härdeman
If an error is generated, nonseekable_open() shouldn't be called. Signed-off-by: David Härdeman --- drivers/media/rc/lirc_dev.c |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/media/rc/lirc_dev.c b/drivers/media/rc/lirc_dev.c index

Re: [PATCH 1/2] em28xx: allow setting the eeprom bus at cards struct

2017-05-01 Thread Devin Heitmueller
On Mon, May 1, 2017 at 10:11 AM, Frank Schäfer wrote: > > Am 01.05.2017 um 13:38 schrieb Mauro Carvalho Chehab: >> Right now, all devices use bus 0 for eeprom. However, newer >> versions of Terratec H6 use a different buffer for eeprom. >> >> So, add support to use a

Re: [PATCH 1/2] em28xx: allow setting the eeprom bus at cards struct

2017-05-01 Thread Frank Schäfer
Am 01.05.2017 um 13:38 schrieb Mauro Carvalho Chehab: > Right now, all devices use bus 0 for eeprom. However, newer > versions of Terratec H6 use a different buffer for eeprom. > > So, add support to use a different I2C address for eeprom. Has this been tested ? Did you read my reply to the

[PATCH] ir-lirc-codec: let lirc_dev handle the lirc_buffer (v3)

2017-05-01 Thread David Härdeman
ir_lirc_register() currently creates its own lirc_buffer before passing the lirc_driver to lirc_register_driver(). When a module is later unloaded, ir_lirc_unregister() gets called which performs a call to lirc_unregister_driver() and then free():s the lirc_buffer. The problem is that: a) there

Re: [PATCH] ir-lirc-codec: let lirc_dev handle the lirc_buffer (v2)

2017-05-01 Thread David Härdeman
On Mon, May 01, 2017 at 01:22:21PM +0100, Sean Young wrote: >On Sat, Apr 29, 2017 at 11:22:28PM +0200, David Härdeman wrote: >> ir_lirc_register() currently creates its own lirc_buffer before >> passing the lirc_driver to lirc_register_driver(). >> >> When a module is later unloaded,

Re: [PATCH] [RFC] rc-core: report protocol information to userspace

2017-05-01 Thread David Härdeman
On Mon, May 01, 2017 at 11:38:30AM +0100, Sean Young wrote: >On Sat, Apr 29, 2017 at 12:52:12PM +0200, David Härdeman wrote: >> Whether we decide to go for any new keytable ioctl():s or not in rc-core, we >> should provide the protocol information of keypresses to userspace. >> >> Note that this

Re: [PATCH] rc-core: export the hardware type to sysfs

2017-05-01 Thread David Härdeman
On Mon, May 01, 2017 at 11:36:13AM +0100, Sean Young wrote: >On Sat, Apr 29, 2017 at 12:03:29PM +0200, David Härdeman wrote: >> Exporting the hardware type makes it possible for userspace applications >> to know what to expect from the hardware. >> >> This makes it possible to write more

Re: [PATCH] ir-lirc-codec: let lirc_dev handle the lirc_buffer (v2)

2017-05-01 Thread Sean Young
On Sat, Apr 29, 2017 at 11:22:28PM +0200, David Härdeman wrote: > ir_lirc_register() currently creates its own lirc_buffer before > passing the lirc_driver to lirc_register_driver(). > > When a module is later unloaded, ir_lirc_unregister() gets called > which performs a call to

[PATCH 2/2] em28xx: add support for new revisions of Terratec H6

2017-05-01 Thread Mauro Carvalho Chehab
There's a new version of Terratec H6 with uses USB ID 0ccd:10b2. This version is similar to the old one (with is supported via the HTC entry), except that this one has the eeprom on the second bus. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 1/2] em28xx: allow setting the eeprom bus at cards struct

2017-05-01 Thread Mauro Carvalho Chehab
Right now, all devices use bus 0 for eeprom. However, newer versions of Terratec H6 use a different buffer for eeprom. So, add support to use a different I2C address for eeprom. Signed-off-by: Mauro Carvalho Chehab --- drivers/media/usb/em28xx/em28xx-cards.c | 1 +

Re: [PATCH] [RFC] rc-core: report protocol information to userspace

2017-05-01 Thread Sean Young
On Sat, Apr 29, 2017 at 12:52:12PM +0200, David Härdeman wrote: > Whether we decide to go for any new keytable ioctl():s or not in rc-core, we > should provide the protocol information of keypresses to userspace. > > Note that this means that the RC_TYPE_* definitions become part of the >

Re: [PATCH] rc-core: export the hardware type to sysfs

2017-05-01 Thread Sean Young
On Sat, Apr 29, 2017 at 12:03:29PM +0200, David Härdeman wrote: > Exporting the hardware type makes it possible for userspace applications > to know what to expect from the hardware. > > This makes it possible to write more user-friendly userspace apps. This duplicates lirc features

Re: [PATCH 0/5] RFC: ADV748x HDMI/Analog video receiver

2017-05-01 Thread Simon Horman
On Fri, Apr 28, 2017 at 09:47:05AM +0100, Kieran Bingham wrote: > Hi Simon, > > On 28/04/17 08:09, Simon Horman wrote: > > On Thu, Apr 27, 2017 at 07:25:59PM +0100, Kieran Bingham wrote: > >> From: Kieran Bingham > >> > >> This is an RFC for the Analog

Re: [PATCH v2 0/3] r8a7793 Gose video input support

2017-05-01 Thread Simon Horman
On Fri, Apr 28, 2017 at 11:40:20AM +0300, Laurent Pinchart wrote: > Hi Simon, > > On Friday 28 Apr 2017 07:16:24 Simon Horman wrote: > > On Wed, Apr 26, 2017 at 06:56:06PM +0300, Laurent Pinchart wrote: > > > On Tuesday 21 Feb 2017 01:42:15 Laurent Pinchart wrote: > > >> On Thursday 20 Oct 2016