Re: [PATCH v11 2/8] usb: phy: omap-usb2: use the new generic PHY framework

2013-09-26 Thread Kishon Vijay Abraham I
Hi Greg, On Friday 27 September 2013 12:21 AM, Greg KH wrote: > On Wed, Aug 21, 2013 at 11:16:07AM +0530, Kishon Vijay Abraham I wrote: >> Used the generic PHY framework API to create the PHY. Now the power off and >> power on are done in omap_usb_power_off and omap_usb_power_on respectively. >> T

[PATCH] [media] uvcvideo: fix data type for pan/tilt control

2013-09-26 Thread Chanho Min
The pan/tilt absolute control value is signed value. If minimum value is minus, It will be changed to plus by clamp_t() as commit 64ae9958a62. ([media] uvcvideo: Fix control value clamping for unsigned integer controls). It leads to wrong setting of the control values. For example, when min and ma

cron job: media_tree daily build: WARNINGS

2013-09-26 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 Sep 27 04:00:13 CEST 2013 git branch: test git hash: ffee921033e64edf8579a3b21c7f15d1a6c3ef71 gcc versi

Re: [PATCH 31/51] DMA-API: media: omap3isp: use dma_coerce_mask_and_coherent()

2013-09-26 Thread Laurent Pinchart
Hi Russell, Thank you for the patch. On Thursday 19 September 2013 22:56:02 Russell King wrote: > The code sequence: > isp->raw_dmamask = DMA_BIT_MASK(32); > isp->dev->dma_mask = &isp->raw_dmamask; > isp->dev->coherent_dma_mask = DMA_BIT_MASK(32); > bypasses the architectures ch

Re: avermedia A306 / PCIe-minicard (laptop)

2013-09-26 Thread remi
Hello The firmware got fixed, the module option, needs a file-name only, no path , lol for once Well, the driver says Firmware OK , I have seen the message of Mauro, he must be right a hundred percent, 'cause we seem to have the initialisations ok, but , for me at lease no data coming out fr

Re: [PATCH 00/51] DMA mask changes

2013-09-26 Thread Rafał Miłecki
2013/9/19 Russell King - ARM Linux : > This email is only being sent to the mailing lists in question, not to > anyone personally. The list of individuals is far to great to do that. > I'm hoping no mailing lists reject the patches based on the number of > recipients. Huh, I think it was enough t

Re: [PATCH v11 2/8] usb: phy: omap-usb2: use the new generic PHY framework

2013-09-26 Thread Greg KH
On Wed, Aug 21, 2013 at 11:16:07AM +0530, Kishon Vijay Abraham I wrote: > Used the generic PHY framework API to create the PHY. Now the power off and > power on are done in omap_usb_power_off and omap_usb_power_on respectively. > The omap-usb2 driver is also moved to driver/phy. > > However using

Re: [PATCH] [media] dvb_demux: fix deadlock in dmx_section_feed_release_filter()

2013-09-26 Thread Mauro Carvalho Chehab
Em Sat, 17 Aug 2013 23:48:51 +0300 Alexey Khoroshilov escreveu: > dmx_section_feed_release_filter() locks dvbdmx->mutex and > if the feed is still filtering, it calls feed->stop_filtering(feed). > stop_filtering() is implemented by dmx_section_feed_stop_filtering() > that first of all try to lock

Re: [PATCH 1/1] cx23885-dvb: fix ds3000 ts2020 split for TEVII S471

2013-09-26 Thread Mauro Carvalho Chehab
Hi Christian, Em Wed, 14 Aug 2013 22:58:47 +0200 Christian Volkmann escreveu: > A split for ds3000/ts2020 code forgot to change the TEVII_S471 code. > Change the TEVII_S471 according the changes to TEVII_S470. > > Signed-off-by: Christian Volkmann > --- > drivers/media/pci/cx23885/cx23885-dvb

[PATCH] [media] stk1135: fix two warnings added by changeset 76e0598

2013-09-26 Thread Mauro Carvalho Chehab
drivers/media/usb/gspca/stk1135.c:615:6: warning: no previous prototype for 'stk1135_try_fmt' [-Wmissing-prototypes] void stk1135_try_fmt(struct gspca_dev *gspca_dev, struct v4l2_format *fmt) ^ drivers/media/usb/gspca/stk1135.c:627:5: warning: no previous prototype for 'stk1135_enum_frames

Capture driver implementation issue/questions

2013-09-26 Thread Rick Ball
Hi All, I'm working on a video capture driver (my first) for a custom board, and I have a few questions about handling 'overflow' conditions (when the application doesn't get back in time to de-queue every frame). I know that one way to avoid this is to allocate additional frame buffers, but I'

Re: Hauppauge HVR-900 HD and HVR 930C-HD with si2165

2013-09-26 Thread Antti Palosaari
On 25.09.2013 07:50, Matthias Schwarzott wrote: On 17.08.2013 13:30, Ulf wrote: Hi, I know the topic Hauppauge HVR-900 HD and HVR 930C-HD with si2165 demodulator was already discussed http://permalink.gmane.org/gmane.linux.drivers.video-input-infrastructure/40982 and http://permalink.gmane.org/

[RFC] omap3isp : fix image corruption after underrun when using resizer

2013-09-26 Thread jean-philippe francois
Hi Laurent, I was able to reliably get corrupted image when placing the pipeline in underrun condition. The pipeline looks like this : YUYV sensor -> CCDC -> Resizer -> V4L output It seems that triggering 'frame sync event' before last line leads to possible corrupted images when using the resiz

Re: [PATCH v4] media: st-rc: Add ST remote control driver

2013-09-26 Thread Srinivas KANDAGATLA
Hi Stephen, On 24/09/13 20:49, Stephen Warren wrote: >>> >> Should those property names be prefixed with "st,"; I assume they're >>> >> specific to this binding rather than something generic that applies to >>> >> all IR controller bindings? If you expect them to be generic, it's fine. >> > >> >

register ov3640 camera on i2c bus

2013-09-26 Thread Tom
Hello community, I tried to use my ov3640 camera along with my gumstix overo board and the linux kernel 3.10. So for that I added the appropriate lines into the board-overo.c file, but when I use "i2cdetect" after startup I get the wrong output: root@overo2:~# i2cdetect -r 3 WARNING! This program

Re: [PATCH v8] s5k5baf: add camera sensor driver

2013-09-26 Thread Andrzej Hajda
Hi Mark, Thanks for the review, sorry for late response. On 09/20/2013 07:06 PM, Mark Rutland wrote: > On Fri, Sep 06, 2013 at 11:31:06AM +0100, Andrzej Hajda wrote: >> Driver for Samsung S5K5BAF UXGA 1/5" 2M CMOS Image Sensor >> with embedded SoC ISP. >> The driver exposes the sensor as two V4L2

Re: What are "Other" video capture settings.

2013-09-26 Thread Hans Verkuil
On Thu 26 September 2013 13:47:26 Steve Cookson wrote: > On 26/09/2013 08:32, Hans Verkuil wrote: > > Typically the video standard needs to be correct (e.g. PAL vs NTSC), > > and the video format produced by the video capture card must be > > supported by the application. > > > > Ask them to r

Re: What are "Other" video capture settings.

2013-09-26 Thread Steve Cookson
On 26/09/2013 08:32, Hans Verkuil wrote: > Typically the video standard needs to be correct (e.g. PAL vs NTSC), > and the video format produced by the video capture card must be > supported by the application. > > Ask them to run 'v4l2-ctl -d /dev/video0 --all', that gives a good > overview of the

Re: What are "Other" video capture settings.

2013-09-26 Thread Hans Verkuil
On Thu 26 September 2013 13:04:00 Steve Cookson wrote: > Hi All, > > Occasionally I see emails suggesting that a video capture card has not > been configured properly on Linux and that the problems the questioner > is experiencing is down to this. > > I have no idea what this means. Without kn

What are "Other" video capture settings.

2013-09-26 Thread Steve Cookson
Hi All, Occasionally I see emails suggesting that a video capture card has not been configured properly on Linux and that the problems the questioner is experiencing is down to this. I have no idea what this means. I usually use a gstreamer command like: gst-launch v4l2src device=/dev/video

Re: [PATCH 39/51] DMA-API: others: use dma_set_coherent_mask()

2013-09-26 Thread Archit Taneja
Hi, On Friday 20 September 2013 04:41 AM, Russell King wrote: The correct way for a driver to specify the coherent DMA mask is not to directly access the field in the struct device, but to use dma_set_coherent_mask(). Only arch and bus code should access this member directly. Convert all direc

RE: [PATCH v2] [media] marvell-ccic: simplify and fix clk handling (a bit)

2013-09-26 Thread Libin Yang
Hi Russell, >-Original Message- >From: Russell King - ARM Linux [mailto:li...@arm.linux.org.uk] >Sent: Thursday, September 26, 2013 4:24 PM >To: Uwe Kleine-König >Cc: Libin Yang; Jonathan Corbet; Mauro Carvalho Chehab; >linux-media@vger.kernel.org; >linux-arm-ker...@lists.infradead.org;

RE: [PATCH v2] [media] marvell-ccic: simplify and fix clk handling (a bit)

2013-09-26 Thread Libin Yang
Hi Uwe, Thanks for your reviewing. Please see the comments below. >On Wed, Sep 25, 2013 at 07:47:19PM -0700, Libin Yang wrote: >> In the clk enable and prepare function, we will check the NULL >> pointer. So it should be no problem. >I'm not sure what you mean here and unfortunately your quoting

Re: [PATCH 1/3] media: i2c: Add ADV761X support

2013-09-26 Thread Laurent Pinchart
Hi Hans, On Thursday 26 September 2013 08:31:58 Hans Verkuil wrote: > On 09/26/2013 12:57 AM, Laurent Pinchart wrote: > > On Wednesday 25 September 2013 18:31:51 Guennadi Liakhovetski wrote: > >> On Wed, 25 Sep 2013, Valentine wrote: > >>> On 09/25/2013 06:08 PM, Guennadi Liakhovetski wrote: > > [

Re: [PATCH 1/3] media: i2c: Add ADV761X support

2013-09-26 Thread Laurent Pinchart
Hi Hans, On Thursday 26 September 2013 08:57:43 Hans Verkuil wrote: > On 09/25/2013 12:21 PM, Laurent Pinchart wrote: > > Hi Valentine, > > > > Thank you for the patch. > > > > Please see below for a couple of comments (in addition to Hans' and > > Guennadi's comments). > > > > On Tuesday 24 Se

Re: [PATCH v2] [media] marvell-ccic: simplify and fix clk handling (a bit)

2013-09-26 Thread Russell King - ARM Linux
On Thu, Sep 26, 2013 at 10:13:56AM +0200, Uwe Kleine-König wrote: > Hi Libin, > > On Wed, Sep 25, 2013 at 07:47:19PM -0700, Libin Yang wrote: > > In the clk enable and prepare function, we will check the NULL > > pointer. So it should be no problem. > I'm not sure what you mean here and unfortunat

Re: [PATCH v2] [media] marvell-ccic: simplify and fix clk handling (a bit)

2013-09-26 Thread Uwe Kleine-König
Hi Libin, On Wed, Sep 25, 2013 at 07:47:19PM -0700, Libin Yang wrote: > In the clk enable and prepare function, we will check the NULL > pointer. So it should be no problem. I'm not sure what you mean here and unfortunately your quoting style makes your statement appear without context. i