[PATCH] documentation: DocBook/media : Fix typo in dvbproperty.xml

2013-03-24 Thread Masanari Iida
Correct spelling typos. Signed-off-by: Masanari Iida standby2...@gmail.com --- Documentation/DocBook/media/dvb/dvbproperty.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/DocBook/media/dvb/dvbproperty.xml

[GIT PULL FOR v3.10] Remove core DV_PRESET support.

2013-03-24 Thread Hans Verkuil
There are no more drivers that use the obsolete DV_PRESET API, so remove it from the V4L2 core code and the V4L2 documentation. This is unchanged from the last 5 patches in this original RFC patch series: http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/60904 except for

[GIT PULL FOR v3.10] vivi: add v4l2_ctrl_modify_range test case.

2013-03-24 Thread Hans Verkuil
Very few drivers use v4l2_ctrl_modify_range. Add it to vivi so applications can use vivi to test their support for v4l2_ctrl_modify_range. Regards, Hans The following changes since commit 69aa6f4ec669b9121057cc9e32cb10b5f744f6d6: [media] drivers: staging: davinci_vpfe: use

Re: [REVIEWv2 PATCH 4/6] v4l2: add const to argument of write-only s_register ioctl.

2013-03-24 Thread Mauro Carvalho Chehab
Em Mon, 18 Mar 2013 15:12:03 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com This ioctl is defined as IOW, so pass the argument as const. Signed-off-by: Hans Verkuil hans.verk...@cisco.com Acked-by: Guennadi Liakhovetski g.liakhovet...@gmx.de

Re: [REVIEWv2 PATCH 4/6] v4l2: add const to argument of write-only s_register ioctl.

2013-03-24 Thread Hans Verkuil
On Sun March 24 2013 11:07:03 Mauro Carvalho Chehab wrote: Em Mon, 18 Mar 2013 15:12:03 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com This ioctl is defined as IOW, so pass the argument as const. Signed-off-by: Hans Verkuil

Re: [REVIEWv2 PATCH 5/6] v4l2-ioctl: simplify debug code.

2013-03-24 Thread Mauro Carvalho Chehab
Em Mon, 18 Mar 2013 15:12:04 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com The core debug code can now be simplified since all the write-only ioctls are now const and will not modify the data they pass to the drivers. So instead of logging

[GIT PULL FOR v3.10] v4l2: add const to argument of write-only s_register ioctl.

2013-03-24 Thread Hans Verkuil
It's identical to: http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/62324 except it's rebased and as requested by Mauro the cx18 and ivtv changes have been split off to their own patches. I did not change the volatile part, changing that is a completely separate issue

Re: [PATCH v2 1/5] em28xx: add support for em25xx i2c bus B read/write/check device operations

2013-03-24 Thread Mauro Carvalho Chehab
Em Sat, 23 Mar 2013 18:27:08 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: The webcam SpeedLink VAD Laplace (em2765 + ov2640) uses a special algorithm for i2c communication with the sensor, which is connected to a second i2c bus. We don't know yet how to find out which devices

[GIT PULL FOR v3.10] v4l2: add const to argument of write-only s_register ioctl

2013-03-24 Thread Hans Verkuil
It's identical to: http://comments.gmane.org/gmane.linux.drivers.video-input-infrastructure/62324 except it's rebased and the ivtv changes have been simplified. Regards, Hans The following changes since commit 27d5a87cf4b44cbcbd0f4706a433e4a68d496236: [media] v4l2-ioctl: add

Re: [PATCH v2 1/5] em28xx: add support for em25xx i2c bus B read/write/check device operations

2013-03-24 Thread Mauro Carvalho Chehab
Em Sat, 23 Mar 2013 18:27:08 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: The webcam SpeedLink VAD Laplace (em2765 + ov2640) uses a special algorithm for i2c communication with the sensor, which is connected to a second i2c bus. We don't know yet how to find out which devices

Re: [PATCH v2 5/5] em28xx: write output frame resolution to regs 0x34+0x35 for em25xx family bridges

2013-03-24 Thread Mauro Carvalho Chehab
Em Sat, 23 Mar 2013 18:27:12 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: The Windows driver writes the output resolution to registers 0x34 (width / 16) and 0x35 (height / 16) always. We don't know yet what these registers are used for. Signed-off-by: Frank Schäfer

Re: [PATCH v2 1/5] em28xx: add support for em25xx i2c bus B read/write/check device operations

2013-03-24 Thread Frank Schäfer
Am 24.03.2013 12:38, schrieb Mauro Carvalho Chehab: Em Sat, 23 Mar 2013 18:27:08 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: The webcam SpeedLink VAD Laplace (em2765 + ov2640) uses a special algorithm for i2c communication with the sensor, which is connected to a second i2c

Re: [PATCH v2 5/5] em28xx: write output frame resolution to regs 0x34+0x35 for em25xx family bridges

2013-03-24 Thread Frank Schäfer
Am 24.03.2013 12:44, schrieb Mauro Carvalho Chehab: Em Sat, 23 Mar 2013 18:27:12 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: The Windows driver writes the output resolution to registers 0x34 (width / 16) and 0x35 (height / 16) always. We don't know yet what these registers

Re: [PATCH v2 1/5] em28xx: add support for em25xx i2c bus B read/write/check device operations

2013-03-24 Thread Frank Schäfer
... Am 24.03.2013 12:22, schrieb Mauro Carvalho Chehab: Please stick with Kernel's coding style, as described on Documentation/CodingStyle and on the common practices. Multi-line comments are like: /* * Foo * bar */ There are also a bunch of

Re: Fwd: Delock 61959

2013-03-24 Thread Antti Palosaari
Maybe it is em28xx + DRX-K + tda18271 based. There is few such devices already supported by em28xx driver. First device to test is 1b80:e425 MaxMedia UB425-TC. Just replace USB ID 0xe425 with 0xe1cc, compile and test. There is some other devices too, especially all those which are using drx-k.

Re: [PATCH v2 1/5] em28xx: add support for em25xx i2c bus B read/write/check device operations

2013-03-24 Thread Mauro Carvalho Chehab
Em Sun, 24 Mar 2013 14:04:47 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: ... Am 24.03.2013 12:22, schrieb Mauro Carvalho Chehab: Please stick with Kernel's coding style, as described on Documentation/CodingStyle and on the common practices. Multi-line comments are like:

Re: [PATCH v2 1/5] em28xx: add support for em25xx i2c bus B read/write/check device operations

2013-03-24 Thread Mauro Carvalho Chehab
Em Sun, 24 Mar 2013 13:53:40 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: Am 24.03.2013 12:38, schrieb Mauro Carvalho Chehab: Em Sat, 23 Mar 2013 18:27:08 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: The webcam SpeedLink VAD Laplace (em2765 + ov2640) uses a

Re: [REVIEW PATCH 01/19] solo6x10: sync to latest code from Bluecherry's git repo.

2013-03-24 Thread Mauro Carvalho Chehab
Em Mon, 18 Mar 2013 13:32:00 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com Synced to commit e9815ac5503ae60cfbf6ff8037035de8f62e2846 from branch next in git repository https://github.com/bluecherrydvr/solo6x10.git Only removed some code under

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

2013-03-24 Thread Mauro Carvalho Chehab
Em Mon, 11 Mar 2013 12:45:47 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com 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

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

2013-03-24 Thread Mauro Carvalho Chehab
Em Mon, 11 Mar 2013 12:45:50 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com 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.

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

2013-03-24 Thread Mauro Carvalho Chehab
Em Mon, 11 Mar 2013 12:45:54 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com 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

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

2013-03-24 Thread Mauro Carvalho Chehab
Em Mon, 11 Mar 2013 12:45:57 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com The v2 of this function doesn't do DMA to objects on the stack like its predecessor does. Signed-off-by: Hans Verkuil hans.verk...@cisco.com ---

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

2013-03-24 Thread Antti Palosaari
On 03/24/2013 05:39 PM, Mauro Carvalho Chehab wrote: Em Mon, 11 Mar 2013 12:45:57 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com The v2 of this function doesn't do DMA to objects on the stack like its predecessor does. Signed-off-by: Hans Verkuil

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

2013-03-24 Thread Mauro Carvalho Chehab
Em Sun, 24 Mar 2013 12:29:32 -0300 Mauro Carvalho Chehab mche...@redhat.com escreveu: Em Mon, 11 Mar 2013 12:45:50 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: From: Hans Verkuil hans.verk...@cisco.com This based on the wis-tw9903.c driver that's part of the go7007 driver. It has

Re: [GIT PULL FOR v3.10] go7007 driver overhaul

2013-03-24 Thread Mauro Carvalho Chehab
Em Fri, 22 Mar 2013 15:36:35 +0100 Hans Verkuil hverk...@xs4all.nl escreveu: Hi all, This pull request updates the staging go7007 driver to the latest V4L2 frameworks and actually makes it work reliably. This pull request assumes that pull request

Re: [PATCH 4/4] [media] dvb-usb/dvb-usb-v2: use IS_ENABLED

2013-03-24 Thread Michael Krufky
Reviewed-by: Michael Krufky mkru...@linuxtv.org On Thu, Mar 21, 2013 at 3:39 PM, Mauro Carvalho Chehab mche...@redhat.com wrote: Instead of checking everywhere there for 3 symbols, use instead IS_ENABLED macro. This replacement was done using this small perl script: my $data; $data .= $_

Re: [PATCH 1/4] [media] dvb-frontends: use IS_ENABLED

2013-03-24 Thread Michael Krufky
...a long awaited cleanup ;-) Reviewed-by: Michael Krufky mkru...@linuxtv.org On Thu, Mar 21, 2013 at 3:39 PM, Mauro Carvalho Chehab mche...@redhat.com wrote: Instead of checking everywhere there for 3 symbols, use instead IS_ENABLED macro. This replacement was done using this small perl

Re: [PATCH 2/4] [media] tuners: use IS_ENABLED

2013-03-24 Thread Michael Krufky
Reviewed-by: Michael Krufky mkru...@linuxtv.org On Thu, Mar 21, 2013 at 3:39 PM, Mauro Carvalho Chehab mche...@redhat.com wrote: Instead of checking everywhere there for 3 symbols, use instead IS_ENABLED macro. This replacement was done using this small perl script: my $data; $data .= $_

Re: [GIT PULL FOR v3.10] au0828 driver overhaul

2013-03-24 Thread Mauro Carvalho Chehab
Em Fri, 22 Mar 2013 14:50:28 -0400 Devin Heitmueller dheitmuel...@kernellabs.com escreveu: On Fri, Mar 22, 2013 at 12:38 PM, Hans Verkuil hverk...@xs4all.nl wrote: It works fine with qv4l2, but there is still a bug causing tvtime to fail. That's caused by commit

Re: [GIT PULL FOR v3.10] au0828 driver overhaul

2013-03-24 Thread Devin Heitmueller
On Sun, Mar 24, 2013 at 1:51 PM, Mauro Carvalho Chehab mche...@redhat.com wrote: drivers/media/dvb-frontends/au8522_decoder.c:static int au8522_g_tuner(struct v4l2_subdev *sd, struct v4l2_tuner *vt) drivers/media/dvb-frontends/au8522_decoder.c-{ drivers/media/dvb-frontends/au8522_decoder.c-

Status of the patches under review at LMML (32 patches)

2013-03-24 Thread Mauro Carvalho Chehab
This is the summary of the patches that are currently under review at Linux Media Mailing List linux-media@vger.kernel.org. Each patch is represented by its submission date, the subject (up to 70 chars) and the patchwork link (if submitted via email). P.S.: This email is c/c to the developers

Re: [PATCH] cxd2820r_t2: Multistream support (MultiPLP)

2013-03-24 Thread Antti Palosaari
Is there anyone who could test that patch? I have no multi PLP signal here. Also there is minor issue on that patch. As stream ID validy is already checked there is no reason for bit AND 0xff. Antti On 03/07/2013 02:16 AM, CrazyCat wrote: MultiPLP filtering support for CXD2820r, not

cron job: media_tree daily build: ERRORS

2013-03-24 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 Mar 24 19:00:24 CET 2013 git branch: test git hash: b781e6be79a394cd6980e9cd8fd5c25822d152b6 gcc

[PATCH] em28xx-i2c: do not break strings across lines

2013-03-24 Thread Frank Schäfer
Signed-off-by: Frank Schäfer fschaefer@googlemail.com --- drivers/media/usb/em28xx/em28xx-i2c.c | 45 +++-- 1 Datei geändert, 20 Zeilen hinzugefügt(+), 25 Zeilen entfernt(-) diff --git a/drivers/media/usb/em28xx/em28xx-i2c.c

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

2013-03-24 Thread Jose Alberto Reguero
On Domingo, 17 de marzo de 2013 23:46:06 Antti Palosaari escribió: On 03/17/2013 08:49 PM, Jose Alberto Reguero wrote: On Martes, 12 de marzo de 2013 00:11:38 Antti Palosaari escribió: On 03/11/2013 10:02 PM, Jose Alberto Reguero wrote: On Lunes, 11 de marzo de 2013 14:57:37 Antti Palosaari

[PATCH] em28xx-i2c: fix coding style of multi line comments

2013-03-24 Thread Frank Schäfer
Signed-off-by: Frank Schäfer fschaefer@googlemail.com --- drivers/media/usb/em28xx/em28xx-i2c.c | 52 + 1 Datei geändert, 34 Zeilen hinzugefügt(+), 18 Zeilen entfernt(-) diff --git a/drivers/media/usb/em28xx/em28xx-i2c.c

Re: [PATCH v2 1/5] em28xx: add support for em25xx i2c bus B read/write/check device operations

2013-03-24 Thread Frank Schäfer
Am 24.03.2013 15:02, schrieb Mauro Carvalho Chehab: Em Sun, 24 Mar 2013 13:53:40 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: Am 24.03.2013 12:38, schrieb Mauro Carvalho Chehab: Em Sat, 23 Mar 2013 18:27:08 +0100 Frank Schäfer fschaefer@googlemail.com escreveu: The webcam

[PATCH] drivers/staging/media/go7007: using strlcpy instead of strncpy

2013-03-24 Thread Chen Gang
for NUL terminated string, need always set '\0' in the end. Signed-off-by: Chen Gang gang.c...@asianux.com --- drivers/staging/media/go7007/saa7134-go7007.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/staging/media/go7007/saa7134-go7007.c

Re: Status of the patches under review at LMML (32 patches)

2013-03-24 Thread Prabhakar Lad
Hi Mauro, On Sun, Mar 24, 2013 at 11:41 PM, Mauro Carvalho Chehab mche...@redhat.com wrote: This is the summary of the patches that are currently under review at Linux Media Mailing List linux-media@vger.kernel.org. Each patch is represented by its submission date, the subject (up to 70

Re: [PATCH 1/2] media: davinci: vpss: enable vpss clocks

2013-03-24 Thread Sekhar Nori
On 3/22/2013 1:23 PM, Prabhakar lad wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com By default the VPSS clocks are only enabled in capture driver for davinci family which creates duplicates. This patch adds support to enable the VPSS clocks in VPSS driver. This avoids duplication of

[PATCH 1/2] [media] tw9906: Remove unneeded version.h header include

2013-03-24 Thread Sachin Kamat
version.h header inclusion is not necessary as detected by versioncheck. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/media/i2c/tw9906.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/media/i2c/tw9906.c b/drivers/media/i2c/tw9906.c index

[PATCH 2/2] [media] go7007: Remove unneeded version.h header include

2013-03-24 Thread Sachin Kamat
version.h header inclusion is not necessary as detected by versioncheck. Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/staging/media/go7007/go7007-v4l2.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/staging/media/go7007/go7007-v4l2.c

Re: [RFC 11/12] media: m5mols: Adding dt support to m5mols driver

2013-03-24 Thread Shaik Ameer Basha
Hi Sylwester, Thanks for the review. Actually I agree with all of your review comments. This was just a temporary test patch, used to test exynos5-mdev. I thought for some one to test exynos5-mdev series patches, i need to provide one working m5mols dt driver. Good to hear you already have one

Re: [PATCH 2/2] media: davinci: vpbe: venc: move the enabling of vpss clocks to driver

2013-03-24 Thread Sekhar Nori
On 3/22/2013 1:23 PM, Prabhakar lad wrote: From: Lad, Prabhakar prabhakar.cse...@gmail.com The vpss clocks were enabled by calling a exported function from a driver in a machine code. calling driver code from platform code is incorrect way. This patch fixes this issue and calls the