cron job: media_tree daily build: WARNINGS

2013-11-02 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: Sun Nov 3 04:00:28 CET 2013 git branch: for-v3.13c git hash: 3adeac2c34cc28e05d0ec52f38f009dcce278555 gcc

[PATCH 12/12] ddbridge: Kconfig and Makefile fixes to build latest ddbridge

2013-11-02 Thread Maik Broemme
Fixed Kconfig and Makefile to build latest version off ddbridge. It adds support for the following devices: - Octopus DVB adapter - Octopus V3 DVB adapter - Octopus LE DVB adapter - Octopus OEM - Octopus Mini - Cine S2 V6 DVB adapter - Cine S2 V6.5 DVB adapter - Octopus CI - Octo

[PATCH 11/12] ddbridge: Update ddbridge header for 0.9.10 changes

2013-11-02 Thread Maik Broemme
Updated ddbridge header for 0.9.10 changes. Signed-off-by: Maik Broemme --- drivers/media/pci/ddbridge/ddbridge.h | 408 -- 1 file changed, 343 insertions(+), 65 deletions(-) diff --git a/drivers/media/pci/ddbridge/ddbridge.h b/drivers/media/pci/ddbridge/ddbridg

[PATCH 09/12] ddbridge: Support for the Digital Devices Resi DVB-C Modulator card

2013-11-02 Thread Maik Broemme
Added support for the Digital Devices Resi DVB-C Modulator card. Signed-off-by: Maik Broemme --- drivers/media/pci/ddbridge/ddbridge-mod.c | 1033 + 1 file changed, 1033 insertions(+) create mode 100644 drivers/media/pci/ddbridge/ddbridge-mod.c diff --git a/drivers/

[PATCH 08/12] ddbridge: Moved i2c interfaces into separate file

2013-11-02 Thread Maik Broemme
Moved i2c interfaces from ddbridge-core.c into separate file. Signed-off-by: Maik Broemme --- drivers/media/pci/ddbridge/ddbridge-i2c.c | 239 ++ 1 file changed, 239 insertions(+) create mode 100644 drivers/media/pci/ddbridge/ddbridge-i2c.c diff --git a/drivers/medi

[PATCH 02/12] tda18271c2dd: Fix description of NXP TDA18271C2 silicon tuner

2013-11-02 Thread Maik Broemme
Added (DD) to NXP TDA18271C2 silicon tuner as this tuner was specifically added for Digital Devices ddbridge driver. Signed-off-by: Maik Broemme --- drivers/media/dvb-frontends/Kconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/dvb-frontends/Kconfig b/

[PATCH 01/12] dvb-frontends: Support for DVB-C2 to DVB frontends

2013-11-02 Thread Maik Broemme
Added support for DVB-C2 to DVB frontends. It will be required by cxd2843 and tda18212dd (Digital Devices) frontends. Signed-off-by: Maik Broemme --- include/uapi/linux/dvb/frontend.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/uapi/linux/dvb/frontend.h b/include/uapi/linux/dvb/

[PATCH 00/12] DDBridge 0.9.10 driver updates

2013-11-02 Thread Maik Broemme
I've updated the current DDBridge to latest version 0.9.10 from Ralph Metzler available at: http://www.metzlerbros.de/dddvb/dddvb-0.9.10.tar.bz2 I've merged the driver to work with current v4l/dvb tree and I will maintain the driver for v4l/dvb in future. The coming patch series is the first vers

[PATCH 07/12] ddbridge: Updated ddbridge registers

2013-11-02 Thread Maik Broemme
Updated ddbridge registers: - Added GPIO and MDIO registers - Added Clock Generator (Sil598 @ 0xAA I2c) - Added DAC (AD9781/AD9783 SPI) - Added Temperature Monitor (2x LM75A @ 0x90,0x92 I2c) - Added CI Interface (only CI-Bridge) - Added output control registers - Added modulator chan

[PATCH 06/12] dvb-core: export dvb_usercopy and new DVB device constants

2013-11-02 Thread Maik Broemme
Added EXPORT_SYMBOL(dvb_usercopy) to allow new ddbridge driver to use it. It is questionable if I should use it in this way or not. If not I will fix it. Added two new DVB device constants DVB_DEVICE_CI and DVB_DEVICE_MOD required by newer ddbridge driver. Again it is questionable to use them or m

[PATCH 05/12] cxd2843: Support for CXD2843ER demodulator for DVB-T/T2/C/C2

2013-11-02 Thread Maik Broemme
Added support for the CXD2843ER demodulator for DVB-T/T2/C/C2 used by recent Digital Devices hardware. Signed-off-by: Maik Broemme --- drivers/media/dvb-frontends/Kconfig |9 + drivers/media/dvb-frontends/Makefile |1 + drivers/media/dvb-frontends/cxd2843.c | 1647

[PATCH 04/12] tda18212dd: Support for NXP TDA18212 (DD) silicon tuner

2013-11-02 Thread Maik Broemme
Added support for the NXP TDA18212 silicon tuner used by recent Digital Devices hardware. This will allow update of ddbridge driver to support newer devices. Signed-off-by: Maik Broemme --- drivers/media/dvb-frontends/Kconfig | 9 + drivers/media/dvb-frontends/Makefile | 1 + driver

Re: [PATCHv2 19/29] tuners: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Em Sat, 02 Nov 2013 22:59:04 +0100 Hans Verkuil escreveu: > On 11/02/2013 10:53 PM, Hans Verkuil wrote: > > On 11/02/2013 10:15 PM, Mauro Carvalho Chehab wrote: > >> Em Sat, 02 Nov 2013 18:25:19 +0100 > >> Hans Verkuil escreveu: > >> > >>> Hi Mauro, > >>> > >>> I'll review this series more caref

Re: [PATCH] tda18271-fe: Fix dvb-c standard selection

2013-11-02 Thread Mauro Carvalho Chehab
Em Sat, 02 Nov 2013 23:31:58 +0200 CrazyCat escreveu: > Mauro Carvalho Chehab пишет: > > This is wrong, as it breaks for 6MHz-spaced channels, like what's used > > in Brazil and Japan. > > > > What happens here is that, if the tuner uses a too wide lowpass filter, > > the interference will be hig

[PATCH] gspca-stk1135: Add delay after configuring clock

2013-11-02 Thread Ondrej Zary
Add a small delay at the end of configure_clock() to allow sensor to initialize. This is needed by Asus VX2S laptop webcam to detect sensor type properly (the already-supported MT9M112). Signed-off-by: Ondrej Zary --- drivers/media/usb/gspca/stk1135.c |3 +++ 1 file changed, 3 insertions(+)

Re: RTL2832U DAB Support?

2013-11-02 Thread Alistair Buxton
Hi, You can listen to DAB with http://www.sdr-j.tk/index.html You need a powerful CPU though, and you might have to unload the DVB driver first. On 27 October 2013 16:45, Christof Meerwald wrote: > Hi, > > http://www.linuxtv.org/wiki/index.php/RealTek_RTL2832U mentions DAB > support. But what e

Re: [PATCHv2 19/29] tuners: Don't use dynamic static allocation

2013-11-02 Thread Hans Verkuil
On 11/02/2013 10:53 PM, Hans Verkuil wrote: > On 11/02/2013 10:15 PM, Mauro Carvalho Chehab wrote: >> Em Sat, 02 Nov 2013 18:25:19 +0100 >> Hans Verkuil escreveu: >> >>> Hi Mauro, >>> >>> I'll review this series more carefully on Monday, >> >> Thanks! >> >>> but for now I want to make >>> a sugges

Re: [PATCHv2 19/29] tuners: Don't use dynamic static allocation

2013-11-02 Thread Hans Verkuil
On 11/02/2013 10:15 PM, Mauro Carvalho Chehab wrote: > Em Sat, 02 Nov 2013 18:25:19 +0100 > Hans Verkuil escreveu: > >> Hi Mauro, >> >> I'll review this series more carefully on Monday, > > Thanks! > >> but for now I want to make >> a suggestion for the array checks: >> >> On 11/02/2013 02:31 P

Re: [PATCH 1/1] as3645a: Remove set_power() from platform data

2013-11-02 Thread Sakari Ailus
On Wed, May 23, 2012 at 03:06:41PM +0300, Sakari Ailus wrote: > On Wed, May 23, 2012 at 01:31:26PM +0200, Laurent Pinchart wrote: > > Hi Sakari, > ... > > > > If the chip is powered on constantly, why do we need a .s_power() subdev > > > > operation at all ? > > > > > > I don't know why was it the

Re: [PATCH] tda18271-fe: Fix dvb-c standard selection

2013-11-02 Thread Mauro Carvalho Chehab
Em Sat, 02 Nov 2013 23:05:14 +0200 CrazyCat escreveu: > Fix dvb-c standard selection - qam8 for ANNEX_AC > > Signed-off-by: Evgeny Plehov > diff --git a/drivers/media/tuners/tda18271-fe.c > b/drivers/media/tuners/tda18271-fe.c > index 4995b89..6a385c8 100644 > --- a/drivers/media/tuners/tda182

Re: [PATCHv2 19/29] tuners: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Em Sat, 02 Nov 2013 18:25:19 +0100 Hans Verkuil escreveu: > Hi Mauro, > > I'll review this series more carefully on Monday, Thanks! > but for now I want to make > a suggestion for the array checks: > > On 11/02/2013 02:31 PM, Mauro Carvalho Chehab wrote: > > Dynamic static allocation is evil,

[PATCH] dw2102: Geniatech T220 support

2013-11-02 Thread CrazyCat
Support for Geniatech T220 DVB-T/T2/C USB stick. Signed-off-by: Evgeny Plehov diff --git a/drivers/media/usb/dvb-usb/dw2102.c b/drivers/media/usb/dvb-usb/dw2102.c index 6136a2c..12e00aa 100644 --- a/drivers/media/usb/dvb-usb/dw2102.c +++ b/drivers/media/usb/dvb-usb/dw2102.c @@ -2,7 +2,7 @@ *

[PATCH] cxd2820r_c: Fix if_ctl calculation

2013-11-02 Thread CrazyCat
Fix tune for DVB-C. Signed-off-by: Evgeny Plehov diff --git a/drivers/media/dvb-frontends/cxd2820r_c.c b/drivers/media/dvb-frontends/cxd2820r_c.c index 125a440..5c6ab49 100644 --- a/drivers/media/dvb-frontends/cxd2820r_c.c +++ b/drivers/media/dvb-frontends/cxd2820r_c.c @@ -78,7 +78,7 @@ int cxd

[PATCH] tda18271-fe: Fix dvb-c standard selection

2013-11-02 Thread CrazyCat
Fix dvb-c standard selection - qam8 for ANNEX_AC Signed-off-by: Evgeny Plehov diff --git a/drivers/media/tuners/tda18271-fe.c b/drivers/media/tuners/tda18271-fe.c index 4995b89..6a385c8 100644 --- a/drivers/media/tuners/tda18271-fe.c +++ b/drivers/media/tuners/tda18271-fe.c @@ -960,16 +960,12 @

Re: [kconfig] update: results of some syntactical checks

2013-11-02 Thread Paul Bolle
On Sat, 2013-11-02 at 17:40 -0200, Mauro Carvalho Chehab wrote: > Em Sat, 02 Nov 2013 20:20:54 +0100 > Paul Bolle escreveu: > > Those are obvious typos. Still present in v3.12-rc7. Perhaps you'd like > > to send the trivial patch to fix this? > > Yes, it is a typo... > > > > Probably, it was mea

[PATCH] [media] az6007: support Technisat Cablestar Combo HDCI (minus remote)

2013-11-02 Thread Roland Scheidegger
This is similar to the Terratec H7. It works with the same az6007 firmware as the former, however the drx-k firmware of the H7 will NOT work. Hence use a different firmware name. The firmware does not need to exist as the one in the eeprom is just fine as long as the h7 one doesn't get loaded, but

Re: [kconfig] update: results of some syntactical checks

2013-11-02 Thread Mauro Carvalho Chehab
Em Sat, 02 Nov 2013 20:20:54 +0100 Paul Bolle escreveu: > On Sun, 2013-10-20 at 00:03 +0200, Martin Walch wrote: > > drivers/media/common/siano/Kconfig:21-26 > > > config SMS_SIANO_DEBUGFS > > > bool "Enable debugfs for smsdvb" > > > depends on SMS_SIANO_MDTV > > > depends on DEBUG_FS > > >

Re: [kconfig] update: results of some syntactical checks

2013-11-02 Thread Paul Bolle
On Sun, 2013-10-20 at 00:03 +0200, Martin Walch wrote: > drivers/media/common/siano/Kconfig:21-26 > > config SMS_SIANO_DEBUGFS > > bool "Enable debugfs for smsdvb" > > depends on SMS_SIANO_MDTV > > depends on DEBUG_FS > > depends on SMS_USB_DRV > > depends on CONFIG_SMS_USB_DRV

status of Technisat CableStar Combi CI HD kernel support

2013-11-02 Thread Janusz S. Bien
I've just subscribed to the list following the advice given at http://www.linuxtv.org/wiki/index.php/DVB-C_USB_Devices: "If you own one or more devices from the following list and you want to help with support development, please contact the Linux-Media Mailing List" Adding the support fo

Re: [PATCHv2 19/29] tuners: Don't use dynamic static allocation

2013-11-02 Thread Hans Verkuil
Hi Mauro, I'll review this series more carefully on Monday, but for now I want to make a suggestion for the array checks: On 11/02/2013 02:31 PM, Mauro Carvalho Chehab wrote: > Dynamic static allocation is evil, as Kernel stack is too low, and > compilation complains about it on some archs: > >

Re: [PATCHv2 27/29] af9035: Don't use dynamic static allocation

2013-11-02 Thread Antti Palosaari
Acked-by: Antti Palosaari Reviewed-by: Antti Palosaari Antti On 02.11.2013 15:31, Mauro Carvalho Chehab wrote: Dynamic static allocation is evil, as Kernel stack is too low, and ompilation complains about it on some archs: drivers/media/usb/dvb-usb-v2/af9035.c:142:1: warning: 'af9035

Re: [PATCHv2 19/29] tuners: Don't use dynamic static allocation

2013-11-02 Thread Antti Palosaari
Acked-by: Antti Palosaari Reviewed-by: Antti Palosaari Antti On 02.11.2013 15:31, Mauro Carvalho Chehab wrote: Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/tuners/e4000.c:50:1: warning: 'e4000_wr_regs'

Re: [PATCHv2 14/29] dvb-frontends: Don't use dynamic static allocation

2013-11-02 Thread Antti Palosaari
Acked-by: Antti Palosaari Reviewed-by: Antti Palosaari Antti On 02.11.2013 15:31, Mauro Carvalho Chehab wrote: Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/dvb-frontends/af9013.c:77:1: warning: 'af9013

Re: [PATCHv2 26/29] af9015: Don't use dynamic static allocation

2013-11-02 Thread Antti Palosaari
ACK. IIRC I did macro optimization here and that used way gives few bytes smaller footprint =) Antti On 02.11.2013 15:31, Mauro Carvalho Chehab wrote: Dynamic static allocation is evil, as Kernel stack is too low, and ompilation complains about it on some archs: drivers/media/usb/dvb-

Re: [linux-media] Patch notification: 1 patch updated

2013-11-02 Thread Mauro Carvalho Chehab
Em Sat, 02 Nov 2013 14:00:13 +0100 Frank Schäfer escreveu: > Am 31.10.2013 13:13, schrieb Patchwork: > > Hello, > > > > The following patch (submitted by you) has been updated in patchwork: > > > > * linux-media: em28xx: make sure that all subdevices are powered on when > > needed > > - ht

[PATCHv2 24/29] dibusb-common: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/usb/dvb-usb/dibusb-common.c:124:1: warning: 'dibusb_i2c_msg' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for the buffer to

[PATCHv2 22/29] v4l2-async: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/v4l2-core/v4l2-async.c:238:1: warning: 'v4l2_async_notifier_unregister' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for th

[PATCHv2 21/29] cimax2: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/pci/cx23885/cimax2.c Instead, let's enforce a limit for the buffer. Considering that I2C transfers are generally limited, and that devices used on USB has a max d

[PATCHv2 17/29] stv090x: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/dvb-frontends/stv090x.c:750:1: warning: 'stv090x_write_regs.constprop.6' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for th

[PATCHv2 00/29] Fix errors/warnings with allmodconfig/allyesconfig on non-x86 archs

2013-11-02 Thread Mauro Carvalho Chehab
To be sure that we're not introducing compilation regressions on media, I'm now using ktest to check for errors/warnings. My current setup is cross-building on several architectures: alpha, arm, avr32, cris (64), frv, i386, ia64, m32r, m68k, mips, openrisc, parisc, s390, sh, sparc, sparc

[PATCHv2 09/29] radio-si470x-i2c: fix a warning on ia64

2013-11-02 Thread Mauro Carvalho Chehab
on ia64, those warnings appear: drivers/media/radio/si470x/radio-si470x-i2c.c:470:12: warning: 'si470x_i2c_suspend' defined but not used [-Wunused-function] drivers/media/radio/si470x/radio-si470x-i2c.c:487:12: warning: 'si470x_i2c_resume' defined but not used [-Wunused-function]

[PATCHv2 01/29] tda9887: remove an warning when compiling for alpha

2013-11-02 Thread Mauro Carvalho Chehab
There's no need to zero the buffer, as if the routine gets an error, rc will be different than one. That fixes the following warning: drivers/media/tuners/tda9887.c: In function 'tda9887_status': drivers/media/tuners/tda9887.c:539:2: warning: value computed is not used [-Wunused-v

[PATCHv2 10/29] rc: Fir warnings on m68k arch

2013-11-02 Thread Mauro Carvalho Chehab
Fix the following warnings: drivers/media/rc/fintek-cir.c: In function 'fintek_cr_write': drivers/media/rc/fintek-cir.c:45:2: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] drivers/media/rc/fintek-cir.c:46:2: warning: cast to pointer from i

[PATCHv2 11/29] uvc/lirc_serial: Fix some warnings on parisc arch

2013-11-02 Thread Mauro Carvalho Chehab
On this arch, usec is not unsigned long. So, we need to typecast, in order to remove those warnings: drivers/media/usb/uvc/uvc_video.c: In function 'uvc_video_clock_update': drivers/media/usb/uvc/uvc_video.c:678:2: warning: format '%lu' expects argument of type 'long unsigned int'

[PATCHv2 05/29] tef6862: fix warning on avr32 arch

2013-11-02 Thread Mauro Carvalho Chehab
On avr32 arch, we get those warnings: drivers/media/radio/tef6862.c:59:1: warning: "MODE_SHIFT" redefined In file included from /devel/v4l/ktest-build/arch/avr32/include/asm/ptrace.h:11, arch/avr32/include/uapi/asm/ptrace.h:41:1: warning: this is the location of the previo

[PATCHv2 28/29] mxl111sf: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/usb/dvb-usb-v2/mxl111sf.c:74:1: warning: 'mxl111sf_ctrl_msg' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for the buffer to

[PATCHv2 20/29] tuner-xc2028: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/tuners/tuner-xc2028.c:651:1: warning: 'load_firmware' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for the buffer. In the s

[PATCHv2 07/29] platform drivers: Fix build on cris and frv archs

2013-11-02 Thread Mauro Carvalho Chehab
On cris and frv archs, the functions below aren't defined: drivers/media/platform/sh_veu.c: In function 'sh_veu_reg_read': drivers/media/platform/sh_veu.c:228:2: error: implicit declaration of function 'ioread32' [-Werror=implicit-function-declaration] drivers/media/platfor

[PATCHv2 23/29] cxusb: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/usb/dvb-usb/cxusb.c:209:1: warning: 'cxusb_i2c_xfer' uses dynamic stack allocation [enabled by default] drivers/media/usb/dvb-usb/cxusb.c:69:1: warning: '

[PATCHv2 14/29] dvb-frontends: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/dvb-frontends/af9013.c:77:1: warning: 'af9013_wr_regs_i2c' uses dynamic stack allocation [enabled by default] drivers/media/dvb-frontends/af9033.c:188:1:

[PATCHv2 16/29] stv0367: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/dvb-frontends/stv0367.c:791:1: warning: 'stv0367_writeregs.constprop.4' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for th

[PATCHv2 18/29] av7110_hw: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/pci/ttpci/av7110_hw.c:510:1: warning: 'av7110_fw_cmd' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for the buffer. In the s

[PATCHv2 19/29] tuners: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/tuners/e4000.c:50:1: warning: 'e4000_wr_regs' uses dynamic stack allocation [enabled by default] drivers/media/tuners/e4000.c:83:1: warning: 'e4000_rd_reg

[PATCHv2 02/29] radio-shark: remove a warning when CONFIG_PM is not defined

2013-11-02 Thread Mauro Carvalho Chehab
On alpha, allyesconfig doesn't have CONFIG_PM, and produces the following warnings: drivers/media/radio/radio-shark.c:274:13: warning: 'shark_resume_leds' defined but not used [-Wunused-function] drivers/media/radio/radio-shark2.c:240:13: warning: 'shark_resume_leds' defined but

[PATCHv2 27/29] af9035: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and ompilation complains about it on some archs: drivers/media/usb/dvb-usb-v2/af9035.c:142:1: warning: 'af9035_wr_regs' uses dynamic stack allocation [enabled by default] drivers/media/usb/dvb-usb-v2/af9035.c:305:1: wa

[PATCHv2 08/29] cx18: disable compilation on frv arch

2013-11-02 Thread Mauro Carvalho Chehab
This driver produces a lot of errors on this arch: In file included from drivers/media/pci/cx18/cx18-driver.c:26:0: /drivers/media/pci/cx18/cx18-io.h: In function 'cx18_raw_readl': drivers/media/pci/cx18/cx18-io.h:40:2: warning: passing argument 1 of '__builtin_read32' disc

[PATCHv2 03/29] zoran: don't build it on alpha

2013-11-02 Thread Mauro Carvalho Chehab
This driver uses virt_to_bus() with is deprecated on Alpha: drivers/media/pci/zoran/zoran_device.c: In function 'zr36057_set_vfe': drivers/media/pci/zoran/zoran_device.c:451:3: warning: 'virt_to_bus' is deprecated (declared at /devel/v4l/ktest-build/arch/alpha/include/asm/io.h:114

[PATCHv2 26/29] af9015: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and ompilation complains about it on some archs: drivers/media/usb/dvb-usb-v2/af9015.c:433:1: warning: 'af9015_eeprom_hash' uses dynamic stack allocation [enabled by default] In this specific case, it is a gcc bug, as the siz

[PATCHv2 15/29] stb0899_drv: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/dvb-frontends/stb0899_drv.c:540:1: warning: 'stb0899_write_regs' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for the buffer

[PATCHv2 25/29] dw2102: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and ompilation complains about it on some archs: drivers/media/usb/dvb-usb/dw2102.c:368:1: warning: 'dw2102_earda_i2c_transfer' uses dynamic stack allocation [enabled by default] drivers/media/usb/dvb-usb/dw2102.c:449:

[PATCHv2 12/29] s5h1420: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/dvb-frontends/s5h1420.c:851:1: warning: 's5h1420_tuner_i2c_tuner_xfer' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for the

[PATCHv2 29/29] lirc_zilog: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and ompilation complains about it on some archs: drivers/staging/media/lirc/lirc_zilog.c:967:1: warning: 'read' uses dynamic stack allocation [enabled by default] Instead, let's enforce a limit for the buffer to be 80. That s

[PATCHv2 13/29] dvb-frontends: Don't use dynamic static allocation

2013-11-02 Thread Mauro Carvalho Chehab
Dynamic static allocation is evil, as Kernel stack is too low, and compilation complains about it on some archs: drivers/media/dvb-frontends/bcm3510.c:230:1: warning: 'bcm3510_do_hab_cmd' uses dynamic stack allocation [enabled by default] drivers/media/dvb-frontends/itd1000.c:69:1

[PATCHv2 06/29] iguanair: shut up a gcc warning on avr32 arch

2013-11-02 Thread Mauro Carvalho Chehab
drivers/media/rc/iguanair.c: In function 'iguanair_set_tx_carrier': drivers/media/rc/iguanair.c:304: warning: 'sevens' may be used uninitialized in this function This is clearly a gcc bug, but it doesn't hurt to add a default line at the switch to shut it up. Signed-off-by: Mauro

Re: [linux-media] Patch notification: 1 patch updated

2013-11-02 Thread Frank Schäfer
Am 31.10.2013 13:13, schrieb Patchwork: > Hello, > > The following patch (submitted by you) has been updated in patchwork: > > * linux-media: em28xx: make sure that all subdevices are powered on when > needed > - http://patchwork.linuxtv.org/patch/20422/ > - for: Linux Media kernel patc

[PATCH v4] videobuf2: Add missing lock held on vb2_fop_relase

2013-11-02 Thread Ricardo Ribalda Delgado
From: Ricardo Ribalda vb2_fop_relase does not held the lock although it is modifying the queue->owner field. This could lead to race conditions on the vb2_perform_io function when multiple applications are accessing the video device via read/write API: [ 308.297741] BUG: unable to handle kernel

Re: [[PATCH v3]] videobuf2: Add missing lock held on vb2_fop_relase

2013-11-02 Thread Ricardo Ribalda Delgado
Hello Sylwester Thanks for your comments. There is a new patch: v4! :) On Fri, Nov 1, 2013 at 11:36 PM, Sylwester Nawrocki wrote: > Hi Ricardo, > > > On 10/31/2013 09:54 PM, Ricardo Ribalda Delgado wrote: >> >> From: Ricardo Ribalda >> >> vb2_fop_relase does not held the lock although it is modi