Re: [PATCH v6 09/17] media: rkisp1: add rockchip isp1 core driver

2018-03-10 Thread Baruch Siach
Hi Jacob, On Thu, Mar 08, 2018 at 05:47:59PM +0800, Jacob Chen wrote: > +config VIDEO_ROCKCHIP_ISP1 > + tristate "Rockchip Image Signal Processing v1 Unit driver" > + depends on VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API > + depends on ARCH_ROCKCHIP || COMPILE_TEST > + select VIDEOBUF2_DM

cron job: media_tree daily build: ERRORS

2018-03-10 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 Mar 11 05:00:11 CET 2018 media-tree git hash:e68854a2588a923b31eebce348f8020374843f8e media_build git

htmldocs: include/net/cfg80211.h:4115: warning: Function parameter or member 'wext.bssid' not described in 'wireless_dev'

2018-03-10 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 3266b5bd97eaa72793df0b6e5a106c69ccc166c4 commit: 84ce5b987783d362ee4e737b653d6e2feacfa40c scripts: kernel-doc: improve nested logic to handle multiple identifiers date: 3 months ago reproduce: make htmldoc

[PATCH] media: ivtv: add parameter to enable ivtvfb on x86 PAT systems

2018-03-10 Thread Nick French
ivtvfb was previously disabled for x86 PAT-enabled systems by commit 1bf1735b4780 ("x86/mm/pat, drivers/media/ivtv: Use arch_phys_wc_add() and require PAT disabled") as a workaround to abstract MTRR code away from device drivers. The driver is not easily upgradable to the PAT-aware ioremap_wc() AP

drivers/media/dvb-frontends/stb0899_drv.h:151:36: error: weak declaration of 'stb0899_attach' being applied to a already existing, static definition

2018-03-10 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 3266b5bd97eaa72793df0b6e5a106c69ccc166c4 commit: 6cdeaed3b1420bd2569891be0c4123ff59628e9e media: dvb_usb_pctv452e: module refcount changes were unbalanced date: 3 months ago config: x86_64-randconfig-a0-03

htmldocs: include/linux/crypto.h:469: warning: Function parameter or member 'cra_u.ablkcipher' not described in 'crypto_alg'

2018-03-10 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 3266b5bd97eaa72793df0b6e5a106c69ccc166c4 commit: 151c468b44a89a9f3173ab8575690014b7249893 scripts: kernel-doc: print the declaration name on warnings date: 3 months ago reproduce: make htmldocs All warnin

[PATCH v2 11/13] media: staging/imx: Rename root notifier

2018-03-10 Thread Steve Longerbeam
Rename the imx-media root async notifier from "subdev_notifier" to simply "notifier", so as not to confuse it with true subdev notifiers. No functional changes. Signed-off-by: Steve Longerbeam --- drivers/staging/media/imx/imx-media-dev.c | 14 +++--- drivers/staging/media/imx/imx-media.

[PATCH v2 12/13] media: staging/imx: Switch to v4l2_async_notifier_add_subdev

2018-03-10 Thread Steve Longerbeam
Switch to v4l2_async_notifier_add_subdev() when adding async subdevs to the imx-media root notifier. This removes the need to check for an already added asd, since v4l2_async_notifier_add_subdev() does this check. Also no need to allocate a subdevs array when registering the root notifier, or keepi

[PATCH v2 13/13] media: staging/imx: TODO: Remove one assumption about OF graph parsing

2018-03-10 Thread Steve Longerbeam
The move to subdev notifiers fixes one assumption of OF graph parsing. If a subdevice has non-video related ports, the subdev driver knows not to follow those ports when adding remote devices to its subdev notifier. Signed-off-by: Steve Longerbeam --- drivers/staging/media/imx/TODO | 29 +++-

[PATCH v2 10/13] media: staging/imx: Loop through all registered subdevs for media links

2018-03-10 Thread Steve Longerbeam
The root imx-media notifier no longer sees all bound subdevices because some of them will be bound to subdev notifiers. So imx_media_create_links() now needs to loop through all subdevices registered with the v4l2-device, not just the ones in the root notifier's done list. This should be safe becau

[PATCH v2 09/13] media: staging/imx: of: Remove recursive graph walk

2018-03-10 Thread Steve Longerbeam
After moving to subdev notifiers, it's no longer necessary to recursively walk the OF graph, because the subdev notifiers will discover and add devices from the graph for us. So the recursive of_parse_subdev() function is gone, replaced with of_add_csi() which adds only the CSI port fwnodes to the

[PATCH v2 08/13] media: imx: mipi csi-2: Register a subdev notifier

2018-03-10 Thread Steve Longerbeam
Parse neighbor remote devices on the MIPI CSI-2 input port, add them to a subdev notifier, and register the subdev notifier for the MIPI CSI-2 receiver, by calling v4l2_async_register_fwnode_subdev(). csi2_parse_endpoints() is modified to be the parse_endpoint callback. Signed-off-by: Steve Longe

[PATCH v2 07/13] media: imx: csi: Register a subdev notifier

2018-03-10 Thread Steve Longerbeam
Parse neighbor remote devices on the CSI port, add them to a subdev notifier, and register the subdev notifier for the CSI, by calling v4l2_async_register_fwnode_subdev(). Signed-off-by: Steve Longerbeam --- drivers/staging/media/imx/imx-media-csi.c | 11 ++- 1 file changed, 10 insertion

[PATCH v2 06/13] media: platform: video-mux: Register a subdev notifier

2018-03-10 Thread Steve Longerbeam
Parse neighbor remote devices on the video muxes input ports, add them to a subdev notifier, and register the subdev notifier for the video mux, by calling v4l2_async_register_fwnode_subdev(). Signed-off-by: Steve Longerbeam --- Changes since v1: - add #include for kcalloc() declaration. drive

[PATCH v2 05/13] media: v4l2-fwnode: Add a convenience function for registering subdevs with notifiers

2018-03-10 Thread Steve Longerbeam
Adds v4l2_async_register_fwnode_subdev(), which is a convenience function for parsing a sub-device's fwnode port endpoints for connected remote sub-devices, registering a sub-device notifier, and then registering the sub-device itself. Signed-off-by: Steve Longerbeam --- Changes since v1: - add #

[PATCH v2 04/13] media: v4l2-fwnode: Switch to v4l2_async_notifier_add_subdev

2018-03-10 Thread Steve Longerbeam
The fwnode endpoint and reference parsing functions in v4l2-fwnode.c are modified to make use of v4l2_async_notifier_add_subdev(). As a result the notifier->subdevs array is no longer allocated or re-allocated, and by extension the max_subdevs value is also no longer needed. Since the notifier->su

[PATCH v2 03/13] media: v4l2: async: Add v4l2_async_notifier_add_subdev

2018-03-10 Thread Steve Longerbeam
v4l2_async_notifier_add_subdev() adds an asd to the notifier. It checks that the asd's match_type is valid and that no other equivalent asd's have already been added to this notifier's asd list, or to other registered notifier's waiting or done lists, and increments num_subdevs. v4l2_async_notifie

[PATCH v2 00/13] media: imx: Switch to subdev notifiers

2018-03-10 Thread Steve Longerbeam
This patchset converts the imx-media driver and its dependent subdevs to use subdev notifiers. There are a couple shortcomings in v4l2-core that prevented subdev notifiers from working correctly in imx-media: 1. v4l2_async_notifier_fwnode_parse_endpoint() treats a fwnode endpoint that is not c

[PATCH v2 02/13] media: v4l2: async: Allow searching for asd of any type

2018-03-10 Thread Steve Longerbeam
Generalize v4l2_async_notifier_fwnode_has_async_subdev() to allow searching for any type of async subdev, not just fwnodes. Rename to v4l2_async_notifier_has_async_subdev() and pass it an asd pointer. TODO: support asd compare with CUSTOM match type in asd_equal(). Signed-off-by: Steve Longerbeam

[PATCH v2 01/13] media: v4l2-fwnode: ignore endpoints that have no remote port parent

2018-03-10 Thread Steve Longerbeam
Documentation/devicetree/bindings/media/video-interfaces.txt states that the 'remote-endpoint' property is optional. So v4l2_async_notifier_fwnode_parse_endpoint() should not return error if the endpoint has no remote port parent. Just ignore the endpoint, skip adding an asd to the notifier and re

Re: [PATCH] media: soc_camera: mt9t112: Update to new interface

2018-03-10 Thread Hans Verkuil
If possible, post a v2. It avoids mistakes. Hans On March 10, 2018 7:35:31 PM GMT+01:00, Jacopo Mondi wrote: >Use in the soc_camera version of mt9t112 driver the new name for the >driver's platform data as defined by the new v4l2 driver for the same >chip. > >Signed-off-by: Jacopo Mondi > >---

Re: ivtv: use arch_phys_wc_add() and require PAT disabled

2018-03-10 Thread Luis R. Rodriguez
On Sat, Mar 10, 2018 at 11:03 AM, Luis R. Rodriguez wrote: > On Sat, Mar 10, 2018 at 8:57 AM, French, Nicholas A. wrote: >> On Wed, Mar 07, 2018 at 11:23:09PM -0600, French, Nicholas A. wrote: >>> On Thu, Mar 08, 2018 at 04:14:11AM +, Luis R. Rodriguez wrote: >>> > On Thu, Mar 08, 2018 at 04:

Re: ivtv: use arch_phys_wc_add() and require PAT disabled

2018-03-10 Thread Luis R. Rodriguez
On Sat, Mar 10, 2018 at 8:57 AM, French, Nicholas A. wrote: > On Wed, Mar 07, 2018 at 11:23:09PM -0600, French, Nicholas A. wrote: >> On Thu, Mar 08, 2018 at 04:14:11AM +, Luis R. Rodriguez wrote: >> > On Thu, Mar 08, 2018 at 04:06:01AM +, Luis R. Rodriguez wrote: >> > > On Thu, Mar 08, 20

Re: ivtv: use arch_phys_wc_add() and require PAT disabled

2018-03-10 Thread Andy Lutomirski
> On Mar 10, 2018, at 8:57 AM, French, Nicholas A. wrote: > >> On Wed, Mar 07, 2018 at 11:23:09PM -0600, French, Nicholas A. wrote: >>> On Thu, Mar 08, 2018 at 04:14:11AM +, Luis R. Rodriguez wrote: On Thu, Mar 08, 2018 at 04:06:01AM +, Luis R. Rodriguez wrote: > On Thu, Mar 0

Re: [PATCH 0/5] Renesas CEU: SH7724 ECOVEC + Aptina mt9t112

2018-03-10 Thread jacopo mondi
Hi Hans, On Sat, Mar 10, 2018 at 06:57:17PM +0100, Hans Verkuil wrote: > Hi Jacopo, > > On 02/03/18 17:35, Jacopo Mondi wrote: > > Hello, > >now that CEU has been picked up for inclusion in v4.17, we can start > > moving > > users of old sh_mobile_ceu_camera driver to use the newly introduced

[PATCH] media: soc_camera: mt9t112: Update to new interface

2018-03-10 Thread Jacopo Mondi
Use in the soc_camera version of mt9t112 driver the new name for the driver's platform data as defined by the new v4l2 driver for the same chip. Signed-off-by: Jacopo Mondi --- Hans: to not break bisect, would you like me to resend the whole series with this commit squashed in: [PATCH 2/5] medi

Re: [PATCH 0/5] Renesas CEU: SH7724 ECOVEC + Aptina mt9t112

2018-03-10 Thread Hans Verkuil
Hi Jacopo, On 02/03/18 17:35, Jacopo Mondi wrote: > Hello, >now that CEU has been picked up for inclusion in v4.17, we can start moving > users of old sh_mobile_ceu_camera driver to use the newly introduced one. > > Migo-R has been first, now it's SH7724 ECOVEC board turn. > > ECOVEC has a c

drivers/media/usb/dvb-usb/pctv452e.c:1102:1: note: in expansion of macro 'MODULE_LICENSE'

2018-03-10 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: cdb06e9d8f520c969676e7d6778cffe5894f079f commit: 6cdeaed3b1420bd2569891be0c4123ff59628e9e media: dvb_usb_pctv452e: module refcount changes were unbalanced date: 3 months ago config: i386-randconfig-sb0-031

Re: [PATCH] media: rc: meson-ir: add timeout on idle

2018-03-10 Thread Matthias Reichl
Hi Sean, On Sat, Mar 10, 2018 at 11:27:45AM +, Sean Young wrote: > Hi Matthias, > > On Fri, Mar 09, 2018 at 04:54:51PM +0100, Matthias Reichl wrote: > > On Thu, Mar 08, 2018 at 04:43:27PM +, Sean Young wrote: > > > On Tue, Mar 06, 2018 at 06:41:22PM +0100, Matthias Reichl wrote: > > > > +

[PATCH] media: i2c: mt9t112: Add TODO note for frame rate control

2018-03-10 Thread Jacopo Mondi
The driver lacks support for frame rate control, and v4l2-compliance complains about that. Add a TODO note to warn driver users that this is expected. While at there, update copyright note to the year we're actually in. Signed-off-by: Jacopo Mondi --- drivers/media/i2c/mt9t112.c | 6 +- 1 f

Re: ivtv: use arch_phys_wc_add() and require PAT disabled

2018-03-10 Thread French, Nicholas A.
On Wed, Mar 07, 2018 at 11:23:09PM -0600, French, Nicholas A. wrote: > On Thu, Mar 08, 2018 at 04:14:11AM +, Luis R. Rodriguez wrote: > > On Thu, Mar 08, 2018 at 04:06:01AM +, Luis R. Rodriguez wrote: > > > On Thu, Mar 08, 2018 at 03:16:29AM +, French, Nicholas A. wrote: > > > > > > > >

Re: [PATCH v3 2/2] media: imx-media-csi: Do not propagate the error when pinctrl is not found

2018-03-10 Thread Fabio Estevam
Hi, On Sat, Mar 3, 2018 at 9:56 AM, Fabio Estevam wrote: > From: Fabio Estevam > > Since commit 52e17089d185 ("media: imx: Don't initialize vars that > won't be used") imx_csi_probe() fails to probe after propagating the > devm_pinctrl_get_select_default() error. > > devm_pinctrl_get_select_defa

[PATCH] [media] ttpci: improve printing of encoded MAC address

2018-03-10 Thread Daniel Scheller
From: Daniel Scheller When loading the budget_av driver for ie. a KNC1 DVB-C TDA10024 card, which makes use of the ttpci eeprom check functionality (that always fails on these cards, but that's no issue at all), this is printed to the kernel log: [ 10.497333] saa7146 (0): dma buffer size 192

Re: [PATCH] media: rc: meson-ir: add timeout on idle

2018-03-10 Thread Sean Young
Hi Matthias, On Fri, Mar 09, 2018 at 04:54:51PM +0100, Matthias Reichl wrote: > On Thu, Mar 08, 2018 at 04:43:27PM +, Sean Young wrote: > > On Tue, Mar 06, 2018 at 06:41:22PM +0100, Matthias Reichl wrote: > > > Meson doesn't seem to be able to generate timeout events > > > in hardware. So inst

Re: [PATCH 09/11] media: vsp1: Provide support for extended command pools

2018-03-10 Thread Kieran Bingham
On 09/03/18 22:04, Kieran Bingham wrote: > VSPD and VSP-DL devices can provide extended display lists supporting > extended command display list objects. > > These extended commands require their own dma memory areas for a header > and body specific to the command type. > > Implement a command po