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: Fri Oct 28 05:00:15 CEST 2016 media-tree git hash:bd676c0c04ec94bd830b9192e2c33f2c4532278d media_build

[PATCH] [media] media: fix platform_no_drv_owner.cocci warnings

drivers/media/platform/mtk-mdp/mtk_mdp_core.c:284:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci CC: Minghsiu Tsai

[linux-next:master 1988/3208] drivers/media/platform/mtk-mdp/mtk_mdp_core.c:284:3-8: No need to set .owner here. The core will do it.

tree: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master head: 58308d9377de18c554d801cb6ff4e0a4750da921 commit: c8eb2d7e8202fd9cb912f5d33cc34ede66dcb24a [1988/3208] [media] media: Add Mediatek MDP Driver coccinelle warnings: (new ones prefixed by >>) >>

Re: Documentation/media/uapi/cec/ sporadically unnecessarily rebuilding

On 27.10.2016 16:52, Jani Nikula wrote: On Thu, 27 Oct 2016, Markus Heiser wrote: Hi Jani, Am 24.10.2016 um 11:04 schrieb Jani Nikula : I think I saw some of this in the past [1], but then couldn't reproduce it after all. Now I'm seeing it

[PATCH next 1/2] media: mtk-mdp: fix video_device_release argument

video_device_release() takes a pointer to struct video_device as argument. Fix two call sites where the address of the pointer is passed instead. Fixes: c8eb2d7e8202fd9c ("[media] media: Add Mediatek MDP Driver") Signed-off-by: Vincent Stehlé Cc: Minghsiu Tsai

[PATCH next 2/2] media: mtk-mdp: NULL-terminate mtk_mdp_comp_dt_ids

The mtk_mdp_comp_dt_ids[] array should be NULL-terminated; add therefore an empty entry in the end. Fixes: c8eb2d7e8202fd9c ("[media] media: Add Mediatek MDP Driver") Signed-off-by: Vincent Stehlé Cc: Minghsiu Tsai Cc: Hans Verkuil

[PATCH v2 3/3] stk1160: Add module param for setting the record gain.

Allow setting a custom record gain for the internal AC97 codec (if available). This can be a value between 0 and 15, 8 is the default and should be suitable for most users. The Windows driver also sets this to 8 without any possibility for changing it. Signed-off-by: Marcel Hasler

[PATCH v2 2/3] stk1160: Check whether to use AC97 codec or internal ADC.

Some STK1160-based devices use the chip's internal 8-bit ADC. This is configured through a strap pin. The value of this and other pins can be read through the POSVA register. If the internal ADC is used, there's no point trying to setup the unavailable AC97 codec. Signed-off-by: Marcel Hasler

[PATCH v2 1/3] stk1160: Remove stk1160-mixer and setup internal AC97 codec automatically.

Exposing all the channels of the device's internal AC97 codec to userspace is unnecessary and confusing. Instead the driver should setup the codec with proper values. This patch removes the mixer and sets up the codec using optimal values, i.e. the same values set by the Windows driver. This

[PATCH v2 0/3] stk1160: Let the driver setup the device's internal AC97 codec

As requested, I've cleaned up my previous patchset for resubmission. This patchset is really a result of my attempt to fix a bug (https://bugzilla.kernel.org/show_bug.cgi?id=180071) that eventually turned out to be caused by a missing quirk in snd-usb-audio. My idea was to remove the AC97

Re: [PATCH 1/2] stk1160: Remove stk1160-mixer and setup AC97 codec automatically.

Hi Marcel, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.9-rc2 next-20161027] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenie

Re: [PATCH 3/3] stk1160: Remove VIDEO_STK1160_AC97 and SND_AC97_CODEC from Kconfig and Makefile.

Marcel, Thanks a lot for all your stk1160 fixes. They are much appreciated! In particular, the click noise was something we really wanted to get rid of: http://mailman.alsa-project.org/pipermail/alsa-devel/2016-October/113981.html Regarding the linux-media fixes, is there any chance you

Re: [PATCH 1/2] stk1160: Remove stk1160-mixer and setup AC97 codec automatically.

Hi Marcel, [auto build test ERROR on linuxtv-media/master] [also build test ERROR on v4.9-rc2 next-20161027] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenie

Re: DocBook documentation on linuxtv.org

Em Thu, 27 Oct 2016 14:27:45 +0300 Sakari Ailus escreveu: > Hi Mauro, > > The link labelled "v4l-dvb-apis/" on the > page still points to the old > DocBook documentation. Could you update it, and perhaps even remove the old > DocBook

[PATCH] [media] rc: print correct variable for z8f0811

A recent rework accidentally left a debugging printk untouched while changing the meaning of the variables, leading to an uninitialized variable being printed: drivers/media/i2c/ir-kbd-i2c.c: In function 'get_key_haup_common': drivers/media/i2c/ir-kbd-i2c.c:62:2: error: 'toggle' may be used

Re: [PATCH v6 2/2] media: Add a driver for the ov5645 camera sensor.

Hi, On 10/26/2016 07:48 PM, Ian Arkver wrote: > On 26/10/16 15:07, Todor Tomov wrote: >> Hi, >> >> On 10/26/2016 03:48 PM, Ian Arkver wrote: >>> [snip] > +static int ov5645_regulators_enable(struct ov5645 *ov5645) > +{ > +int ret; > + > +ret =

Re: [PATCH] [media] dvb: avoid warning in dvb_net

On Thursday, October 27, 2016 10:13:27 AM CEST Jarod Wilson wrote: > On Thu, Oct 27, 2016 at 03:57:41PM +0200, Arnd Bergmann wrote: > > With gcc-5 or higher on x86, we can get a bogus warning in the > > dvb-net code: > > > > drivers/media/dvb-core/dvb_net.c: In function ‘dvb_net_ule’: > >

Re: Documentation/media/uapi/cec/ sporadically unnecessarily rebuilding

Hi Jani, Am 24.10.2016 um 11:04 schrieb Jani Nikula : > I think I saw some of this in the past [1], but then couldn't reproduce > it after all. Now I'm seeing it again. Sporadically > Documentation/media/uapi/cec/ gets rebuilt on successive runs of make > htmldocs, even

Re: [PATCH] [media] dvb: avoid warning in dvb_net

On Thu, Oct 27, 2016 at 05:09:28PM +0200, Arnd Bergmann wrote: > On Thursday, October 27, 2016 10:13:27 AM CEST Jarod Wilson wrote: > > On Thu, Oct 27, 2016 at 03:57:41PM +0200, Arnd Bergmann wrote: > > > With gcc-5 or higher on x86, we can get a bogus warning in the > > > dvb-net code: > > > > >

[PATCH v2] [media] dvb: avoid warning in dvb_net

With gcc-5 or higher on x86, we can get a bogus warning in the dvb-net code: drivers/media/dvb-core/dvb_net.c: In function ‘dvb_net_ule’: arch/x86/include/asm/string_32.h:77:14: error: ‘dest_addr’ may be used uninitialized in this function [-Werror=maybe-uninitialized]

[PATCH 3/3] stk1160: Remove VIDEO_STK1160_AC97 and SND_AC97_CODEC from Kconfig and Makefile.

The VIDEO_STK1160_AC97 option is no longer needed after the removal of stk1160-mixer. For the same reason SND and SND_AC97_CODEC are no longer required. Signed-off-by: Marcel Hasler --- drivers/media/usb/stk1160/Kconfig | 3 +-- drivers/media/usb/stk1160/Makefile | 4 +---

Re: [RFC PATCH v2 8/9] drm: writeback: Add out-fences for writeback connectors

Hi Gustavo, On Wed, Oct 26, 2016 at 07:40:29PM -0200, Gustavo Padovan wrote: 2016-10-26 Brian Starkey : Add the OUT_FENCE_PTR property to writeback connectors, to enable userspace to get a fence which will signal once the writeback is complete. A timeline is added to

[PATCH 1/1] v4l: videodev2: Include linux/time.h for timeval and timespec structs

struct timeval and struct timespec are defined in linux/time.h. Explicitly include the header if __KERNEL__ is defined. Signed-off-by: Sakari Ailus --- include/uapi/linux/videodev2.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/videodev2.h

Re: [RFC PATCH v2 9/9] drm: mali-dp: Add writeback out-fence support

2016-10-27 Brian Starkey : > On Wed, Oct 26, 2016 at 07:43:57PM -0200, Gustavo Padovan wrote: > > 2016-10-26 Brian Starkey : > > > > > If userspace has asked for an out-fence for the writeback, we add a > > > fence to malidp_mw_job, to be signaled

[PATCH/RFC] v4l: vsp1: Use FCP device for DisplayList and VB2 queue

From: Magnus Damm Incrementally fix up parts of the code not yet covered by the IOMMU patches by Laurent: [PATCH 0/6] R-Car DU: Fix IOMMU operation when connected to VSP This patch simply uses the recently introduced function rcar_fcp_get_device() on the VSP device

[PATCH] dtv-core: get rid of duplicated kernel-doc include

Somehow, two DVB headers were included twice. Remove the duplication. Reported-by: Markus Heiser Signed-off-by: Mauro Carvalho Chehab --- Documentation/media/kapi/dtv-core.rst | 8 1 file changed, 8 deletions(-) diff --git

[PATCH] [media] lirc: introduce LIRC_SET_TRANSMITTER_WAIT ioctl

lirc transmit waits for the IR to complete, since existing versions of lircd (prior to 0.9.4) rely on this. Allows this to be configurable if this is not desirable. Signed-off-by: Sean Young --- Documentation/media/uapi/rc/lirc-func.rst | 1 +

Re: [PATCH v2 5/7] [media] ir-lirc-codec: don't wait any transmitting time for tx only devices

On Thu, Oct 27, 2016 at 04:44:01PM +0900, Andi Shyti wrote: > Hi Sean, > > it's been a while :) > > I was going through your review fixing what needs to be fixed, > but... > > > > @@ -153,7 +153,7 @@ static ssize_t ir_lirc_transmit_ir(struct file *file, > > > const char __user *buf, > > > }

Re: [RFC PATCH v2 7/9] drm: atomic: factor out common out-fence operations

On Wed, Oct 26, 2016 at 07:45:14PM -0200, Gustavo Padovan wrote: %p should be kept for your internal debug only. Make sure to remove anything that exposes kernel address when sending patches. Gustavo Noted, thanks! -Brian -- To unsubscribe from this list: send the line "unsubscribe

[PATCH 1/2] stk1160: Remove stk1160-mixer and setup AC97 codec automatically.

Exposing the device's internal AC97 codec to userspace seems rather pointless. Instead the driver should setup the codec with proper values. This patch removes the mixer and sets up the codec using optimal values, i.e. the same values set by the Windows driver. This also makes the device work

Re: [PATCH 3/3] stk1160: Remove VIDEO_STK1160_AC97 and SND_AC97_CODEC from Kconfig and Makefile.

Hi Sure, I'll clean up the patches later on today and resubmit. The last one should be merged with the first one anyway, I missed that at first. Regards Marcel 2016-10-27 14:38 GMT+02:00 Ezequiel Garcia : > > Marcel, > > Thanks a lot for all your stk1160 fixes.

Re: [PATCH] [media] smiapp: make PM functions as __maybe_unused

Hi Arnd, On Thu, Oct 27, 2016 at 09:43:16AM +0200, Arnd Bergmann wrote: > On Thursday, October 27, 2016 10:28:18 AM CEST Sakari Ailus wrote: > > > > On Wed, Oct 26, 2016 at 10:38:01PM +0200, Arnd Bergmann wrote: > > > The rework of the PM support has caused two functions to > > > be orphaned

[RFC 3/3] v4l: vsp1: Provide a writeback video device

When the VSP1 is used in an active display pipeline, the output of the WPF can supply the LIF entity directly and simultaneously write to memory. Support this functionality in the VSP1 driver, by extending the WPF source pads, and establishing a V4L2 video device node connected to the new source.

[RFC 2/3] v4l: vsp1: allow entities to have multiple source pads

The upcoming writeback feature of the VSP1 WPF, allows the active output of the DU to be written back to memory. On Generation 3 hardware, the DU is fed by the LIF, which is in turn fed by the WPF. It is the WPF which will perform memory writeback functionality, and this brings in a second output

[RFC 0/3] vsp1 writeback prototype

This series presents an initial version of a write back video node for the VSP DRM pipeline, utilising the existing vsp1_video base where possible. The current version limits the output of the video node to match the frames presented by updates from the CRTC atomic flush events. Matching the

[RFC 1/3] Revert "[media] v4l: vsp1: Supply frames to the DU continuously"

This reverts commit 3299ba5c0b213be5d911752d40251c1abc1004f7. The DU output mode does not rely on frames being supplied on the WPF as its pipeline is supplied from DRM. For the upcoming WPF writeback functionality, we will choose to enable writeback mode if there is an output buffer, or disable

DocBook documentation on linuxtv.org

Hi Mauro, The link labelled "v4l-dvb-apis/" on the page still points to the old DocBook documentation. Could you update it, and perhaps even remove the old DocBook documentation so people do not accidentally continue to use it? -- Kind regards, Sakari

Re: [PATCH] [media] dvb: avoid warning in dvb_net

On Thu, Oct 27, 2016 at 03:57:41PM +0200, Arnd Bergmann wrote: > With gcc-5 or higher on x86, we can get a bogus warning in the > dvb-net code: > > drivers/media/dvb-core/dvb_net.c: In function ‘dvb_net_ule’: > arch/x86/include/asm/string_32.h:77:14: error: ‘dest_addr’ may be used >

Re: [RFC PATCH v2 9/9] drm: mali-dp: Add writeback out-fence support

On Thu, Oct 27, 2016 at 09:25:19AM -0200, Gustavo Padovan wrote: > 2016-10-27 Brian Starkey : > > > On Wed, Oct 26, 2016 at 07:43:57PM -0200, Gustavo Padovan wrote: > > > 2016-10-26 Brian Starkey : > > > > > > > If userspace has asked for an

[PATCH] [media] dvb: avoid warning in dvb_net

With gcc-5 or higher on x86, we can get a bogus warning in the dvb-net code: drivers/media/dvb-core/dvb_net.c: In function ‘dvb_net_ule’: arch/x86/include/asm/string_32.h:77:14: error: ‘dest_addr’ may be used uninitialized in this function [-Werror=maybe-uninitialized]

Re: [RFC PATCH v2 9/9] drm: mali-dp: Add writeback out-fence support

On Wed, Oct 26, 2016 at 07:43:57PM -0200, Gustavo Padovan wrote: 2016-10-26 Brian Starkey : If userspace has asked for an out-fence for the writeback, we add a fence to malidp_mw_job, to be signaled when the writeback job has completed. Signed-off-by: Brian Starkey

Re: [PATCH v2 5/7] [media] ir-lirc-codec: don't wait any transmitting time for tx only devices

Hi Sean, it's been a while :) I was going through your review fixing what needs to be fixed, but... > > @@ -153,7 +153,7 @@ static ssize_t ir_lirc_transmit_ir(struct file *file, > > const char __user *buf, > > } > > > > ret = dev->tx_ir(dev, txbuf, count); > > - if (ret < 0) > > +

[PATCH 2/2] stk1160: Check whether to use AC97 codec or internal ADC.

Some STK1160-based devices use the chip's internal 8-bit ADC. This is configured through a strap pin. The value of this and other pins can be read through the POSVA register. If the internal ADC is used, there's no point trying to setup the unavailable AC97 codec. Signed-off-by: Marcel Hasler

Re: [PATCH] [media] smiapp: make PM functions as __maybe_unused

On Thursday, October 27, 2016 10:28:18 AM CEST Sakari Ailus wrote: > > On Wed, Oct 26, 2016 at 10:38:01PM +0200, Arnd Bergmann wrote: > > The rework of the PM support has caused two functions to > > be orphaned when CONFIG_PM is disabled: > > > > media/i2c/smiapp/smiapp-core.c:1352:12: error:

[RFC] V4L2 unified low-level decoder API

Hi, This RFC aims to start discussions in order to define the codec specific controls structures to fulfill the low-level decoder API needed by non "Stream API" based decoders ("stateless" or "Frame API" based decoders). Several implementation exists now which runs on several SoC and various

Re: [PATCH] [media] smiapp: make PM functions as __maybe_unused

Hi Arnd, On Wed, Oct 26, 2016 at 10:38:01PM +0200, Arnd Bergmann wrote: > The rework of the PM support has caused two functions to > be orphaned when CONFIG_PM is disabled: > > media/i2c/smiapp/smiapp-core.c:1352:12: error: 'smiapp_power_off' defined but > not used [-Werror=unused-function] >

YUV444 contradicting wikipedia

Hi, Looks like the Linux definition of the (packed) YUV444 format contradicts wikipedia. According to https://linuxtv.org/downloads/v4l-dvb-apis-new/uapi/v4l/pixfmt-packed-yuv.html The Linux V4L2_PIX_FMT_YUV444 format takes 16 bits per pixel, whereas the wikipedia