Re: [PATCHv10 08/26] v4l: vb2-dma-contig: add support for scatterlist in userptr mode

2012-10-29 Thread Marek Szyprowski
Hello, On 10/26/2012 6:24 PM, Pawel Osciak wrote: Hi Tomasz, On Wed, Oct 10, 2012 at 7:46 AM, Tomasz Stanislawski wrote: This patch introduces usage of dma_map_sg to map memory behind a userspace pointer to a device as dma-contiguous mapping. Perhaps I'm missing something, but I don't unde

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

2012-10-29 Thread Mauro Carvalho Chehab
Em Tue, 30 Oct 2012 01:00:12 -0200 Mauro Carvalho Chehab escreveu: > Em Mon, 29 Oct 2012 23:14:55 +0200 > Frank Schäfer escreveu: > > > Am 29.10.2012 22:03, schrieb Mauro Carvalho Chehab: > > > Em Mon, 29 Oct 2012 17:33:12 +0200 > > > Frank Schäfer escreveu: > > > > > >> Am 28.10.2012 21:57, s

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

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 23:14:55 +0200 Frank Schäfer escreveu: > Am 29.10.2012 22:03, schrieb Mauro Carvalho Chehab: > > Em Mon, 29 Oct 2012 17:33:12 +0200 > > Frank Schäfer escreveu: > > > >> Am 28.10.2012 21:57, schrieb Mauro Carvalho Chehab: > >>> Em Sun, 21 Oct 2012 19:52:05 +0300 > >>> Frank Sc

Re: [PATCH 10/23] V4L: Add auto focus targets to the selections API

2012-10-29 Thread Sylwester Nawrocki
Hi Sakari, On 10/29/2012 09:00 PM, Sakari Ailus wrote: > On Thu, May 10, 2012 at 12:30:45PM +0200, Sylwester Nawrocki wrote: >> The camera automatic focus algorithms may require setting up >> a spot or rectangle coordinates or multiple such parameters. >> >> The automatic focus selection targets a

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

2012-10-29 Thread Frank Schäfer
Am 29.10.2012 22:03, schrieb Mauro Carvalho Chehab: > Em Mon, 29 Oct 2012 17:33:12 +0200 > Frank Schäfer escreveu: > >> Am 28.10.2012 21:57, schrieb Mauro Carvalho Chehab: >>> Em Sun, 21 Oct 2012 19:52:05 +0300 >>> Frank Schäfer escreveu: >>> This patch series adds support for USB bulk trans

cron job: media_tree daily build: WARNINGS

2012-10-29 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:Mon Oct 29 19:00:27 CET 2012 git hash:8f7e91a31fb95c50880c76505b416630c0326d93 gcc version: i686-linux-gcc (GCC

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

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 17:33:12 +0200 Frank Schäfer escreveu: > Am 28.10.2012 21:57, schrieb 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 f

Re: [PATCH 10/23] V4L: Add auto focus targets to the selections API

2012-10-29 Thread Sakari Ailus
Hi Sylwester, On Thu, May 10, 2012 at 12:30:45PM +0200, Sylwester Nawrocki wrote: > The camera automatic focus algorithms may require setting up > a spot or rectangle coordinates or multiple such parameters. > > The automatic focus selection targets are introduced in order > to allow applications

tidspbridge: ARM common zImage?

2012-10-29 Thread Omar Ramirez Luna
Hi, On 26 October 2012 13:00, Tony Lindgren wrote: ... >> > I would also like to move the tidspbridge to the DMA API, but I think we'll >> > need to move step by step there, and using the OMAP IOMMU and IOVMM APIs >> > as an >> > intermediate step would allow splitting patches in reviewable chun

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

2012-10-29 Thread Frank Schäfer
Am 28.10.2012 21:57, schrieb 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) >> t

[PULL] video_visstrim staging/ov7670_for_v3.7

2012-10-29 Thread Javier Martin
Hi Mauro, since there was some confusion regarding my two last series for the ov7670 I've decided to send this pull request to make things more clear and avoid merging order issues. It should apply properly in your current tree. The following changes since commit 1534e55974c7e2f57623457c0f6b4108

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Simon Farnsworth
On Monday 29 October 2012 13:44:45 Mauro Carvalho Chehab wrote: > Thanks for digging into it and getting more data. Do you know if this change > it also needed with USB devices that do DMA (isoc and/or bulk)? Or the USB > core already handles that? > I'm not a huge expert - the linux-pm list (cc'd

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 14:11 + Simon Farnsworth escreveu: > On Monday 29 October 2012 09:32:27 Andy Walls wrote: > > On Mon, 2012-10-29 at 13:02 +, Simon Farnsworth wrote: > > > It will affect other drivers as well; the basic cause is that modern chips > > > can enter a package deep sleep st

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 15:12:03 +0100 Sylwester Nawrocki escreveu: > On 10/29/2012 12:32 PM, Mauro Carvalho Chehab wrote: > > Em Mon, 29 Oct 2012 12:19:32 +0100 > > Sylwester Nawrocki escreveu: > > > >> On 10/29/2012 11:21 AM, Mauro Carvalho Chehab wrote: > >>> Hans Verkuil yesterday's build still

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Andy Walls
On Mon, 2012-10-29 at 13:02 +, Simon Farnsworth wrote: > On Monday 29 October 2012 09:58:17 Mauro Carvalho Chehab wrote: > > I prefer if you don't c/c me on that ;) Patchwork is the main source that I > > use > > on my patch reviews. > > > Noted. > > > Btw, I saw your patch yesterday (and sk

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Sylwester Nawrocki
On 10/29/2012 12:32 PM, Mauro Carvalho Chehab wrote: > Em Mon, 29 Oct 2012 12:19:32 +0100 > Sylwester Nawrocki escreveu: > >> On 10/29/2012 11:21 AM, Mauro Carvalho Chehab wrote: >>> Hans Verkuil yesterday's build still got two warnings at the >>> generic drivers: >>> http://hverkuil.home

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Simon Farnsworth
On Monday 29 October 2012 09:32:27 Andy Walls wrote: > On Mon, 2012-10-29 at 13:02 +, Simon Farnsworth wrote: > > It will affect other drivers as well; the basic cause is that modern chips > > can enter a package deep sleep state that affects both CPU speed and latency > > to start of DMA. On o

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Andy Walls
On Mon, 2012-10-29 at 13:02 +, Simon Farnsworth wrote: > On Monday 29 October 2012 09:58:17 Mauro Carvalho Chehab wrote: > > I prefer if you don't c/c me on that ;) Patchwork is the main source that I > > use > > on my patch reviews. > > > Noted. > > > Btw, I saw your patch yesterday (and sk

Re: [PATCH 35/68] [media] pwc-if: must check vb2_queue_init() success

2012-10-29 Thread Ezequiel Garcia
On Mon, Oct 29, 2012 at 8:44 AM, Mauro Carvalho Chehab wrote: > Em Mon, 29 Oct 2012 08:37:31 -0300 > Ezequiel Garcia escreveu: > >> On Sat, Oct 27, 2012 at 5:40 PM, Mauro Carvalho Chehab >> wrote: >> > drivers/media/usb/pwc/pwc-if.c: In function 'usb_pwc_probe': >> > drivers/media/usb/pwc/pwc-if

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Simon Farnsworth
On Monday 29 October 2012 09:58:17 Mauro Carvalho Chehab wrote: > I prefer if you don't c/c me on that ;) Patchwork is the main source that I > use > on my patch reviews. > Noted. > Btw, I saw your patch yesterday (and skipped it, for now), as I never played > with those pm QoS stuff before, nor

[PATCH v2] media: coda: Fix H.264 header alignment.

2012-10-29 Thread javier . martin
From: Javier Martin Length of H.264 headers is variable and thus it might not be aligned for the coda to append the encoded frame. This causes the first frame to overwrite part of the H.264 PPS. In order to solve that, a filler NAL must be added between the headers and the first frame to preserv

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 11:25:38 + Simon Farnsworth escreveu: > On Monday 22 October 2012 12:50:11 Simon Farnsworth wrote: > > The SAA7134 appears to have trouble buffering more than one line of video > > when doing DMA. Rather than try to fix the driver to cope (as has been done > > by Andy Wall

Re: [PATCH 35/68] [media] pwc-if: must check vb2_queue_init() success

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 08:37:31 -0300 Ezequiel Garcia escreveu: > On Sat, Oct 27, 2012 at 5:40 PM, Mauro Carvalho Chehab > wrote: > > drivers/media/usb/pwc/pwc-if.c: In function 'usb_pwc_probe': > > drivers/media/usb/pwc/pwc-if.c:1003:16: warning: ignoring return value of > > 'vb2_queue_init', dec

Re: [PATCH 35/68] [media] pwc-if: must check vb2_queue_init() success

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 08:37:31 -0300 Ezequiel Garcia escreveu: > On Sat, Oct 27, 2012 at 5:40 PM, Mauro Carvalho Chehab > wrote: > > drivers/media/usb/pwc/pwc-if.c: In function 'usb_pwc_probe': > > drivers/media/usb/pwc/pwc-if.c:1003:16: warning: ignoring return value of > > 'vb2_queue_init', dec

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 11:38:39 +0100 Laurent Pinchart escreveu: > Hi Mauro, > > On Monday 29 October 2012 08:21:56 Mauro Carvalho Chehab wrote: > > Hans Verkuil yesterday's build still got two warnings at the > > generic drivers: > > http://hverkuil.home.xs4all.nl/logs/Sunday.log > > > >

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Ezequiel Garcia
On Mon, Oct 29, 2012 at 8:32 AM, Mauro Carvalho Chehab wrote: > Em Mon, 29 Oct 2012 12:19:32 +0100 > Sylwester Nawrocki escreveu: > >> On 10/29/2012 11:21 AM, Mauro Carvalho Chehab wrote: >> > Hans Verkuil yesterday's build still got two warnings at the >> > generic drivers: >> > http://h

Re: [PATCH 35/68] [media] pwc-if: must check vb2_queue_init() success

2012-10-29 Thread Ezequiel Garcia
On Sat, Oct 27, 2012 at 5:40 PM, Mauro Carvalho Chehab wrote: > drivers/media/usb/pwc/pwc-if.c: In function 'usb_pwc_probe': > drivers/media/usb/pwc/pwc-if.c:1003:16: warning: ignoring return value of > 'vb2_queue_init', declared with attribute warn_unused_result [-Wunused-result] > In the past,

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Mauro Carvalho Chehab
Em Mon, 29 Oct 2012 12:19:32 +0100 Sylwester Nawrocki escreveu: > On 10/29/2012 11:21 AM, Mauro Carvalho Chehab wrote: > > Hans Verkuil yesterday's build still got two warnings at the > > generic drivers: > > http://hverkuil.home.xs4all.nl/logs/Sunday.log > > > > They didn't appear at i3

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Ezequiel Garcia
On Mon, Oct 29, 2012 at 8:19 AM, Sylwester Nawrocki wrote: > On 10/29/2012 11:21 AM, Mauro Carvalho Chehab wrote: >> Hans Verkuil yesterday's build still got two warnings at the >> generic drivers: >> http://hverkuil.home.xs4all.nl/logs/Sunday.log >> >> They didn't appear at i386 build pro

Re: [PATCH] saa7134: Add pm_qos_request to fix video corruption

2012-10-29 Thread Simon Farnsworth
On Monday 22 October 2012 12:50:11 Simon Farnsworth wrote: > The SAA7134 appears to have trouble buffering more than one line of video > when doing DMA. Rather than try to fix the driver to cope (as has been done > by Andy Walls for the cx18 driver), put in a pm_qos_request to limit deep > sleep ex

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Sylwester Nawrocki
On 10/29/2012 11:21 AM, Mauro Carvalho Chehab wrote: > Hans Verkuil yesterday's build still got two warnings at the > generic drivers: > http://hverkuil.home.xs4all.nl/logs/Sunday.log > > They didn't appear at i386 build probably because of some > optimization done there. > > Anyway, fixi

Re: [PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Laurent Pinchart
Hi Mauro, On Monday 29 October 2012 08:21:56 Mauro Carvalho Chehab wrote: > Hans Verkuil yesterday's build still got two warnings at the > generic drivers: > http://hverkuil.home.xs4all.nl/logs/Sunday.log > > They didn't appear at i386 build probably because of some > optimization done th

Re: [PATCH] media: coda: Fix H.264 header alignment.

2012-10-29 Thread Philipp Zabel
Hi Javier, Am Montag, den 29.10.2012, 10:20 +0100 schrieb javier.mar...@vista-silicon.com: > From: Javier Martin > > Length of H.264 headers is variable and thus it might not be > aligned for the coda to append the encoded frame. This causes > the first frame to overwrite part of the H.264 PPS.

[PATCH 0/2] Fix a few more warnings

2012-10-29 Thread Mauro Carvalho Chehab
Hans Verkuil yesterday's build still got two warnings at the generic drivers: http://hverkuil.home.xs4all.nl/logs/Sunday.log They didn't appear at i386 build probably because of some optimization done there. Anyway, fixing them are trivial, so let's do it. After applying those patches, t

[PATCH 2/2] [media] xc4000: Fix a few warnings

2012-10-29 Thread Mauro Carvalho Chehab
drivers/media/tuners/xc4000.c: In function ‘check_firmware’: drivers/media/tuners/xc4000.c:1048:45: warning: ‘fw_minor’ may be used uninitialized in this function [-Wmaybe-uninitialized] drivers/media/tuners/xc4000.c:1048:39: warning: ‘fw_major’ may be used uninitialized in this function [-Wmaybe

[PATCH 1/2] [media] drxk_hard: fix the return code from an error handler

2012-10-29 Thread Mauro Carvalho Chehab
While it is very unlikely, if the number of parameters for QAMDemodulatorCommand is not 2 or 4, status become undefined: /home/hans/work/build/v4l-dvb-git/drivers/media/dvb-frontends/drxk_hard.c: In function ‘QAMDemodulatorCommand’: /home/hans/work/build/v4l-dvb-git/drivers/media/dvb-frontends/dr

[PATCH] media: coda: Fix H.264 header alignment.

2012-10-29 Thread javier . martin
From: Javier Martin Length of H.264 headers is variable and thus it might not be aligned for the coda to append the encoded frame. This causes the first frame to overwrite part of the H.264 PPS. In order to solve that, a filler NAL must be added between the headers and the first frame to preserv

[PATCHv2] Improve media Kconfig menu

2012-10-29 Thread Hans Verkuil
(Sent again, this time with a SoB) 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 closely to the "Media ancillary driver

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

2012-10-29 Thread Guennadi Liakhovetski
On Sun, 28 Oct 2012, Sylwester Nawrocki wrote: > 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.