Re: [GIT PULL FOR v3.9] Move cx2341x from media/i2c to media/common

2013-02-05 Thread Hans Verkuil
On Tue February 5 2013 19:49:41 Mauro Carvalho Chehab wrote: > Hi Hans, > > Em Tue, 29 Jan 2013 09:56:20 +0100 > Hans Verkuil escreveu: > > > Hi Mauro, > > > > The cx2341x module is a helper module for conexant-based MPEG encoders. > > It isn't an i2c module at all, instead it should be in comm

Re: [RFC PATCH 1/8] stk-webcam: various fixes.

2013-02-05 Thread Hans Verkuil
On Tue February 5 2013 19:41:31 Hans Verkuil wrote: > > Buffer ioctls: > > fail: v4l2-test-buffers.cpp(132): ret != -1 > > I need to look into this a bit more. I probably need to improve > v4l2-compliance > itself so I get better feedback as to which error is actually returned here.

RE: [PATCH v2 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-02-05 Thread Inki Dae
> -Original Message- > From: Sachin Kamat [mailto:sachin.ka...@linaro.org] > Sent: Wednesday, February 06, 2013 2:30 PM > To: linux-media@vger.kernel.org; dri-de...@lists.freedesktop.org; > devicetree-disc...@lists.ozlabs.org > Cc: k.deb...@samsung.com; sachin.ka...@linaro.org; inki@sam

Re: [PATCH] Correctly set data for USB request in case of a previous failure.

2013-02-05 Thread Hans Petter Selasky
On Wednesday 06 February 2013 00:23:08 Mauro Carvalho Chehab wrote: > Em Mon, 14 Jan 2013 16:06:20 +0100 > > Hans Petter Selasky escreveu: > > Improved patch follows: > It would be even more improved if you send it to the right ML ;) > > I suspect that your original intention were to send it to

[PATCH] s5p-fimc: send valid m2m ctx to fimc_m2m_job_finish

2013-02-05 Thread Shaik Ameer Basha
fimc_m2m_job_finish() has to be called with the m2m context for the necessary cleanup while resume. But currently fimc_m2m_job_finish() always passes m2m context as NULL. This patch preserves the context before making it null, for necessary cleanup. Signed-off-by: Shaik Ameer Basha --- drivers/

[PATCH] [media] exynos-gsc: send valid m2m ctx to gsc_m2m_job_finish

2013-02-05 Thread Shaik Ameer Basha
gsc_m2m_job_finish() has to be called with the m2m context for the necessary cleanup while resume. But currently gsc_m2m_job_finish() always passes m2m context as NULL. This patch preserves the context before making it null, for necessary cleanup. Use gsc_m2m_opened() instead gsc_m2m_active() in g

[PATCH v2 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-02-05 Thread Sachin Kamat
From: Ajay Kumar This patch adds device tree match table for Exynos G2D controller. Signed-off-by: Ajay Kumar Signed-off-by: Sachin Kamat --- Patch based on exynos-drm-fixes branch of Inki Dae's tree: git://git.kernel.org/pub/scm/linux/kernel/git/daeinki/drm-exynos.git Changes since v1: Modif

[PATCH v2 1/2] [media] s5p-g2d: Add DT based discovery support

2013-02-05 Thread Sachin Kamat
This patch adds device tree based discovery support to G2D driver Signed-off-by: Sachin Kamat --- Based on for_v3.9 branch of below tree: git://linuxtv.org/snawrocki/samsung.git Changes since v1: * Addressed review comments from Sylwester . * Modified the compatible string as per the discussions

[PATCH v5 0/1] Add display-timing node parsing to exynos drm fimd

2013-02-05 Thread Vikas Sajjan
Add display-timing node parsing to drm fimd and depends on the display helper patchset at http://lists.freedesktop.org/archives/dri-devel/2013-January/033998.html It also adds pinctrl support for drm fimd. changes since v4: - addressed comments from Paul Menzel , to modify the co

[PATCH v5 1/1] video: drm: exynos: Add display-timing node parsing using video helper function

2013-02-05 Thread Vikas Sajjan
Add support for parsing the display-timing node using video helper function. The DT node parsing and pinctrl selection is done only if 'dev.of_node' exists and the NON-DT logic is still maintained under the 'else' part. Signed-off-by: Leela Krishna Amudala Signed-off-by: Vikas Sajjan --- drive

Re: [PATCH 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-02-05 Thread Sachin Kamat
On 5 February 2013 15:03, Sylwester Nawrocki wrote: > On 02/05/2013 04:03 AM, Inki Dae wrote: > [...] >>> Exynos4210 has same g2d IP (v3.0) as C110 or V210; so the same >>> comptible string will be used for this one too. >>> And please check if exynos4212 and 4412 SoCs have same fimg-2d ip. >

RE: [PATCH V3 10/15] [media] marvell-ccic: split mcam-core into 2 parts for soc_camera support

2013-02-05 Thread Albert Wang
Hi, Jonathan Thanks a lot for explaining it. :) >-Original Message- >From: Jonathan Corbet [mailto:cor...@lwn.net] >Sent: Tuesday, 05 February, 2013 11:14 >To: Albert Wang >Cc: g.liakhovet...@gmx.de; linux-media@vger.kernel.org; Libin Yang >Subject: Re: [PATCH V3 10/15] [media] marvell-c

Re: [PATCH] Correctly set data for USB request in case of a previous failure.

2013-02-05 Thread Mauro Carvalho Chehab
Em Mon, 14 Jan 2013 16:06:20 +0100 Hans Petter Selasky escreveu: > Improved patch follows: It would be even more improved if you send it to the right ML ;) I suspect that your original intention were to send it to linux-input ML, instead of linux-media :) Regards, Mauro > > --HPS > > From a

Re: [media] tuners/xc5000: fix MODE_AIR in xc5000_set_params()

2013-02-05 Thread Mauro Carvalho Chehab
Em Sun, 13 Jan 2013 22:31:33 +0300 Dan Carpenter escreveu: > There is a missing break so we use XC_RF_MODE_CABLE instead of > XC_RF_MODE_AIR. > > Signed-off-by: Dan Carpenter > --- > Static checker stuff. Untested. > > diff --git a/drivers/media/tuners/xc5000.c b/drivers/media/tuners/xc5000.c

Re: [PATCH] em28xx: overhaul em28xx_capture_area_set()

2013-02-05 Thread Mauro Carvalho Chehab
Em Sun, 20 Jan 2013 14:26:47 +0100 Frank Schäfer escreveu: > - move the bit shifting of width+height values inside the function > - fix the debug message format and output values > - add comment about the size limit (e.g. EM277x supports >2MPix) > - make void, because error checking is incomplete

Re: [media] dvb-usb: reading before start of array

2013-02-05 Thread Mauro Carvalho Chehab
Em Wed, 9 Jan 2013 10:36:32 +0300 Dan Carpenter escreveu: > This is a static checker fix. In the ttusb_process_muxpack() we do: > > cc = (muxpack[len - 4] << 8) | muxpack[len - 3]; > > That means if we pass a number less than 4 then we will either trigger a > checksum error message or re

Re: Patch update notification: 2 patches updated

2013-02-05 Thread Mauro Carvalho Chehab
Em Tue, 05 Feb 2013 22:46:10 +0100 Frank Schäfer escreveu: > Am 05.02.2013 22:33, schrieb Patchwork: > > Hello, > > > > The following patches (submitted by you) have been updated in patchwork: > ... > > * [RFC] em28xx: fix analog streaming with USB bulk transfers > > - http://patchwork.linu

Re: [PATCH] em28xx: fix usb alternate setting for analog and digital video endpoints > 0

2013-02-05 Thread Mauro Carvalho Chehab
Em Tue, 05 Feb 2013 22:37:50 +0100 Frank Schäfer escreveu: > Am 05.02.2013 21:57, schrieb Mauro Carvalho Chehab: > > Em Fri, 18 Jan 2013 18:25:48 +0100 > > Frank Schäfer escreveu: > > > >> While the current code handles sound interfaces with a number > 0 > >> correctly, it > >> assumes that the

Re: Patch update notification: 2 patches updated

2013-02-05 Thread Frank Schäfer
Am 05.02.2013 22:33, schrieb Patchwork: > Hello, > > The following patches (submitted by you) have been updated in patchwork: ... > * [RFC] em28xx: fix analog streaming with USB bulk transfers > - http://patchwork.linuxtv.org/patch/16197/ > was: New > now: RFC What's your plan with t

Re: [GIT PULL FOR 3.8] Exynos/s5p driver fixes

2013-02-05 Thread Sylwester Nawrocki
On 02/05/2013 09:43 PM, Mauro Carvalho Chehab wrote: Error: Importing patches from git://linuxtv.org/snawrocki/samsung.git v3.8-rc5-fixes fatal: Couldn't find remote ref v3.8-rc5-fixes Oops, too bad. I had some accident with this repository and forgot to restore that branch. It's pushed back n

Re: [PATCH] em28xx: fix usb alternate setting for analog and digital video endpoints > 0

2013-02-05 Thread Frank Schäfer
Am 05.02.2013 21:57, schrieb Mauro Carvalho Chehab: > Em Fri, 18 Jan 2013 18:25:48 +0100 > Frank Schäfer escreveu: > >> While the current code handles sound interfaces with a number > 0 correctly, >> it >> assumes that the interface number for analog + digital video is always 0 when >> changing t

cron job: media_tree daily build: WARNINGS

2013-02-05 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: Tue Feb 5 19:00:24 CET 2013 git branch: for_v3.9 git hash: f1927479554bf19d2ac54c4b1a38538e54e1881b gcc ve

Re: [PATCH] em28xx: fix usb alternate setting for analog and digital video endpoints > 0

2013-02-05 Thread Mauro Carvalho Chehab
Em Fri, 18 Jan 2013 18:25:48 +0100 Frank Schäfer escreveu: > While the current code handles sound interfaces with a number > 0 correctly, > it > assumes that the interface number for analog + digital video is always 0 when > changing the alternate setting. > > (NOTE: the "SpeedLink VAD Laplace

Re: [GIT PULL FOR 3.8] Exynos/s5p driver fixes

2013-02-05 Thread Mauro Carvalho Chehab
Em Sat, 19 Jan 2013 14:59:32 +0100 Sylwester Nawrocki escreveu: > Hi Mauro, > > The following changes since commit 7d1f9aeff1ee4a20b1aeb377dd0f579fe9647619: > >Linux 3.8-rc4 (2013-01-17 19:25:45 -0800) > > are available in the git repository at: >git://linuxtv.org/snawrocki/samsung.git

[PATCH v2] media: ths7353: add support for ths7353 video amplifier

2013-02-05 Thread Prabhakar Lad
From: Lad, Prabhakar The patch adds support for THS7353 video amplifier. The the THS7353 amplifier is very much similar to the existing THS7303 video amplifier driver. This patch appropriately makes changes to the existing ths7303 driver and adds support for the THS7353. This patch also adds V4L2

re: V4L/DVB (8986): cx24116: Adding DVB-S2 demodulator support

2013-02-05 Thread Dan Carpenter
Hello Steven Toth, The patch 0d46748c3f87: "V4L/DVB (8986): cx24116: Adding DVB-S2 demodulator support" from Sep 4, 2008, leads to the following warning: "drivers/media/dvb-frontends/cx24116.c:983 cx24116_send_diseqc_msg() error: buffer overflow 'd->msg' 6 <= 23" drivers/media/dvb-front

Re: [GIT PULL FOR v3.9] Move cx2341x from media/i2c to media/common

2013-02-05 Thread Mauro Carvalho Chehab
Hi Hans, Em Tue, 29 Jan 2013 09:56:20 +0100 Hans Verkuil escreveu: > Hi Mauro, > > The cx2341x module is a helper module for conexant-based MPEG encoders. > It isn't an i2c module at all, instead it should be in common since it is > used by 7 pci and usb drivers to handle the MPEG setup. >

Re: [RFC PATCH 1/8] stk-webcam: various fixes.

2013-02-05 Thread Hans Verkuil
On Tue February 5 2013 18:08:31 Arvydas Sidorenko wrote: > On Tue, Feb 5, 2013 at 5:56 PM, Ezequiel Garcia wrote: > > On Tue, Feb 5, 2013 at 12:35 PM, Hans Verkuil wrote: > >> On Tue February 5 2013 16:28:17 Arvydas Sidorenko wrote: > >>> > >>> I have tested the patches using STK-1135 webcam. Eve

Re: [PATCH v17 4/7] fbmon: add videomode helpers

2013-02-05 Thread Steffen Trumtrar
Hi! On Fri, Feb 01, 2013 at 06:29:50PM +0900, Jingoo Han wrote: > On Friday, January 25, 2013 6:02 PM, Steffen Trumtrar wrote > > > > + fbmode->sync = 0; > > + fbmode->vmode = 0; > > + if (vm->dmt_flags & VESA_DMT_HSYNC_HIGH) > > + fbmode->sync |= FB_SYNC_HOR_HIGH_ACT; > > + if

[PATCH v6 1/2] sta2x11_vip: convert to videobuf2, control framework, file handler

2013-02-05 Thread Federico Vaga
This patch re-write the driver and use the videobuf2 interface instead of the old videobuf. Moreover, it uses also the control framework which allows the driver to inherit controls from its subdevice (ADV7180). Finally the driver does not implement custom file operation but it uses the generic ones

Re: [PATCH] davinci: dm644x: fix enum ccdc_gama_width and enum ccdc_data_size comparision warning

2013-02-05 Thread Prabhakar Lad
Hi Mauro, On Tue, Feb 5, 2013 at 9:54 PM, Mauro Carvalho Chehab wrote: > Em Wed, 2 Jan 2013 17:23:50 +0530 > "Lad, Prabhakar" escreveu: > >> while the effect is harmless this patch > > I disagree that this is a harmless warning. It is here for a reason: > you should not be relying on the enum "

Re: [RFC PATCH 1/8] stk-webcam: various fixes.

2013-02-05 Thread Arvydas Sidorenko
On Tue, Feb 5, 2013 at 5:56 PM, Ezequiel Garcia wrote: > On Tue, Feb 5, 2013 at 12:35 PM, Hans Verkuil wrote: >> On Tue February 5 2013 16:28:17 Arvydas Sidorenko wrote: >>> >>> I have tested the patches using STK-1135 webcam. Everything works well. >>> >>> $ v4l2-compliance -d /dev/video0 >>> Dr

Re: [RFC PATCH 1/8] stk-webcam: various fixes.

2013-02-05 Thread Ezequiel Garcia
On Tue, Feb 5, 2013 at 12:35 PM, Hans Verkuil wrote: > On Tue February 5 2013 16:28:17 Arvydas Sidorenko wrote: >> On Mon, Feb 4, 2013 at 2:35 PM, Hans Verkuil wrote: >> > >> > Hi Arvydas, >> > >> > Yes indeed, it would be great if you could test this! >> > >> > Note that the patch series is also

Re: [PATCH] davinci: dm644x: fix enum ccdc_gama_width and enum ccdc_data_size comparision warning

2013-02-05 Thread Mauro Carvalho Chehab
Em Wed, 2 Jan 2013 17:23:50 +0530 "Lad, Prabhakar" escreveu: > while the effect is harmless this patch I disagree that this is a harmless warning. It is here for a reason: you should not be relying on the enum "magic" value, since the main reason to use an enum is to fill/compare the enum field

Re: [GIT PULL for 3.7-rc7] media fixes

2013-02-05 Thread Mauro Carvalho Chehab
Em Tue, 5 Feb 2013 13:35:19 -0200 Mauro Carvalho Chehab escreveu: Heh, obviously the subject is wrong... I meant to write "GIT PULL for 3.8-rc7" :) Regards, Mauro > Hi Linus, > > Please pull from: > git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media > v4l_for_linus >

[GIT PULL for 3.7-rc7] media fixes

2013-02-05 Thread Mauro Carvalho Chehab
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media v4l_for_linus For a regression fix on a few radio drivers that were preventing radio TX to work on those devices. Regards, Mauro - The following changes since commit 68d6f84ba0c47e658beff3a

Re: [RFC PATCH 1/8] stk-webcam: various fixes.

2013-02-05 Thread Hans Verkuil
On Tue February 5 2013 16:28:17 Arvydas Sidorenko wrote: > On Mon, Feb 4, 2013 at 2:35 PM, Hans Verkuil wrote: > > > > Hi Arvydas, > > > > Yes indeed, it would be great if you could test this! > > > > Note that the patch series is also available in my git tree: > > > > http://git.linuxtv.org/hverk

Re: [RFC PATCH 1/8] stk-webcam: various fixes.

2013-02-05 Thread Arvydas Sidorenko
On Mon, Feb 4, 2013 at 2:35 PM, Hans Verkuil wrote: > > Hi Arvydas, > > Yes indeed, it would be great if you could test this! > > Note that the patch series is also available in my git tree: > > http://git.linuxtv.org/hverkuil/media_tree.git/shortlog/refs/heads/stkwebcam > > Besides the normal tes

Re: WinTV-HVR-1400: scandvb (and kaffeine) fails to find any channels

2013-02-05 Thread Chris Clayton
On 02/03/13 00:03, Chris Clayton wrote: On 02/01/13 21:07, Devin Heitmueller wrote: On Fri, Feb 1, 2013 at 4:03 PM, Chris Clayton wrote: Yes, I noticed that but even with the tuning timeout set at medium or longest, I doesn't find any channels. However, I've been following the debug messag

[GIT PULL FOR v3.9] v4l2-compliance fixes for tm6000

2013-02-05 Thread Hans Verkuil
These patches fix various v4l2-compliance issues for the tm6000. Tested with my tm6000. Unchanged from the RFC patches posted a few days ago. Regards, Hans The following changes since commit a32f7d1ad3744914273c6907204c2ab3b5d496a0: Merge branch 'v4l_for_linus' into staging/for_v3.

[GIT PULL FOR v3.9] meye: convert to the control framework

2013-02-05 Thread Hans Verkuil
These patches upgrade meye to the control framework. Unchanged from the RFC patches posted a week ago. Regards, Hans The following changes since commit a32f7d1ad3744914273c6907204c2ab3b5d496a0: Merge branch 'v4l_for_linus' into staging/for_v3.9 (2013-01-24 18:49:18 -0200) are availa

[GIT PULL FOR v3.9] v4l2-compliance fixes for c-qcam

2013-02-05 Thread Hans Verkuil
These patches fix various compliance issues for c-qcam. Tested on more-or-less (mostly less) working actual hardware. Unchanged from the RFC patches posted a week ago. Regards, HansThe following changes since commit a32f7d1ad3744914273c6907204c2ab3b5d496a0: Merge branch 'v4l_for_linu

[GIT PULL FOR v3.9] v4l2-compliance fixes for bw-qcam

2013-02-05 Thread Hans Verkuil
These patches upgrade bw-qcam to vb2 and fix various compliance issues. Tested on actual hardware. Unchanged from the RFC patches posted a week ago. Regards, Hans The following changes since commit a32f7d1ad3744914273c6907204c2ab3b5d496a0: Merge branch 'v4l_for_linus' into staging/fo

[GIT PULL FOR v3.9] v4l2-compliance fixes for radio-miropcm20

2013-02-05 Thread Hans Verkuil
These patches upgrade radio-miropcm20 to the latest frameworks. Tested on actual hardware. Unchanged from the RFC patches posted a week ago. Regards, Hans The following changes since commit a32f7d1ad3744914273c6907204c2ab3b5d496a0: Merge branch 'v4l_for_linus' into staging/for_v3.9 (

[PATCH] [media] saa7164: silence GCC warnings

2013-02-05 Thread Paul Bolle
Compiling the saa7164 driver without CONFIG_VIDEO_ADV_DEBUG set triggers these GCC warnings: drivers/media/pci/saa7164/saa7164-encoder.c:1301:12: warning: ‘saa7164_g_register’ defined but not used [-Wunused-function] drivers/media/pci/saa7164/saa7164-encoder.c:1314:12: warning: ‘saa7164_s

ttusb2: Kconfig patch to auto-select frontends for TechnoTrend CT-3650

2013-02-05 Thread Matti Kurkela
The ttusb2 module is already updated to recognize the TechnoTrend CT-3650 CI DVB C/T USB2.0 receiver in addition to the Pinnacle 400e. But if MEDIA_SUBDRV_AUTOSELECT is used, the required tuner and demodulator modules are not automatically selected. Here's a patch to fix that and add a note o

Re: [PATCH v4 1/1] video: drm: exynos: Adds display-timing node parsing using video helper function

2013-02-05 Thread Paul Menzel
Dear Vikas, thank you for the patch. Please send a fifth iteration with the following changes to the commit message. Am Dienstag, den 05.02.2013, 11:02 +0530 schrieb Vikas Sajjan: The summary should not implicitly assume »patch« written before it. So do not add third person s to »Add«.

Re: [PATCH 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-02-05 Thread Sylwester Nawrocki
On 02/05/2013 04:03 AM, Inki Dae wrote: [...] >> Exynos4210 has same g2d IP (v3.0) as C110 or V210; so the same >> comptible string will be used for this one too. >> >>> And please check if exynos4212 and 4412 SoCs have same fimg-2d ip. >>> If it's different, we might need to add ip version proper

Re: [PATCH] uvcvideo: Implement videobuf2 .wait_prepare and .wait_finish operations

2013-02-05 Thread Laurent Pinchart
Hi Sean, On Monday 04 February 2013 12:36:47 Sean V Kelley wrote: > On Mon, Jan 21, 2013 at 2:51 AM, Laurent Pinchart wrote: > > Those optional operations are used to release and reacquire the queue > > lock when videobuf2 needs to perform operations that sleep for a long > > time, such as waiting

Re: [PATCH 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-02-05 Thread Joonyoung Shim
On 02/05/2013 12:03 PM, Inki Dae wrote: > 2013/2/4 Sachin Kamat : >> On 1 February 2013 18:28, Inki Dae wrote: >>> >>> >>> >>> 2013. 2. 1. 오후 8:52 Inki Dae 작성: >>> > -Original Message- > From: linux-media-ow...@vger.kernel.org [mailto:linux-media- > ow...@vger.kernel.org]