Re: [PATCH 02/14] pvrusb2: fix sparse warning

2013-10-04 Thread Mike Isely
Acked-by: Mike Isely -Mike On Fri, 4 Oct 2013, Hans Verkuil wrote: > From: Hans Verkuil > > drivers/media/usb/pvrusb2/pvrusb2-hdw.c:2871:13: warning: symbol > 'pvr2_hdw_get_detected_std' was not declared. Should it be static? > > Signed-off-by: Hans Verkuil > Cc: Mike Isely > --- > dr

cron job: media_tree daily build: WARNINGS

2013-10-04 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: Sat Oct 5 04:00:20 CEST 2013 git branch: test git hash: d10e8280c4c2513d3e7350c27d8e6f0fa03a5f71 gcc versi

Re: cx23885: Add basic analog radio support

2013-10-04 Thread Alfredo Jesús Delaiti
Hi all El 14/01/12 15:25, Miroslav Slugeň escribió: New version of patch, fixed video modes for DVR3200 tuners and working audio mux. I tested this patch (https://linuxtv.org/patch/9498/) with the latest versions of git (September 28, 2013) with my TV card (Mygica X8507) and it works. I fo

[GIT PULL BUG FIX] cx24117: prevent mutex to be stuck on locked state if FE init fails

2013-10-04 Thread Michael Krufky
The following changes since commit d10e8280c4c2513d3e7350c27d8e6f0fa03a5f71: [media] cx24117: use hybrid_tuner_request/release_state to share state between multiple instances (2013-10-03 07:40:12 -0300) are available in the git repository at: git://linuxtv.org/mkrufky/dvb cx24117 for you

Re: [PATCH 07/14] drxk_hard: fix sparse warnings

2013-10-04 Thread Michael Krufky
On Fri, Oct 4, 2013 at 10:40 AM, Antti Palosaari wrote: > On 04.10.2013 17:01, Hans Verkuil wrote: >> >> From: Hans Verkuil >> >> drivers/media/dvb-frontends/drxk_hard.c:1086:62: warning: Using plain >> integer as NULL pointer >> drivers/media/dvb-frontends/drxk_hard.c:2784:63: warning: Using pla

Re: [PATCH 06/14] drxd_hard: fix sparse warnings

2013-10-04 Thread Michael Krufky
On Fri, Oct 4, 2013 at 10:44 AM, Antti Palosaari wrote: > On 04.10.2013 17:01, Hans Verkuil wrote: >> >> From: Hans Verkuil >> >> drivers/media/dvb-frontends/drxd_hard.c:1017:70: warning: Using plain >> integer as NULL pointer >> drivers/media/dvb-frontends/drxd_hard.c:1038:69: warning: Using pla

Re: [PATCH 04/14] tuner-xs2028.c: fix sparse warnings

2013-10-04 Thread Michael Krufky
On Fri, Oct 4, 2013 at 10:01 AM, Hans Verkuil wrote: > From: Hans Verkuil > > drivers/media/tuners/tuner-xc2028.c:575:24: warning: cast to restricted __le16 > drivers/media/tuners/tuner-xc2028.c:686:21: warning: cast to restricted __le16 > > Signed-off-by: Hans Verkuil > Cc: Mauro Carvalho Cheha

Re: [PATCH 05/14] cxd2820r_core: fix sparse warnings

2013-10-04 Thread Michael Krufky
On Fri, Oct 4, 2013 at 10:36 AM, Antti Palosaari wrote: > On 04.10.2013 17:01, Hans Verkuil wrote: >> >> From: Hans Verkuil >> >> drivers/media/dvb-frontends/cxd2820r_core.c:34:32: error: cannot size >> expression >> drivers/media/dvb-frontends/cxd2820r_core.c:68:32: error: cannot size >> express

Re: [PATCH] cx24117: Prevent mutex to be stuck on locked state if FE init fails.

2013-10-04 Thread Antti Palosaari
On 04.10.2013 17:48, Luis Alves wrote: Hi, This patch will fix the situation where the mutex was left in a locked state if for some reason the FE init failed. Regards, Luis Signed-off-by: Luis Alves Reviewed-by: Antti Palosaari --- drivers/media/dvb-frontends/cx24117.c |9 ---

[PATCH] cx24117: Prevent mutex to be stuck on locked state if FE init fails.

2013-10-04 Thread Luis Alves
Hi, This patch will fix the situation where the mutex was left in a locked state if for some reason the FE init failed. Regards, Luis Signed-off-by: Luis Alves --- drivers/media/dvb-frontends/cx24117.c |9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/med

Re: [PATCH 06/14] drxd_hard: fix sparse warnings

2013-10-04 Thread Antti Palosaari
On 04.10.2013 17:01, Hans Verkuil wrote: From: Hans Verkuil drivers/media/dvb-frontends/drxd_hard.c:1017:70: warning: Using plain integer as NULL pointer drivers/media/dvb-frontends/drxd_hard.c:1038:69: warning: Using plain integer as NULL pointer drivers/media/dvb-frontends/drxd_hard.c:2836:

Re: [PATCH 07/14] drxk_hard: fix sparse warnings

2013-10-04 Thread Antti Palosaari
On 04.10.2013 17:01, Hans Verkuil wrote: From: Hans Verkuil drivers/media/dvb-frontends/drxk_hard.c:1086:62: warning: Using plain integer as NULL pointer drivers/media/dvb-frontends/drxk_hard.c:2784:63: warning: Using plain integer as NULL pointer Signed-off-by: Hans Verkuil Reviewed-by:

Re: [PATCH 05/14] cxd2820r_core: fix sparse warnings

2013-10-04 Thread Antti Palosaari
On 04.10.2013 17:01, Hans Verkuil wrote: From: Hans Verkuil drivers/media/dvb-frontends/cxd2820r_core.c:34:32: error: cannot size expression drivers/media/dvb-frontends/cxd2820r_core.c:68:32: error: cannot size expression Signed-off-by: Hans Verkuil Acked-by: Antti Palosaari Reviewed-by: A

[PATCH] media: rcar_vin: Add preliminary r8a7790 support

2013-10-04 Thread Valentine Barshak
Signed-off-by: Valentine Barshak --- drivers/media/platform/soc_camera/rcar_vin.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/media/platform/soc_camera/rcar_vin.c b/drivers/media/platform/soc_camera/rcar_vin.c index d02a7e0..b21f777 100644 --- a/drivers/media/

Re: [PATCH 0/3] media: Add SH-Mobile RCAR-H2 Lager board support

2013-10-04 Thread Valentine
On 10/04/2013 02:13 PM, Hans Verkuil wrote: On 09/24/2013 06:14 PM, Guennadi Liakhovetski wrote: Hi Valentine, Hi, Since patches 2 and 3 here are for soc-camera, I can offer to take all 3 via my tree after all comments to patch 1/3 are addressed and someone (Laurent?) has acked it. Alternat

Re: [PATCH] vb2: Allow STREAMOFF for io emulator

2013-10-04 Thread Ricardo Ribalda Delgado
Hello Hans I am implementing a test application for our camera, think of v4l2-compliance but for testing the hardware (average of pixels, rotation...) . I am implementing it using python (because of numpy and matplotlib). I dont really care about perferomance, I only care about the data correctne

[PATCH 08/14] fmdrv_common: fix sparse warning

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/radio/wl128x/fmdrv_common.c:178:6: warning: symbol 'g_st_write' was not declared. Should it be static? Signed-off-by: Hans Verkuil Cc: Manjunatha Halli --- drivers/media/radio/wl128x/fmdrv_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

[PATCH 13/14] cx25821: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/pci/cx25821/cx25821-cards.c:49:20: warning: symbol 'cx25821_bcount' was not declared. Should it be static? drivers/media/pci/cx25821/cx25821-video-upstream.c:162:33: warning: incorrect type in assignment (different base types) drivers/media/pci/cx25821/cx25821-v

[PATCH 09/14] radio-keene: fix sparse warning

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/radio/radio-keene.c:126:45: warning: dubious: !x | y Signed-off-by: Hans Verkuil --- drivers/media/radio/radio-keene.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/radio/radio-keene.c b/drivers/media/radio/radio-keene.c ind

Re: [PATCH] vb2: Allow STREAMOFF for io emulator

2013-10-04 Thread Hans Verkuil
Hi Ricardo, On 10/04/2013 03:49 PM, Ricardo Ribalda Delgado wrote: > A video device opened and streaming in io emulator mode can only stop > streamming if its file descriptor is closed. > > There are some parameters that can only be changed if the device is not > streaming. Also, the power consum

[PATCH 02/14] pvrusb2: fix sparse warning

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/usb/pvrusb2/pvrusb2-hdw.c:2871:13: warning: symbol 'pvr2_hdw_get_detected_std' was not declared. Should it be static? Signed-off-by: Hans Verkuil Cc: Mike Isely --- drivers/media/usb/pvrusb2/pvrusb2-hdw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 07/14] drxk_hard: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/dvb-frontends/drxk_hard.c:1086:62: warning: Using plain integer as NULL pointer drivers/media/dvb-frontends/drxk_hard.c:2784:63: warning: Using plain integer as NULL pointer Signed-off-by: Hans Verkuil Cc: Antti Palosaari --- drivers/media/dvb-frontends/drxk

[PATCH 11/14] cx231xx: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/usb/cx231xx/cx231xx-pcb-cfg.c:31:19: warning: symbol 'cx231xx_Scenario' was not declared. Should it be static? drivers/media/usb/cx231xx/cx231xx-pcb-cfg.c:675:23: warning: cast to restricted __le32 Signed-off-by: Hans Verkuil --- drivers/media/usb/cx231xx/cx2

[PATCH 12/14] tlg2300: fix sparse warning

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/usb/tlg2300/pd-main.c:235:25: warning: incorrect type in assignment (different base types) Signed-off-by: Hans Verkuil Cc: Huang Shijie --- drivers/media/usb/tlg2300/pd-main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/u

[PATCH 14/14] siano: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/common/siano/smsdvb-main.c:47:5: warning: symbol 'sms_to_guard_interval_table' was not declared. Should it be static? drivers/media/common/siano/smsdvb-main.c:54:5: warning: symbol 'sms_to_code_rate_table' was not declared. Should it be static? drivers/media/com

[PATCH 10/14] az6027: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/usb/dvb-usb/az6027.c:257:23: warning: symbol 'az6027_stb0899_config' was not declared. Should it be static? drivers/media/usb/dvb-usb/az6027.c:294:23: warning: symbol 'az6027_stb6100_config' was not declared. Should it be static? Signed-off-by: Hans Verkuil --

[PATCH 00/14] Fixes for sparse warnings

2013-10-04 Thread Hans Verkuil
This is a first set of fixes for sparse warnings. There are lots more, but you have to start somewhere... Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kern

[PATCH 01/14] hdpvr: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/usb/hdpvr/hdpvr-core.c:110:54: warning: incorrect type in argument 1 (different base types) drivers/media/usb/hdpvr/hdpvr-core.c:112:39: warning: invalid assignment: += drivers/media/usb/hdpvr/hdpvr-core.c:304:26: warning: Using plain integer as NULL pointer Si

[PATCH 06/14] drxd_hard: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/dvb-frontends/drxd_hard.c:1017:70: warning: Using plain integer as NULL pointer drivers/media/dvb-frontends/drxd_hard.c:1038:69: warning: Using plain integer as NULL pointer drivers/media/dvb-frontends/drxd_hard.c:2836:33: warning: Using plain integer as NULL p

[PATCH 05/14] cxd2820r_core: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/dvb-frontends/cxd2820r_core.c:34:32: error: cannot size expression drivers/media/dvb-frontends/cxd2820r_core.c:68:32: error: cannot size expression Signed-off-by: Hans Verkuil Cc: Antti Palosaari --- drivers/media/dvb-frontends/cxd2820r_core.c | 4 ++-- 1 file

[PATCH 04/14] tuner-xs2028.c: fix sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/tuners/tuner-xc2028.c:575:24: warning: cast to restricted __le16 drivers/media/tuners/tuner-xc2028.c:686:21: warning: cast to restricted __le16 Signed-off-by: Hans Verkuil Cc: Mauro Carvalho Chehab --- drivers/media/tuners/tuner-xc2028.c | 4 ++-- 1 file chang

[PATCH 03/14] timblogiw: fix two sparse warnings

2013-10-04 Thread Hans Verkuil
From: Hans Verkuil drivers/media/platform/timblogiw.c:763:43: warning: incorrect type in initializer (incompatible argument 3 (different signedness)) drivers/media/platform/timblogiw.c:764:43: warning: incorrect type in initializer (incompatible argument 3 (different signedness)) Signed-off-by

[PATCH] vb2: Allow STREAMOFF for io emulator

2013-10-04 Thread Ricardo Ribalda Delgado
A video device opened and streaming in io emulator mode can only stop streamming if its file descriptor is closed. There are some parameters that can only be changed if the device is not streaming. Also, the power consumption of a device streaming could be different than one not streaming. With t

[PATCH v4 2/4] [media] exynos-scaler: Add core functionality for the SCALER driver

2013-10-04 Thread Shaik Ameer Basha
This patch adds the core functionality for the SCALER driver. Signed-off-by: Shaik Ameer Basha --- drivers/media/platform/exynos-scaler/scaler.c | 1238 + drivers/media/platform/exynos-scaler/scaler.h | 375 2 files changed, 1613 insertions(+) create mode 10064

[PATCH v4 1/4] [media] exynos-scaler: Add new driver for Exynos5 SCALER

2013-10-04 Thread Shaik Ameer Basha
This patch adds support for SCALER device which is a new device for scaling, blending, color fill and color space conversion on EXYNOS5410 and EXYNOS5420 SoCs. This device supports the followings as key feature. input image format - YCbCr420 2P(UV/VU), 3P - YCbCr422 1P(YUYV/UY

[PATCH v4 4/4] [media] exynos-scaler: Add DT bindings for SCALER driver

2013-10-04 Thread Shaik Ameer Basha
This patch adds the DT binding documentation for the Exynos5420/5410 based SCALER device driver. Signed-off-by: Shaik Ameer Basha --- .../devicetree/bindings/media/exynos5-scaler.txt | 22 1 file changed, 22 insertions(+) create mode 100644 Documentation/devicetree/bind

[PATCH v4 3/4] [media] exynos-scaler: Add m2m functionality for the SCALER driver

2013-10-04 Thread Shaik Ameer Basha
This patch adds the Makefile and memory to memory (m2m) interface functionality for the SCALER driver. Signed-off-by: Shaik Ameer Basha --- drivers/media/platform/Kconfig|8 + drivers/media/platform/Makefile |1 + drivers/media/platform/exynos-scaler

[PATCH v4 0/4] Exynos5 Series SCALER Driver

2013-10-04 Thread Shaik Ameer Basha
This patch adds support for SCALER device which is a new device for scaling, blending, color fill and color space conversion on EXYNOS5410/5420 SoCs. This device supports the following as key features. input image format - YCbCr420 2P(UV/VU), 3P - YCbCr422 1P(YUYV/UYVY/YVYU),

[PATCH] dvb-apps: fix compiler warnings

2013-10-04 Thread Hans Verkuil
This patch fixes all remaining dvb-apps compiler warnings: test_video.c:322:2: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘ssize_t’ [-Wformat=] dvbscan.c:128:6: warning: variable ‘output_type’ set but not used [-Wunused-but-set-variable] dvbscan.

libtool warning in libdvbv5

2013-10-04 Thread Hans Verkuil
Hi Gregor, When linking libdvbv5.la I get the following warning from libtool: CCLD libdvbv5.la libtool: link: warning: `-version-info/-version-number' is ignored for convenience libraries Other libs don't give that warning, but I don't see any obvious differences. Do you know what might

[GIT PULL FOR v3.12] Various fixes

2013-10-04 Thread Hans Verkuil
The following changes since commit d10e8280c4c2513d3e7350c27d8e6f0fa03a5f71: [media] cx24117: use hybrid_tuner_request/release_state to share state between multiple instances (2013-10-03 07:40:12 -0300) are available in the git repository at: git://linuxtv.org/hverkuil/media_tree.git for-v

[GIT PULL FOR v3.13] Various fixes

2013-10-04 Thread Hans Verkuil
Hi Mauro, Just a bunch of various fixes. Most notably the solo fixes for big-endian systems: these fixes were removed when I did the large sync to the Bluecherry code base for the solo driver. Many thanks to Krzysztof for doing this work again. Regards, Hans The following changes sinc

Re: [PATCH 0/3] media: Add SH-Mobile RCAR-H2 Lager board support

2013-10-04 Thread Hans Verkuil
On 09/24/2013 06:14 PM, Guennadi Liakhovetski wrote: > Hi Valentine, > > Since patches 2 and 3 here are for soc-camera, I can offer to take all 3 > via my tree after all comments to patch 1/3 are addressed and someone > (Laurent?) has acked it. Alternatively I can ack the two patches and let >

Re: [PATCH 2/6] v4l: omap4iss: Add support for OMAP4 camera interface - Video devices

2013-10-04 Thread Dan Carpenter
On Thu, Oct 03, 2013 at 01:55:29AM +0200, Laurent Pinchart wrote: > + > + ret = vb2_streamon(&vfh->queue, type); > + if (ret < 0) > + goto err_iss_video_check_format; > + > + /* In sensor-to-memory mode, the stream can be started synchronously > + * to the stream on com

Re: Suspected cache coherency problem on V4L2 and AR7100 CPU

2013-10-04 Thread Krzysztof Hałasa
> I'm debugging a problem with a SOLO6110-based H.264 PCI video encoder on > Atheros AR7100-based (MIPS, big-endian) platform. BTW this CPU obviously has VIPT data cache, this means a physical page with multiple virtual addresses (e.g. mapped multiple times) may and will be cached multiple times.