[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 t

[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 resource_si

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 escreveu: > From: Hans Verkuil > > This ioctl is defined as IOW, so pass the argument as const. > > Signed-off-by: Hans Verkuil > Acked-by: Guennadi Liakhovetski > Acked-by: Lad, Prabhakar ... > diff --git a/drivers/media/pci/ivtv/ivtv-ioctl

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 escreveu: > > > From: Hans Verkuil > > > > This ioctl is defined as IOW, so pass the argument as const. > > > > Signed-off-by: Hans Verkuil > > Acked-by: Guennadi Liakhovetski > > A

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 escreveu: > From: Hans Verkuil > > 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 write-only ioctls before the driver is c

[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 and

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 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 support/use it. > It's v

[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 precisi

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 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 support/use it. > It's v

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 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 > --- > drivers/media/usb/em28xx/

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 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

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 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. >> >>

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 scripts/c

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 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 escreveu: > Am 24.03.2013 12:38, schrieb Mauro Carvalho Chehab: > > Em Sat, 23 Mar 2013 18:27:08 +0100 > > Frank Schäfer escreveu: > > > >> The webcam "SpeedLink VAD Laplace" (em2765 + ov2640) uses a special > >> algorithm > >> for i2c communicat

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 escreveu: > From: Hans Verkuil > > Synced to commit e9815ac5503ae60cfbf6ff8037035de8f62e2846 from > branch next in git repository https://github.com/bluecherrydvr/solo6x10.git > > Only removed some code under #if LINUX_VERSION_CODE < some-kernel-

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 escreveu: > 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 d

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 escreveu: > 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 >

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 escreveu: > 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 inpu

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 escreveu: > 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/me

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 escreveu: 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/Makefil

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 escreveu: > Em Mon, 11 Mar 2013 12:45:50 +0100 > Hans Verkuil escreveu: > > > 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 Eb

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 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 > http://patchwork.linuxtv.org/patch/17568/ was merged

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

2013-03-24 Thread Michael Krufky
Reviewed-by: Michael Krufky On Thu, Mar 21, 2013 at 3:39 PM, Mauro Carvalho Chehab 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 .= $_ while (<>); > if ($data =~ m/CON

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

2013-03-24 Thread Michael Krufky
...a long awaited cleanup ;-) Reviewed-by: Michael Krufky On Thu, Mar 21, 2013 at 3:39 PM, Mauro Carvalho Chehab 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 .= $_ w

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

2013-03-24 Thread Michael Krufky
Reviewed-by: Michael Krufky On Thu, Mar 21, 2013 at 3:39 PM, Mauro Carvalho Chehab 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 .= $_ while (<>); > if ($data =~ m/CON

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 escreveu: > On Fri, Mar 22, 2013 at 12:38 PM, Hans Verkuil wrote: > > It works fine with qv4l2, but there is still a bug causing tvtime to fail. > > That's caused by commit e58071f024aa337b7ce41682578b33895b024f8b, applied > > August last year

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 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- int val = 0;

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 . 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 where some action is expected.

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 tested

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 versio

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

2013-03-24 Thread Frank Schäfer
Signed-off-by: Frank Schäfer --- 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 b/drivers/media/usb/em28xx/em28xx-i2c.c index 9e2fa41..db40835 10

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

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

2013-03-24 Thread Frank Schäfer
Signed-off-by: Frank Schäfer --- 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 b/drivers/media/usb/em28xx/em28xx-i2c.c index db40835..79ed1da 10

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 escreveu: > >> Am 24.03.2013 12:38, schrieb Mauro Carvalho Chehab: >>> Em Sat, 23 Mar 2013 18:27:08 +0100 >>> Frank Schäfer escreveu: >>> The webcam "SpeedLink VAD Laplace" (em2765 + ov26

[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 --- 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 b/drivers/staging/media/go7007/saa713

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 wrote: > This is the summary of the patches that are currently under review at > Linux Media Mailing List . > Each patch is represented by its submission date, the subject (up to 70 > chars) and the patchwork link (if submitted via

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 > > 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 code and also adding

[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 --- 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 2263a91..d94325b 100644 --- a

[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 --- 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 b/drivers/staging/media/go7007/

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 p

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 > > 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 function from driver code it