[PATCH 8/9] mem2mem_testdev: Don't check vb2_queue_init() return value

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Signed-off-by: Ezequiel Garcia --- drivers/media/platform/mem2mem_testdev.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/mem2mem_testdev.c b/drivers/media/

[PATCH 9/9] videobuf2-core: Change vb2_queue_init return type to void

2012-08-24 Thread Ezequiel Garcia
Signed-off-by: Ezequiel Garcia --- drivers/media/v4l2-core/videobuf2-core.c |3 +-- include/media/videobuf2-core.h |2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c index 4da

[PATCH 7/9] stk1160: Don't check vb2_queue_init() return

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Signed-off-by: Ezequiel Garcia --- drivers/media/usb/stk1160/stk1160-core.c |4 +--- drivers/media/usb/stk1160/stk1160-v4l.c | 12 +++- drivers/media/usb/stk1160/stk1160.h |2 +- 3 files ch

[PATCH 6/9] soc-camera: Don't check vb2_queue_init() return value

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Cc: Josh Wu Cc: Javier Martin Cc: Guennadi Liakhovetski Cc: Magnus Damm Signed-off-by: Ezequiel Garcia --- drivers/media/platform/soc_camera/atmel-isi.c |3 ++- drivers/media/platform/soc_camera/mx2_c

[PATCH 5/9] marvel-cam: Don't check vb2_queue_init() return value

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Cc: Jonathan Corbet Signed-off-by: Ezequiel Garcia --- drivers/media/platform/marvell-ccic/mcam-core.c |9 +++-- 1 files changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/media/platform/marve

[PATCH 4/9] mem2mem-deinterlace: Don't check vb2_queue_init() return value

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Cc: Javier Martin Signed-off-by: Ezequiel Garcia --- drivers/media/platform/m2m-deinterlace.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/m2m-deinterlace.

[PATCH 3/9] mem2mem-emmaprp: Don't check vb2_queue_init() return value

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Cc: Javier Martin Signed-off-by: Ezequiel Garcia --- drivers/media/platform/mx2_emmaprp.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/mx2_emmaprp.c b/dri

[PATCH 2/9] coda: Don't check vb2_queue_init() return value

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Cc: Javier Martin Signed-off-by: Ezequiel Garcia --- drivers/media/platform/coda.c |7 +++ 1 files changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/coda.c b/drivers/media/plat

[PATCH 1/9] s5p: Don't check vb2_queue_init() return value

2012-08-24 Thread Ezequiel Garcia
Right now vb2_queue_init() returns always 0 and it will be changed to return void. Cc: Sylwester Nawrocki Signed-off-by: Ezequiel Garcia --- drivers/media/platform/s5p-fimc/fimc-m2m.c |7 +++ drivers/media/platform/s5p-g2d/g2d.c|7 +++ drivers/media/platform/s5p-jpeg/jp

Re: RFC: Core + Radio profile

2012-08-24 Thread Andy Walls
On Wed, 2012-08-22 at 11:40 +0200, Hans Verkuil wrote: > Hi all! > > The v4l2-compliance utility checks whether device drivers follows the V4L2 > specification. What is missing are 'profiles' detailing what device drivers > should and shouldn't implement for particular device classes. > >This has

Re: [PATCH 10/12] [media] move i2c files into drivers/media/i2c

2012-08-24 Thread Sylwester Nawrocki
Hi Mauro, On 08/15/2012 03:48 PM, Mauro Carvalho Chehab wrote: > Move ancillary I2C drivers into drivers/media/i2c, in order to > better organize them. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/Kconfig | 9 +- > drivers/media/Makefile

[RFC v4] V4L DT bindings

2012-08-24 Thread Guennadi Liakhovetski
Hi all After an initial RFC [1] and taking into consideration an even earlier patch-set [2], Sylwester and I have spent some time discussing V4L DT bindings, below is a result of those discussions. We have chosen to try to design a DT example, documentation and implementation should follow. I'

Re: [PATCH] [media] rc: do not sleep when the driver blocks on IR completion

2012-08-24 Thread Sean Young
On Sat, Aug 25, 2012 at 12:05:18AM +0200, David Härdeman wrote: > On Thu, Aug 23, 2012 at 10:18:35PM +0100, Sean Young wrote: > >Some drivers wait for the IR device to complete sending before > >returning, so sleeping should not be done. > > I'm not quite sure what the purpose is. Even if a driver

Re: [media] rc-core: move timeout and checks to lirc

2012-08-24 Thread Sean Young
On Sat, Aug 25, 2012 at 12:16:04AM +0200, David Härdeman wrote: > On Thu, Aug 16, 2012 at 11:15:14PM +0100, Sean Young wrote: > > > >> The lirc TX functionality expects the process which writes (TX) data to > >> the lirc dev to sleep until the actual data has been transmitted by the > >> hardware.

Re: [PATCH RFC 1/4] V4L: Add V4L2_CID_FRAMESIZE image source class control

2012-08-24 Thread Sakari Ailus
Hi Sylwester, On Fri, Aug 24, 2012 at 10:15:58AM +0200, Sylwester Nawrocki wrote: > On 08/24/2012 12:41 AM, Laurent Pinchart wrote: > > On Thursday 23 August 2012 21:24:12 Sakari Ailus wrote: > >> Sylwester Nawrocki wrote: > On Thu, Aug 23, 2012 at 11:51:26AM +0200, Sylwester Nawrocki wrote:

Re: [media] rc-core: move timeout and checks to lirc

2012-08-24 Thread David Härdeman
On Thu, Aug 16, 2012 at 11:15:14PM +0100, Sean Young wrote: > >> The lirc TX functionality expects the process which writes (TX) data to >> the lirc dev to sleep until the actual data has been transmitted by the >> hardware. >> >> Since the same timeout calculation is duplicated in more than one

Re: [PATCH] [media] rc: fix buffer overrun

2012-08-24 Thread David Härdeman
On Thu, Aug 23, 2012 at 10:18:25PM +0100, Sean Young wrote: >"[media] rc-core: move timeout and checks to lirc" introduced a buffer >overrun by passing the number of bytes, rather than the number of samples, >to the transmit function. > >Signed-off-by: Sean Young Acked-by: David Härdeman Thanks

Re: [PATCH] [media] rc: do not sleep when the driver blocks on IR completion

2012-08-24 Thread David Härdeman
On Thu, Aug 23, 2012 at 10:18:35PM +0100, Sean Young wrote: >Some drivers wait for the IR device to complete sending before >returning, so sleeping should not be done. I'm not quite sure what the purpose is. Even if a driver waits for TX to finish, the lirc imposed sleep isn't harmful in any way.

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Guennadi Liakhovetski
On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > Hi Guennadi, > > On Fri, 24 Aug 2012 15:35:59 +0200 (CEST) > Guennadi Liakhovetski wrote: > ... > > Below I asked to provide details about how you have to change this > > register value: toggle dynamically at run-time or just set once at > > ini

Re: [PATCHv2 7/8] ir-rx51: Convert latency constraints to PM QoS API

2012-08-24 Thread Tony Lindgren
* Timo Kokkonen [120824 08:11]: > Convert the driver from the obsolete omap_pm_set_max_mpu_wakeup_lat > API to the new PM QoS API. This allows the callback to be removed from > the platform data structure. > > The latency requirements are also adjusted to prevent the MPU from > going into sleep m

Re: [Workshop-2011] Media summit at the Kernel Summit - was: Fwd: Re: [Ksummit-2012-discuss] Organising Mini Summits within the Kernel Summit

2012-08-24 Thread Guennadi Liakhovetski
On Fri, 24 Aug 2012, Mauro Carvalho Chehab wrote: > Em 24-08-2012 09:41, Guennadi Liakhovetski escreveu: > > Hi Laurent > > > > On Fri, 24 Aug 2012, Laurent Pinchart wrote: > > > >> Hi Guennadi, > >> > >> On Friday 24 August 2012 14:11:58 Guennadi Liakhovetski wrote: > >>> On Fri, 24 Aug 2012, M

cron job: media_tree daily build: WARNINGS

2012-08-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:Fri Aug 24 19:00:29 CEST 2012 git hash:79e8c7bebb467bbc3f2514d75bba669a3f354324 gcc version: i686-linux-gcc (GC

Re: [Workshop-2011] Media summit at the Kernel Summit - was: Fwd: Re: [Ksummit-2012-discuss] Organising Mini Summits within the Kernel Summit

2012-08-24 Thread Mauro Carvalho Chehab
Em 24-08-2012 09:41, Guennadi Liakhovetski escreveu: > Hi Laurent > > On Fri, 24 Aug 2012, Laurent Pinchart wrote: > >> Hi Guennadi, >> >> On Friday 24 August 2012 14:11:58 Guennadi Liakhovetski wrote: >>> On Fri, 24 Aug 2012, Mauro Carvalho Chehab wrote: Em 22-08-2012 18:39, Naveen KRISHNAM

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Anatolij Gustschin
Hi Guennadi, On Fri, 24 Aug 2012 15:35:59 +0200 (CEST) Guennadi Liakhovetski wrote: ... > Below I asked to provide details about how you have to change this > register value: toggle dynamically at run-time or just set once at > initialisation? Even if toggle: are this certain moments, related t

[PATCH 04/12] media: coda: allocate internal framebuffers separately from v4l2 buffers

2012-08-24 Thread Philipp Zabel
Some codecs running on CODA need internal framebuffers for reference and reconstructed frames. Allocate them separately, and do not use the input vb2_buffers: those will be handed off to userspace regularly, and there is no way to signal to the CODA which of the registered framebuffers are off limi

[PATCH 05/12] media: coda: ignore coda busy status in coda_job_ready

2012-08-24 Thread Philipp Zabel
job_ready is supposed to signal whether a context is ready to be added to the job queue, not whether the CODA is ready to run it immediately. Calling v4l2_m2m_job_finish at the end of coda_irq_handler already guarantees that the coda is ready when v4l2-mem2mem eventually tries to run the next queue

[PATCH 06/12] media: coda: keep track of active instances

2012-08-24 Thread Philipp Zabel
Determining the next free instance just by incrementing and decrementing an instance counter does not work: if there are two instances opened, 0 and 1, and instance 0 is released, the next call to coda_open will create a new instance with index 1, but instance 1 is already in use. Instead, scan a

[PATCH 07/12] media: coda: stop all queues in case of lockup

2012-08-24 Thread Philipp Zabel
Add a 1 second timeout for each PIC_RUN command to the CODA. In case it locks up, stop all queues and dequeue remaining buffers. Signed-off-by: Philipp Zabel --- drivers/media/video/coda.c | 21 + 1 file changed, 21 insertions(+) diff --git a/drivers/media/video/coda.c b/d

[PATCH 08/12] coda: enable user pointer support

2012-08-24 Thread Philipp Zabel
USERPTR buffer support is provided by the videobuf2 framework. Signed-off-by: Philipp Zabel --- drivers/media/video/coda.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/coda.c b/drivers/media/video/coda.c index e2c4585..700df4b 100644 --- a/driver

[PATCH 09/12] media: coda: wait for picture run completion in start/stop_streaming

2012-08-24 Thread Philipp Zabel
While the CODA is running a PIC_RUN command, its registers are not to be touched. Signed-off-by: Philipp Zabel --- drivers/media/video/coda.c | 34 +++--- 1 file changed, 27 insertions(+), 7 deletions(-) diff --git a/drivers/media/video/coda.c b/drivers/media/video

[PATCH 10/12] media: coda: fix sizeimage setting in try_fmt

2012-08-24 Thread Philipp Zabel
VIDIOC_TRY_FMT would incorrectly return bytesperline * height, instead of width * height * 3 / 2. Signed-off-by: Philipp Zabel --- drivers/media/video/coda.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/drivers/media/video/coda.c b/drivers/media/video/coda.c i

[PATCH 11/12] ARM i.MX5: Fix CODA7 clock lookup for device tree on i.MX51 and i.MX53

2012-08-24 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- arch/arm/mach-imx/clk-imx51-imx53.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/mach-imx/clk-imx51-imx53.c b/arch/arm/mach-imx/clk-imx51-imx53.c index a2200c7..887ede9 100644 --- a/arch/arm/mach-imx/clk-imx51-imx53.c +++ b/

[PATCH 12/12] ARM i.MX5: Add CODA7 to device tree for i.MX51 and i.MX53

2012-08-24 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- arch/arm/boot/dts/imx51.dtsi |6 ++ arch/arm/boot/dts/imx53.dtsi |7 +++ 2 files changed, 13 insertions(+) diff --git a/arch/arm/boot/dts/imx51.dtsi b/arch/arm/boot/dts/imx51.dtsi index bfa65ab..8e23068 100644 --- a/arch/arm/boot/dts/imx51.dtsi ++

[PATCH 01/12] coda: firmware loading for 64-bit AXI bus width

2012-08-24 Thread Philipp Zabel
Add support for loading a raw firmware with 16-bit chars ordered in little-endian 64-bit words, corresponding to the memory access pattern of CODA7 and above: When writing the boot code into the code download register, the chars have to be reordered back. Signed-off-by: Philipp Zabel --- drivers

[PATCH 03/12] coda: fix IRAM/AXI handling for i.MX53

2012-08-24 Thread Philipp Zabel
This uses the genalloc API to allocate a work buffer in the SoC's on-chip SRAM and sets up the AXI_SRAM_USE register. Signed-off-by: Philipp Zabel --- drivers/media/video/Kconfig |3 ++- drivers/media/video/coda.c | 61 --- drivers/media/video/coda.

[PATCH 02/12] coda: add i.MX53 / CODA7541 platform support

2012-08-24 Thread Philipp Zabel
Signed-off-by: Philipp Zabel --- drivers/media/video/coda.c | 37 + 1 file changed, 37 insertions(+) diff --git a/drivers/media/video/coda.c b/drivers/media/video/coda.c index 86dae17..aa12b7b 100644 --- a/drivers/media/video/coda.c +++ b/drivers/media/video

[PATCH 0/12] Initial i.MX5/CODA7 support for the CODA driver

2012-08-24 Thread Philipp Zabel
These patches contain initial firmware loading and encoding support for the CODA7 series VPU contained in i.MX51 and i.MX53 SoCs, and fix some multi-instance issues. regards Philipp --- arch/arm/boot/dts/imx51.dtsi|6 + arch/arm/boot/dts/imx53.dtsi|7 ++ arch/arm

[PATCH] Corrected Oops on omap_vout when no manager is connected

2012-08-24 Thread Federico Fuga
If no manager is connected to the vout device, the omapvid_init() function fails. No error condition is checked, and the device is started. Later on, when irq is serviced, a NULL pointer dereference occurs. Also, the isr routine must be registered only if no error occurs, otherwise the isr triggers

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Detlev Zundel
Hi Guennadi, > Hi Detlev > > On Fri, 24 Aug 2012, Detlev Zundel wrote: > >> Hello Guennadi, >> >> > Hi Anatolij >> > >> > On Fri, 24 Aug 2012, Anatolij Gustschin wrote: >> > >> >> Add controls for horizontal and vertical blanking, analog control >> >> and control for undocumented register 32. >>

Re: RFC: Core + Radio profile

2012-08-24 Thread Hans Verkuil
On Fri August 24 2012 16:51:56 Mauro Carvalho Chehab wrote: > Em 24-08-2012 09:31, Hans Verkuil escreveu: > > Hi Mauro, > > > > Thanks for your review! > > > > On Wed August 22 2012 15:42:26 Mauro Carvalho Chehab wrote: > >> Em 22-08-2012 07:11, Hans Verkuil escreveu: > >>> I've added some more c

[PATCHv2 5/8] ir-rx51: Move platform data checking into probe function

2012-08-24 Thread Timo Kokkonen
This driver is useless without proper platform data. If data is not available, we should not register the driver at all. Once this check is done, the BUG_ON check during device open is no longer needed. Signed-off-by: Timo Kokkonen --- drivers/media/rc/ir-rx51.c | 4 +++- 1 file changed, 3 inser

[PATCHv2 6/8] ir-rx51: Replace module_{init,exit} macros with module_platform_driver

2012-08-24 Thread Timo Kokkonen
No reason to avoid using the existing helpers. Signed-off-by: Timo Kokkonen --- drivers/media/rc/ir-rx51.c | 12 +--- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/drivers/media/rc/ir-rx51.c b/drivers/media/rc/ir-rx51.c index 16b3c1f..6e1ffa6 100644 --- a/drivers/media/rc

[PATCHv2 7/8] ir-rx51: Convert latency constraints to PM QoS API

2012-08-24 Thread Timo Kokkonen
Convert the driver from the obsolete omap_pm_set_max_mpu_wakeup_lat API to the new PM QoS API. This allows the callback to be removed from the platform data structure. The latency requirements are also adjusted to prevent the MPU from going into sleep mode. This is needed as the GP timers have no

[PATCHv2 8/8] ir-rx51: Remove useless variable from struct lirc_rx51

2012-08-24 Thread Timo Kokkonen
As clearly visible from the patch, this variable has no useful purpose what so ever. Thus, it can be removed altogether without any side effects. Signed-off-by: Timo Kokkonen --- drivers/media/rc/ir-rx51.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/drivers/media/

[PATCHv2 4/8] ir-rx51: Clean up timer initialization code

2012-08-24 Thread Timo Kokkonen
Remove a redundant macro definition. This is unneeded and becomes more readable once the actual timer code is refactored a little. Signed-off-by: Timo Kokkonen --- drivers/media/rc/ir-rx51.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/drivers/media/rc/ir-rx51.

[PATCHv2 2/8] ir-rx51: Handle signals properly

2012-08-24 Thread Timo Kokkonen
The lirc-dev expects the ir-code to be transmitted when the write call returns back to the user space. We should not leave TX ongoing no matter what is the reason we return to the user space. Easiest solution for that is to simply remove interruptible sleeps. The first wait_event_interruptible is

[PATCHv2 1/8] ir-rx51: Adjust dependencies

2012-08-24 Thread Timo Kokkonen
Although this kind of IR diode circuitry is known to exist only in N900 hardware, nothing prevents making similar circuitry on any OMAP based board. The MACH_NOKIA_RX51 dependency is thus not something we want to be there. Also, this should depend on LIRC as it is a LIRC driver. Signed-off-by: Ti

[PATCHv2 3/8] ir-rx51: Trivial fixes

2012-08-24 Thread Timo Kokkonen
-Fix typo -Change pwm_timer_num type to match type in platform data -Remove extra parenthesis -Replace magic constant with proper bit defintions -Remove duplicate exit pointer Signed-off-by: Timo Kokkonen --- drivers/media/rc/Kconfig | 2 +- drivers/media/rc/ir-rx51.c | 10 ++ 2 f

[PATCHv2 0/8] ir-rx51: Fixes in response to review comments

2012-08-24 Thread Timo Kokkonen
These patches fix most of the issues pointed out in the patch review by Sean Young and Sakari Ailus. The most noticeable change after these patch set is that the IR transmission no longer times out even if the timers are not waking up the MPU as it should be. Now that Jean Pihet kindly instructed

Re: RFC: Core + Radio profile

2012-08-24 Thread Mauro Carvalho Chehab
Em 24-08-2012 09:31, Hans Verkuil escreveu: > Hi Mauro, > > Thanks for your review! > > On Wed August 22 2012 15:42:26 Mauro Carvalho Chehab wrote: >> Em 22-08-2012 07:11, Hans Verkuil escreveu: >>> I've added some more core profile requirements. >>> >>> Regards, >>> >>> Hans >>> >>> On Wed A

[PATCH v3] V4L: soc_camera: allow reading from video device if supported

2012-08-24 Thread Anatolij Gustschin
Try reading on video device. If the camera bus driver supports reading we can try it and return the result. Also add a debug line. Signed-off-by: Anatolij Gustschin --- v3: - simplified as suggested by Guennadi v2: - rebased on current staging/for_v3.7 branch drivers/media/platform/soc_camer

Re: [PATCH 2/3] mt9v022: fix the V4L2_CID_EXPOSURE control

2012-08-24 Thread Guennadi Liakhovetski
On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > Hi Guennadi, > > On Fri, 24 Aug 2012 13:22:18 +0200 (CEST) > Guennadi Liakhovetski wrote: > ... > > > --- a/drivers/media/i2c/soc_camera/mt9v022.c > > > +++ b/drivers/media/i2c/soc_camera/mt9v022.c > > > @@ -274,9 +274,9 @@ static int mt9v022_s_cr

[git:v4l-dvb/for_v3.7] [media] video: mx1_camera: Use clk_prepare_enable/clk_disable_unprepare

2012-08-24 Thread Mauro Carvalho Chehab
This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/media_tree.git tree: Subject: [media] video: mx1_camera: Use clk_prepare_enable/clk_disable_unprepare Author: Fabio Estevam Date:Fri May 25 20:14:47 2012 -0300 Prepare t

[git:v4l-dvb/for_v3.7] [media] video: mx2_camera: Use clk_prepare_enable/clk_disable_unprepare

2012-08-24 Thread Mauro Carvalho Chehab
This is an automatic generated email to let you know that the following patch were queued at the http://git.linuxtv.org/media_tree.git tree: Subject: [media] video: mx2_camera: Use clk_prepare_enable/clk_disable_unprepare Author: Fabio Estevam Date:Fri May 25 20:14:48 2012 -0300 Prepare t

Re: [PATCH 2/3] mt9v022: fix the V4L2_CID_EXPOSURE control

2012-08-24 Thread Anatolij Gustschin
Hi Guennadi, On Fri, 24 Aug 2012 13:22:18 +0200 (CEST) Guennadi Liakhovetski wrote: ... > > --- a/drivers/media/i2c/soc_camera/mt9v022.c > > +++ b/drivers/media/i2c/soc_camera/mt9v022.c > > @@ -274,9 +274,9 @@ static int mt9v022_s_crop(struct v4l2_subdev *sd, > > struct v4l2_crop *a) > >

Re: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Federico Vaga
> You can take the patch which adds prepare/finish methods to memory > allocators. It should not have any dependency on the other stuff from > that thread. I'm fine with merging it either together with Your patch > or via Tomasz's patchset, whatever comes first. Thank you. I'll do the job -- Fed

RE: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Marek Szyprowski
Hello, On Friday, August 24, 2012 3:23 PM Federico Vaga wrote: > > Getting back to your patch - in your approach cpu cache handling is > > missing. I suspect that it worked fine only because it has been > > tested on some simple platform without any cpu caches (or with very > > small ones). > >

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Guennadi Liakhovetski
Hi Detlev On Fri, 24 Aug 2012, Detlev Zundel wrote: > Hello Guennadi, > > > Hi Anatolij > > > > On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > > > >> Add controls for horizontal and vertical blanking, analog control > >> and control for undocumented register 32. > > > > Sorry, I don't think th

Re: [PATCH 3/3] mt9v022: set y_skip_top field to zero

2012-08-24 Thread Anatolij Gustschin
On Fri, 24 Aug 2012 13:23:22 +0200 (CEST) Guennadi Liakhovetski wrote: > On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > > > Set "y_skip_top" to zero and remove comment as I do not see this > > line corruption on two different mt9v022 setups. The first read-out > > line is perfectly fine. > >

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Anatolij Gustschin
Hi Guennadi, On Fri, 24 Aug 2012 13:08:52 +0200 (CEST) Guennadi Liakhovetski wrote: ... > > +#define MT9V022_HORIZONTAL_BLANKING_MIN43 > > +#define MT9V022_HORIZONTAL_BLANKING_MAX1023 > > +#define MT9V022_HORIZONTAL_BLANKING_DEF94 > > +#define MT9V022_VERTICAL_BLANKING_MIN 2 > >

Re: [PATCH 2/3] [media] videobuf2-dma-streaming: new videobuf2 memory allocator

2012-08-24 Thread Federico Vaga
> Getting back to your patch - in your approach cpu cache handling is > missing. I suspect that it worked fine only because it has been > tested on some simple platform without any cpu caches (or with very > small ones). Is missing from the memory allocator because I do it on the device driver. T

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Detlev Zundel
Hello Guennadi, > Hi Anatolij > > On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > >> Add controls for horizontal and vertical blanking, analog control >> and control for undocumented register 32. > > Sorry, I don't think this is a good idea to export an undocumented > register as a control. Why

Re: [Workshop-2011] Media summit at the Kernel Summit - was: Fwd: Re: [Ksummit-2012-discuss] Organising Mini Summits within the Kernel Summit

2012-08-24 Thread Guennadi Liakhovetski
Hi Laurent On Fri, 24 Aug 2012, Laurent Pinchart wrote: > Hi Guennadi, > > On Friday 24 August 2012 14:11:58 Guennadi Liakhovetski wrote: > > On Fri, 24 Aug 2012, Mauro Carvalho Chehab wrote: > > > Em 22-08-2012 18:39, Naveen KRISHNAMURTHY escreveu: > > > > Hi Mauro, > > > > > > > > Can you ple

Re: RFC: Core + Radio profile

2012-08-24 Thread Hans Verkuil
Hi Mauro, Thanks for your review! On Wed August 22 2012 15:42:26 Mauro Carvalho Chehab wrote: > Em 22-08-2012 07:11, Hans Verkuil escreveu: > > I've added some more core profile requirements. > > > > Regards, > > > > Hans > > > > On Wed August 22 2012 11:40:25 Hans Verkuil wrote: > >> Hi a

Re: [Workshop-2011] Media summit at the Kernel Summit - was: Fwd: Re: [Ksummit-2012-discuss] Organising Mini Summits within the Kernel Summit

2012-08-24 Thread Laurent Pinchart
Hi Guennadi, On Friday 24 August 2012 14:11:58 Guennadi Liakhovetski wrote: > On Fri, 24 Aug 2012, Mauro Carvalho Chehab wrote: > > Em 22-08-2012 18:39, Naveen KRISHNAMURTHY escreveu: > > > Hi Mauro, > > > > > > Can you please provide the schedule for the linuxTV discussions this > > > time at Sa

Re: [Workshop-2011] Media summit at the Kernel Summit - was: Fwd: Re: [Ksummit-2012-discuss] Organising Mini Summits within the Kernel Summit

2012-08-24 Thread Guennadi Liakhovetski
Hi Mauro On Fri, 24 Aug 2012, Mauro Carvalho Chehab wrote: > Hi Naveen, > > Em 22-08-2012 18:39, Naveen KRISHNAMURTHY escreveu: > > Hi Mauro, > > > > Can you please provide the schedule for the linuxTV discussions this time > > at San Diego? > > Which day will it be on? If you have a detailed

[PATCH] [media] mem2mem_testdev: Make m2mtest_dev_release function static

2012-08-24 Thread Sachin Kamat
Fixes the following warning: drivers/media/platform/mem2mem_testdev.c:73:6: warning: symbol 'm2mtest_dev_release' was not declared. Should it be static? Signed-off-by: Sachin Kamat --- drivers/media/platform/mem2mem_testdev.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --gi

Re: [PATCH v2] V4L: soc_camera: allow reading from video device if supported

2012-08-24 Thread Anatolij Gustschin
Hi Guennadi, On Fri, 24 Aug 2012 13:37:20 +0200 (CEST) Guennadi Liakhovetski wrote: ... > How about a slightly simpler > > > + struct soc_camera_host *ici = to_soc_camera_host(icd->parent); > > - int err = -EINVAL; > > > > - dev_err(icd->pdev, "camera device read not implemented\n"); > >

Re: [PATCH v2] V4L: soc_camera: allow reading from video device if supported

2012-08-24 Thread Guennadi Liakhovetski
On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > Try reading on video device. If the camera bus driver supports reading > we can try it and return the result. Also add a debug line. > > Signed-off-by: Anatolij Gustschin > --- > v2: - rebased on current staging/for_v3.7 branch > > drivers/med

Re: [PATCH 3/3] mt9v022: set y_skip_top field to zero

2012-08-24 Thread Guennadi Liakhovetski
On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > Set "y_skip_top" to zero and remove comment as I do not see this > line corruption on two different mt9v022 setups. The first read-out > line is perfectly fine. On what systems have you checked this? Thanks Guennadi > > Signed-off-by: Anatolij G

Re: [PATCH 2/3] mt9v022: fix the V4L2_CID_EXPOSURE control

2012-08-24 Thread Guennadi Liakhovetski
On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > Since the MT9V022_TOTAL_SHUTTER_WIDTH register is controlled in manual > mode by V4L2_CID_EXPOSURE control, it shouldn't be written directly in > mt9v022_s_crop(). In manual mode this register should be set to the > V4L2_CID_EXPOSURE control value.

Re: [PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Guennadi Liakhovetski
Hi Anatolij On Fri, 24 Aug 2012, Anatolij Gustschin wrote: > Add controls for horizontal and vertical blanking, analog control > and control for undocumented register 32. Sorry, I don't think this is a good idea to export an undocumented register as a control. At most I would add a platform par

Re: [PATCH 7/8] ir-rx51: Remove MPU wakeup latency adjustments

2012-08-24 Thread Timo Kokkonen
On 08/24/12 12:04, Jean Pihet wrote: > Hi Timo, > > On Fri, Aug 24, 2012 at 10:14 AM, Timo Kokkonen > wrote: >> Hi Jean, >> >> On 08/23/12 14:58, Jean Pihet wrote: >>> Hi Timo, >>> >>> On Wed, Aug 22, 2012 at 9:50 PM, Timo Kokkonen >>> wrote: >>> That is correct. The API to use is the PM QoS A

Re: [Workshop-2011] Media summit at the Kernel Summit - was: Fwd: Re: [Ksummit-2012-discuss] Organising Mini Summits within the Kernel Summit

2012-08-24 Thread Mauro Carvalho Chehab
Hi Naveen, Em 22-08-2012 18:39, Naveen KRISHNAMURTHY escreveu: > Hi Mauro, > > Can you please provide the schedule for the linuxTV discussions this time at > San Diego? > Which day will it be on? If you have a detailed schedule, please pass on. I > am planning > to be in only on that day as I

Re: [PATCH 2/8] ir-rx51: Handle signals properly

2012-08-24 Thread Timo Kokkonen
On 08/24/12 13:03, Sean Young wrote: > On Wed, Aug 22, 2012 at 10:50:35PM +0300, Timo Kokkonen wrote: >> The lirc-dev expects the ir-code to be transmitted when the write call >> returns back to the user space. We should not leave TX ongoing no >> matter what is the reason we return to the user spa

Re: [PATCH 2/8] ir-rx51: Handle signals properly

2012-08-24 Thread Sean Young
On Wed, Aug 22, 2012 at 10:50:35PM +0300, Timo Kokkonen wrote: > The lirc-dev expects the ir-code to be transmitted when the write call > returns back to the user space. We should not leave TX ongoing no > matter what is the reason we return to the user space. Easiest > solution for that is to simp

[PATCH v2] V4L: soc_camera: allow reading from video device if supported

2012-08-24 Thread Anatolij Gustschin
Try reading on video device. If the camera bus driver supports reading we can try it and return the result. Also add a debug line. Signed-off-by: Anatolij Gustschin --- v2: - rebased on current staging/for_v3.7 branch drivers/media/platform/soc_camera/soc_camera.c |9 - 1 files ch

[PATCH 2/3] mt9v022: fix the V4L2_CID_EXPOSURE control

2012-08-24 Thread Anatolij Gustschin
Since the MT9V022_TOTAL_SHUTTER_WIDTH register is controlled in manual mode by V4L2_CID_EXPOSURE control, it shouldn't be written directly in mt9v022_s_crop(). In manual mode this register should be set to the V4L2_CID_EXPOSURE control value. Changing this register directly and outside of the actua

[PATCH 3/3] mt9v022: set y_skip_top field to zero

2012-08-24 Thread Anatolij Gustschin
Set "y_skip_top" to zero and remove comment as I do not see this line corruption on two different mt9v022 setups. The first read-out line is perfectly fine. Signed-off-by: Anatolij Gustschin --- drivers/media/i2c/soc_camera/mt9v022.c |6 +- 1 files changed, 1 insertions(+), 5 deletions(-

[PATCH 0/3] various updates for mt9v022 driver

2012-08-24 Thread Anatolij Gustschin
Anatolij Gustschin (3): mt9v022: add v4l2 controls for blanking and other register settings mt9v022: fix the V4L2_CID_EXPOSURE control mt9v022: set y_skip_top field to zero drivers/media/i2c/soc_camera/mt9v022.c | 117 1 files changed, 104 insertions(+), 13

[PATCH 1/3] mt9v022: add v4l2 controls for blanking and other register settings

2012-08-24 Thread Anatolij Gustschin
Add controls for horizontal and vertical blanking, analog control and control for undocumented register 32. Also add an error message for case that the control handler init failed. Since setting the blanking registers is done by controls now, we should't change these registers outside of the contro

Re: [PATCH 7/8] ir-rx51: Remove MPU wakeup latency adjustments

2012-08-24 Thread Jean Pihet
Hi Timo, On Fri, Aug 24, 2012 at 10:14 AM, Timo Kokkonen wrote: > Hi Jean, > > On 08/23/12 14:58, Jean Pihet wrote: >> Hi Timo, >> >> On Wed, Aug 22, 2012 at 9:50 PM, Timo Kokkonen >> wrote: >> That is correct. The API to use is the PM QoS API which cpuidle uses >> to determine the next MPU sta

Re: [PATCH RFC 1/4] V4L: Add V4L2_CID_FRAMESIZE image source class control

2012-08-24 Thread Sylwester Nawrocki
Hi, On 08/24/2012 12:41 AM, Laurent Pinchart wrote: > On Thursday 23 August 2012 21:24:12 Sakari Ailus wrote: >> Sylwester Nawrocki wrote: On Thu, Aug 23, 2012 at 11:51:26AM +0200, Sylwester Nawrocki wrote: > The V4L2_CID_FRAMESIZE control determines maximum number > of media bus samp

Re: [PATCH 7/8] ir-rx51: Remove MPU wakeup latency adjustments

2012-08-24 Thread Timo Kokkonen
Hi Jean, On 08/23/12 14:58, Jean Pihet wrote: > Hi Timo, > > On Wed, Aug 22, 2012 at 9:50 PM, Timo Kokkonen wrote: > That is correct. The API to use is the PM QoS API which cpuidle uses > to determine the next MPU state based on the allowed latency. > >> A more appropriate fix for the problem w