cron job: media_tree daily build: ABI WARNING

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 Sep 13 04:00:20 CEST 2016 git branch: test git hash: c3b809834db8b1a8891c7ff873a216eac119628d gcc

Re: [PATCH 2/2] [media] atmel-isc: mark PM functions as __maybe_unused

Hi Arnd, Thank you for your patch. I think it's better to add switch CONFIG_PM, but the PM feature is a must, or the ISC can not work, maybe the best choice is to add 'depends on PM' in Kconfig. #ifdef CONFIG_PM isc_runtime_suspend { XXX } isc_runtime_resume { XXX } static

Re: [PATCH 0/2] [media] tw5864 constify some structures

On Tue, Sep 13, 2016 at 02:02:36AM +0300, Andrey Utkin wrote: > tw5864 is a recently submitted driver and it is currently present only > in media tree. Oops, have just fetched linux-next updates, tw5864 is already in next. -- To unsubscribe from this list: send the line "unsubscribe linux-media"

[PATCH 1/2] [media] tw5864: constify vb2_ops structure

Inspired by "[media] pci: constify vb2_ops structures" patch from Julia Lawall (dated 9 Sep 2016). Signed-off-by: Andrey Utkin --- drivers/media/pci/tw5864/tw5864-video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/2] [media] tw5864: constify struct video_device template

tw5864_video_template is used for filling of actual video_device structures. It is copied by value, and is not used for anything else. Signed-off-by: Andrey Utkin --- drivers/media/pci/tw5864/tw5864-video.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 0/2] [media] tw5864 constify some structures

tw5864 is a recently submitted driver and it is currently present only in media tree. Recent patches submitted by Julia Lawall urged me to make similar changes in this driver. Andrey Utkin (2): [media] tw5864: constify vb2_ops structure [media] tw5864: constify struct video_device template

Re: [PATCH 00/26] constify local structures

On Mon, 12 Sep 2016, Jarkko Sakkinen wrote: > On Mon, Sep 12, 2016 at 04:43:58PM +0300, Felipe Balbi wrote: > > > > Hi, > > > > Jarkko Sakkinen writes: > > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > >> > > >> > > >> On Sun, 11 Sep 2016,

Re: [PATCH 00/26] constify local structures

On Mon, Sep 12, 2016 at 04:43:58PM +0300, Felipe Balbi wrote: > > Hi, > > Jarkko Sakkinen writes: > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > >> > >> > >> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > >> > >> > On Sun, Sep 11, 2016 at

Re: musb: isoc pkt loss with pwc

2016-09-12 21:57 GMT+03:00 Bin Liu : > Hi, > > On Mon, Sep 12, 2016 at 11:52:46AM +0300, Matwey V. Kornilov wrote: >> 2016-09-12 6:28 GMT+03:00 Bin Liu : >> > Hi, >> > >> > On Tue, Aug 30, 2016 at 11:44:33PM +0300, Matwey V. Kornilov wrote: >> >> 2016-08-30 21:30

Re: musb: isoc pkt loss with pwc

Hi, On Mon, Sep 12, 2016 at 11:52:46AM +0300, Matwey V. Kornilov wrote: > 2016-09-12 6:28 GMT+03:00 Bin Liu : > > Hi, > > > > On Tue, Aug 30, 2016 at 11:44:33PM +0300, Matwey V. Kornilov wrote: > >> 2016-08-30 21:30 GMT+03:00 Bin Liu : > >> > Hi, > >> > > >> > On Sun,

Re: [PATCH 00/26] constify local structures

On Mon, Sep 12, 2016 at 03:52:08PM +0200, Julia Lawall wrote: > > > On Mon, 12 Sep 2016, Felipe Balbi wrote: > > > > > Hi, > > > > Jarkko Sakkinen writes: > > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > >> > > >> > > >> On Sun, 11 Sep

Re: [PATCH 1/2] [media] ad5820: use __maybe_unused for PM functions

On Mon 2016-09-12 17:32:57, Arnd Bergmann wrote: > The new ad5820 driver uses #ifdef to hide the suspend/resume functions, > but gets it wrong when CONFIG_PM_SLEEP is disabled: > > drivers/media/i2c/ad5820.c:286:12: error: 'ad5820_resume' defined but not > used [-Werror=unused-function] >

[PATCH v1 2/2] st-hva: add debug file system

This patch creates 4 static debugfs entries to dump: - the device-related information ("st-hva/device") - the list of registered encoders ("st-hva/encoders") - the current values of the hva registers ("st-hva/regs") - the information about the last closed instance ("st-hva/last") It also creates

[PATCH v1 0/2] add debug capabilities to v4l2 encoder for STMicroelectronics SOC

version 1: - Initial submission With the first patch, a short summary about the encoding operation is unconditionnaly printed at each instance closing: - information about the stream (format, profile, level, resolution) - performance information (number of encoded frames, maximum framerate) -

[PATCH v1 1/2] st-hva: encoding summary at instance release

This patch prints unconditionnaly a short summary about the encoding operation at each instance closing, for debug purpose: - information about the stream (format, profile, level, resolution) - performance information (number of encoded frames, maximum framerate) - potential (system, encoding...)

Re: [PATCH v2] [media] vivid: support for contiguous DMA buffers

Hello Vincent, On Mon, Sep 12, 2016 at 4:47 AM, Vincent Abriou wrote: > It allows to simulate the behavior of hardware with such limitations or > to connect vivid to real hardware with such limitations. > > Add the "allocators" module parameter option to let vivid use the

[PATCH 2/2] [media] Input: synaptics-rmi4: disallow impossible configuration

The newly added debug mode for the synaptics-rmi4 driver relies on the v4l2 interface and vb2_vmalloc, but those might be configured as loadable modules when the driver itself is built-in, resulting in a link failure: drivers/input/rmi4/rmi_core.o: In function `rmi_f54_remove':

[PATCH] usb: gadget: uvc: add V4L2 dependency

Building the UVC gadget into the kernel fails to build when CONFIG_VIDEO_V4L2 is a loadable module: drivers/usb/gadget/function/usb_f_uvc.o: In function `uvc_function_ep0_complete': uvc_configfs.c:(.text.uvc_function_ep0_complete+0x84): undefined reference to `v4l2_event_queue'

[PATCH 1/2] [media] Input: atmel_mxt: disallow impossible configuration

The newnly added debug mode for the atmel_mxt_ts driver relies on the v4l2 interface and vb2_vmalloc, but those might be configured as loadable modules when the driver itself is built-in, resulting in a link failure: drivers/input/touchscreen/atmel_mxt_ts.o: In function `mxt_vidioc_querycap':

[PATCH 1/2] [media] ad5820: use __maybe_unused for PM functions

The new ad5820 driver uses #ifdef to hide the suspend/resume functions, but gets it wrong when CONFIG_PM_SLEEP is disabled: drivers/media/i2c/ad5820.c:286:12: error: 'ad5820_resume' defined but not used [-Werror=unused-function] drivers/media/i2c/ad5820.c:274:12: error: 'ad5820_suspend' defined

[PATCH 2/2] [media] atmel-isc: mark PM functions as __maybe_unused

The newly added atmel-isc driver uses SET_RUNTIME_PM_OPS() to refer to its suspend/resume functions, causing a warning when CONFIG_PM is not set: media/platform/atmel/atmel-isc.c:1477:12: error: 'isc_runtime_resume' defined but not used [-Werror=unused-function]

Re: [PATCH 00/26] constify local structures

On Mon, Sep 12, 2016 at 3:43 PM, Felipe Balbi wrote: > Jarkko Sakkinen writes: >> On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: >>> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: >>> > On Sun, Sep 11, 2016 at 03:05:42PM

Re: [PATCH 00/26] constify local structures

On Mon, 12 Sep 2016, Felipe Balbi wrote: > > Hi, > > Jarkko Sakkinen writes: > > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > >> > >> > >> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > >> > >> > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia

Re: [PATCH 00/26] constify local structures

Hi, Jarkko Sakkinen writes: > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: >> >> >> On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: >> >> > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: >> > > Constify local structures. >> > > >>

Re: [PATCH v2 6/7] Documentation: bindings: add documentation for ir-spi device driver

On Fri, Sep 02, 2016 at 02:33:20PM +0900, Andi Shyti wrote: > Hi Rob, > > > > Document the ir-spi driver's binding which is a IR led driven > > > through the SPI line. > > > > > > Signed-off-by: Andi Shyti > > > --- > > >

Re: [PATCH 00/26] constify local structures

On Mon, 12 Sep 2016, Jarkko Sakkinen wrote: > On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > > > > > On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > > > > > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: > > > > Constify local structures. > > > > > > > > The

Re: [PATCH 00/26] constify local structures

On Mon, Sep 12, 2016 at 10:54:07AM +0200, Julia Lawall wrote: > > > On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > > > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: > > > Constify local structures. > > > > > > The semantic patch that makes this change is as follows: > > >

[PATCH] [media] MAINTAINERS: add vimc entry

Signed-off-by: Helen Koike --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 0a16a82..43e0eb4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -12540,6 +12540,14 @@ W: https://linuxtv.org S: Maintained F:

Re: [PATCH V2] [media] staging/media/cec: fix coding style error

On Sun, Sep 11, 2016 at 08:42:21PM +0300, Antti Palosaari wrote: > On 09/11/2016 07:07 PM, Richard wrote: > > Greetings Linux Kernel Developers, > > > > This is Task 10 of the Eudyptula Challenge, i fix few line over 80 > > characters, hope you will accept this pacth. > > > > /Richard > > > > For

Re: [PATCH 00/26] constify local structures

On Sun, 11 Sep 2016, Jarkko Sakkinen wrote: > On Sun, Sep 11, 2016 at 03:05:42PM +0200, Julia Lawall wrote: > > Constify local structures. > > > > The semantic patch that makes this change is as follows: > > (http://coccinelle.lip6.fr/) > > Just my two cents but: > > 1. You *can* use a static

Re: musb: isoc pkt loss with pwc

2016-09-12 6:28 GMT+03:00 Bin Liu : > Hi, > > On Tue, Aug 30, 2016 at 11:44:33PM +0300, Matwey V. Kornilov wrote: >> 2016-08-30 21:30 GMT+03:00 Bin Liu : >> > Hi, >> > >> > On Sun, Aug 28, 2016 at 01:13:55PM +0300, Matwey V. Kornilov wrote: >> >> Hello Bin, >> >> >> >>

[PATCH v2] [media] vivid: support for contiguous DMA buffers

It allows to simulate the behavior of hardware with such limitations or to connect vivid to real hardware with such limitations. Add the "allocators" module parameter option to let vivid use the dma-contig instead of vmalloc. Signed-off-by: Philipp Zabel Signed-off-by:

[PATCH] [media] pulse8-cec: avoid uninitialized data use

Building with -Wmaybe-uninitialized reveals the use on an uninitialized variable containing the physical address of the device whenever firmware before version 2 is used: drivers/staging/media/pulse8-cec/pulse8-cec.c: In function 'pulse8_connect':

[PATCH] [media] atmel-isc: set the format on the first open

Set the current format on the first open. Signed-off-by: Songjun Wu --- drivers/media/platform/atmel/atmel-isc.c | 30 ++ 1 file changed, 18 insertions(+), 12 deletions(-) diff --git a/drivers/media/platform/atmel/atmel-isc.c

[GIT PULL FOR v4.9] Fix two compilation issues

Hi Mauro, These two patches fix compilation issues for 4.9. One simple warning and one fixing duplicate functions. Regards, Hans The following changes since commit 8a5a2ba86ab8fc12267fea974b9cd730ad2dee24: [media] v4l: vsp1: Add R8A7792 VSP1V support (2016-09-09 11:32:43 -0300) are