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

2012-10-28 Thread Hiremath, Vaibhav
On Sat, Oct 27, 2012 at 17:01:09, Laurent Pinchart wrote: > Hi Vaibhav, > > On Friday 26 October 2012 09:13:20 Hiremath, Vaibhav wrote: > > On Thu, Oct 25, 2012 at 19:30:58, Valkeinen, Tomi wrote: > > > On 2012-03-09 10:03, Hiremath, Vaibhav wrote: > > > > On Fri, Mar 09, 2012 at 05:17:41, Laurent

Re: [PATCH 53/68] [media] tlg2300: index is unsigned, so never below zero

2012-10-28 Thread Huang Shijie
On Sun, Oct 28, 2012 at 4:41 AM, Mauro Carvalho Chehab wrote: > drivers/media/usb/tlg2300/pd-video.c:891:2: warning: comparison of unsigned > expression < 0 is always false [-Wtype-limits] > drivers/media/usb/tlg2300/pd-video.c:926:2: warning: comparison of unsigned > expression < 0 is always fa

[patch review 02/02] usb hid quirks for Masterkit MA901 usb radio

2012-10-28 Thread Alexey Klimov
Don't let Masterkit MA901 USB radio be handled by usb hid drivers. This device will be handled by radio-ma901.c driver. Signed-off-by: Alexey Klimov diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 5de3bb3..8e06569 100644 --- a/drivers/hid/hid-core.c +++ b/drivers/hid/hid-core

[patch review 01/02] add driver for Masterkit MA901 usb radio

2012-10-28 Thread Alexey Klimov
This patch creates a new usb-radio driver, radio-ma901.c, that supports Masterkit MA 901 USB FM radio devices. This device plugs into both the USB and an analog audio input or headphones, so this thing only deals with initialization and frequency setting. Signed-off-by: Alexey Klimov diff --git

[patch review 00/02] driver for Masterkit MA901 usb radio

2012-10-28 Thread Alexey Klimov
Hello all, Well, i accidentally bought usb radio device Masterkit MA901 and wrote driver for it (looks like V4L doesn't have support for this device). It is selled, for example, on this web site (Russian language, sorry): http://www.masterkit.ru/main/set.php?code_id=607729 I googled to find if i

Re: [PATCH] Add chipid to fc2580.c

2012-10-28 Thread Antti Palosaari
su 28.10.2012 22:07 Mauro Carvalho Chehab kirjoitti: > Em Mon, 22 Oct 2012 10:17:26 +0200 > Oliver Schinagl escreveu: > >> diff --git a/drivers/media/tuners/fc2580.c >> b/drivers/media/tuners/fc2580.c >> index aff39ae..102d942 100644 >> I found a fellow Asus U3100+ user (mentioned him before with

cron job: media_tree daily build: WARNINGS

2012-10-28 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 Oct 28 19:00:25 CET 2012 git hash:895507c19f88b7bbf82f24737d86cca0398647cb gcc version: i686-linux-gcc (GCC

Re: [PATCH] Add chipid to fc2580.c

2012-10-28 Thread Mauro Carvalho Chehab
Em Mon, 22 Oct 2012 10:17:26 +0200 Oliver Schinagl escreveu: > diff --git a/drivers/media/tuners/fc2580.c b/drivers/media/tuners/fc2580.c > index aff39ae..102d942 100644 > I found a fellow Asus U3100+ user (mentioned him before with the > firmware issue) that even when using the latest firmware,

[PATCH 2/2] [media] rc: Fix double free in gpio_ir_recv_remove()

2012-10-28 Thread Jesper Juhl
Since rc_unregister_device() frees its argument there's no need to subsequently call rc_free_device() on the same variable - in fact it's a double free bug. Easily fixed by just removing the rc_free_device() call. Signed-off-by: Jesper Juhl --- drivers/media/rc/gpio-ir-recv.c |1 - 1 files c

Re: [PATCH 00/23] em28xx: add support fur USB bulk transfers

2012-10-28 Thread Mauro Carvalho Chehab
Em Sun, 21 Oct 2012 19:52:05 +0300 Frank Schäfer escreveu: > This patch series adds support for USB bulk transfers to the em28xx driver. > > Patch 1 is a bugfix for the image data processing with non-interlaced devices > (webcams) > that should be considered for stable (see commit message). >

[PATCH 1/2] [media] rc: Fix double free in gpio_ir_recv_probe()

2012-10-28 Thread Jesper Juhl
At the 'err_request_irq' label, rc_unregister_device(rcdev) frees its argument. So when we fall through to the 'err_gpio_request' label further down and call rc_free_device(rcdev) then that's a double free. Fix that by moving 'rcdev = NULL' from after the call to rc_free_device() to after rc_unreg

Re: [PATCH] it913x [BUG] Enable endpoint 3 on devices with HID interface.

2012-10-28 Thread Mauro Carvalho Chehab
Em Sat, 20 Oct 2012 22:03:15 +0100 Malcolm Priestley escreveu: > > On some USB controllers when endpoint 3 (used by HID) is not enabled > this causes a USB reset. > > Signed-off-by: Malcolm Priestley Hi Malcolm, Patch looks ok. Yet, your name is missed at MAINTAINERS file. Please add it the

Re: [PATCH] Improve media Kconfig menu

2012-10-28 Thread Mauro Carvalho Chehab
Em Fri, 19 Oct 2012 13:04:22 +0200 Hans Verkuil escreveu: > I've always found it very confusing that the "Media ancillary drivers (tuners, > sensors, i2c, frontends)" comment came after the "Autoselect" option. This > patch > moves it up and changes the "Autoselect" text to correspond more close

Re: [PATCH 4/4] [media] rc: Call rc_register_device before irq setup

2012-10-28 Thread Mauro Carvalho Chehab
Em Mon, 15 Oct 2012 13:13:47 +0200 Matthijs Kooijman escreveu: > This should fix a potential race condition, when the irq handler > triggers while rc_register_device is still setting up the rdev->raw > device. > > This crash has not been observed in practice, but there should be a very > small w

Re: [PATCH 3/4] [media] rc: Set rdev before irq setup

2012-10-28 Thread Mauro Carvalho Chehab
Em Mon, 15 Oct 2012 13:13:46 +0200 Matthijs Kooijman escreveu: > This fixes a problem in fintek-cir, ite-cir and nuvoton-cir where the > irq handler would trigger during module load before the rdev member was > set, causing a NULL pointer crash. > > It seems this crash is very reproducible (just

Re: [PATCH 2/4] [media] rc: Make probe cleanup goto labels more verbose

2012-10-28 Thread Mauro Carvalho Chehab
Em Mon, 15 Oct 2012 13:13:45 +0200 Matthijs Kooijman escreveu: > Before, labels were simply numbered. Now, the labels are named after the > cleanup action they'll perform (first), based on how the winbond-cir > driver does it. This makes the code a bit more clear and makes changes > in the orderi

Re: [PATCH 1/4] [media] ene-ir: Fix cleanup on probe failure

2012-10-28 Thread Mauro Carvalho Chehab
Em Mon, 15 Oct 2012 13:13:44 +0200 Matthijs Kooijman escreveu: > This makes the cleanup on probe failure more consistent with other > drivers. This is similar to what commit > f27b853ea24a9b70585f9251384d97929e6551c3 ("[media] rc: Fix invalid > free_region and/or free_irq on probe failure") did f

Re: [PATCH 00/23] em28xx: add support fur USB bulk transfers

2012-10-28 Thread Frank Schäfer
Am 28.10.2012 16:08, schrieb Mauro Carvalho Chehab: > Em 28-10-2012 11:00, Frank Schäfer escreveu: >> Am 21.10.2012 19:52, schrieb Frank Schäfer: >>> This patch series adds support for USB bulk transfers to the em28xx >>> driver. >> >> Mauro, >> I sent this patches to you because you are listed as

Re: [PATCH 2/2] media: V4L2: support asynchronous subdevice registration

2012-10-28 Thread Sylwester Nawrocki
Hi, On 10/24/2012 03:54 PM, Guennadi Liakhovetski wrote: > On Sat, 20 Oct 2012, Guennadi Liakhovetski wrote: > >> Currently bridge device drivers register devices for all subdevices >> synchronously, tupically, during their probing. E.g. if an I2C CMOS sensor >> is attached to a video bridge devi

Re: [PATCH 00/23] em28xx: add support fur USB bulk transfers

2012-10-28 Thread Mauro Carvalho Chehab
Em 28-10-2012 11:00, Frank Schäfer escreveu: Am 21.10.2012 19:52, schrieb Frank Schäfer: This patch series adds support for USB bulk transfers to the em28xx driver. Mauro, I sent this patches to you because you are listed as the maintainer and it would be nice to get at least a reply from you.

Re: [PATCH v2, RESEND] libv4lconvert: clarify the behavior and resulting restrictions of v4lconvert_convert()

2012-10-28 Thread Frank Schäfer
Am 21.10.2012 19:58, schrieb Frank Schäfer: > Signed-off-by: Frank Schäfer > --- > lib/include/libv4lconvert.h | 20 ++-- > 1 Datei geändert, 18 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) > > diff --git a/lib/include/libv4lconvert.h b/lib/include/libv4lconvert.h > index 167b57d

Re: [PATCH 00/23] em28xx: add support fur USB bulk transfers

2012-10-28 Thread Frank Schäfer
Am 21.10.2012 19:52, schrieb Frank Schäfer: > This patch series adds support for USB bulk transfers to the em28xx driver. > > Patch 1 is a bugfix for the image data processing with non-interlaced devices > (webcams) > that should be considered for stable (see commit message). > > Patches 2-21 exte

[PATCH 1/1] v4l, smiapp, smiapp-pll, adp1653: Update contact information

2012-10-28 Thread Sakari Ailus
Update contact information to correspond my e-mail address changes. Signed-off-by: Sakari Ailus --- drivers/media/i2c/adp1653.c|4 ++-- drivers/media/i2c/smiapp-pll.c |4 ++-- drivers/media/i2c/smiapp-pll.h |2 +- drivers/media/i2c/smiapp/smiap