Re: [PATCH] isdn: Return -EINTR in gigaset_start() if locking attempts fails.

2012-03-16 Thread David Miller
From: santosh nayak Date: Fri, 16 Mar 2012 18:40:13 +0530 > We have 3 callers: gigaset_probe(), gigaset_tty_open() and > gigaset_probe(). Each caller tries to free allocated memory > if lock fails. This is possible if we returns -EINTR. Look again at the callers. They interpret "0" as an error,

Re: [Linaro-mm-sig] [PATCH] dma-buf: add get_dma_buf()

2012-03-16 Thread Kyungmin Park
Reviewed-by: Kyungmin Park On 3/17/12, Rob Clark wrote: > From: Rob Clark > > Works in a similar way to get_file(), and is needed in cases such as > when the exporter needs to also keep a reference to the dmabuf (that > is later released with a dma_buf_put()), and possibly other similar > cases

[PATCH 2/7] marvell-cam: Remove broken "owner" logic

2012-03-16 Thread Jonathan Corbet
The marvell cam driver retained just enough of the owner-tracking logic from cafe_ccic to be broken; it could, conceivably, cause the driver to release DMA memory while the controller is still active. Simply remove the remaining pieces and ensure that the controller is stopped before we free thing

[PATCH] A series of Marvell camera fixes

2012-03-16 Thread Jonathan Corbet
Hi, Mauro, Here is a set of relatively straightforward fixes for some Marvell camera bugs, some of which were rather less straightforward to find. If you prefer, they can be pulled from: git://git.lwn.net/linux-2.6.git marvell-fixes Therein you'll find: Jonathan Corbet (7): marve

[PATCH 1/7] marvell-cam: ensure that the camera stops when requested

2012-03-16 Thread Jonathan Corbet
The controller stop/restart logic could possibly restart DMA after the driver things things have stopped, with suitably ugly results. Make sure that we only restart the hardware if we're supposed to be streaming. Signed-off-by: Jonathan Corbet --- drivers/media/video/marvell-ccic/mcam-core.c |

[PATCH 5/7] marvell-cam: Don't signal multiple frame completions in scatter/gather mode

2012-03-16 Thread Jonathan Corbet
There is only one frame known to the DMA engine in scatter/gather mode, but it still tells us that any or all of frames 1-3 are done at each completion interrupt. Avoid the creation of junk frames by being sure to only "complete" one on each interrupt. Signed-off-by: Jonathan Corbet --- drivers

[PATCH 6/7] mmp-camera: Don't power up the sensor on resume

2012-03-16 Thread Jonathan Corbet
We still need to power up the controller to avoid unsightly self-immolation should something try to access its registers, but the sensor can stay powered down unless the camera was actually operating at suspend time. This gets rid of the camera LED flash on resume, fixing OLPC bug #11644. Signed-o

[PATCH 7/7] marvell-cam: Demote the "release" print to debug level

2012-03-16 Thread Jonathan Corbet
We've spammed enough logfiles at this point. Signed-off-by: Jonathan Corbet --- drivers/media/video/marvell-ccic/mcam-core.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/video/marvell-ccic/mcam-core.c b/drivers/media/video/marvell-ccic/mcam-core.c index 0

[PATCH 4/7] marvell-cam: fix the green screen of death

2012-03-16 Thread Jonathan Corbet
I had learned through hard experience that dinking around with the DMA descriptors while the C1_DESC_ENA enable bit was set is a recipe for all kinds of truly malicious behavior on the hardware's part, regardless of whether the DMA engine is actually operating at the time. That notwithstanding, th

[PATCH 3/7] marvell-cam: Increase the DMA shutdown timeout

2012-03-16 Thread Jonathan Corbet
Experience shows that, on the Armada platform, it can take as much as 120ms for the DMA engine to actually shut down after it has been told to. So a 40ms timeout is not adequate; use 150ms instead. Also make sure we don't leave the DMA_ACTIVE flag set once things are down. Signed-off-by: Jonatha

[PATCH] media: em28xx: Unused macro cleanup

2012-03-16 Thread Ezequiel Garcia
Signed-off-by: Ezequiel Garcia --- drivers/media/video/em28xx/em28xx-i2c.c |8 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/drivers/media/video/em28xx/em28xx-i2c.c b/drivers/media/video/em28xx/em28xx-i2c.c index 36f5a9b..a88e169 100644 --- a/drivers/media/video/em2

Re: Move em27xx/em28xx webcams to a gspca subdriver ?

2012-03-16 Thread Frank Schäfer
[Was: eMPIA EM2710 Webcam (em28xx) and LIRC] Continue this part of the discussion in a new thread... Am 15.03.2012 14:05, schrieb Mauro Carvalho Chehab: > Em 15-03-2012 09:34, Frank Schäfer escreveu: >> ... >> >> I would like to bring up the question, if it wouldn't make sense to move >> support

Bit rate of a transport stream

2012-03-16 Thread Bhasker C V
Hi all, In finding the bit rate of a transport stream as a whole, I had a doubt if someone can help me please I did try to google but could not get any definite information on this topic Say I have 2 programs in a transport stream each with 1 audio and 1 video Can I safely assume the total b

[PATCH] dma-buf: add get_dma_buf()

2012-03-16 Thread Rob Clark
From: Rob Clark Works in a similar way to get_file(), and is needed in cases such as when the exporter needs to also keep a reference to the dmabuf (that is later released with a dma_buf_put()), and possibly other similar cases. Signed-off-by: Rob Clark --- Minor update on original to add a mis

Re: My Microdia (SN9C201) webcam succumbs to glare in Linux

2012-03-16 Thread Jean-Francois Moine
On Fri, 16 Mar 2012 08:53:51 +1100 Xavion wrote: > As you can probably gather from the attached screenshots, I'm > attempting to use my SN9C201 webcam for home security.  The problem is > that it succumbs to external glare during the middle hours of sunny > days when used in Linux. > > The same

Re: [PATCH] RFC: dma-buf: userspace mmap support

2012-03-16 Thread Rob Clark
On Fri, Mar 16, 2012 at 12:24 PM, Tom Cooksey wrote: > >> From: Rob Clark >> >> Enable optional userspace access to dma-buf buffers via mmap() on the >> dma-buf file descriptor.  Userspace access to the buffer should be >> bracketed with DMA_BUF_IOCTL_{PREPARE,FINISH}_ACCESS ioctl calls to >> giv

cron job: media_tree daily build: ERRORS

2012-03-16 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 Mar 16 19:00:18 CET 2012 git hash:632fba4d012458fd5fedc678fb9b0f8bc59ceda2 gcc version: i686-linux-gcc (GCC

Re: [PATCH] [media] staging: Return -EINTR in s2250_probe() if fails to get lock.

2012-03-16 Thread Oliver Neukum
Am Freitag, 16. März 2012, 17:56:20 schrieb santosh prasad nayak: > On Fri, Mar 16, 2012 at 10:02 PM, Oliver Neukum wrote: > > > > Indeed there's a lot wrong here. The idea of having an interruptible > > sleep in probe() is arcane. You need a very, very, very good reason for > > that. > > Can yo

Re: [Linaro-mm-sig] [PATCH] dma-buf: add get_dma_buf()

2012-03-16 Thread Dave Airlie
On Fri, Mar 16, 2012 at 4:04 PM, Rob Clark wrote: > From: Rob Clark > > Works in a similar way to get_file(), and is needed in cases such as > when the exporter needs to also keep a reference to the dmabuf (that > is later released with a dma_buf_put()), and possibly other similar > cases. > > Si

[PATCH v3] media: rc: Pospone ir raw decoders loading until really needed

2012-03-16 Thread Ezequiel Garcia
This changes rc_core to not load the IR decoders at load time, postponing it to load only if a RC_DRIVER_IR_RAW device is registered via rc_register_device. We use a static boolean variable, to ensure decoders modules are only loaded once. Tested with rc-loopback device only. Signed-off-by: Ezequi

Re: [PATCH] [media] staging: Return -EINTR in s2250_probe() if fails to get lock.

2012-03-16 Thread santosh prasad nayak
On Fri, Mar 16, 2012 at 10:02 PM, Oliver Neukum wrote: > Am Freitag, 16. März 2012, 17:23:58 schrieb santosh nayak: >> From: Santosh Nayak >> >> In s2250_probe(), If locking attempt is interrupted by a signal then >> it should return -EINTR after unregistering audio device and making free >> the

Re: [PATCH] [media] staging: Return -EINTR in s2250_probe() if fails to get lock.

2012-03-16 Thread Oliver Neukum
Am Freitag, 16. März 2012, 17:23:58 schrieb santosh nayak: > From: Santosh Nayak > > In s2250_probe(), If locking attempt is interrupted by a signal then > it should return -EINTR after unregistering audio device and making free > the allocated memory. > > At present, if locking is interrupted b

[PATCH] [media] staging: Return -EINTR in s2250_probe() if fails to get lock.

2012-03-16 Thread santosh nayak
From: Santosh Nayak In s2250_probe(), If locking attempt is interrupted by a signal then it should return -EINTR after unregistering audio device and making free the allocated memory. At present, if locking is interrupted by signal it will display message "initialized successfully" and return su

Re: [PATCH V2]NEXT:drivers:staging:media Fix comments and some typos in staging/media/*

2012-03-16 Thread Justin P. Mattock
On 03/16/2012 09:06 AM, Justin P. Mattock wrote: On 03/16/2012 08:58 AM, Greg KH wrote: On Fri, Mar 16, 2012 at 04:45:03PM +0100, Jiri Kosina wrote: On Fri, 16 Mar 2012, Justin P. Mattock wrote: before I forget about this patch, what was the status of this one? As previously announced multi

Re: [PATCH V2]NEXT:drivers:staging:media Fix comments and some typos in staging/media/*

2012-03-16 Thread Justin P. Mattock
On 03/16/2012 08:58 AM, Greg KH wrote: On Fri, Mar 16, 2012 at 04:45:03PM +0100, Jiri Kosina wrote: On Fri, 16 Mar 2012, Justin P. Mattock wrote: before I forget about this patch, what was the status of this one? As previously announced multiple times, I am ignoring patches that go to driver

[PATCH] dma-buf: add get_dma_buf()

2012-03-16 Thread Rob Clark
From: Rob Clark Works in a similar way to get_file(), and is needed in cases such as when the exporter needs to also keep a reference to the dmabuf (that is later released with a dma_buf_put()), and possibly other similar cases. Signed-off-by: Rob Clark --- include/linux/dma-buf.h | 14 +

Re: [PATCH V2]NEXT:drivers:staging:media Fix comments and some typos in staging/media/*

2012-03-16 Thread Greg KH
On Fri, Mar 16, 2012 at 04:45:03PM +0100, Jiri Kosina wrote: > On Fri, 16 Mar 2012, Justin P. Mattock wrote: > > > before I forget about this patch, what was the status of this one? > > As previously announced multiple times, I am ignoring patches that go to > drivers/staging and letting Greg pi

Re: [PATCH] em28xx: pre-allocate DVB isoc transfer buffers

2012-03-16 Thread Andy Furniss
Gianluca Gennari wrote: Andy Furniss wrote: Mar 16 14:56:58 noki kernel: ehci_hcd :01:0a.2: shutdown urb ced2c800 ep4in-iso Looks like some innocuous logging from the ehci_hcd driver. I've never seen it because I'm not using the ehci_hcd module on my systems. When you tune a new channel, th

Re: [PATCH V2]NEXT:drivers:staging:media Fix comments and some typos in staging/media/*

2012-03-16 Thread Justin P. Mattock
On 03/16/2012 08:45 AM, Jiri Kosina wrote: On Fri, 16 Mar 2012, Justin P. Mattock wrote: before I forget about this patch, what was the status of this one? As previously announced multiple times, I am ignoring patches that go to drivers/staging and letting Greg pick them up if he wants to. G

Re: [PATCH V2]NEXT:drivers:staging:media Fix comments and some typos in staging/media/*

2012-03-16 Thread Jiri Kosina
On Fri, 16 Mar 2012, Justin P. Mattock wrote: > before I forget about this patch, what was the status of this one? As previously announced multiple times, I am ignoring patches that go to drivers/staging and letting Greg pick them up if he wants to. Greg, if you want to have this changed, just

Re: [PATCH] em28xx: pre-allocate DVB isoc transfer buffers

2012-03-16 Thread Gianluca Gennari
Il 16/03/2012 16:05, Andy Furniss ha scritto: > Andy Furniss wrote: >> Gianluca Gennari wrote: > >>> Hi Andy, >>> the patch is already in the current media_build tree and is queued for >>> kernel 3.4. >> >> Ahh, I'll give that a try, thanks. > > Seems to be working OK so far, I had to avoid build

Re: [PATCH V2]NEXT:drivers:staging:media Fix comments and some typos in staging/media/*

2012-03-16 Thread Justin P. Mattock
before I forget about this patch, what was the status of this one? On 03/06/2012 06:55 AM, Justin P. Mattock wrote: From: "Justin P. Mattock" linux-next: I like to spend some time reading code, in doing so I have found some typos in some of the comments. The patch below fixes what I have found

Re: [PATCH] em28xx: pre-allocate DVB isoc transfer buffers

2012-03-16 Thread Andy Furniss
Andy Furniss wrote: Gianluca Gennari wrote: Hi Andy, the patch is already in the current media_build tree and is queued for kernel 3.4. Ahh, I'll give that a try, thanks. Seems to be working OK so far, I had to avoid building radio to get it to compile. Are these just informational - I

Re: [Linaro-mm-sig] [PATCH] RFC: dma-buf: userspace mmap support

2012-03-16 Thread Rob Clark
On Fri, Mar 16, 2012 at 5:50 AM, Marcus Lorentzon wrote: > On 03/15/2012 02:32 AM, Rob Clark wrote: >> >> From: Rob Clark >> [snip] >> >> In all cases, the mmap() call is allowed to fail, and the associated >> dma_buf_ops are optional (mmap() will fail if at least the mmap() >> op is not implement

reading config parameters of omap3-isp subdevs

2012-03-16 Thread Michael Jones
Hi all, I am playing around with some parameters in the previewer on the ISP. With ioctl VIDIOC_OMAP3ISP_PRV_CFG I am able to write the various parameters but what I'm missing is a way to read them. For example, I have no way to adjust only coef2 in 'struct omap3isp_prev_wbal' while leaving

RE: [Q] media: V4L2 compressed frames and s_fmt.

2012-03-16 Thread Kamil Debski
Hi Javier, hi Jeongtae, Please see my comments below. > -Original Message- > From: javier Martin [mailto:javier.mar...@vista-silicon.com] > Sent: 15 March 2012 15:31 > To: Kamil Debski > Cc: Sakari Ailus; linux-media@vger.kernel.org > Subject: Re: [Q] media: V4L2 compressed frames and s_f

[PULL] soc-camera for 3.4 take 2

2012-03-16 Thread Guennadi Liakhovetski
Hi Mauro A couple more soc-camera patches for 3.4. See - I'm trying to comply: several smaller pull-requests;-) The following changes since commit 632fba4d012458fd5fedc678fb9b0f8bc59ceda2: [media] cx25821: Add a card definition for "No brand" cards that have: subvendor = 0x subdevice = 0

[PATCH] isdn: Return -EINTR in gigaset_start() if locking attempts fails.

2012-03-16 Thread santosh nayak
From: Santosh Nayak If locking attempt was interrupted by a signal then we should return -EINTR so that caller can take appropriate action. We have 3 callers: gigaset_probe(), gigaset_tty_open() and gigaset_probe(). Each caller tries to free allocated memory if lock fails. This is possible if we

Re: [PATCH 1/1] mt9v032: Provide pixel rate control

2012-03-16 Thread Sakari Ailus
On Fri, Mar 16, 2012 at 01:55:13PM +0100, Laurent Pinchart wrote: > Hi Sakari, > > On Friday 16 March 2012 14:36:53 Sakari Ailus wrote: > > On Fri, Mar 16, 2012 at 01:31:39PM +0100, Laurent Pinchart wrote: > > > On Friday 16 March 2012 14:12:11 Sakari Ailus wrote: > > > > On Fri, Mar 16, 2012 at 1

Re: [PATCH 1/1] mt9v032: Provide pixel rate control

2012-03-16 Thread Laurent Pinchart
Hi Sakari, On Friday 16 March 2012 14:36:53 Sakari Ailus wrote: > On Fri, Mar 16, 2012 at 01:31:39PM +0100, Laurent Pinchart wrote: > > On Friday 16 March 2012 14:12:11 Sakari Ailus wrote: > > > On Fri, Mar 16, 2012 at 12:58:30PM +0100, Laurent Pinchart wrote: > > > > On Thursday 15 March 2012 23:

[PATCH] [media] dib0700: Return -EINTR and unlock mutex if locking attempts fails.

2012-03-16 Thread santosh nayak
From: Santosh Nayak In 'dib0700_i2c_xfer_new()' and 'dib0700_i2c_xfer_legacy()' we are taking two locks: 1. i2c_mutex 2. usb_mutex If attempt to take 'usb_mutex' lock fails then the previously taken lock 'i2c_mutex' should be unlocked and -EINTR should be return s

Re: [PATCH 1/1] mt9v032: Provide pixel rate control

2012-03-16 Thread Sakari Ailus
Hi Laurent, On Fri, Mar 16, 2012 at 01:31:39PM +0100, Laurent Pinchart wrote: > On Friday 16 March 2012 14:12:11 Sakari Ailus wrote: > > On Fri, Mar 16, 2012 at 12:58:30PM +0100, Laurent Pinchart wrote: > > > On Thursday 15 March 2012 23:01:39 Sakari Ailus wrote: > > > > Provide pixel rate control

Re: [PATCH 1/1] mt9v032: Provide pixel rate control

2012-03-16 Thread Laurent Pinchart
Hi Sakari, On Friday 16 March 2012 14:12:11 Sakari Ailus wrote: > On Fri, Mar 16, 2012 at 12:58:30PM +0100, Laurent Pinchart wrote: > > On Thursday 15 March 2012 23:01:39 Sakari Ailus wrote: > > > Provide pixel rate control calculated from external clock and horizontal > > > binning factor. > > >

Re: [PATCH 1/1] mt9v032: Provide pixel rate control

2012-03-16 Thread Sakari Ailus
Hi Laurent, Thanks for the review! On Fri, Mar 16, 2012 at 12:58:30PM +0100, Laurent Pinchart wrote: > On Thursday 15 March 2012 23:01:39 Sakari Ailus wrote: > > Provide pixel rate control calculated from external clock and horizontal > > binning factor. > > > > Signed-off-by: Sakari Ailus > >

Re: [PATCH 1/1] mt9v032: Provide pixel rate control

2012-03-16 Thread Laurent Pinchart
Hi Sakari, Thanks for the patch. On Thursday 15 March 2012 23:01:39 Sakari Ailus wrote: > Provide pixel rate control calculated from external clock and horizontal > binning factor. > > Signed-off-by: Sakari Ailus > --- > drivers/media/video/mt9v032.c | 35 ++-

media_build fail

2012-03-16 Thread Andy Furniss
Building on an old 32bit PC with gcc 4.0.3, though I did manage to build some weeks ago. Today I get - C [M] /home/andy/Src/media_build/v4l/radio-isa.o /home/andy/Src/media_build/v4l/radio-isa.c: In function 'radio_isa_probe': /home/andy/Src/media_build/v4l/radio-isa.c:246: error: implicit de

Re: [PATCH] omap_vout: Set DSS overlay_info only if paddr is non zero

2012-03-16 Thread Archit Taneja
Hi, On Friday 16 March 2012 03:46 PM, Archit Taneja wrote: On Monday 12 March 2012 03:34 PM, Hiremath, Vaibhav wrote: On Wed, Mar 07, 2012 at 14:31:16, Taneja, Archit wrote: The omap_vout driver tries to set the DSS overlay_info using set_overlay_info() when the physical address for the overla

Re: [Linaro-mm-sig] [PATCH] RFC: dma-buf: userspace mmap support

2012-03-16 Thread Marcus Lorentzon
On 03/15/2012 02:32 AM, Rob Clark wrote: From: Rob Clark [snip] In all cases, the mmap() call is allowed to fail, and the associated dma_buf_ops are optional (mmap() will fail if at least the mmap() op is not implemented by the exporter, but in either case the {prepare,finish}_access() ops are op

Re: cxd2820r: i2c wr failed (PCTV Nanostick 290e)

2012-03-16 Thread Andy Furniss
Andy Furniss wrote: (maybe echo 3 > /proc/sys/vm/drop_caches would also work I don't know if it makes any difference, but I missed a bit from that - maybe this would be safer - sync;echo 3 > /proc/sys/vm/drop_caches -- To unsubscribe from this list: send the line "unsubscribe linux-media" in

Re: [PATCH] omap_vout: Set DSS overlay_info only if paddr is non zero

2012-03-16 Thread Archit Taneja
On Monday 12 March 2012 03:34 PM, Hiremath, Vaibhav wrote: On Wed, Mar 07, 2012 at 14:31:16, Taneja, Archit wrote: The omap_vout driver tries to set the DSS overlay_info using set_overlay_info() when the physical address for the overlay is still not configured. This happens in omap_vout_probe()

Re: [PATCH] em28xx: pre-allocate DVB isoc transfer buffers

2012-03-16 Thread Andy Furniss
Gianluca Gennari wrote: Il 16/03/2012 01:25, Andy Furniss ha scritto: Does this patch have a chance of getting in? I am still having to flush caches before use. If you want more testing I can give it a go. I didn't earlier as I didn't have a git to apply it to and thought it was going to get i

Fwd: ADS InstantFM Music RDX-155-EF

2012-03-16 Thread Daniele Rogora
Hi everyone, I need your help. I can't get any sound from the device in the subject. I'm actually on Gentoo 64bit, vanilla kernel 3.3.0-rc2 with usb-audio support and radio-si470x with usb support built-in. I'm using alsa 1.0.25. When i plug the device I get the following dmesg: [ 1704.922092

RE: [PATCH] omap_vout: Fix "DMA transaction error" issue when rotation is enabled

2012-03-16 Thread Hiremath, Vaibhav
On Wed, Mar 14, 2012 at 21:00:25, Laurent Pinchart wrote: > Hi Vaibhav, > > On Friday 09 March 2012 17:41:57 Vaibhav Hiremath wrote: > > When rotation is enabled and driver is configured in USERPTR > > buffer exchange mechanism, in specific use-case driver reports > > an error, > >"DMA transac

RE: [PATCH] omap_vout: Fix the build warning and section miss-match warning

2012-03-16 Thread Hiremath, Vaibhav
On Wed, Mar 14, 2012 at 20:22:37, Laurent Pinchart wrote: > Hi Vaibhav, > > Thanks for the patch. > > On Friday 09 March 2012 17:44:03 Vaibhav Hiremath wrote: > > Patch fixes below build warning and section miss-match warning > > from omap_vout driver - > > You should probably not refer to "patc

Re: eMPIA EM2710 Webcam (em28xx) and LIRC

2012-03-16 Thread Rui Salvaterra
Hi again, I'm terribly sorry for the delay 2012/3/15 Mauro Carvalho Chehab : > Em 15-03-2012 14:15, Ezequiel García escreveu: >> Hi Mauro, >> >> On 3/15/12, Mauro Carvalho Chehab wrote: >>> If you won't take it, it is likely that some day someone will do it, >>> but, as this is just a cleanup, t

Re: cxd2820r: i2c wr failed (PCTV Nanostick 290e)

2012-03-16 Thread Keith Edmunds
Thanks for the suggestions; I'll investigate further. One other oddity I noticed: the last time the i2c errors were logged, rather than immediately rebooting I unplugged the 290e. There is a blue LED on the 290e, which of course went out when it was unplugged. However, it didn't come back on when