cron job: media_tree daily build: ERRORS

2017-09-29 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: Sat Sep 30 05:00:18 CEST 2017 media-tree git hash:cf09e3c904bf424f8b6a8203958e09bf7d9bcbc0 media_build

[PATCH] media: usb: s2255drv: update to firmware loading

2017-09-29 Thread dean
fixes intermittent soft reboot issue with firmware load increases wait time of reset, as required by HW Signed-off-by: Dean Anderson --- drivers/media/usb/s2255/s2255drv.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git

Re: [PATCH v2 13/17] media: v4l2-async: simplify v4l2_async_subdev structure

2017-09-29 Thread Sakari Ailus
Hi Mauro, (Removing the non-list recipients.) On Fri, Sep 29, 2017 at 06:27:13AM -0300, Mauro Carvalho Chehab wrote: > Em Thu, 28 Sep 2017 15:09:21 +0300 > Sakari Ailus escreveu: > > > Hi Mauro, > > > > On Wed, Sep 27, 2017 at 06:46:56PM -0300, Mauro Carvalho Chehab

[PATCH] imx-csi: fix burst size

2017-09-29 Thread Russell King
Setting a burst size of "8" doesn't work for IMX219 with 8-bit bayer, but a burst size of "16" does. Fix this. Signed-off-by: Russell King --- drivers/staging/media/imx/imx-media-csi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH RFC] media: staging/imx: fix complete handler

2017-09-29 Thread Russell King
The complete handler walks all entities, expecting to find an imx subdevice for each and every entity. However, camera drivers such as smiapp can themselves contain multiple entities, for which there will not be an imx subdevice. This causes imx_media_find_subdev_by_sd() to fail, making the imx

[PATCH RFC] [media] v4l: async: don't bomb out on ->complete failure

2017-09-29 Thread Russell King
When a subdev is being registered via v4l2_async_register_subdev(), we test to see if we have all the components, and if so, we call the ->complete() handler. However, the error handling is broken - if notifier->complete() returns an error, we propagate the error out of

Re: [PATCH 2/2] v4l: cadence: Add Cadence MIPI-CSI2 TX driver

2017-09-29 Thread Benoit Parrot
Maxime, Thank you for the patch. Maxime Ripard wrote on Fri [2017-Sep-22 13:47:03 +0200]: > The Cadence MIPI-CSI2 TX controller is an hardware block meant to be used > as a bridge between pixel interfaces and a CSI-2 bus. > > It supports operating with an

Re: [PATCH v4 2/2] v4l: cadence: Add Cadence MIPI-CSI2 RX driver

2017-09-29 Thread Benoit Parrot
Maxime Ripard wrote on Fri [2017-Sep-22 12:08:23 +0200]: > The Cadence CSI-2 RX Controller is an hardware block meant to be used as a > bridge between a CSI-2 bus and pixel grabbers. > > It supports operating with internal or external D-PHY, with up to 4 lanes,

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Markus Heiser
> Am 29.09.2017 um 17:29 schrieb Mauro Carvalho Chehab > : > > Em Thu, 28 Sep 2017 18:28:32 +0200 > Markus Heiser escreveu: > >> Hi Mauro, >> >>> Am 27.09.2017 um 23:10 schrieb Mauro Carvalho Chehab >>> : > >>>

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Mauro Carvalho Chehab
Em Fri, 29 Sep 2017 12:29:31 -0300 Mauro Carvalho Chehab escreveu: > Em Thu, 28 Sep 2017 18:28:32 +0200 > Markus Heiser escreveu: > > > Hi Mauro, > > > > > Am 27.09.2017 um 23:10 schrieb Mauro Carvalho Chehab > > >

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Mauro Carvalho Chehab
Em Thu, 28 Sep 2017 18:28:32 +0200 Markus Heiser escreveu: > Hi Mauro, > > > Am 27.09.2017 um 23:10 schrieb Mauro Carvalho Chehab > > : > > +It is possible to document nested structs unions, like:: > > + > > + /** > > + * struct

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Mauro Carvalho Chehab
Em Fri, 29 Sep 2017 17:00:36 +0200 Markus Heiser escreveu: > > None to argue here. If it is part of the language, either comply or use > > some other language that it isn't position oriented. > > Just for info; when Guido van Rossum created python he thought, that >

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Markus Heiser
> Am 29.09.2017 um 15:33 schrieb Mauro Carvalho Chehab > : >> Sorry if I was unclear. I mean we should not use tabs in reST or in py. > > Ah, OK. > >> In python the indentation is a part of the language syntax, same in >> reST; the indentation is the markup. It's not

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Mauro Carvalho Chehab
Em Fri, 29 Sep 2017 15:07:05 +0200 Markus Heiser escreveu: > > Am 29.09.2017 um 14:08 schrieb Mauro Carvalho Chehab > > : > > > > Em Thu, 28 Sep 2017 18:28:32 +0200 > > Markus Heiser escreveu: > > > >> Hi

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Markus Heiser
> Am 29.09.2017 um 14:08 schrieb Mauro Carvalho Chehab > : > > Em Thu, 28 Sep 2017 18:28:32 +0200 > Markus Heiser escreveu: > >> Hi Mauro, >> >>> Am 27.09.2017 um 23:10 schrieb Mauro Carvalho Chehab >>> : >>> >>

Re: [PATCH] dma-fence: fix dma_fence_get_rcu_safe

2017-09-29 Thread Joonas Lahtinen
On Wed, 2017-09-20 at 20:20 +0200, Daniel Vetter wrote: > On Mon, Sep 11, 2017 at 01:06:32PM +0200, Christian König wrote: > > Am 11.09.2017 um 12:01 schrieb Chris Wilson: > > > [SNIP] > > > > Yeah, but that is illegal with a fence objects. > > > > > > > > When anybody allocates fences this way

Re: [PATCH v2 09/13] scripts: kernel-doc: parse next structs/unions

2017-09-29 Thread Mauro Carvalho Chehab
Em Thu, 28 Sep 2017 18:28:32 +0200 Markus Heiser escreveu: > Hi Mauro, > > > Am 27.09.2017 um 23:10 schrieb Mauro Carvalho Chehab > > : > > > I also untabified the example since tabs in reST are > a nightmare, especially in code blocks ...

Re: [PATCH v2 16/17] media: videobuf2-core: improve kernel-doc markups

2017-09-29 Thread Marek Szyprowski
Hi, On 2017-09-27 23:46, Mauro Carvalho Chehab wrote: Now that nested structs are supported, change the documentation to use it. While here, add cross-references where pertinent and use monotonic fonts where pertinent, using the right markup tags. Signed-off-by: Mauro Carvalho Chehab

[bug report] V4L/DVB (13717): [MB86A16] Statistics Updates

2017-09-29 Thread Dan Carpenter
Hello Manu Abraham, The patch 77557abef0de: "V4L/DVB (13717): [MB86A16] Statistics Updates" from Dec 3, 2009, leads to the following static checker warning: drivers/media/dvb-frontends/mb86a16.c:1690 mb86a16_read_ber() error: uninitialized symbol 'timer'.

[PATCH 2/2] smiapp: Rely on runtime PM

2017-09-29 Thread Sakari Ailus
Instead of relying on a mix of runtime PM and the s_power() callback, drop the s_power() callback altogether and use runtime PM solely. As device access is required during device power-on and power-off sequences, runtime PM alone cannot tell whether the device is available. Thus the "active"

[PATCH 1/2] smiapp: Use __v4l2_ctrl_handler_setup()

2017-09-29 Thread Sakari Ailus
Use unlocked __v4l2_ctrl_handler_setup() in order to make the control setup atomic. Signed-off-by: Sakari Ailus --- drivers/media/i2c/smiapp/smiapp-core.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git

[PATCH 0/2] smiapp: Don't rely on s_power()

2017-09-29 Thread Sakari Ailus
Hi everyone, These two patches make the smiapp driver independent of the s_power() core sub-device callback, and instead use runtime PM to control the device's power state. This should be the model for future sensor drivers: the s_power() callback is really something we want to get rid of. A

[PATCH v3 0/2] Improve generic DT binding documentation for media devices

2017-09-29 Thread Sakari Ailus
Hi folks, This set improves the DT binding documentation for media devices where device specific documentation is lacking and also documents that explicit documentation of all properties supported by the device is required. - Port and endpoint numbering - lane numbering for the data-lanes

[PATCH v3 1/2] dt: bindings: media: Document practices for DT bindings, ports, endpoints

2017-09-29 Thread Sakari Ailus
Port and endpoint numbering has been omitted in DT binding documentation for a large number of devices. Also common properties the device uses have been missed in binding documentation. Make it explicit that these things need to be documented. Signed-off-by: Sakari Ailus

[PATCH v3 2/2] dt: bindings: media: Document data lane numbering without lane reordering

2017-09-29 Thread Sakari Ailus
Most devices do not support lane reordering and in many cases the documentation of the data-lanes property is incomplete for such devices. Document that in case the lane reordering isn't supported, monotonically incremented values from 0 or 1 shall be used. Signed-off-by: Sakari Ailus

Re: [PATCH v7 6/7] media: videodev2: add a flag for MC-centric devices

2017-09-29 Thread Sakari Ailus
On Wed, Sep 27, 2017 at 07:23:48PM -0300, Mauro Carvalho Chehab wrote: > As both vdev-centric and MC-centric devices may implement the > same APIs, we need a flag to allow userspace to distinguish > between them. > > Signed-off-by: Mauro Carvalho Chehab Acked-by:

Re: [PATCH v7 7/7] media: open.rst: add a notice about subdev-API on vdev-centric

2017-09-29 Thread Sakari Ailus
On Wed, Sep 27, 2017 at 07:23:49PM -0300, Mauro Carvalho Chehab wrote: > The documentation doesn't mention if vdev-centric hardware > control would have subdev API or not. > > Add a notice about that, reflecting the current status, where > three drivers use it, in order to support some