RE: Prof DVB-S2 USB device

2013-07-24 Thread Krishna Kishore
Dear Oliver, Sometimes, I am seeing the following errors also. ... ... [ 4150.160583] stv0900_read_reg: i2c error -11, reg[0xf1a8] [ 4150.183959] stv0900_read_reg [ 4150.183990] stv0900_read_reg: i2c error -11, reg[0xf1a8] [ 4150.207427] stv0900_read_reg [ 4150.207458] stv0900_read_reg: i2c

Re: Prof DVB-S2 USB device

2013-07-24 Thread Oliver Schinagl
On 24-07-13 08:56, Krishna Kishore wrote: Dear Oliver, Thanks for your response. Here are more details. Please help me in making this work. Linux version: -sh-4.1# uname -a Linux (none) 3.4.0 #28 SMP PREEMPT Tue Jul 23 16:24:14 IST 2013 armv7l GNU/Linux Your kernel is ancient. The

RE: width and height of JPEG compressed images

2013-07-24 Thread Thomas Vajzovic
Hi Sakiri, On 23 July 2013 23:21 Sakari Ailus wrote: On Sun, Jul 21, 2013 at 10:38:18PM +0200, Sylwester Nawrocki wrote: On 07/19/2013 10:28 PM, Sakari Ailus wrote: On Sat, Jul 06, 2013 at 09:58:23PM +0200, Sylwester Nawrocki wrote: On 07/05/2013 10:22 AM, Thomas Vajzovic wrote: The

Re: width and height of JPEG compressed images

2013-07-24 Thread Sylwester Nawrocki
Hi, On 07/24/2013 09:47 AM, Thomas Vajzovic wrote: On 23 July 2013 23:21 Sakari Ailus wrote: On Sun, Jul 21, 2013 at 10:38:18PM +0200, Sylwester Nawrocki wrote: On 07/19/2013 10:28 PM, Sakari Ailus wrote: On Sat, Jul 06, 2013 at 09:58:23PM +0200, Sylwester Nawrocki wrote: On 07/05/2013

RE: Prof DVB-S2 USB device

2013-07-24 Thread Krishna Kishore
Dear Oliver, Thanks for your response. I tried with 3.10.1. As you rightly pointed out, it does not seem to work on my board (pandaboard). It gets stuck at Starting kernel Now, I am trying with 3.4.47 version now. Let me see if it works. The delay of creating

Re: width and height of JPEG compressed images

2013-07-24 Thread Sylwester Nawrocki
Hi Thomas, On 07/22/2013 10:40 AM, Thomas Vajzovic wrote: On 21 July 2013 21:38 Sylwester Nawrocki wrote: On 07/19/2013 10:28 PM, Sakari Ailus wrote: On Sat, Jul 06, 2013 at 09:58:23PM +0200, Sylwester Nawrocki wrote: On 07/05/2013 10:22 AM, Thomas Vajzovic wrote: The hardware reads AxB

Re: [PATCH RFC 4/5] V4L2: Rename subdev field of struct v4l2_async_notifier

2013-07-24 Thread Sylwester Nawrocki
Hi Prabhakar, On 07/23/2013 05:50 PM, Prabhakar Lad wrote: On Mon, Jul 22, 2013 at 11:34 PM, Sylwester Nawrocki s.nawro...@samsung.com wrote: This is a purely cosmetic change. Since the 'subdev' member points to an array of subdevs it seems more intuitive to name it in plural form.

Re: [PATCH v4 1/3] [media] coda: Fix error paths

2013-07-24 Thread Philipp Zabel
Hi Fabio, Am Dienstag, den 23.07.2013, 15:04 -0300 schrieb Fabio Estevam: Some resources were not being released in the error path and some were released in the incorrect order. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- Changes since v3: - Only disable the clocks

RE: Prof DVB-S2 USB device

2013-07-24 Thread Krishna Kishore
Please see the delay between (1) and (2)... Any idea about the reason for the delay. Though I am not sure, it may not be specific to Linux kernel version. () [3.254455] dvb-usb: will pass the complete MPEG2 transport stream to the software demuxer. [

RE: Prof DVB-S2 USB device

2013-07-24 Thread Krishna Kishore
I am continuously getting following error. I assume that this is expected behavior. At some correct frequency, it will get locked and channel data will be provided. Is it correct? [ 1819.146484] stv0900_init [ 1850.112823] stv0900_sleep [ 1859.615112] stv0900_init [ 1884.433593]

Re: [PATCH RFC 0/5] v4l2-async DT support improvement and cleanups

2013-07-24 Thread Laurent Pinchart
Hi Sylwester, Thanks for the patches. On Monday 22 July 2013 20:04:42 Sylwester Nawrocki wrote: Hello, This is a few patches for the v4l2-async API I wrote while adding the asynchronous subdev registration support to the exynos4-is driver. The most significant change is addition of

Re: [PATCH RFC 0/5] v4l2-async DT support improvement and cleanups

2013-07-24 Thread Hans Verkuil
On Mon 22 July 2013 20:04:42 Sylwester Nawrocki wrote: Hello, This is a few patches for the v4l2-async API I wrote while adding the asynchronous subdev registration support to the exynos4-is driver. The most significant change is addition of V4L2_ASYNC_MATCH_OF subdev matching method,

Re: [PATCH v2 5/5] v4l: Renesas R-Car VSP1 driver

2013-07-24 Thread Katsuya MATSUBARA
Hi Laurent, Thank you for your great work for VSP1. Some comments below. From: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com Date: Wed, 17 Jul 2013 16:54:42 +0200 (snip) +++ b/drivers/media/platform/vsp1/vsp1_drv.c +/* + * vsp1_drv.c -- R-Car VSP1 Driver + * + * Copyright

Re: Prof DVB-S2 USB device

2013-07-24 Thread Oliver Schinagl
On 24-07-13 10:59, Krishna Kishore wrote: Dear Oliver, Thanks for your response. I tried with 3.10.1. As you rightly pointed out, it does not seem to work on my board (pandaboard). It gets stuck at Starting kernel Now, I am trying with 3.4.47 version now. Let me see if

RE: Prof DVB-S2 USB device

2013-07-24 Thread Krishna Kishore
On Desktop PC (Ubuntu 12.04 which has 3.2.0 Kernel) also, I am not getting the list of channels when I scan. I am using Kaffeine. -Original Message- From: Oliver Schinagl [mailto:oliver+l...@schinagl.nl] Sent: Wednesday, July 24, 2013 4:34 PM To: Krishna Kishore Cc:

Re: [PATCH RFC 3/5] V4L2: Add V4L2_ASYNC_MATCH_OF subdev matching type

2013-07-24 Thread Guennadi Liakhovetski
Hi Sylwester On Mon, 22 Jul 2013, Sylwester Nawrocki wrote: Add support for matching by device_node pointer. This allows the notifier user to simply pass a list of device_node pointers corresponding to sub-devices. Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com Signed-off-by:

Re: [PATCH RFC 4/5] V4L2: Rename subdev field of struct v4l2_async_notifier

2013-07-24 Thread Guennadi Liakhovetski
Hi Sylwester On Mon, 22 Jul 2013, Sylwester Nawrocki wrote: This is a purely cosmetic change. Since the 'subdev' member points to an array of subdevs it seems more intuitive to name it in plural form. Well, I was aware of the fact, that subdev is an array and that the plural form of subdev

RE: Prof DVB-S2 USB device

2013-07-24 Thread Krishna Kishore
Any idea on the following error? scanning /stbref/dvb-apps-f3a70b206f0f/util/scan/dvb-s/Insat4B_C-93.5E using '/dev/dvb/adapter0/frontend0' and '/dev/dvb/adapter0/demux0' initial transponder 3725000 H 2750 3 initial transponder 375 H 425 3 initial transponder 3762000 H 425 3

Re: [PATCH RFC 0/5] v4l2-async DT support improvement and cleanups

2013-07-24 Thread Guennadi Liakhovetski
Hi Sylwester On Mon, 22 Jul 2013, Sylwester Nawrocki wrote: Hello, This is a few patches for the v4l2-async API I wrote while adding the asynchronous subdev registration support to the exynos4-is driver. The most significant change is addition of V4L2_ASYNC_MATCH_OF subdev matching

Re: Prof DVB-S2 USB device

2013-07-24 Thread Oliver Schinagl
On 24-07-13 13:20, Krishna Kishore wrote: On Desktop PC (Ubuntu 12.04 which has 3.2.0 Kernel) also, I am not getting the list of channels when I scan. I am using Kaffeine. While I understand you prefer to run a LTS distro, 3.2.0 is old! The reason why I keep bringing this up, media drivers

Re: Prof DVB-S2 USB device

2013-07-24 Thread Oliver Schinagl
On 24-07-13 13:31, Krishna Kishore wrote: Any idea on the following error? scanning /stbref/dvb-apps-f3a70b206f0f/util/scan/dvb-s/Insat4B_C-93.5E using '/dev/dvb/adapter0/frontend0' and '/dev/dvb/adapter0/demux0' initial transponder 3725000 H 2750 3 initial transponder 375 H 425 3

Re: cron job: media_tree daily build: ERRORS

2013-07-24 Thread Hans Verkuil
On Tue 23 July 2013 19:27:47 Hans Verkuil wrote: This message is generated daily by a cron job that builds media_tree for the kernels and architectures in the list below. I've fixed the build issues after the 3.11 merge. So hopefully today's daily build will be back to 'WARNINGS'. Regards,

[PATCH] cx23885[v3]: Fix interrupt storm when enabling IR receiver.

2013-07-24 Thread Luis Alves
Hi, New patch for this issue. Changes: - Added flatiron readreg and writereg functions prototypes (new header file). - Modified the av work handler to preserve all other register bits when dealing with the interrupt flag. Regards, Luis Signed-off-by: Luis Alves lja...@gmail.com ---

[PATCH] cx23885[v4]: Fix interrupt storm when enabling IR receiver.

2013-07-24 Thread Luis Alves
Hi, Removed wrong description in the header file. Sorry about that... New patch for this issue. Changes: - Added flatiron readreg and writereg functions prototypes (new header file). - Modified the av work handler to preserve all other register bits when dealing with the interrupt flag.

RE: Prof DVB-S2 USB device

2013-07-24 Thread Krishna Kishore
Thanks a lot for your response. On PC (Ubuntu 12.04) also, channel list is not seen. I saw a youtube video (http://www.youtube.com/watch?v=lv-Hel3DUkY) that Prof USB 7500 DVB-S2 device works. Looks like driver is trying to lock at the frequency which is out of range of what driver is

Re: [PATCH v2 1/5] media: Fix circular graph traversal

2013-07-24 Thread Laurent Pinchart
Hi Sakari, On Thursday 18 July 2013 13:22:09 Sakari Ailus wrote: On Thu, Jul 18, 2013 at 01:06:40AM +0200, Laurent Pinchart wrote: On Wednesday 17 July 2013 22:47:03 Sakari Ailus wrote: On Wed, Jul 17, 2013 at 04:54:38PM +0200, Laurent Pinchart wrote: The graph traversal API

Re: PROBLEM: dvb-usb-rtl28xxu and Terratec Cinergy TStickRC (rev3) - no signal on some frequencies

2013-07-24 Thread Antti Palosaari
Could you test attached patch? It enhances reception a little bit, you should be able to receive more weak signals. I was able to made test setup against modulator. Modulator + attenuator + attenuator + TV-stick, where I got picture using Windows driver at signal level -29dBm whilst on

Benachrichtigung von UBS AG

2013-07-24 Thread UBS AG
Sehr geehrter Kunde, Kürzlich zeigen unsere Aufzeichnungen, dass Ihr UBS-Konto möglich durch einen Dritten unbefugten Zutritt. Die Sicherheit Ihres Kontos ist unser wichtigstes Anliegen, deshalb haben wir beschlossen, den Zugang zu Ihrem Konto vorübergehend zu begrenzen. Für den vollen Zugang

[PATCH] V4L: s5c73m3: Add format propagation for TRY formats

2013-07-24 Thread Sylwester Nawrocki
From: Andrzej Hajda a.ha...@samsung.com Resolution set on ISP pad of S5C73M3-OIF subdev should be propagated to source pad for TRY and ACTIVE formats. The patch adds missing propagation for TRY format. Signed-off-by: Andrzej Hajda a.ha...@samsung.com Signed-off-by: Kyungmin Park

Re: [PATCH v2 5/5] v4l: Renesas R-Car VSP1 driver

2013-07-24 Thread Laurent Pinchart
Hi Matsubara-san, On Wednesday 24 July 2013 19:38:39 Katsuya MATSUBARA wrote: Hi Laurent, Thank you for your great work for VSP1. Some comments below. Thank you for the review. From: Laurent Pinchart laurent.pinchart+rene...@ideasonboard.com Date: Wed, 17 Jul 2013 16:54:42 +0200

[PATCH 1/2] cx24117[v4]: Add new dvb-frontend.

2013-07-24 Thread Luis Alves
v4: Patch order fixed. Changed some msleep's to clear checkpatch warning. Signed-off-by: Luis Alves lja...@gmail.com --- drivers/media/dvb-frontends/Kconfig |7 + drivers/media/dvb-frontends/Makefile |1 + drivers/media/dvb-frontends/cx24117.c | 1621 +

[PATCH 2/2] cx24117[v4]: Add new dvb-frontend: add supported cards to cx23885. Currently tested with TBS6980 and TBS6981.

2013-07-24 Thread Luis Alves
v4: Patch order fixed. Changed some msleep's to clear checkpatch warnings. Signed-off-by: Luis Alves lja...@gmail.com --- drivers/media/pci/cx23885/Kconfig |1 + drivers/media/pci/cx23885/cx23885-cards.c | 67 + drivers/media/pci/cx23885/cx23885-dvb.c

[PATCH] smiapp: re-use clamp_t instead of min(..., max(...))

2013-07-24 Thread Andy Shevchenko
clamp_t does the job to put a variable into the given range. Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com --- drivers/media/i2c/smiapp/smiapp-core.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/media/i2c/smiapp/smiapp-core.c

[PATCH] V4L: s5c73m3: Add device tree support

2013-07-24 Thread Sylwester Nawrocki
From: Andrzej Hajda a.ha...@samsung.com This patch adds the V4L2 asychronous subdev registration and device tree support. Common clock API is used to control the sensor master clock from within the subdev driver. Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com Signed-off-by: Andrzej

Re: [PATCH 2/4] [media] em28xx: i2s 5 sample rates is a subset of 3 one.

2013-07-24 Thread Frank Schäfer
[2nd try - vger.kernel.org rejects html content] Please, don't top-post and don't drop any recipients. Am 24.07.2013 15:40, schrieb Alban Browaeys: if 0xf0 , then the if (0x20 == ) else if (0x30 ==) first match 0x20 succeed and bail out . 0x20 is a subset if 0x30. (chipcfg

Benachrichtigung von UBS AG

2013-07-24 Thread UBS AG
Sehr geehrter Kunde, Kürzlich zeigen unsere Aufzeichnungen, dass Ihr UBS-Konto möglich durch einen Dritten unbefugten Zutritt. Die Sicherheit Ihres Kontos ist unser wichtigstes Anliegen, deshalb haben wir beschlossen, den Zugang zu Ihrem Konto vorübergehend zu begrenzen. Für den vollen Zugang

Re: [PATCH 3/4] [media] em28xx: usb power config is in the low byte.

2013-07-24 Thread Frank Schäfer
[2nd try - vger.kernel.org rejects html content] Am 24.07.2013 16:16, schrieb Alban Browaeys: sorry the weater is getting really warm there. I will take a closer look at that . But mind that we grab two bytes and 'usb config for audio and power usb transfer rates' thus I guessed we had to

Re: [PATCH 1/2] cx24117[v4]: Add new dvb-frontend.

2013-07-24 Thread Antti Palosaari
On 07/24/2013 06:18 PM, Luis Alves wrote: v4: Patch order fixed. Changed some msleep's to clear checkpatch warning. Signed-off-by: Luis Alves lja...@gmail.com Reviewed-by: Antti Palosaari cr...@iki.fi --- drivers/media/dvb-frontends/Kconfig |7 +

Re: [PATCH] smiapp: re-use clamp_t instead of min(..., max(...))

2013-07-24 Thread Sakari Ailus
Hi Andy, Thanks for the patch. (Drop Mauro from cc.) On Wed, Jul 24, 2013 at 06:21:18PM +0300, Andy Shevchenko wrote: clamp_t does the job to put a variable into the given range. Signed-off-by: Andy Shevchenko andriy.shevche...@linux.intel.com --- drivers/media/i2c/smiapp/smiapp-core.c |

Re: [PATCH] smiapp: re-use clamp_t instead of min(..., max(...))

2013-07-24 Thread Andy Shevchenko
On Wed, Jul 24, 2013 at 6:45 PM, Sakari Ailus sakari.ai...@iki.fi wrote: [] + max_m = clamp_t(u32, max_m, sensor-limits[SMIAPP_LIMIT_SCALER_M_MIN], + sensor-limits[SMIAPP_LIMIT_SCALER_M_MAX]); Do you need clamp_t()? Wouldn't plain clamp() do? The *_t variants are

Re: [PATCH] smiapp: re-use clamp_t instead of min(..., max(...))

2013-07-24 Thread Sakari Ailus
On Wed, Jul 24, 2013 at 06:49:24PM +0300, Andy Shevchenko wrote: On Wed, Jul 24, 2013 at 6:45 PM, Sakari Ailus sakari.ai...@iki.fi wrote: [] + max_m = clamp_t(u32, max_m, sensor-limits[SMIAPP_LIMIT_SCALER_M_MIN], + sensor-limits[SMIAPP_LIMIT_SCALER_M_MAX]);

[PATCH 2/2] stv090x: on tuning lock return correct tuned paramaters like freq/sr/fec/rolloff/etc

2013-07-24 Thread Chris Lee
If you need it broken up more just let me know, I look forward to comments, thanks Chris --- drivers/media/dvb-frontends/stv090x.c | 182 -- drivers/media/dvb-frontends/stv090x_reg.h | 2 + 2 files changed, 172 insertions(+), 12 deletions(-) diff --git

Re: [PATCH v8] V4L2: soc_camera: Renesas R-Car VIN driver

2013-07-24 Thread Guennadi Liakhovetski
Hi Sergei, Vladimir So, looks like we're almost there. checkpatch.pl looks pretty good too, I don't care about 80 chars, Kconfig seems to be a dull one, have a look at msleep(1) warning whether it bothers you. On Sat, 20 Jul 2013, Sergei Shtylyov wrote: From: Vladimir Barinov

stv090x vs stv0900 support

2013-07-24 Thread Chris Lee
Im looking for comments on these two modules, they overlap support for the same demods. stv0900 supporting stv0900 and stv090x supporting stv0900 and stv0903. Ive flipped a few cards from one to the other and they function fine. In some ways stv090x is better suited. Its a pain supporting two

RE: stv090x vs stv0900 support

2013-07-24 Thread Krishna Kishore
My opinion is that, it is better to have only stv090x. Apart from minimizing the number of patches and ease of maintenance, it will avoid the confusion that I had When I started using prof 7500. I had to enable stv0900 and stb6100. I got confused on whether to enable stv0900 or to enable

Re: [PATCH 2/2] cx24117[v4]: Add new dvb-frontend: add supported cards to cx23885. Currently tested with TBS6980 and TBS6981.

2013-07-24 Thread Antti Palosaari
On 07/24/2013 06:18 PM, Luis Alves wrote: v4: Patch order fixed. Changed some msleep's to clear checkpatch warnings. Signed-off-by: Luis Alves lja...@gmail.com Reviewed-by: Antti Palosaari cr...@iki.fi --- drivers/media/pci/cx23885/Kconfig |1 +

Re: stv090x vs stv0900 support

2013-07-24 Thread Antti Palosaari
On 07/24/2013 08:21 PM, Krishna Kishore wrote: My opinion is that, it is better to have only stv090x. Apart from minimizing the number of patches and ease of maintenance, it will avoid the confusion that I had When I started using prof 7500. I had to enable stv0900 and stb6100. I got confused

Re: linux-next: Tree for Jul 24 (media/usb/stk1160)

2013-07-24 Thread Ezequiel Garcia
Hello Randy, On Wed, Jul 24, 2013 at 10:32:57AM -0700, Randy Dunlap wrote: On 07/23/13 23:32, Stephen Rothwell wrote: Hi all, Changes since 20130723: on x86_64: drivers/built-in.o: In function `stk1160_ac97_register': (.text+0x414e77): undefined reference to `snd_card_create'

cron job: media_tree daily build: WARNINGS

2013-07-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 Jul 24 19:00:17 CEST 2013 git branch: test git hash: c859e6ef33ac0c9a5e9e934fe11a2232752b4e96 gcc

Re: [PATCH 01/15] drivers: phy: add generic PHY framework

2013-07-24 Thread Arnd Bergmann
On Tuesday 23 July 2013, Tomasz Figa wrote: On Tuesday 23 of July 2013 17:14:20 Alan Stern wrote: On Tue, 23 Jul 2013, Tomasz Figa wrote: Where would you want to have those phy_address arrays stored? There are no board files when booting with DT. Not even saying that you don't need to

Re: [PATCH v8] V4L2: soc_camera: Renesas R-Car VIN driver

2013-07-24 Thread Vladimir Barinov
Hi Guennadi, Thank you for the v8 review. On 07/24/2013 08:14 PM, Guennadi Liakhovetski wrote: [snip] + /* output format */ + switch (icd-current_fmt-host_fmt-fourcc) { + case V4L2_PIX_FMT_NV16: + iowrite32(ALIGN(cam-width * cam-height, 0x80), +

Re: [PATCH v8] V4L2: soc_camera: Renesas R-Car VIN driver

2013-07-24 Thread Sergei Shtylyov
Hello. On 07/24/2013 08:14 PM, Guennadi Liakhovetski wrote: From: Vladimir Barinov vladimir.bari...@cogentembedded.com Add Renesas R-Car VIN (Video In) V4L2 driver. Based on the patch by Phil Edworthy phil.edwor...@renesas.com. Signed-off-by: Vladimir Barinov

Re: [PATCH v8] V4L2: soc_camera: Renesas R-Car VIN driver

2013-07-24 Thread Guennadi Liakhovetski
On Wed, 24 Jul 2013, Sergei Shtylyov wrote: Hello. On 07/24/2013 08:14 PM, Guennadi Liakhovetski wrote: From: Vladimir Barinov vladimir.bari...@cogentembedded.com Add Renesas R-Car VIN (Video In) V4L2 driver. Based on the patch by Phil Edworthy phil.edwor...@renesas.com.

Re: [PATCH v8] V4L2: soc_camera: Renesas R-Car VIN driver

2013-07-24 Thread Guennadi Liakhovetski
Hi Vladimir On Wed, 24 Jul 2013, Vladimir Barinov wrote: Hi Guennadi, Thank you for the v8 review. On 07/24/2013 08:14 PM, Guennadi Liakhovetski wrote: [snip] + /* output format */ + switch (icd-current_fmt-host_fmt-fourcc) { + case V4L2_PIX_FMT_NV16: +

Re: [PATCH v2 4/5] v4l: Add V4L2_PIX_FMT_NV16M and V4L2_PIX_FMT_NV61M formats

2013-07-24 Thread Sylwester Nawrocki
Hi Laurent, On 07/17/2013 04:54 PM, Laurent Pinchart wrote: NV16M and NV61M are planar YCbCr 4:2:2 and YCrCb 4:2:2 formats with a luma plane followed by an interleaved chroma plane. The planes are not required to be contiguous in memory, and the formats can only be used with the multi-planar

Re: [PATCH v2 3/5] v4l: Add media format codes for ARGB8888 and AYUV8888 on 32-bit busses

2013-07-24 Thread Sylwester Nawrocki
Hi Laurent, On 07/17/2013 04:54 PM, Laurent Pinchart wrote: Signed-off-by: Laurent Pinchartlaurent.pinchart+rene...@ideasonboard.com --- Documentation/DocBook/media/v4l/subdev-formats.xml | 609 + Documentation/DocBook/media_api.tmpl | 6 +

Re: stv090x vs stv0900 support

2013-07-24 Thread Oliver Schinagl
On 07/24/13 19:52, Antti Palosaari wrote: On 07/24/2013 08:21 PM, Krishna Kishore wrote: My opinion is that, it is better to have only stv090x. Apart from minimizing the number of patches and ease of maintenance, it will avoid the confusion that I had When I started using prof 7500. I had to

Re: [PATCH v2 5/5] v4l: Renesas R-Car VSP1 driver

2013-07-24 Thread Sakari Ailus
Hi Laurent, What a nice driver! A few minor comments below: On Wed, Jul 17, 2013 at 04:54:42PM +0200, Laurent Pinchart wrote: ... +static void vsp1_device_init(struct vsp1_device *vsp1) +{ + unsigned int i; + u32 status; + + /* Reset any channel that might be running. */ +

Re: [PATCH v8] V4L2: soc_camera: Renesas R-Car VIN driver

2013-07-24 Thread Katsuya MATSUBARA
Hi Vladimir, Thank you for the revised patch. From: Sergei Shtylyov sergei.shtyl...@cogentembedded.com Date: Sat, 20 Jul 2013 03:14:34 +0400 From: Vladimir Barinov vladimir.bari...@cogentembedded.com Add Renesas R-Car VIN (Video In) V4L2 driver. Based on the patch by Phil Edworthy