Re: Patches

2012-06-15 Thread Randy Dunlap
On 06/14/2012 05:23 PM, Peter Senna Tschudin wrote: Hello Randy, ./scripts/get_maintainer.pl -f drivers/media/video/cx231xx/ Mauro Carvalho Chehab mche...@infradead.org (maintainer:MEDIA INPUT INFRA...,commit_signer:23/24=96%) Devin Heitmueller dheitmuel...@kernellabs.com

Re: [media] DRX-K: Initial check-in

2012-06-15 Thread Ralph Metzler
Dan Carpenter writes: On Sun, Jun 10, 2012 at 09:42:22PM +0200, Ralph Metzler wrote: Dan Carpenter writes: Hello Ralph Metzler, The patch 43dd07f758d8: [media] DRX-K: Initial check-in from Jul 3, 2011, leads to the following warning:

Re: [patch RFC] [media] staging: solo6x10: fix | vs

2012-06-15 Thread Ralph Metzler
Dan Carpenter writes: On Sat, Jun 09, 2012 at 10:47:32AM +0300, Dan Carpenter wrote: The test here is never true because '' was used instead of '|'. It was the same as: if (status ((116) (117)) ... Signed-off-by: Dan Carpenter dan.carpen...@oracle.com --- I don't

Re: [PATCH v4 7/7] v4l: Correct conflicting V4L2 subdev selection API documentation

2012-06-15 Thread Laurent Pinchart
Hi Sakari, On Friday 15 June 2012 18:31:50 Sakari Ailus wrote: On Fri, Jun 15, 2012 at 04:14:21PM +0200, Laurent Pinchart wrote: On Friday 15 June 2012 16:44:40 Sakari Ailus wrote: The API reference documents that the KEEP_CONFIG flag tells the configuration should not be propatgated by

Re: [PATCH v4 2/7] v4l: Remove _ACTUAL from subdev selection API target definition names

2012-06-15 Thread Sakari Ailus
Hi Laurent, Laurent Pinchart wrote: Hi Sakari, Thanks for the patch. Thanks for the comments! On Friday 15 June 2012 16:44:35 Sakari Ailus wrote: The string _ACTUAL does not say anything more about the target names. Drop it. V4L2 selection API was changed by V4L: Remove _ACTIVE from the

Re: [PATCH v4 7/7] v4l: Correct conflicting V4L2 subdev selection API documentation

2012-06-15 Thread Sakari Ailus
Hi Laurent, Laurent Pinchart wrote: Hi Sakari, On Friday 15 June 2012 18:31:50 Sakari Ailus wrote: On Fri, Jun 15, 2012 at 04:14:21PM +0200, Laurent Pinchart wrote: On Friday 15 June 2012 16:44:40 Sakari Ailus wrote: The API reference documents that the KEEP_CONFIG flag tells the

cron job: media_tree daily build: WARNINGS

2012-06-15 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:Fri Jun 15 19:00:15 CEST 2012 git hash:5472d3f17845c4398c6a510b46855820920c2181 gcc version: i686-linux-gcc

[GIT PULL FOR 3.5] V4L2 and V4L2 subdev selection target and flag changes

2012-06-15 Thread Sakari Ailus
Hi Mauro, This patchset contains changes to unify the selection targets and flags across the V4L2 and V4L2 subdev interfaces, as well as a fix to conflicting information in the documentation. Since the patchset (v4) I've addressed Laurent's comments: - Fixed badly formatted xref references -

dvb_usb_v2: use pointers to properties[REGRESSION]

2012-06-15 Thread Malcolm Priestley
Hi Antti You can't have dvb_usb_device_properties as constant structure pointer. At run time it needs to be copied to a private area. Two or more devices of the same type on the system will be pointing to the same structure. Any changes they make to the structure will be common to all.

Re: dvb_usb_v2: use pointers to properties[REGRESSION]

2012-06-15 Thread Antti Palosaari
Hello Malcolm, On 06/16/2012 01:11 AM, Malcolm Priestley wrote: Hi Antti You can't have dvb_usb_device_properties as constant structure pointer. At run time it needs to be copied to a private area. Having constant structure for properties was one of main idea of whole change. Earlier it

Re: dvb_usb_v2: use pointers to properties[REGRESSION]

2012-06-15 Thread Malcolm Priestley
On Sat, 2012-06-16 at 01:54 +0300, Antti Palosaari wrote: Hello Malcolm, On 06/16/2012 01:11 AM, Malcolm Priestley wrote: Hi Antti You can't have dvb_usb_device_properties as constant structure pointer. At run time it needs to be copied to a private area. Having constant structure

Flaky init for HVR850 (cx231xx, 2040:b140)

2012-06-15 Thread David Dillow
I'm trying to track down the root cause of my HVR850 failing to reliably initialize on the 3.4 kernel. The issue does not seem to be new, as the same behavior appears under CentOS 2.6.32-220.17.1. Both kernels are running on x86_64. The HVR850 is a recent model 1230 from NewEgg. Upon being

Re: dvb_usb_v2: use pointers to properties[REGRESSION]

2012-06-15 Thread Antti Palosaari
On 06/16/2012 03:35 AM, Malcolm Priestley wrote: On Sat, 2012-06-16 at 01:54 +0300, Antti Palosaari wrote: Hello Malcolm, On 06/16/2012 01:11 AM, Malcolm Priestley wrote: Hi Antti You can't have dvb_usb_device_properties as constant structure pointer. At run time it needs to be copied to a

Re: dvb_usb_v2: use pointers to properties[REGRESSION]

2012-06-15 Thread Antti Palosaari
I did example for you, here you are :) On 06/16/2012 03:55 AM, Antti Palosaari wrote: On 06/16/2012 03:35 AM, Malcolm Priestley wrote: On Sat, 2012-06-16 at 01:54 +0300, Antti Palosaari wrote: Hello Malcolm, On 06/16/2012 01:11 AM, Malcolm Priestley wrote: Hi Antti You can't have

dmb-th problem

2012-06-15 Thread Choi Wing Chan
i have two cards both of them are for dmb-th (mainly used in china and hong kong). however, these two cards stop working after upgraded the kernel to 3.3. sadly the drivers were writen by a person David Wong who has been passed away. i traced the code and found a function set_delivery_system which

Re: [PATCH 3/8] [RESEND] saa7146: Variable set but not used

2012-06-15 Thread Michael Hunold
Hello Peter, on 14.06.2012 19:58 Peter Senna Tschudin said the following: In function fops_open variable type was set but not used. thanks for your patch, but I think it does not work. Tested by compilation only. Signed-off-by: Peter Senna Tschudin peter.se...@gmail.com ---

Re: [PATCH 3/8] [RESEND] saa7146: Variable set but not used

2012-06-15 Thread Hans Verkuil
On 15/06/12 08:02, Michael Hunold wrote: Hello Peter, on 14.06.2012 19:58 Peter Senna Tschudin said the following: In function fops_open variable type was set but not used. thanks for your patch, but I think it does not work. Tested by compilation only. Signed-off-by: Peter Senna

Re: [PATCH 3/8] [RESEND] saa7146: Variable set but not used

2012-06-15 Thread Michael Hunold
Hello Hans, on 15.06.2012 08:24 Hans Verkuil said the following: On 15/06/12 08:02, Michael Hunold wrote: A few lines below fh is allocated and fh-type is set to type. Simply removing type will result in a compilation error IMO, so I wonder if your compile-test really worked. Can you have

Re: [PATCH RFC 00/10] media file tree reorg - part 1

2012-06-15 Thread Hans Verkuil
On 14/06/12 22:35, Mauro Carvalho Chehab wrote: As discussed a while ago, breaking media drivers by V4L or DVB is confusing, as: - hybrid devices are at V4L drivers; - DVB-only devices for chips that support analog are at V4L drivers; - Analog support addition

[GIT PULL FOR 3.5] SMIA++ compile fix

2012-06-15 Thread Sakari Ailus
Hi Mauro, This pull contains a simple compile fix for the SMIA++ driver. 3.5 would be preferred. The following changes since commit 5472d3f17845c4398c6a510b46855820920c2181: [media] mt9m032: Implement V4L2_CID_PIXEL_RATE control (2012-05-24 09:27:24 -0300) are available in the git repository

[PATCH] s5p-mfc: Fix setting controls

2012-06-15 Thread Kamil Debski
Fixed s_ctrl function when setting the following controls: - V4L2_CID_MPEG_VIDEO_DECODER_MPEG4_DEBLOCK_FILTER - V4L2_CID_MPEG_MFC51_VIDEO_DECODER_H264_DISPLAY_DELAY Signed-off-by: Kamil Debski k.deb...@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com ---

RE: [PATCH 3/5] usb: gadget/uvc: Add super-speed support to UVC webcam gadget

2012-06-15 Thread Bhupesh SHARMA
Hi Laurent, -Original Message- From: Laurent Pinchart [mailto:laurent.pinch...@ideasonboard.com] Sent: Monday, June 11, 2012 12:55 PM To: Bhupesh SHARMA Cc: linux-...@vger.kernel.org; ba...@ti.com; linux- me...@vger.kernel.org Subject: Re: [PATCH 3/5] usb: gadget/uvc: Add

Re: [PATCH RFC 00/10] media file tree reorg - part 1

2012-06-15 Thread Mauro Carvalho Chehab
Em 15-06-2012 04:39, Hans Verkuil escreveu: On 14/06/12 22:35, Mauro Carvalho Chehab wrote: As discussed a while ago, breaking media drivers by V4L or DVB is confusing, as: - hybrid devices are at V4L drivers; - DVB-only devices for chips that support analog are at V4L drivers;

[PATCH for 3.6] DocBook fix: wrong type in documentation

2012-06-15 Thread Hans Verkuil
Just a quick one I came across just now. Regards, Hans Signed-off-by: Hans Verkuil hans.verk...@cisco.com diff --git a/Documentation/DocBook/media/v4l/dev-raw-vbi.xml b/Documentation/DocBook/media/v4l/dev-raw-vbi.xml index b788c72..2a3c91a 100644 ---

Re: Patches

2012-06-15 Thread Ezequiel Garcia
Hi Peter, On Thu, Jun 14, 2012 at 9:23 PM, Peter Senna Tschudin peter.se...@gmail.com wrote: When I run it pointing to the patch, it also includes Julia Lawall and Greg Kroah-Hartman. See: $ scripts/get_maintainer.pl outgoing/0001-RESEND-cx231xx-Paranoic-stack-memory-save.patch Mauro

Re: extend v4l2_mbus_framefmt

2012-06-15 Thread Hans Verkuil
On 14/06/12 05:17, Scott Jiang wrote: I would expect that the combination of v4l2_mbus_framefmt + v4l2_dv_timings gives you the information you need. I can solve this problem in HD, but how about SD? Add a fake dv_timings ops in SD decoder driver? No, you add g/s_std instead. SD timings

[GIT PULL] Samsung fixes for v3.5

2012-06-15 Thread Marek Szyprowski
Hello Mauro, Please pull various fixes for V4L drivers for v3.5 kernel series. Best regards, Marek Szyprowski Samsung Poland RD Center The following changes since commit f8f5701bdaf9134b1f90e5044a82c66324d2073f: Linux 3.5-rc1 (2012-06-02 18:29:26 -0700) are available in the git repository

Re: [PATCH 5/8] nuvoton-cir: Code cleanup: remove unused variable and function

2012-06-15 Thread Jarod Wilson
On Thu, Jun 14, 2012 at 02:58:13PM -0300, Peter Senna Tschudin wrote: Tested by compilation only. Making use of this code is on the TODO list, possibly even happens by way of David's pending patches for lirc interface parity, so I'm against removing it. -- Jarod Wilson ja...@redhat.com -- To

[PATCH v4 0/7] V4L2 and V4L2 subdev selection target and flag changes

2012-06-15 Thread Sakari Ailus
Hi, Compared to the previous version of the patchset, I've fixed a few spelling errors, changed X/O to tell whether a target or a flag is valid on an interface to more clear Yes/No. There's also a new patch to V4L2 subdev selection API documentation fixing conflicting definition of the

[PATCH v4 2/7] v4l: Remove _ACTUAL from subdev selection API target definition names

2012-06-15 Thread Sakari Ailus
The string _ACTUAL does not say anything more about the target names. Drop it. V4L2 selection API was changed by V4L: Remove _ACTIVE from the selection target name definitions by Sylwester Nawrocki. This patch does the same for the V4L2 subdev API. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi

[PATCH v4 5/7] v4l: Unify selection flags

2012-06-15 Thread Sakari Ailus
Unify flags on the selection interfaces on V4L2 and V4L2 subdev. Flags are very similar to targets in this case: there are more similarities than differences between the two interfaces. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi --- drivers/media/video/omap3isp/ispccdc.c|2 +-

[PATCH v4 4/7] v4l: Common documentation for selection targets

2012-06-15 Thread Sakari Ailus
Both V4L2 and V4L2 subdev interface have very similar selection APIs with differences foremost related to in-memory and media bus formats. However, the selection targets are the same for both. Most targets are and in the future will likely continue to be more the same than with any differences.

[PATCH v4 7/7] v4l: Correct conflicting V4L2 subdev selection API documentation

2012-06-15 Thread Sakari Ailus
The API reference documents that the KEEP_CONFIG flag tells the configuration should not be propatgated by the driver whereas the interface documentation (dev-subdev.xml) categorically prohibited any changes to the rest of the pipeline. The latter makes no sense, since it would severely limit the

[PATCH v4 3/7] v4l: Unify selection targets across V4L2 and V4L2 subdev interfaces

2012-06-15 Thread Sakari Ailus
Signed-off-by: Sakari Ailus sakari.ai...@iki.fi Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com --- drivers/media/video/omap3isp/ispccdc.c |6 ++-- drivers/media/video/omap3isp/isppreview.c |6 ++-- drivers/media/video/omap3isp/ispresizer.c |6 ++--

[PATCH v4 6/7] v4l: Unify selection flags documentation

2012-06-15 Thread Sakari Ailus
As for the selection targets, the selection flags are now the same on V4L2 and V4L2 subdev interfaces. Also document them so. Signed-off-by: Sakari Ailus sakari.ai...@iki.fi --- Documentation/DocBook/media/v4l/dev-subdev.xml |6 +- Documentation/DocBook/media/v4l/selection-api.xml |

[PATCH v4 1/7] V4L: Remove _ACTIVE from the selection target name definitions

2012-06-15 Thread Sakari Ailus
From: Sylwester Nawrocki sylvester.nawro...@gmail.com This patch drops the _ACTIVE part from the selection target names as a prerequisite to unify the selection target names across the subdev and regular video node API. The meaning of V4L2_SEL_TGT_*_ACTIVE and V4L2_SUBDEV_SEL_TGT_*_ACTUAL

Re: [PATCH v4 2/7] v4l: Remove _ACTUAL from subdev selection API target definition names

2012-06-15 Thread Laurent Pinchart
Hi Sakari, Thanks for the patch. On Friday 15 June 2012 16:44:35 Sakari Ailus wrote: The string _ACTUAL does not say anything more about the target names. Drop it. V4L2 selection API was changed by V4L: Remove _ACTIVE from the selection target name definitions by Sylwester Nawrocki. This

STK1160 has audio now (was Re: [Q] Why is it needed to add an alsa module to v4l audio capture devices?)

2012-06-15 Thread Ezequiel Garcia
Hi again, On Mon, Jun 11, 2012 at 1:39 PM, Ezequiel Garcia some setup is required to configure the audio input associated with a video input, and to enable clock for the audio sampler. Such setup is made when a video input is selected. You likely need something similar for stk1160. I've

Re: [PATCH v4 7/7] v4l: Correct conflicting V4L2 subdev selection API documentation

2012-06-15 Thread Laurent Pinchart
Hi Sakari, Thanks for the patch. On Friday 15 June 2012 16:44:40 Sakari Ailus wrote: The API reference documents that the KEEP_CONFIG flag tells the configuration should not be propatgated by the driver whereas the interface s/propatgated/propagated/ documentation (dev-subdev.xml)

Re: [PATCH v4 7/7] v4l: Correct conflicting V4L2 subdev selection API documentation

2012-06-15 Thread Sakari Ailus
Hi Laurent, Thanks for the comments! On Fri, Jun 15, 2012 at 04:14:21PM +0200, Laurent Pinchart wrote: Hi Sakari, Thanks for the patch. On Friday 15 June 2012 16:44:40 Sakari Ailus wrote: The API reference documents that the KEEP_CONFIG flag tells the configuration should not be

[samsung:media-for3.5-vb2-dmabuf-v7] warning: (VIDEO_VIVI DRM) selects DMA_SHARED_BUFFER which has unmet direct dependencies (EXPERIMENTAL)

2012-06-15 Thread wfg
Hi Tomasz, There are new compile warnings show up in tree: git://git.infradead.org/users/kmpark/linux-samsung media-for3.5-vb2-dmabuf-v7 head: ae2f5efcd4c8f89fec68807eaadfd653a0d75a9f commit: 757c994793a6fb902a8f22029d4b5a4a1f66e51f [163/194] v4l: vivi: support for dmabuf importing config: