Re: [PATCH RFC v3 2/6] v4l2-ctrl: Add helper function for control range update

2013-03-11 Thread Hans Verkuil
Hi Sylwester, On Wed January 23 2013 23:21:57 Sylwester Nawrocki wrote: > This patch adds a helper function that allows to modify range, > i.e. minimum, maximum, step and default value of a v4l2 control, > after the control has been created and initialized. This is helpful > in situations when ran

Re: [PATCH] media: davinci: kconfig: fix incorrect selects

2013-03-11 Thread Prabhakar Lad
Hi Sekhar, Thanks for the patch! few nits below also version number for patch is missing as this should have been v2 :) BTW this patch still is not present in media list. On Mon, Mar 11, 2013 at 5:52 PM, Sekhar Nori wrote: > drivers/media/platform/davinci/Kconfig uses selects where > it should

Re: [PATCH -next] [media] davinci: vpfe: fix return value check in vpfe_enable_clock()

2013-03-11 Thread Prabhakar Lad
Hi Wei, Thanks for the patch! I'll queue it up for v3.10 On Mon, Mar 11, 2013 at 7:27 PM, Wei Yongjun wrote: > From: Wei Yongjun > > In case of error, the function clk_get() returns ERR_PTR() > and never returns NULL. The NULL test in the return value > check should be replaced with IS_ERR(). >

Re: [REVIEW PATCH 11/15] au0828: fix disconnect sequence.

2013-03-11 Thread Devin Heitmueller
On Mon, Mar 11, 2013 at 5:00 PM, Hans Verkuil wrote: > From: Hans Verkuil > > The driver crashed when the device was disconnected while an application > still had a device node open. Fixed by using the release() callback of struct > v4l2_device. This is all obviously good stuff. I actually spen

Re: [PATCH v7] [media] Add a V4L2 OF parser

2013-03-11 Thread Sylwester Nawrocki
Hi Guennadi, On 03/11/2013 04:03 PM, Guennadi Liakhovetski wrote: Hi Sylwester Thanks for continuing this work! You have made a great progress compared to my initial version, and I should really have looked at each your submitted new revision, unfortunately, I haven't managed that. So, sorry fo

Re: Fw: [patch 02/03 v2] usb hid quirks for Masterkit MA901 usb radio

2013-03-11 Thread Alexey Klimov
Hi Jiri and Mauro, all, On Fri, Dec 28, 2012 at 4:29 PM, Mauro Carvalho Chehab wrote: > Hi Jiri, > > There's another radio device that it is incorrectly detected as an HID driver. > As I'll be applying the driver's patch via the media tree, do you mind if I > also > apply this hid patch there? >

Re: [PATCH] m920x: silence compiler warning

2013-03-11 Thread Antonio Ospite
On Mon, 11 Mar 2013 19:25:10 +0200 Antti Palosaari wrote: > drivers/media/usb/dvb-usb/m920x.c: In function ‘m920x_probe’: > drivers/media/usb/dvb-usb/m920x.c:91:6: warning: ‘ret’ may be used > uninitialized in this function [-Wuninitialized] > drivers/media/usb/dvb-usb/m920x.c:70:6: note: ‘ret’

Re: [PATCH] block i2c tuner reads for Avermedia Twinstar in the af9035 driver

2013-03-11 Thread Antti Palosaari
On 03/11/2013 10:02 PM, Jose Alberto Reguero wrote: On Lunes, 11 de marzo de 2013 14:57:37 Antti Palosaari escribió: On 03/11/2013 01:51 PM, Jose Alberto Reguero wrote: On Lunes, 11 de febrero de 2013 14:48:18 Jose Alberto Reguero escribió: On Domingo, 10 de febrero de 2013 22:11:53 Antti Palo

Re: [REVIEW PATCH 00/42] go7007: complete overhaul

2013-03-11 Thread Hans Verkuil
On Mon March 11 2013 12:45:38 Hans Verkuil wrote: > Hi all, > > This patch series updates the staging go7007 driver to the latest > V4L2 frameworks and actually makes it work reliably. > > Some highlights: > > - moved the custom i2c drivers to media/i2c. > - replaced the s2250-loader by a common

[PATCH] [media] soc_camera: remove two outdated selects

2013-03-11 Thread Paul Bolle
Release v2.6.30 removed the MT9M001_PCA9536_SWITCH and MT9V022_PCA9536_SWITCH Kconfig symbols, in commits 36034dc325ecab63c8cfb992fbf9a1a8e94738a2 ("V4L/DVB (11032): mt9m001: allow setting of bus width from board code") and e958e27adeade7fa085dd396a8a0dfaef7e338c1 ("V4L/DVB (11033): mt9v022: allow

Re: Custom device names for v4l2 devices

2013-03-11 Thread Laurent Pinchart
Hi Vinay, On Monday 11 March 2013 10:55:37 vka...@codeaurora.org wrote: > > Names of V4L2 device nodes keep on varying depending on target, on some > > targets, the device node assigned to my device is /dev/video21 and on some > > it is /dev/video15. In order to determine my device, i am opening i

cron job: media_tree daily build: WARNINGS

2013-03-11 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: Mon Mar 11 19:00:22 CET 2013 git branch: test git hash: 457ba4ce4f435d0b4dd82a0acc6c796e541a2ea7 gcc versio

[REVIEW PATCH 03/15] au0828: frequency handling fixes.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil - define an initial frequency - return an error if g_frequency is called for an invalid tuner index - get the clamped frequency value after setting it: i.e. the tuner driver may clamp the given frequency to a valid frequency range and ctrl_freq should get that actual clampe

[REVIEW PATCH 04/15] au0828: fix intendation coding style issue.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil No code change, just fixing a wrong indentation. Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au0828/au0

[REVIEW PATCH 08/15] au0828: add try_fmt_vbi support, zero vbi.reserved, pix.priv.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Also get rid of unnecessary format type check. Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c |7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au0828/au0828-vid

[REVIEW PATCH 05/15] au0828: fix audio input handling.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil - V4L2_CAP_AUDIO was set, but enumaudio was not implemented. - audioset was never filled by enum_input - ctrl_ainput was never updated when switching the video input - g_audio was broken due to faulty logic: g_audio should set the index, it doesn't receive it from the user.

[REVIEW PATCH 09/15] au0828: replace deprecated current_norm by g_std.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c | 15 +-- drivers/media/usb/au0828/au0828.h |1 + 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au082

[REVIEW PATCH 12/15] au0828: simplify i2c_gate_ctrl.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Turn it into a simple function. Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au0828/au0828

[REVIEW PATCH 11/15] au0828: fix disconnect sequence.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil The driver crashed when the device was disconnected while an application still had a device node open. Fixed by using the release() callback of struct v4l2_device. Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-core.c | 48 ---

[REVIEW PATCH 15/15] au0828: improve firmware loading & locking.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil - open/close/read and poll need to take the core lock as well. - when the tuner goes to sleep we should set std_set_in_tuner_core to 0 since the tuner loses the firmware at that time. - initialize the tuner if std_set_in_tuner_core == 0 whenever: 1) g/s_tuner, s_std or s_fr

[REVIEW PATCH 14/15] au0828: fix initial video routing.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil After loading the module the initial video routing is not setup. Explicitly call s_input to get this right. Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c | 30 ++ 1 file changed, 18 insertions(+), 12 deletions(-) diff

[REVIEW PATCH 06/15] au0828: convert to the control framework.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-core.c | 15 ++-- drivers/media/usb/au0828/au0828-video.c | 39 ++- drivers/media/usb/au0828/au0828.h |2 ++ 3 files changed, 17 insertions(+), 39 deletions(-)

[REVIEW PATCH 13/15] au0828: don't change global state information on open().

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Just opening a device shouldn't have any side-effects. Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c |9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au0828/

[REVIEW PATCH 02/15] au0828: fix querycap.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-video.c b/drivers/media/usb/au0828/au0828-video.c index 8b9e826..4254b2c 100644 --

[REVIEW PATCH 10/15] au8522_decoder: remove obsolete control ops.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Now that au0828 has been converted to the control framework these compatilibity ops are no longer needed. Signed-off-by: Hans Verkuil --- drivers/media/dvb-frontends/au8522_decoder.c |7 --- 1 file changed, 7 deletions(-) diff --git a/drivers/media/dvb-frontends/au8

[REVIEW PATCH 01/15] au8522_decoder: convert to the control framework.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/dvb-frontends/au8522_decoder.c | 130 +- drivers/media/dvb-frontends/au8522_priv.h|6 +- 2 files changed, 46 insertions(+), 90 deletions(-) diff --git a/drivers/media/dvb-frontends/au8522_decoder.

[REVIEW PATCH 07/15] au0828: add prio, control event and log_status support

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/media/usb/au0828/au0828-video.c | 42 --- drivers/media/usb/au0828/au0828.h |4 +++ 2 files changed, 37 insertions(+), 9 deletions(-) diff --git a/drivers/media/usb/au0828/au0828-video.c b/drive

[REVIEW PATCH 00/15] au0828: v4l2-compliance cleanups

2013-03-11 Thread Hans Verkuil
Hi all, This patch series converts the au0828/au8522 drivers to the latest frameworks, except for vb2 as usual. Tested with a WinTV aero generously donated by Hauppauge some time ago. I also did a lot of fixes in the disconnect handling and setting up the right routing/std information at the rig

Re: omap3isp: iommu register problem.

2013-03-11 Thread Laurent Pinchart
Hi Javier, On Monday 11 March 2013 16:28:58 javier Martin wrote: > On 11 March 2013 16:01, Laurent Pinchart wrote: > > On Monday 11 March 2013 13:18:12 javier Martin wrote: > >> I've just found the following thread where te problem is explained: > >> http://lists.infradead.org/pipermail/linux-arm-

Re: [PATCH] block i2c tuner reads for Avermedia Twinstar in the af9035 driver

2013-03-11 Thread Jose Alberto Reguero
On Lunes, 11 de marzo de 2013 14:57:37 Antti Palosaari escribió: > On 03/11/2013 01:51 PM, Jose Alberto Reguero wrote: > > On Lunes, 11 de febrero de 2013 14:48:18 Jose Alberto Reguero escribió: > >> On Domingo, 10 de febrero de 2013 22:11:53 Antti Palosaari escribió: > >>> On 02/10/2013 09:43 PM,

[RFC PATCH 8/8] s5p-fimc: Create media links for the FIMC-IS entities

2013-03-11 Thread Sylwester Nawrocki
Create disabled links from the FIMC-LITE subdevs to the FIMC-IS-ISP subdev and from FIMC-IS-ISP to all FIMC subdevs. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/s5p-fimc/fimc-mdevice.c | 78 +++- 1 file changed, 62 insertions(+

[RFC PATCH 6/8] fimc-is: Add Exynos4x12 FIMC-IS device tree bindings documentation

2013-03-11 Thread Sylwester Nawrocki
Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- .../devicetree/bindings/media/exynos4-fimc-is.txt | 41 1 file changed, 41 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/exynos4-fimc-is.txt diff --git a/Documentation/devicetr

[RFC PATCH 7/8] s5p-fimc: Add fimc-is subdevs registration

2013-03-11 Thread Sylwester Nawrocki
This patch allows to register FIMC-IS device represented by FIMC-IS-ISP subdev to the top level media device driver. The use_isp platform data structure field allows to select whether the fimc-is ISP subdev should be tried to be registered or not. Signed-off-by: Sylwester Nawrocki Signed-off-by:

[RFC PATCH 5/8] s5p-fimc: Add ISP video capture driver stubs

2013-03-11 Thread Sylwester Nawrocki
This patch adds a video capture node for the FIMC-IS ISP IP block and Makefile/Kconfig to actually enable the driver's compilation. The ISP video capture driver is still a work in progress. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/s5p-fimc/Kconf

[RFC PATCH 4/8] s5p-fimc: Add common FIMC-IS image sensor driver

2013-03-11 Thread Sylwester Nawrocki
This subdev driver currently only handles an image sensor's power supplies and reset signal. There is no any I2C communication here as it is handled by the ISP's firmware. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/s5p-fimc/fimc-is-sensor.c | 308

[RFC PATCH 3/8] s5p-fimc: Add FIMC-IS parameter region definitions

2013-03-11 Thread Sylwester Nawrocki
This patch adds ISP processing parameters interface files. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/s5p-fimc/fimc-is-param.c | 971 + drivers/media/platform/s5p-fimc/fimc-is-param.h | 1018 +++ 2 files cha

[RFC PATCH 2/8] s5p-fimc: Add FIMC-IS ISP I2C bus driver

2013-03-11 Thread Sylwester Nawrocki
This patch adds the ISP I2C bus controller driver files. Creating a standard I2C bus adapter, even if the driver doesn't actually communicates with the hardware and it is instead used by the ISP firmware running on the Cortex-A5, allows to use standard hardware description in the device tree. As t

[RFC PATCH 0/8] A V4L2 driver for Exynos4x12 Imaging Subsystem

2013-03-11 Thread Sylwester Nawrocki
This patch series is an initial version of a driver for the camera ISP subsystem (FIMC-IS) embedded in Samsung Exynos4x12 SoCs. The FIMC-IS subsystem is build around a ARM Cortex-A5 CPU that controls its dedicated peripherals, like I2C, SPI, UART, PWM, ADC,... and the ISP chain. There are 3 hardw

[PATCH RFC 11/11] V4L: Add MATRIX option to V4L2_CID_EXPOSURE_METERING control

2013-03-11 Thread Sylwester Nawrocki
This patch adds a menu option to the V4L2_CID_EXPOSURE_METERING control for multi-zone metering. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- Documentation/DocBook/media/v4l/controls.xml |9 - drivers/media/v4l2-core/v4l2-ctrls.c |1 + include/uapi/

[PATCH RFC 10/11] s5p-fimc: Remove dependency on fimc-core.h in fimc-lite driver

2013-03-11 Thread Sylwester Nawrocki
Drop fimc-lite.h header inclusion to make the exynos-fimc-lite module independent on other modules. Move struct fimc_fmt declaration to the driver's private headers as it is used in multiple modules. Reported-by: Shaik Ameer Basha Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park -

[PATCH RFC 09/11] s5p-fimc: Ensure proper s_power() call order in the ISP datapaths

2013-03-11 Thread Sylwester Nawrocki
Since the FIMC-IS firmware communicates with an image sensor directly through the ISP I2C bus controllers the sub-devices power supplies cannot be simply enabled from left to right or disabled from right to left along the processing pipeline. Thus a subdev index to call s_power() on is looked up fr

[PATCH RFC 08/11] s5p-fimc: Ensure proper s_stream() call order in the ISP datapaths

2013-03-11 Thread Sylwester Nawrocki
Since the FIMC-IS firmware communicates with an image sensor directly through the ISP I2C bus controllers data streaming cannot be simply enabled from left to right or disabled from right to left along the processing pipeline. Thus a subdev index to call s_stream() on is looked up from a table, rat

[PATCH RFC 07/11] s5p-fimc: Ensure CAMCLK clock can be enabled by FIMC-LITE devices

2013-03-11 Thread Sylwester Nawrocki
In configurations where FIMC-LITE is used to capture image signal from an external sensor only we need to ensure one of FIMC devices is in active power state and the "fimc" gate clock is enabled. Otherwise the CAMCLK clock output signal will be masked off preventing an external sensor's operation.

[PATCH RFC 06/11] s5p-fimc: Add the FIMC ISP writeback input support

2013-03-11 Thread Sylwester Nawrocki
A second sink pad is added to each FIMC.N subdev that will be used to link it to the ISP subdev. Only V4L2_MBUS_FMT_YUV10_1X30 format is supported at the pad FIMC_SD_PAD_SINK_FIFO. TODO: - Implement the FIMC input bus type selection based on state of media link from FIMC-IS-ISP to FIMC.N subde

[PATCH RFC 05/11] s5p-fimc: Add support for PIXELASYNCMx clocks

2013-03-11 Thread Sylwester Nawrocki
This patch ads handling of clocks for the CAMBLK subsystem which is a glue logic for FIMC-IS or LCD controller and FIMC IP. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/s5p-fimc/fimc-mdevice.c | 41 +++- drivers/media/platform/s

[PATCH RFC 04/11] s5p-fimc: Update graph traversal for entities with multiple source pads

2013-03-11 Thread Sylwester Nawrocki
We cannot assume that the passed entity the fimc_pipeline_prepare() function is supposed to start the media graph traversal from will always have its sink pad at pad index 0. Find the starting media entity's sink pad by iterating over its all pads and checking the pad flags. This ensures proper han

[PATCH RFC 03/11] s5p-csis: Add parent clock setup

2013-03-11 Thread Sylwester Nawrocki
With this patch the driver will set "parent" clock as a parent clock of "mux" clock. When the samsung clocks driver is reworked to use new composite clock type, the "mux" clock can be removed. "parent" clock should be set in relevant dtsi file and can be overwritten in a board dts file. This way i

[PATCH RFC 02/11] s5p-fimc: Add parent clock setup

2013-03-11 Thread Sylwester Nawrocki
With this patch the driver will set "parent" clock as a parent clock of "mux" clock. When the samsung clocks driver is reworked to use new composite clock type, the "mux" clock can be removed. "parent" clock should be set in related dtsi file and can be overwritten in a board dts file. This way it

[PATCH RFC 01/11] s5p-fimc: Added error checks for pipeline stream on callbacks

2013-03-11 Thread Sylwester Nawrocki
From: Andrzej Hajda set_stream error for pipelines is logged or reported to user space if possible. Signed-off-by: Andrzej Hajda Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- drivers/media/platform/s5p-fimc/fimc-capture.c | 15 ++- 1 file changed, 10 insert

[PATCH RFC 00/11] s5p-fimc: Exynos4x12 FIMC-IS support prerequisite

2013-03-11 Thread Sylwester Nawrocki
This patch series contains couple fixes to the s5p-fimc driver and changes necessary for the Exynos4x12 FIMC-IS support. Andrzej Hajda (1): s5p-fimc: Added error checks for pipeline stream on callbacks Sylwester Nawrocki (10): s5p-fimc: Add parent clock setup s5p-csis: Add parent clock setu

Re: Custom device names for v4l2 devices

2013-03-11 Thread vkalia
Please suggest. Thanks Vinay > Hi > > Names of V4L2 device nodes keep on varying depending on target, on some > targets, the device node assigned to my device is /dev/video21 and on some > it is /dev/video15. In order to determine my device, i am opening it, > reading the capabilities, enumeratin

[PATCH] m920x: silence compiler warning

2013-03-11 Thread Antti Palosaari
drivers/media/usb/dvb-usb/m920x.c: In function ‘m920x_probe’: drivers/media/usb/dvb-usb/m920x.c:91:6: warning: ‘ret’ may be used uninitialized in this function [-Wuninitialized] drivers/media/usb/dvb-usb/m920x.c:70:6: note: ‘ret’ was declared here Signed-off-by: Antti Palosaari --- drivers/medi

[PATCH] [media] ts2020: use customise option correctly

2013-03-11 Thread Paul Bolle
The Kconfig entry for "TS2020 based tuners" defaults to modular if DVB_FE_CUSTOMISE is set. But that Kconfig symbol was replaced with MEDIA_SUBDRV_AUTOSELECT as of v3.7. So use the new symbol. And negate the logic, so we are in line with all the similar entries in this file. Signed-off-by: Paul Bo

Re: omap3isp: iommu register problem.

2013-03-11 Thread javier Martin
Hi Laurent, thank you for your answer. On 11 March 2013 16:01, Laurent Pinchart wrote: > Hi Javier, > > On Monday 11 March 2013 13:18:12 javier Martin wrote: >> I've just found the following thread where te problem is explained: >> http://lists.infradead.org/pipermail/linux-arm-kernel/2012-Februa

Re: [PATCH v7] [media] Add a V4L2 OF parser

2013-03-11 Thread Guennadi Liakhovetski
Hi Sylwester Thanks for continuing this work! You have made a great progress compared to my initial version, and I should really have looked at each your submitted new revision, unfortunately, I haven't managed that. So, sorry for chiming back in so late in the game, but maybe we still manage t

Re: omap3isp: iommu register problem.

2013-03-11 Thread Laurent Pinchart
Hi Javier, On Monday 11 March 2013 13:18:12 javier Martin wrote: > I've just found the following thread where te problem is explained: > http://lists.infradead.org/pipermail/linux-arm-kernel/2012-February/086364.h > tml > > The problem is related with the order iommu and omap3isp are probed > whe

Re: [PATCH] block i2c tuner reads for Avermedia Twinstar in the af9035 driver

2013-03-11 Thread Antti Palosaari
On 03/11/2013 01:51 PM, Jose Alberto Reguero wrote: On Lunes, 11 de febrero de 2013 14:48:18 Jose Alberto Reguero escribió: On Domingo, 10 de febrero de 2013 22:11:53 Antti Palosaari escribió: On 02/10/2013 09:43 PM, Jose Alberto Reguero wrote: This patch block the i2c tuner reads for Avermedi

Re: omap3isp: iommu register problem.

2013-03-11 Thread javier Martin
I've just found the following thread where te problem is explained: http://lists.infradead.org/pipermail/linux-arm-kernel/2012-February/086364.html The problem is related with the order iommu and omap3isp are probed when both are built-in. If I load omap3isp as a module the problem is gone. Howev

Re: dvb-apps: Additional channels for Netherlands

2013-03-11 Thread Oliver Schinagl
On 09-03-13 15:45, Dmitry Katsubo wrote: Dear LinuxTV developers, Hoi Dimitry, When I was playing with "scan" utility, I paid attention to comments in file /usr/share/dvb/dvb-t/nl-All # The Netherlands, whole country # Created from http://radio-tv-nederland.nl/TV%20zenderlijst%20Nederland.xl

[REVIEW PATCH 19/42] s2250-loader: use usbv2_cypress_load_firmware

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil The v2 of this function doesn't do DMA to objects on the stack like its predecessor does. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/Makefile |4 ++-- drivers/staging/media/go7007/s2250-loader.c |7 --- 2 files changed, 6 insertions(+), 5

[REVIEW PATCH 22/42] go7007: fix DMA related errors.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil - Don't pass data allocated on the stack to usb_control_msg. - Use dma_mapping_error after calling dma_map_page(). Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-priv.h|1 + drivers/staging/media/go7007/go7007-usb.c | 36 +++

Re: [PATCH] block i2c tuner reads for Avermedia Twinstar in the af9035 driver

2013-03-11 Thread Jose Alberto Reguero
On Lunes, 11 de febrero de 2013 14:48:18 Jose Alberto Reguero escribió: > On Domingo, 10 de febrero de 2013 22:11:53 Antti Palosaari escribió: > > On 02/10/2013 09:43 PM, Jose Alberto Reguero wrote: > > > This patch block the i2c tuner reads for Avermedia Twinstar. If it's > > > needed other pids c

Re: [REVIEW PATCH 01/42] v4l2-ctrls: eliminate lockdep false alarms for struct v4l2_ctrl_handler.lock

2013-03-11 Thread Hans Verkuil
On Mon March 11 2013 12:45:39 Hans Verkuil wrote: > From: Andy Walls Oops, somehow I messed up Andy's email. I've corrected it in my git tree. Regards, Hans > > When calling v4l2_ctrl_add_handler(), lockdep would detect a potential > recursive locking problem on a situation that is by

[REVIEW PATCH 15/42] go7007: i2c initialization changes for tw2804

2013-03-11 Thread Hans Verkuil
From: Volokh Konstantin Do i2c initialization via struct item as tw2804 has a 0x00 i2c address, so we need to use the I2C_CLIENT_TEN flag for validity. Signed-off-by: Volokh Konstantin Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-driver.c | 20 +++- dr

[REVIEW PATCH 10/42] ov7640: add new ov7640 driver

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil This based on the wis-ov7640.c driver that's part of the go7007 driver. It has been converted to a v4l subdev driver by Pete Eberlein, and I made additional cleanups. Based on work by: Pete Eberlein Signed-off-by: Hans Verkuil Cc: Pete Eberlein --- drivers/media/i2c/Kconf

[REVIEW PATCH 11/42] uda1342: add new uda1342 audio codec driver

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil This based on the wis-uda1342.c driver that's part of the go7007 driver. It has been converted to a v4l subdev driver by Pete Eberlein, and I made additional cleanups. Based on work by: Pete Eberlein Signed-off-by: Hans Verkuil Cc: Pete Eberlein --- drivers/media/i2c/Kcon

[REVIEW PATCH 18/42] go7007: add audio input ioctls.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Since we now know what audio inputs there are, we can also get/set and enumerate them. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-v4l2.c | 51 ++-- 1 file changed, 48 insertions(+), 3 deletions(-) diff --git a/drivers/stagi

[REVIEW PATCH 20/42] go7007: go7007: add device_caps and bus_info support to querycap.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil And don't set the version field, the core does that for you. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-priv.h|1 + drivers/staging/media/go7007/go7007-usb.c |1 + drivers/staging/media/go7007/go7007-v4l2.c| 10 +++--- drive

[REVIEW PATCH 16/42] go7007: switch to standard tuner/i2c subdevs.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Instead of using the wis-* drivers we now use the standard 'proper' subdev drivers. The board configuration tables now also list the possible audio inputs, this will be used later to implement audio inputs. Special mention deserves a little change in set_capture_size() where

[REVIEW PATCH 07/42] go7007: fix i2c_xfer return codes.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil The i2c_xfer functions didn't return the proper error codes and (especially important) on success they returned 0 instead of the number of transferred messages. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-i2c.c | 20 ++-- drivers/sta

[REVIEW PATCH 12/42] tw9903: add new tw9903 video decoder.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil This based on the wis-tw9903.c driver that's part of the go7007 driver. It has been converted to a v4l subdev driver by Pete Eberlein, and I made additional cleanups. Based on work by: Pete Eberlein Signed-off-by: Hans Verkuil Cc: Pete Eberlein --- drivers/media/i2c/Kconf

[REVIEW PATCH 25/42] go7007: convert to the control framework and remove obsolete JPEGCOMP support.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Just add a read-only V4L2_CID_JPEG_ACTIVE_MARKER control to replace the JPEGCOMP support. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-driver.c | 11 +- drivers/staging/media/go7007/go7007-priv.h |3 + drivers/staging/media/go7007/go7007-v4l2.c

[REVIEW PATCH 28/42] go7007: add log_status support.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-v4l2.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/staging/media/go7007/go7007-v4l2.c b/drivers/staging/media/go7007/go7007-v4l2.c index 186a56b..d3eef8d 100644 --

[REVIEW PATCH 31/42] go7007: simplify the PX-TV402U board ID handling.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil There really is no need to split out the board IDs for each tuner. That's what the tuner_type is for, after all. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-priv.h | 15 ++- drivers/staging/media/go7007/go7007-usb.c |9 +++-- dr

[REVIEW PATCH 33/42] go7007: drop struct go7007_file

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Remove struct go7007_file: all fields contained in that struct are moved to the go7007 struct since they are really global values. The lock has just been deleted (what's the point of a per-fh lock??). Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-priv.h

[REVIEW PATCH 21/42] go7007: remove current_norm.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil It's deprecated and replaced by g_std. Since this driver already implements g_std the use of current_norm can just be removed. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-v4l2.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/med

[REVIEW PATCH 14/42] tw2804: modify ADC power control

2013-03-11 Thread Hans Verkuil
From: Volokh Konstantin Switch off all ADC (max 4) with first init, we control it when starting/stopping streaming. Signed-off-by: Volokh Konstantin Signed-off-by: Hans Verkuil --- drivers/media/i2c/tw2804.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/i2c/tw2804.c b/d

[REVIEW PATCH 34/42] go7007: convert to core locking and vb2.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Convert this driver to videobuf2 and core locking. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/Kconfig |2 +- drivers/staging/media/go7007/go7007-driver.c | 156 --- drivers/staging/media/go7007/go7007-priv.h| 27 +- drivers/staging/

[REVIEW PATCH 39/42] go7007-loader: renamed from s2250-loader

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil All s2250 names are renamed to go7007. This will be the generic go7007 firmware loader for any go7007 device, not just for the s2250/1. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/Kconfig | 24 ++-- drivers/staging/media/go7007/Makefile|

[REVIEW PATCH 40/42] go7007-loader: add support for the other devices and move fw files

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Add support for the other devices that need to load the boot firmware. All firmware files are now placed in a single go7007 directory. Also remove the device_extension_s stuff: this is clearly a left-over from the olden days. Signed-off-by: Hans Verkuil --- drivers/staging/

[REVIEW PATCH 37/42] saa7134-go7007: add support for this combination.

2013-03-11 Thread Hans Verkuil
From: Pete Eberlein Add support for the Sensoray model 614 board, which is a saa7134 with a go7007 MPEG encoder. Signed-off-by: Pete Eberlein [hans.verk...@cisco.com: updated to make it merge correctly] Signed-off-by: Hans Verkuil --- drivers/media/pci/saa7134/saa7134-cards.c | 29 +

[REVIEW PATCH 42/42] MAINTAINERS: add the go7007 driver.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- MAINTAINERS |5 + 1 file changed, 5 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index ff2fcc9..5703edd 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7550,6 +7550,11 @@ M: David Täht S: Odd Fixes F: drivers/st

[REVIEW PATCH 41/42] go7007: update the README

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/README | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/staging/media/go7007/README b/drivers/staging/media/go7007/README index aeba132..779fe3f 100644 --- a/drivers/sta

[REVIEW PATCH 38/42] s2250: add comment describing the hardware.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/s2250-board.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/staging/media/go7007/s2250-board.c b/drivers/staging/media/go7007/s2250-board.c index cb36427..beaa98b 100644 --- a/drivers/stagin

[REVIEW PATCH 36/42] go7007: remove cropping functions

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Remove these dummy cropping functions: cropping was never implemented. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-v4l2.c | 92 1 file changed, 92 deletions(-) diff --git a/drivers/staging/media/go7007/go7007-v4l2.c b/

[REVIEW PATCH 35/42] go7007: embed struct video_device

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Do not allocate it, but just embed in the go7007 struct. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-driver.c |1 - drivers/staging/media/go7007/go7007-priv.h|2 +- drivers/staging/media/go7007/go7007-usb.c |2 +- drivers/staging/

[REVIEW PATCH 32/42] go7007: set up the saa7115 audio clock correctly.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil The s_crystal_freq operation has to be called for the saa7115 to set up the audio clock correctly. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-priv.h |1 + drivers/staging/media/go7007/go7007-usb.c |5 - drivers/staging/media/go7007/go700

[REVIEW PATCH 30/42] go7007: standardize MPEG handling support.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil The go7007 produces elementary streams, so we shouldn't use the STREAM_TYPE control, since that is for multiplexed streams. Instead use V4L2_PIX_FMT_MPEG1/2/4. Initially set up all the values for MPEG-2 dvd-mode, and select the dvd_mode field if those values match that setup

[REVIEW PATCH 29/42] go7007: tuner/std related fixes.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil - The Adlink is initially detected as a sensor board, so the driver config is set up as if it was a sensor. Later the driver discovers that it really is a video capture card and that means that the driver configuration has to be updated for a PAL/NTSC capture card. - Set

[REVIEW PATCH 26/42] s2250: convert to the control framework.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/s2250-board.c | 149 +--- 1 file changed, 46 insertions(+), 103 deletions(-) diff --git a/drivers/staging/media/go7007/s2250-board.c b/drivers/staging/media/go7007/s2250-board.c index 2266

[REVIEW PATCH 27/42] go7007: add prio and control event support.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go7007-priv.h |2 ++ drivers/staging/media/go7007/go7007-v4l2.c | 23 ++- 2 files changed, 20 insertions(+), 5 deletions(-) diff --git a/drivers/staging/media/go7007/go7007-priv.h b/drive

[REVIEW PATCH 24/42] go7007: fix unregister/disconnect handling.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil - use the v4l2_device's release() callback - remove the unnecessary ref_count - don't free usb data structures on disconnect, only do that in the final release callback. This is the correct way in order to safely handle disconnect and removal of modules. Signed-off-by: Hans

[REVIEW PATCH 23/42] go7007: remember boot firmware.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Don't load it everytime you stop encoding. Instead remember it. Another reason for not loading it every time is that this could be called from within the release() file operation, which turns out to be deadly. Signed-off-by: Hans Verkuil --- drivers/staging/media/go7007/go70

[REVIEW PATCH 09/42] sony-btf-mpx: the MPX driver for the sony BTF PAL/SECAM tuner

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil The Sony BTF PG472Z has an internal MPX to deal with mono/stereo/bilingual audio. This is split off from the wis-sony-tuner driver that is part of the go7007 driver as it should be a separate i2c sub-device driver. The wis-sony-tuner is really three i2c devices: a standard tun

[REVIEW PATCH 06/42] saa7115: add support for double-rate ASCLK

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil Some devices expect a double rate ASCLK. Add a flag to let the driver know through the s_crystal_freq call. Signed-off-by: Hans Verkuil --- drivers/media/i2c/saa7115.c | 16 +++- include/media/saa7115.h |7 --- 2 files changed, 15 insertions(+), 8 d

[REVIEW PATCH 13/42] tw2804: add support for the Techwell tw2804.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil This is based on the wis-tw2804.c driver that's part of the go7007 driver. It has been converted to a v4l subdev driver by Volokh Konstantin, and I made additional cleanups. Based on work by: Volokh Konstantin Signed-off-by: Hans Verkuil Cc: Volokh Konstantin --- drivers/

[REVIEW PATCH 08/42] tuner: add Sony BTF tuners

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil This adds support for three Sony BTF tuners: TUNER_SONY_BTF_PG472Z: PAL+SECAM TUNER_SONY_BTF_PK467Z: NTSC-M-JP TUNER_SONY_BTF_PB463Z: NTSC-M These come from the go7007 staging driver where they were implemented in the wis-sony-tuner i2c driver. Adding support for these tuner

[REVIEW PATCH 05/42] saa7115: improve querystd handling for the saa7115.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil The saa7115 has better PAL/NTSC detection, so it can detect PAL even though the chip is currently set up for NTSC. Signed-off-by: Hans Verkuil --- drivers/media/i2c/saa7115.c | 56 +-- 1 file changed, 28 insertions(+), 28 deletions(-

[REVIEW PATCH 02/42] v4l2-core: add code to check for specific ops.

2013-03-11 Thread Hans Verkuil
From: Hans Verkuil This patch adds a v4l2_subdev_has_op() macro and a v4l2_device_has_op macro to quickly check if a specific subdev or any subdev supports a particular subdev operation. This makes it easy for drivers to disable certain ioctls if none of the subdevs supports the necessary functi

[REVIEW PATCH 01/42] v4l2-ctrls: eliminate lockdep false alarms for struct v4l2_ctrl_handler.lock

2013-03-11 Thread Hans Verkuil
From: Andy Walls When calling v4l2_ctrl_add_handler(), lockdep would detect a potential recursive locking problem on a situation that is by design intended and not a recursive lock. This happened because all struct v4l2_ctrl_handler.lock mutexes were created as members of the same lock class in

  1   2   >