[PATCH 2/4] [media] exynos-gsc: don't clear format when freeing buffers with REQBUFS(0)

2016-09-30 Thread Javier Martinez Canillas
User-space applications can use the VIDIOC_REQBUFS ioctl to determine if a memory mapped, user pointer or DMABUF based I/O is supported by a driver. For example, GStreamer attempts to determine the I/O methods supported by the driver by doing many VIDIOC_REQBUFS ioctl calls with different memory t

[PATCH 1/4] [media] exynos-gsc: change spamming try_fmt log message to debug

2016-09-30 Thread Javier Martinez Canillas
The driver try_fmt handler prints a message each time that the image size has been changed due the maximum and minimum width and height. Since user-space can try different format and sizes, this logs a lot of unnecessary messages. Change the message log level to debug and while being there, also a

[PATCH 3/4] [media] exynos-gsc: fix supported RGB pixel format

2016-09-30 Thread Javier Martinez Canillas
The driver exposes 32-bit A/XRGB 8-8-8-8 as supported format but testing shows that using this format produces frames with wrong colors. The test was done with the following GStreamer pipeline: $ gst-launch-1.0 videotestsrc num-buffers=20 ! video/x-raw,format=UYVY \ ! v4l2video3convert ! video/x-r

[PATCH 4/4] [media] exynos-gsc: do proper bytesperline and sizeimage calculation

2016-09-30 Thread Javier Martinez Canillas
The driver don't take into account the differences between packed, semi planar and multi planar formats when calculating the pixel format bytes per lines and image size values. This makes GStreamer to fail when the following formats are used NV12, NV21, NV16, NV61, YV12, I420 and Y42B: "gst_video_

[PATCH 0/4] [media] exynos-gsc: Cleanup and fixes

2016-09-30 Thread Javier Martinez Canillas
Hello, This patch series contains a cleanup and some fixes for the exynos-gsc driver. I found these issues when trying to use the driver with the GStreamer v4l2videoconvert element. I wasn't able to display to Exynos DRM/KMS driver and only a small set of formats were working correctly. After the

[PATCH 1/2] rc: core: add managed versions of rc_allocate_device and rc_register_device

2016-09-30 Thread Heiner Kallweit
Introduce managed versions of both functions. They allows to simplify the error path in the probe function of rc drivers, and usually also to simplify the remove function. New element managed_alloc in struct rc_dev is needed to correctly handle mixed use, e.g. managed version of rc_register_device

[PATCH 2/2] rc: nuvoton: use managed versions of rc_allocate_device and rc_register_device

2016-09-30 Thread Heiner Kallweit
Simplify the remove function and the error path in the probe function by using the managed versions of rc_allocate_device and rc_register_device. Signed-off-by: Heiner Kallweit --- drivers/media/rc/nuvoton-cir.c | 50 +- 1 file changed, 20 insertions(+), 3

Re: [PATCH 2/3] ARM: dts: gose: add HDMI input

2016-09-30 Thread Laurent Pinchart
On Friday 30 Sep 2016 15:00:59 Geert Uytterhoeven wrote: > On Fri, Sep 30, 2016 at 2:40 PM, Laurent Pinchart wrote: > >> --- a/arch/arm/boot/dts/r8a7793-gose.dts > >> +++ b/arch/arm/boot/dts/r8a7793-gose.dts > >> @@ -374,6 +374,11 @@ > >> groups = "audio_clk_a"; > >> fun

Re: [PATCH 2/3] ARM: dts: gose: add HDMI input

2016-09-30 Thread Geert Uytterhoeven
On Fri, Sep 30, 2016 at 2:40 PM, Laurent Pinchart wrote: >> --- a/arch/arm/boot/dts/r8a7793-gose.dts >> +++ b/arch/arm/boot/dts/r8a7793-gose.dts >> @@ -374,6 +374,11 @@ >> groups = "audio_clk_a"; >> function = "audio_clk"; >> }; >> + >> + vin0_pins: vin0 { >>

Re: [PATCH 3/3] ARM: dts: gose: add composite video input

2016-09-30 Thread Laurent Pinchart
Hi Ulrich, Thank you for the patch. On Friday 16 Sep 2016 15:09:35 Ulrich Hecht wrote: > Signed-off-by: Ulrich Hecht > --- > arch/arm/boot/dts/r8a7793-gose.dts | 36 ++ > 1 file changed, 36 insertions(+) > > diff --git a/arch/arm/boot/dts/r8a7793-gose.dts > b/ar

Re: [PATCH 2/3] ARM: dts: gose: add HDMI input

2016-09-30 Thread Laurent Pinchart
Hi Ulrich, Thank you for the patch. On Friday 16 Sep 2016 15:09:34 Ulrich Hecht wrote: > Identical to the setup on Lager. > > Signed-off-by: Ulrich Hecht > --- > arch/arm/boot/dts/r8a7793-gose.dts | 41 +++ > 1 file changed, 41 insertions(+) > > diff --git a/ar

Re: [PATCH 1/3] ARM: dts: r8a7793: Enable VIN0, VIN1

2016-09-30 Thread Laurent Pinchart
Hi Ulrich, On Friday 30 Sep 2016 13:55:50 Laurent Pinchart wrote: > On Friday 16 Sep 2016 15:09:33 Ulrich Hecht wrote: > > Signed-off-by: Ulrich Hecht > > --- > > > > arch/arm/boot/dts/r8a7793.dtsi | 20 > > 1 file changed, 20 insertions(+) > > > > diff --git a/arch/arm/bo

Re: [PATCH 1/3] ARM: dts: r8a7793: Enable VIN0, VIN1

2016-09-30 Thread Laurent Pinchart
Hi Ulrich, Thanks for the patch. On Friday 16 Sep 2016 15:09:33 Ulrich Hecht wrote: > Signed-off-by: Ulrich Hecht > --- > arch/arm/boot/dts/r8a7793.dtsi | 20 > 1 file changed, 20 insertions(+) > > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dts

[PATCH] uvcvideo: uvc_scan_fallback() for webcams with broken chain

2016-09-30 Thread Laurent Pinchart
From: Henrik Ingo Some devices have invalid baSourceID references, causing uvc_scan_chain() to fail, but if we just take the entities we can find and put them together in the most sensible chain we can think of, turns out they do work anyway. Note: This heuristic assumes there is a single chain.

Re: [PATCH] [media] omap3isp: don't call of_node_put

2016-09-30 Thread Laurent Pinchart
Hello Nikolaus, On Friday 30 Sep 2016 10:28:40 H. Nikolaus Schaller wrote: > > Am 29.09.2016 um 10:54 schrieb Laurent Pinchart: > > On Thursday 08 Sep 2016 17:48:33 H. Nikolaus Schaller wrote: > >> of_node_put() has already been called inside > >> of_graph_get_next_endpoint(). > >> > >> Otherwise

Re: [PATCH] [media] omap3isp: don't call of_node_put

2016-09-30 Thread H. Nikolaus Schaller
Hi Laurent, > Am 29.09.2016 um 10:54 schrieb Laurent Pinchart > : > > Hi Nikolaus, > > On Thursday 08 Sep 2016 17:48:33 H. Nikolaus Schaller wrote: >> of_node_put() has already been called inside of_graph_get_next_endpoint(). >> >> Otherwise we may get warnings like >> >> [ 10.118286] omap3