cron job: media_tree daily build: WARNINGS

2017-09-09 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: Sun Sep 10 05:00:17 CEST 2017 media-tree git hash:1efdf1776e2253b77413c997bed862410e4b6aaf media_build

dvb_usb_rtl2832u vs dvb_usb_rtl28xxu

2017-09-09 Thread Eyal Lebedinsky
I was running f19 and had dvb_usb_rtl2832u built from git handling my "Leadtek Winfast DTV2000 DS PLUS TV" tuners. https://www.linuxtv.org/wiki/index.php/Leadtek_WinFast_DTV2000DS_PLUS I am using jaredquinn's driver Yesterday I upgraded to f22 and the tuners stopped working. I

Re: [PATCH v9 24/24] arm: dts: omap3: N9/N950: Add flash references to the camera

2017-09-09 Thread Sakari Ailus
Hi Pavel, On Sat, Sep 09, 2017 at 08:32:40PM +0200, Pavel Machek wrote: > Hi! > > > Add flash and indicator LED phandles to the sensor node. > > > > Signed-off-by: Sakari Ailus > > I'm adding similar support to et8ek8 and wonder.. why don't you also > add support

Re: [RFC] et8ek8: Add support for flash and lens devices

2017-09-09 Thread Sakari Ailus
Hi Pavel, On Sat, Sep 09, 2017 at 11:47:24PM +0200, Pavel Machek wrote: > > Parse async sub-devices by using > v4l2_subdev_fwnode_reference_parse_sensor_common(). > > These types devices aren't directly related to the sensor, but are > nevertheless handled by the et8ek8 driver due to the

[RFC] et8ek8: Add support for flash and lens devices

2017-09-09 Thread Pavel Machek
Parse async sub-devices by using v4l2_subdev_fwnode_reference_parse_sensor_common(). These types devices aren't directly related to the sensor, but are nevertheless handled by the et8ek8 driver due to the relationship of these component to the main part of the camera module --- the sensor.

Re: [PATCH v9 18/24] as3645a: Switch to fwnode property API

2017-09-09 Thread Sakari Ailus
Hi Jacek, On Sat, Sep 09, 2017 at 09:06:41PM +0200, Jacek Anaszewski wrote: > Hi Sakari, > > I've come across this patch only by a chance. I believe that merging > leds-as3645a.c patches via media tree is not going to be a persistent > pattern. At least we haven't agreed on that, and in any case

Re: [GIT PULL] SAA716x DVB driver

2017-09-09 Thread Mauro Carvalho Chehab
Em Sat, 9 Sep 2017 14:52:18 +0200 Soeren Moch escreveu: > Hi Mauro, > > sorry for the late reply, unfortunately your email was filtered out as spam. I'll focus talking about saa716x/av7110 here, as this is out of of scope of the documentation patches. > > On 27.08.2017 12:30,

[PATCH 3/3] [media] xc2028: Use common error handling code in load_firmware()

2017-09-09 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 9 Sep 2017 22:07:04 +0200 Add jump targets so that a bit of exception handling can be better reused at the end of this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring

[PATCH 2/3] [media] xc2028: Adjust two null pointer checks in load_all_firmwares()

2017-09-09 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 9 Sep 2017 21:48:58 +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

[PATCH 1/3] [media] xc2028: Delete two error messages for a failed memory allocation in load_all_firmwares()

2017-09-09 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 9 Sep 2017 21:30:11 +0200 Omit extra messages for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 0/3] [media] xc2028: Adjustments for two function implementations

2017-09-09 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 9 Sep 2017 22:18:22 +0200 Three update suggestions were taken into account from static source code analysis. Markus Elfring (3): Delete two error messages for a failed memory allocation Adjust two null pointer checks Use

Re: [RFC PATCH v4 3/6] i2c: add docs to clarify DMA handling

2017-09-09 Thread Mauro Carvalho Chehab
Em Sat, 9 Sep 2017 17:27:06 +0200 Wolfram Sang escreveu: > > Yes, but the statistics that 10% of I2C bus master drivers > > are DMA-safe is not true, if you take those into account ;-) > > > > Perhaps you could write it as (or something similar): > > > > At this time of

[PATCH 2/2] [media] xc4000: Adjust two null pointer checks in xc4000_fwupload()

2017-09-09 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 9 Sep 2017 20:55:14 +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

[PATCH 1/2] [media] xc4000: Delete two error messages for a failed memory allocation in xc4000_fwupload()

2017-09-09 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 9 Sep 2017 20:46:35 +0200 Omit extra messages for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

[PATCH 0/2] [media] xc4000: Adjustments for xc4000_fwupload()

2017-09-09 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 9 Sep 2017 21:00:12 +0200 Two update suggestions were taken into account from static source code analysis. Markus Elfring (2): Delete two error messages for a failed memory allocation Adjust two null pointer checks

Re: [PATCH v9 18/24] as3645a: Switch to fwnode property API

2017-09-09 Thread Jacek Anaszewski
Hi Sakari, I've come across this patch only by a chance. I believe that merging leds-as3645a.c patches via media tree is not going to be a persistent pattern. At least we haven't agreed on that, and in any case I should have a possibility to give my ack for this patch. Would you mind also adding

Re: [PATCH v9 24/24] arm: dts: omap3: N9/N950: Add flash references to the camera

2017-09-09 Thread Pavel Machek
Hi! > Add flash and indicator LED phandles to the sensor node. > > Signed-off-by: Sakari Ailus I'm adding similar support to et8ek8 and wonder.. why don't you also add support for autofocus? Driver not yet available? Thanks,

Re: [PATCH v9 04/24] v4l: async: Add V4L2 async documentation to the documentation build

2017-09-09 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday, 8 September 2017 16:11:55 EEST Sakari Ailus wrote: > The V4L2 async wasn't part of the documentation build. Fix this. > > Signed-off-by: Sakari Ailus > Reviewed-by: Niklas Söderlund

Re: [PATCH v9 02/24] v4l: async: Remove re-probing support

2017-09-09 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday, 8 September 2017 16:11:53 EEST Sakari Ailus wrote: > Remove V4L2 async re-probing support. The re-probing support has been > there to support cases where the sub-devices require resources provided by > the main driver's hardware to function, such as

Re: [PATCH v9 03/24] v4l: async: Use more intuitive names for internal functions

2017-09-09 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Friday, 8 September 2017 16:11:54 EEST Sakari Ailus wrote: > Rename internal functions to make the names of the functions better > describe what they do. > > Old nameNew name > v4l2_async_test_notify

Re: [RFC PATCH v4 3/6] i2c: add docs to clarify DMA handling

2017-09-09 Thread Wolfram Sang
> Yes, but the statistics that 10% of I2C bus master drivers > are DMA-safe is not true, if you take those into account ;-) > > Perhaps you could write it as (or something similar): > > At this time of writing, only +10% of I2C bus master > drivers for non-remote boards have DMA

Re: [GIT PULL] SAA716x DVB driver

2017-09-09 Thread Soeren Moch
Hi Mauro, sorry for the late reply, unfortunately your email was filtered out as spam. On 27.08.2017 12:30, Mauro Carvalho Chehab wrote: > Em Fri, 21 Jul 2017 22:44:42 +0200 > Soeren Moch escreveu: > >> Hi Mauro, >> >> On 20.07.2017 07:49:36 -0300, Mauro Carvalho Chehab wrote: >>>

Re: [PATCH 00/15] Improve DVB documentation and reduce its gap

2017-09-09 Thread Soeren Moch
On 04.09.2017 13:29, Mauro Carvalho Chehab wrote: > Em Mon, 4 Sep 2017 02:55:15 +0200 > Soeren Moch escreveu: > >> Hi Mauro, >> >> On 01.09.2017 11:32, Mauro Carvalho Chehab wrote: >>> Em Fri, 1 Sep 2017 10:40:28 +0200 >>> Honza Petrouš escreveu: >>>

Re: [PATCH] [media] pxa_camera: Delete an error message for a failed memory allocation in pxa_camera_probe()

2017-09-09 Thread Robert Jarzmik
SF Markus Elfring writes: > From: Markus Elfring > Date: Fri, 8 Sep 2017 22:05:14 +0200 > > Omit an extra message for a memory allocation failure in this function. > > This issue was detected by using the Coccinelle software. > >

Re: [PATCH v9 00/23] Unified fwnode endpoint parser, async sub-device notifier support, N9 flash DTS

2017-09-09 Thread Pavel Machek
On Fri 2017-09-08 16:25:07, Sakari Ailus wrote: > On Fri, Sep 08, 2017 at 04:11:51PM +0300, Sakari Ailus wrote: > > With this, the as3645a driver successfully registers a sub-device to the > > media device created by the omap3isp driver. The kernel also has the > > information it's related to the

Re: [PATCH v9 18/24] as3645a: Switch to fwnode property API

2017-09-09 Thread Pavel Machek
On Fri 2017-09-08 16:18:16, Sakari Ailus wrote: > Switch the as3645a from OF to the fwnode property API. Also add ACPI > support. > > Signed-off-by: Sakari Ailus Acked-by: Pavel Machek -- (english) http://www.livejournal.com/~pavelmachek (cesky,

Re: [PATCH v9 09/24] omap3isp: Print the name of the entity where no source pads could be found

2017-09-09 Thread Pavel Machek
On Fri 2017-09-08 16:18:07, Sakari Ailus wrote: > If no source pads are found in an entity, print the name of the entity. > > Signed-off-by: Sakari Ailus > Acked-by: Hans Verkuil Acked-by: Pavel Machek -- (english)

Re: [PATCH v9 08/24] omap3isp: Fix check for our own sub-devices

2017-09-09 Thread Pavel Machek
On Fri 2017-09-08 16:18:06, Sakari Ailus wrote: > We only want to link sub-devices that were bound to the async notifier the > isp driver registered but there may be other sub-devices in the > v4l2_device as well. Check for the correct async notifier. > > Signed-off-by: Sakari Ailus

Re: [PATCH v9 03/24] v4l: async: Use more intuitive names for internal functions

2017-09-09 Thread Pavel Machek
On Fri 2017-09-08 16:11:54, Sakari Ailus wrote: > Rename internal functions to make the names of the functions better > describe what they do. > > Old nameNew name > v4l2_async_test_notify v4l2_async_match_notify > v4l2_async_belongs

Re: [PATCH v9 15/24] dt: bindings: Add a binding for flash LED devices associated to a sensor

2017-09-09 Thread Pavel Machek
On Fri 2017-09-08 16:18:13, Sakari Ailus wrote: > Camera flash drivers (and LEDs) are separate from the sensor devices in > DT. In order to make an association between the two, provide the > association information to the software. > > Signed-off-by: Sakari Ailus >

Re: [PATCH v9 17/24] ACPI: Document how to refer to LEDs from remote nodes

2017-09-09 Thread Pavel Machek
On Fri 2017-09-08 16:18:15, Sakari Ailus wrote: > Document referring to LEDs from remote device nodes, such as from camera > sensors. > > Signed-off-by: Sakari Ailus > diff --git a/Documentation/acpi/dsd/leds.txt b/Documentation/acpi/dsd/leds.txt > new file mode