cron job: media_tree daily build: ERRORS

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 Aug 15 05:00:15 CEST 2017 media-tree git hash:ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0 media_build

Re: [PATCH] [media] staging/imx: always select VIDEOBUF2_DMA_CONTIG

Thanks, Reviewed-by: Steve Longerbeam Steve On 08/07/2017 03:49 AM, Arnd Bergmann wrote: I ran into a rare build error during randconfig testing: drivers/staging/media/imx/imx-media-capture.o: In function `capture_stop_streaming':

Re: [PATCH 6/6] [media] media: imx: capture: constify vb2_ops structures

Thanks, Reviewed-by: Steve Longerbeam Steve On 08/05/2017 03:47 AM, Julia Lawall wrote: These vb2_ops structures are only stored in the ops field of a vb2_queue structure, which is declared as const. Thus the vb2_ops structures themselves can be const. Done

Re: [PATCH] [media] media: imx: use setup_timer

Thanks, Reviewed-by: Steve Longerbeam Tested-by: Steve Longerbeam Steve On 08/13/2017 11:39 AM, Cihangir Akturk wrote: Use setup_timer function instead of initializing timer with the function and data fields. Generated by:

Unknown symbol errors with latest RH7 kernel using meda_build

I'm new to mailing lists, so I'm sorry if this message is poorly formatted. I have been compiling v4l on RH7 using very minor changes (made it into an rpm, have to remove 'v3.16_wait_on_bit.patch' from the backports) for a while now. I had been using an older set of firmware and git hash from the

Re: [PATCH 0/2] More s5p-jpeg fixes

Hi Andrzej, Thanks for the patches. On 08/11/2017 01:49 PM, Andrzej Pietrasiewicz wrote: > Hi All, > > The first patch in the series fixes decoding path, > the second patch fixes encoding path. Please see > appropriate commit messages. > > Andrzej Pietrasiewicz (2): > media: s5p-jpeg: don't

Re: [PATCH] [media] media: imx: depends on V4L2 sub-device userspace API

On Mon, Aug 14, 2017 at 10:57:46AM -0700, Steve Longerbeam wrote: > Hi Akturk, this has already been fixed, see Sorry. Apparently, I missed to update my tree, somehow. > > 4560cb4a0c ("media: imx: add VIDEO_V4L2_SUBDEV_API dependency"). > > Steve > > On 08/14/2017 04:00 AM, Cihangir Akturk

Re: [PATCH 0/5] s5p-jpeg fixes

Hi Andrzej, Thanks for the patch set. On 08/08/2017 01:22 PM, Andrzej Pietrasiewicz wrote: > Hi All, > > This series contains a number of fixes to the s5p-jpeg driver. > > There are two patches from Tony K Nadackal, which got lost long time ago. > > @Thierry: > The patch changing the software

Re: [PATCH] v4l2-compat-ioctl32.c: make ctrl_is_pointer generic

Hi Hans, On Fri, Aug 11, 2017 at 03:26:03PM +0200, Hans Verkuil wrote: > The ctrl_is_pointer used a hard-coded list of control IDs that besides being > outdated also wouldn't work for custom driver controls. > > Replaced by calling queryctrl and checking if the V4L2_CTRL_FLAG_HAS_PAYLOAD > flag

[PATCH v1.3 1/1] omap3isp: Skip CSI-2 receiver initialisation in CCP2 configuration

If the CSI-2 receiver isn't part of the pipeline (or isn't there to begin with), skip its initialisation. Signed-off-by: Sakari Ailus Tested-by: Laurent Pinchart # on Beagleboard-xM + MPT9P031 Acked-by: Pavel Machek

Re: [PATCH v1.2 1/1] omap3isp: Skip CSI-2 receiver initialisation in CCP2 configuration

On Mon, Aug 14, 2017 at 04:33:39PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > On Monday 14 Aug 2017 13:53:27 Sakari Ailus wrote: > > On Fri, Aug 11, 2017 at 02:32:00PM +0300, Laurent Pinchart wrote: > > > On Friday 11 Aug 2017 12:57:09 Sakari Ailus wrote: > > >> If the CSI-2 receiver isn't

Re: [PATCH] [media] media: imx: depends on V4L2 sub-device userspace API

Hi Akturk, this has already been fixed, see 4560cb4a0c ("media: imx: add VIDEO_V4L2_SUBDEV_API dependency"). Steve On 08/14/2017 04:00 AM, Cihangir Akturk wrote: This driver uses various v4l2_subdev_get_try_*() functions provided by V4L2 sub-device userspace API. Current configuration of

[PATCH v2] media: ddbridge: constify stv0910_p and lnbh25_cfg

These structures are only copied into other structures, so they can be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- v2: fix typo in the commit message drivers/media/pci/ddbridge/ddbridge-core.c |4 ++-- 1 file changed, 2 insertions(+), 2

[PATCH] [media] dvb-frontends/stv0367: remove QAM_AUTO from ddb_fe_ops

From: Daniel Scheller Since the cab_* codepath doesn't recognize QAM_AUTO, don't announce that it is supported when it really isn't. Fixes ie. w_scan from unconditionally using QAM_AUTO on DVB-C scans. Signed-off-by: Daniel Scheller ---

[PATCH] media: ddbridge: constify stv0910_p and lnbh25_cfg

These structures are only copied into other stuructures, so they can be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/pci/ddbridge/ddbridge-core.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [PATCH 3/4] arm: dts: renesas: add cec clock for Koelsch board

On Sun, Jul 30, 2017 at 3:07 PM, Hans Verkuil wrote: > From: Hans Verkuil Probably the one-line summary should be ARM: dts: koelsch: Add CEC clock for HDMI transmitter > The adv7511 on the Koelsch board has a 12 MHz fixed clock > for the CEC

[PATCH v2 2/8] v4l: vsp1: Provide a fragment pool

Each display list allocates a body to store register values in a dma accessible buffer from a dma_alloc_wc() allocation. Each of these results in an entry in the TLB, and a large number of display list allocations adds pressure to this resource. Reduce TLB pressure on the IPMMUs by allocating

[PATCH v2 8/8] v4l: vsp1: Reduce display list body size

The display list originally allocated a body of 256 entries to store all of the register lists required for each frame. This has now been separated into fragments for constant stream setup, and runtime updates. Empirical testing shows that the body0 now uses a maximum of 41 registers for each

[PATCH v2 5/8] v4l: vsp1: Refactor display list configure operations

The entities provide a single .configure operation which configures the object into the target display list, based on the vsp1_entity_params selection. This restricts us to a single function prototype for both static configuration (the pre-stream INIT stage) and the dynamic runtime stages for

[PATCH v2 4/8] v4l: vsp1: Use reference counting for fragments

Extend the display list body with a reference count, allowing bodies to be kept as long as a reference is maintained. This provides the ability to keep a cached copy of bodies which will not change, so that they can be re-applied to multiple display lists. Signed-off-by: Kieran Bingham

[PATCH v2 7/8] v4l: vsp1: Move video configuration to a cached dlb

We are now able to configure a pipeline directly into a local display list body. Take advantage of this fact, and create a cacheable body to store the configuration of the pipeline in the video object. vsp1_video_pipeline_run() is now the last user of the pipe->dl object. Convert this function to

[PATCH v2 1/8] v4l: vsp1: Protect fragments against overflow

The fragment write function relies on the code never asking it to write more than the entries available in the list. Currently with each list body containing 256 entries, this is fine, but we can reduce this number greatly saving memory. In preparation of this - add a level of protection to

[PATCH v2 6/8] v4l: vsp1: Adapt entities to configure into a body

Currently the entities store their configurations into a display list. Adapt this such that the code can be configured into a body fragment directly, allowing greater flexibility and control of the content. All users of vsp1_dl_list_write() are removed in this process, thus it too is removed. A

[PATCH v2 0/8] vsp1: TLB optimisation and DL caching

Each display list currently allocates an area of DMA memory to store register settings for the VSP1 to process. Each of these allocations adds pressure to the IPMMU TLB entries. We can reduce the pressure by pre-allocating larger areas and dividing the area across multiple bodies represented as a

[PATCH v2 3/8] v4l: vsp1: Convert display lists to use new fragment pool

Adapt the dl->body0 object to use an object from the fragment pool. This greatly reduces the pressure on the TLB for IPMMU use cases, as all of the lists use a single allocation for the main body. The CLU and LUT objects pre-allocate a pool containing two bodies, allowing a userspace update

[GIT PULL FOR v4.14] More constify, uvc and some random patches

The following changes since commit ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0: media: ddbridge: split code into multiple files (2017-08-09 12:17:01 -0400) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v4.14h for you to fetch changes up to

Re: [PATCH v1.2 1/1] omap3isp: Skip CSI-2 receiver initialisation in CCP2 configuration

Hi Sakari, On Monday 14 Aug 2017 13:53:27 Sakari Ailus wrote: > On Fri, Aug 11, 2017 at 02:32:00PM +0300, Laurent Pinchart wrote: > > On Friday 11 Aug 2017 12:57:09 Sakari Ailus wrote: > >> If the CSI-2 receiver isn't part of the pipeline (or isn't there to > >> begin with), skip its

[GIT PULL FOR v4.14] constify patches

The following changes since commit ec0c3ec497cabbf3bfa03a9eb5edcc252190a4e0: media: ddbridge: split code into multiple files (2017-08-09 12:17:01 -0400) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v4.14g for you to fetch changes up to

si2168 b40 and lack of auto plp

Hello a few questions is there any card out there with a si2168 b40 in a country that uses plp!=0 that dvb-t2 actually works for? i suspect the answer to this is no and that it is broken on all cards. auto plp appears broken anyway on my ct2-4650 with si2168 b40 and has been broken for a

[PATCH] [media] media: imx: depends on V4L2 sub-device userspace API

This driver uses various v4l2_subdev_get_try_*() functions provided by V4L2 sub-device userspace API. Current configuration of Kconfig file allows us to enable VIDEO_IMX_MEDIA without enabling this API. This breaks the build of driver. Depend on VIDEO_V4L2_SUBDEV_API to fix this issue.

Re: [PATCH v1.2 1/1] omap3isp: Skip CSI-2 receiver initialisation in CCP2 configuration

On Fri, Aug 11, 2017 at 02:32:00PM +0300, Laurent Pinchart wrote: > > + if (!pipe) > > + return -EBUSY; > > When can this happen ? And this test should be for phy->entity instead. -- Sakari Ailus e-mail: sakari.ai...@iki.fi

Re: [PATCH v1.2 1/1] omap3isp: Skip CSI-2 receiver initialisation in CCP2 configuration

Hi Laurent, On Fri, Aug 11, 2017 at 02:32:00PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. > > On Friday 11 Aug 2017 12:57:09 Sakari Ailus wrote: > > If the CSI-2 receiver isn't part of the pipeline (or isn't there to begin > > with), skip its initialisation. > > I

[PATCH 3/3] v4l: fwnode: Use a less clash-prone name for MAX_DATA_LANES macro

Avoid using a generic name such as MAX_DATA_LANES in a header file widely included in drivers. Instead, call it V4L2_FWNODE_CSI2_MAX_DATA_LANES. Fixes: 4ee236219f6d ("media: v4l2-fwnode: suppress a warning at OF parsing logic") Signed-off-by: Sakari Ailus ---

[PATCH 2/2] v4l: fwnode: The clock lane is the first lane in lane_polarities

The clock lane is the first lane in the lane_polarities array. Reflect this consistently by putting the number of data lanes after the number of clock lanes. Fixes: 4ee236219f6d ("media: v4l2-fwnode: suppress a warning at OF parsing logic") Signed-off-by: Sakari Ailus

[PATCH 0/2] [PATCH 0/2] Bugfix for the fwnode smatch warning fix

Hi folks, Here's a few patches to address a subtle but serious bug introduced in the fwnode smatch warning fix. The first patch fixes a bug whereas the second one returns the way the lane polarity array size is calculatated consistent across V4L2 fwnode. Sakari Ailus (2): v4l: fwnode: Fix

[PATCH 1/2] v4l: fwnode: Fix lane-polarities property parsing

fwnode_property_read_u32_array() only returns the number of array elements if the array argument is NULL. The assumption that it always did so lead to lane-polarities properties never being read. Fixes: 4ee236219f6d ("media: v4l2-fwnode: suppress a warning at OF parsing logic") Signed-off-by:

[RFC PATCH] media: vb2: add bidirectional flag in vb2_queue

Hi, This RFC patch is intended to give to the drivers a choice to change the default behavior of the v4l2-core DMA mapping direction from DMA_TO/FROM_DEVICE (depending on the buffer type CAPTURE or OUTPUT) to DMA_BIDIRECTIONAL during queue_init time. Initially the issue with DMA mapping

Re: [PATCH] [media] media: imx: use setup_timer

On Sun, 2017-08-13 at 21:39 +0300, Cihangir Akturk wrote: > Use setup_timer function instead of initializing timer with the > function and data fields. > > Generated by: scripts/coccinelle/api/setup_timer.cocci. > > Signed-off-by: Cihangir Akturk > --- >  

[PATCH] [media] pxa_camera: constify v4l2_clk_ops structure

This v4l2_clk_ops structure is only passed as the first argument of v4l2_clk_register, which is const, so the v4l2_clk_ops structure can also be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/platform/pxa_camera.c |2 +- 1 file

[PATCH] [media] v4l2: av7110_v4l: constify v4l2_audio structure

This v4l2_audio structure is only copied into other structures, so it can be const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/media/pci/ttpci/av7110_v4l.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git