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

2013-09-24 Thread Matthias Schwarzott
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/gmane.linux.drivers.video-input-infrastructur

cron job: media_tree daily build: ERRORS

2013-09-24 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: Wed Sep 25 04:00:48 CEST 2013 git branch: test git hash: ffee921033e64edf8579a3b21c7f15d1a6c3ef71 gcc versi

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

2013-09-24 Thread Stephen Warren
On 09/24/2013 02:08 AM, Srinivas KANDAGATLA wrote: > Thanks Stephen, > On 23/09/13 21:40, Stephen Warren wrote: >> On 09/19/2013 02:59 AM, Srinivas KANDAGATLA wrote: >>> This patch adds support to ST RC driver, which is basically a IR/UHF >>> receiver and transmitter. This IP (IRB) is common across

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

2013-09-24 Thread Uwe Kleine-König
The marvell-ccic does several things wrong or ineffectively in the clock handling and it's usage of the devm_* stuff - it assumes clk_get doesn't return NULL - it explicitly calls devm_clk_put instead just keeping the reference during it's lifetime and let the driver core call it - it calls

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

2013-09-24 Thread Uwe Kleine-König
The marvell-ccic does several things wrong or ineffectively in the clock handling and it's usage of the devm_* stuff - it assumes clk_get doesn't return NULL - it explicitly calls devm_clk_put instead just keeping the reference during it's lifetime and let the driver core call it - it calls

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

2013-09-24 Thread Andy Walls
Hans Verkuil wrote: >On 09/24/2013 06:19 PM, Guennadi Liakhovetski wrote: >> Hi Hans >> >> On Tue, 24 Sep 2013, Hans Verkuil wrote: >> >>> Shouldn't the interrupt_service_routine() op be implemented as well? >>> Usually these drivers will generate interrupts if e.g. the format >changes. >> >> S

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

2013-09-24 Thread Hans Verkuil
On 09/24/2013 06:19 PM, Guennadi Liakhovetski wrote: > Hi Hans > > On Tue, 24 Sep 2013, Hans Verkuil wrote: > >> Shouldn't the interrupt_service_routine() op be implemented as well? >> Usually these drivers will generate interrupts if e.g. the format changes. > > Should it? AFAIU, .interrupt_ser

Re: [PATCH v5 1/4] media: s5p-tv: Replace mxr_ macro by default dev_

2013-09-24 Thread Joe Perches
On Tue, 2013-09-24 at 14:52 +0200, Tomasz Stanislawski wrote: > On 09/23/2013 07:44 PM, Joe Perches wrote: > > On Mon, 2013-09-23 at 17:48 +0200, Bartlomiej Zolnierkiewicz wrote: > >> On Monday, September 23, 2013 04:50:01 PM Tomasz Stanislawski wrote: > >>> May I ask what is the rationale for this

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

2013-09-24 Thread Guennadi Liakhovetski
Hi Hans On Tue, 24 Sep 2013, Hans Verkuil wrote: > Shouldn't the interrupt_service_routine() op be implemented as well? > Usually these drivers will generate interrupts if e.g. the format changes. Should it? AFAIU, .interrupt_service_routine() is a subde operation to be called by a bridge drive

Re: [PATCH 0/3] media: Add SH-Mobile RCAR-H2 Lager board support

2013-09-24 Thread Guennadi Liakhovetski
Hi Valentine, Since patches 2 and 3 here are for soc-camera, I can offer to take all 3 via my tree after all comments to patch 1/3 are addressed and someone (Laurent?) has acked it. Alternatively I can ack the two patches and let all 3 go via another tree, or we can split this series too - no p

Re: [PATCH 3/3] media: rcar_vin: Add RGB888_1X24 input format support

2013-09-24 Thread Guennadi Liakhovetski
On Tue, 24 Sep 2013, Valentine Barshak wrote: > This adds V4L2_MBUS_FMT_RGB888_1X24 input format support > which is used by the ADV7612 chip. > > Signed-off-by: Valentine Barshak > --- > drivers/media/platform/soc_camera/rcar_vin.c | 12 ++-- > 1 file changed, 10 insertions(+), 2 deleti

Re: [PATCH] Revert "V4L/DVB: uvc: Enable USB autosuspend by default on uvcvideo"

2013-09-24 Thread VDR User
On Tue, Sep 24, 2013 at 4:34 AM, Laurent Pinchart wrote: > I've discussed this issue during LPC last week, and I still believe we should > enable auto-suspend. The feature really saves power, without it my C910 > Logitech webcam gets hot even when unused. > > If we disable auto-suspend by default

Re: [PATCH] media: i2c: adv7343: fix the DT binding properties

2013-09-24 Thread Mark Brown
On Mon, Sep 23, 2013 at 01:50:51PM +0200, Laurent Pinchart wrote: > Isn't regulator_get_optional() intended for devices that can have supplies > unconnected in normal use ? The ADV7343 supplies are mandatory from a > hardware > point of view, so I think we should use regulator_get(). Otherwise

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

2013-09-24 Thread Guennadi Liakhovetski
Hi Valentine, On Tue, 24 Sep 2013, Valentine Barshak wrote: > This adds ADV7611/ADV7612 Dual Port Xpressview > 225 MHz HDMI Receiver support. > > The code is based on the ADV7604 driver, and ADV7612 patch > by Shinobu Uehara > > Signed-off-by: Valentine Barshak IIRC, Laurent is reviewing all

Re: [PATCH] media: i2c: adv7343: fix the DT binding properties

2013-09-24 Thread Mark Brown
On Tue, Sep 24, 2013 at 11:44:57AM +0200, Laurent Pinchart wrote: > On Monday 23 September 2013 15:33:10 Stephen Warren wrote: > > So I think you want to make the supply properties mandatory in DT (since > > some form of supply is mandatory in HW), yet make the driver support > > broken DTs which

Re: [PATCH v5 1/4] media: s5p-tv: Replace mxr_ macro by default dev_

2013-09-24 Thread Bartlomiej Zolnierkiewicz
Hi Tomasz, On Tuesday, September 24, 2013 02:52:44 PM Tomasz Stanislawski wrote: > On 09/23/2013 07:44 PM, Joe Perches wrote: > > On Mon, 2013-09-23 at 17:48 +0200, Bartlomiej Zolnierkiewicz wrote: > >> On Monday, September 23, 2013 04:50:01 PM Tomasz Stanislawski wrote: > >>> May I ask what is t

[RFC PATCH 4/4] ARM: dts: exynos4210-trats: add panel and dsi nodes

2013-09-24 Thread Andrzej Hajda
The patch adds mipi-dsi-exynos bus master node and s6e8aa0 panel subnode to trats device. Signed-off-by: Andrzej Hajda Signed-off-by: Kyungmin Park Conflicts: arch/arm/boot/dts/exynos4210-trats.dts --- arch/arm/boot/dts/exynos4210-trats.dts | 54 ++ 1 fi

[RFC PATCH 1/4] mipi-dsi-bus: add MIPI DSI bus support

2013-09-24 Thread Andrzej Hajda
MIPI DSI is a high-speed serial interface to transmit data from/to host to display module. Signed-off-by: Andrzej Hajda Signed-off-by: Kyungmin Park --- drivers/video/display/Kconfig| 4 + drivers/video/display/Makefile | 1 + drivers/video/display/mipi-dsi-bus.c | 332 +++

[RFC PATCH 3/4] panel-s6e8aa0: add driver

2013-09-24 Thread Andrzej Hajda
The patch adds mipi-dsi-bus slave driver for s6e8aa0 familiy panels. Signed-off-by: Donghwa Lee Signed-off-by: Inki Dae Signed-off-by: Joongmock Shin Signed-off-by: Eunchul Kim Signed-off-by: Tomasz Figa Signed-off-by: Andrzej Hajda Signed-off-by: Kyungmin Park --- drivers/video/display/Kc

[RFC PATCH 0/4] CDFv3: MIPI DSI bus implementation

2013-09-24 Thread Andrzej Hajda
Hi Laurent, This is my MIPI DSI bus implementation. The patchset contains also two drivers: - DSI bus master driver for Exynos, - DSI slave driver for s6e8aa0 panel family. All code has been tested on real device - Exynos 4210 based 'Trats'. This is not final version, there are still some things

[RFC PATCH 2/4] mipi-dsi-exynos: add driver

2013-09-24 Thread Andrzej Hajda
This patch adds mipi-dsi-bus master driver for Exynos chipset family. Signed-off-by: Tomasz Figa Signed-off-by: Donghwa Lee Signed-off-by: Sylwester Nawrocki Signed-off-by: Andrzej Hajda Signed-off-by: Kyungmin Park --- drivers/video/display/Kconfig |4 + drivers/video/display/

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

2013-09-24 Thread Hans Verkuil
Hi Valentine, On Tue 24 September 2013 15:38:34 Valentine Barshak wrote: > This adds ADV7611/ADV7612 Dual Port Xpressview > 225 MHz HDMI Receiver support. > > The code is based on the ADV7604 driver, and ADV7612 patch > by Shinobu Uehara Thanks for the patch! I have a few review comments below

[PATCH 2/3] media: rcar_vin: Add preliminary r8a7790 H2 support

2013-09-24 Thread Valentine Barshak
Signed-off-by: Valentine Barshak --- drivers/media/platform/soc_camera/rcar_vin.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/media/platform/soc_camera/rcar_vin.c b/drivers/media/platform/soc_camera/rcar_vin.c index d02a7e0..cf81e02 100644 --- a/drivers/media/

[PATCH 3/3] media: rcar_vin: Add RGB888_1X24 input format support

2013-09-24 Thread Valentine Barshak
This adds V4L2_MBUS_FMT_RGB888_1X24 input format support which is used by the ADV7612 chip. Signed-off-by: Valentine Barshak --- drivers/media/platform/soc_camera/rcar_vin.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/soc_camera/rcar_v

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

2013-09-24 Thread Valentine Barshak
This adds ADV7611/ADV7612 Dual Port Xpressview 225 MHz HDMI Receiver support. The code is based on the ADV7604 driver, and ADV7612 patch by Shinobu Uehara Signed-off-by: Valentine Barshak --- drivers/media/i2c/Kconfig | 11 + drivers/media/i2c/Makefile |1 + drivers/media/i2c/adv761x.

[PATCH 0/3] media: Add SH-Mobile RCAR-H2 Lager board support

2013-09-24 Thread Valentine Barshak
The following patches add ADV7611/ADV7612 HDMI receiver I2C driver and add RCAR H2 SOC support along with ADV761x output format support to rcar_vin soc_camera driver. These changes are needed for SH-Mobile R8A7790 Lager board video input support. Valentine Barshak (3): media: i2c: Add ADV761X s

Re: [PATCH v5 1/4] media: s5p-tv: Replace mxr_ macro by default dev_

2013-09-24 Thread Mauro Carvalho Chehab
Em Tue, 24 Sep 2013 12:33:34 +0200 Bartlomiej Zolnierkiewicz escreveu: > > Hi Tomasz, > > On Tuesday, September 24, 2013 11:43:53 AM Tomasz Stanislawski wrote: > > Hi, > > > > On 09/23/2013 05:48 PM, Bartlomiej Zolnierkiewicz wrote: > > > > > > Hi Tomasz, > > > > > > On Monday, September 23,

Re: [PATCH] V4L: Drop meaningless video_is_registered() call in v4l2_open()

2013-09-24 Thread Hans Verkuil
On Thu 12 September 2013 12:19:29 Sylwester Nawrocki wrote: > Hi Hans, > > On 09/11/2013 04:01 PM, Hans Verkuil wrote: > > On 09/11/2013 03:07 PM, Sylwester Nawrocki wrote: > >> On 09/09/2013 11:07 AM, Hans Verkuil wrote: > >>> On 09/06/2013 12:33 AM, Sylwester Nawrocki wrote: > On 08/07/2013

Re: [PATCH v5 1/4] media: s5p-tv: Replace mxr_ macro by default dev_

2013-09-24 Thread Tomasz Stanislawski
On 09/23/2013 07:44 PM, Joe Perches wrote: > On Mon, 2013-09-23 at 17:48 +0200, Bartlomiej Zolnierkiewicz wrote: >> On Monday, September 23, 2013 04:50:01 PM Tomasz Stanislawski wrote: >>> May I ask what is the rationale for this patch? >>> To reduce a few lines of code? >> This patch makes source

Re: [PATCH] driver: firewire: remove unnecessary #if 0 code

2013-09-24 Thread Stefan Richter
On Sep 22 Govindarajulu Varadarajan wrote: > Signed-off-by: Govindarajulu Varadarajan > --- > drivers/media/firewire/firedtv-avc.c | 41 > > 1 file changed, 41 deletions(-) > > diff --git a/drivers/media/firewire/firedtv-avc.c > b/drivers/media/firewire/fir

Re: [PATCH] Revert "V4L/DVB: uvc: Enable USB autosuspend by default on uvcvideo"

2013-09-24 Thread Laurent Pinchart
Hi Adam, On Thursday 13 June 2013 17:56:47 Adam Lee wrote: > On Thu, Apr 25, 2013 at 02:33:06PM +0800, Adam Lee wrote: > > On Wed, Apr 24, 2013 at 11:17:52AM +0200, Laurent Pinchart wrote: > > > On Wednesday 24 April 2013 15:57:19 adam@canonical.com wrote: > > > > From: Adam Lee > > > > > >

Re: [media-workshop] V2: Agenda for the Edinburgh mini-summit

2013-09-24 Thread Hans Verkuil
On Tue 24 September 2013 12:59:44 Laurent Pinchart wrote: > Hi Hans, > > On Tuesday 10 September 2013 11:34:32 Hans Verkuil wrote: > > I have collected all the ideas up to now in a V2 of the agenda. > > > > The items are grouped by the person(s) that suggested them. As done in the > > past those

Re: [media-workshop] V2: Agenda for the Edinburgh mini-summit

2013-09-24 Thread Laurent Pinchart
Hi Hans, On Tuesday 10 September 2013 11:34:32 Hans Verkuil wrote: > I have collected all the ideas up to now in a V2 of the agenda. > > The items are grouped by the person(s) that suggested them. As done in the > past those who suggested a topic and who will attend the mini-summit are > expected

Re: [media-workshop] V2: Agenda for the Edinburgh mini-summit

2013-09-24 Thread Laurent Pinchart
Hi Oliver, On Monday 23 September 2013 18:37:48 Oliver Schinagl wrote: > On 09/23/13 16:45, Sylwester Nawrocki wrote: > > Hi, > > > > I would like to have a short discussion on LED flash devices support > > in the kernel. Currently there are two APIs: the V4L2 and LED class > > API exposed by the

Re: [PATCH v5 1/4] media: s5p-tv: Replace mxr_ macro by default dev_

2013-09-24 Thread Bartlomiej Zolnierkiewicz
Hi Tomasz, On Tuesday, September 24, 2013 11:43:53 AM Tomasz Stanislawski wrote: > Hi, > > On 09/23/2013 05:48 PM, Bartlomiej Zolnierkiewicz wrote: > > > > Hi Tomasz, > > > > On Monday, September 23, 2013 04:50:01 PM Tomasz Stanislawski wrote: > >> Hello, > >> May I ask what is the rationale f

Re: [PATCH] media: i2c: adv7343: fix the DT binding properties

2013-09-24 Thread Laurent Pinchart
On Monday 23 September 2013 15:33:10 Stephen Warren wrote: > On 09/23/2013 05:50 AM, Laurent Pinchart wrote: > > On Monday 23 September 2013 08:18:52 Prabhakar Lad wrote: > >> On Fri, Sep 20, 2013 at 3:22 PM, Sylwester Nawrocki wrote: > >>> On 09/20/2013 10:11 AM, Prabhakar Lad wrote: > OK I w

Re: [PATCH v5 1/4] media: s5p-tv: Replace mxr_ macro by default dev_

2013-09-24 Thread Tomasz Stanislawski
Hi, On 09/23/2013 05:48 PM, Bartlomiej Zolnierkiewicz wrote: > > Hi Tomasz, > > On Monday, September 23, 2013 04:50:01 PM Tomasz Stanislawski wrote: >> Hello, >> May I ask what is the rationale for this patch? >> To reduce a few lines of code? > > This patch makes source code more generic-like

Re: V2: Agenda for the Edinburgh mini-summit

2013-09-24 Thread Thierry Reding
Resending using Bryan's correct email address. On Mon, Sep 23, 2013 at 10:27:06PM +0200, Sylwester Nawrocki wrote: > On 09/23/2013 06:37 PM, Oliver Schinagl wrote: > >On 09/23/13 16:45, Sylwester Nawrocki wrote: > >>Hi, > >> > >>I would like to have a short discussion on LED flash devices support

Re: V2: Agenda for the Edinburgh mini-summit

2013-09-24 Thread Thierry Reding
On Mon, Sep 23, 2013 at 10:27:06PM +0200, Sylwester Nawrocki wrote: > On 09/23/2013 06:37 PM, Oliver Schinagl wrote: > >On 09/23/13 16:45, Sylwester Nawrocki wrote: > >>Hi, > >> > >>I would like to have a short discussion on LED flash devices support > >>in the kernel. Currently there are two APIs:

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

2013-09-24 Thread Srinivas KANDAGATLA
Thanks Stephen, On 23/09/13 21:40, Stephen Warren wrote: > On 09/19/2013 02:59 AM, Srinivas KANDAGATLA wrote: >> This patch adds support to ST RC driver, which is basically a IR/UHF >> receiver and transmitter. This IP (IRB) is common across all the ST >> parts for settop box platforms. IRB is embe