Re: [PATCH 07/10] mem2mem-emmaprp: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread javier Martin
On 23 August 2012 15:08, Ezequiel Garcia wrote: > queue_init() is always called by v4l2_m2m_ctx_init(), which allocates > a context struct v4l2_m2m_ctx with kzalloc. > Therefore, there is no need to clear vb2_queue src/dst structs. > > Cc: Javier Martin > Signed-off-by: Ezequiel Garcia > --- >

Re: [PATCH 05/10] coda: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread javier Martin
On 23 August 2012 15:08, Ezequiel Garcia wrote: > queue_init() is always called by v4l2_m2m_ctx_init(), which allocates > a context struct v4l2_m2m_ctx with kzalloc. > Therefore, there is no need to clear vb2_queue src/dst structs. > > Cc: Javier Martin > Signed-off-by: Ezequiel Garcia > --- >

[v3 1/1] driver-core: Shut up dev_dbg_reatelimited() without DEBUG

2012-08-23 Thread Hiroshi Doyu
dev_dbg_reatelimited() without DEBUG printed "217078 callbacks suppressed". This shouldn't print anything without DEBUG. With CONFIG_DYNAMIC_DEBUG, the print should be configured as expected. Signed-off-by: Hiroshi Doyu Reported-by: Hin-Tak Leung Tested-by: Antti Palosaari Acked-by: Hin-Tak Le

RE: [PATCH v4 1/4] [media] s5p-mfc: Update MFCv5 driver for callback based architecture

2012-08-23 Thread Arun Kumar K
Hi Kamil, Thank you for the review comments. Will post v5 patches incorporating your comments. Regards Arun --- Original Message --- Sender : Kamil Debski Software Engineer/SPRC-Linux Platform (SSD)/Samsung Electronics Date : Aug 22, 2012 21:48 (GMT+05:30) Title : RE: [PATCH v4 1/4]

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

2012-08-23 Thread Laurent Pinchart
Hi Sylwester, 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 samples transmitted within a single data fram

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

2012-08-23 Thread Sean Young
Some drivers wait for the IR device to complete sending before returning, so sleeping should not be done. Signed-off-by: Sean Young --- drivers/media/rc/iguanair.c | 1 + drivers/media/rc/ir-lirc-codec.c | 5 + drivers/media/rc/winbond-cir.c | 1 + include/media/rc-core.h | 2

[PATCH] [media] ttusbir: Add USB dependency

2012-08-23 Thread Sean Young
This patch fixes the error: ERROR: "usb_speed_string" [drivers/usb/core/usbcore.ko] undefined! Reported-by: Guenter Roeck Signed-off-by: Sean Young --- drivers/media/rc/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/media/rc/Kconfig b/drivers/media/rc/Kconfig index 4682a5a.

[PATCH] [media] rc: fix buffer overrun

2012-08-23 Thread Sean Young
"[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 --- drivers/media/rc/ir-lirc-codec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

[PATCH] stk1160: Remove unused 'ifnum' variable

2012-08-23 Thread Ezequiel Garcia
Since ifnum is not used anywhere it is safe to remove it. This was spotted by Hans's media_tree daily build. Signed-off-by: Ezequiel Garcia --- drivers/media/usb/stk1160/stk1160-core.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/media/usb/stk1160/stk1160-cor

cron job: media_tree daily build: WARNINGS

2012-08-23 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:Thu Aug 23 19:00:21 CEST 2012 git hash:a47b6118e134b51562de520d644d3979b3d99e44 gcc version: i686-linux-gcc (GC

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

2012-08-23 Thread Sakari Ailus
Hi Sylwester, Sylwester Nawrocki wrote: On 08/23/2012 02:13 PM, Sakari Ailus wrote: Hi Sylwester, Thanks for the patch. Thanks for your review. On Thu, Aug 23, 2012 at 11:51:26AM +0200, Sylwester Nawrocki wrote: The V4L2_CID_FRAMESIZE control determines maximum number of media bus samples

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

2012-08-23 Thread Sylwester Nawrocki
Hi Sakari, On 08/23/2012 02:13 PM, Sakari Ailus wrote: > Hi Sylwester, > > Thanks for the patch. Thanks for your review. > On Thu, Aug 23, 2012 at 11:51:26AM +0200, Sylwester Nawrocki wrote: >> The V4L2_CID_FRAMESIZE control determines maximum number >> of media bus samples transmitted within a

Re: [PATCH 04/10] mem2mem_testdev: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Sylwester Nawrocki
On 08/23/2012 03:49 PM, Sylwester Nawrocki wrote: > Hi Ezequiel, > > On 08/23/2012 03:08 PM, Ezequiel Garcia wrote: >> queue_init() is always called by v4l2_m2m_ctx_init(), which allocates >> a context struct v4l2_m2m_ctx with kzalloc. >> Therefore, there is no need to clear vb2_queue src/dst stru

Re: [PATCH 04/10] mem2mem_testdev: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Sylwester Nawrocki
Hi Ezequiel, On 08/23/2012 03:08 PM, Ezequiel Garcia wrote: > queue_init() is always called by v4l2_m2m_ctx_init(), which allocates > a context struct v4l2_m2m_ctx with kzalloc. > Therefore, there is no need to clear vb2_queue src/dst structs. > > Signed-off-by: Ezequiel Garcia Looks good to me

[PATCH 10/10] s5p-g2d: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Ezequiel Garcia
queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Kamil Debski Signed-off-by: Ezequiel Garcia --- drivers/media/platform/s5p-g2d/g2d.c |2 -- 1 files changed,

[PATCH 09/10] s5p-jpeg: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Ezequiel Garcia
queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Andrzej Pietrasiewicz Signed-off-by: Ezequiel Garcia --- drivers/media/platform/s5p-jpeg/jpeg-core.c |2 -- 1

[PATCH 08/10] s5p-fimc: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Ezequiel Garcia
queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Sylwester Nawrocki Signed-off-by: Ezequiel Garcia --- drivers/media/platform/s5p-fimc/fimc-m2m.c |2 -- 1 fil

[PATCH 07/10] mem2mem-emmaprp: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Ezequiel Garcia
queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Javier Martin Signed-off-by: Ezequiel Garcia --- drivers/media/platform/mx2_emmaprp.c |2 -- 1 files changed,

[PATCH 06/10] mem2mem-deinterlace: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Ezequiel Garcia
queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Javier Martin Signed-off-by: Ezequiel Garcia --- drivers/media/platform/m2m-deinterlace.c |2 -- 1 files chan

[PATCH 05/10] coda: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Ezequiel Garcia
queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Cc: Javier Martin Signed-off-by: Ezequiel Garcia --- drivers/media/platform/coda.c |2 -- 1 files changed, 0 inse

[PATCH 04/10] mem2mem_testdev: Remove unneeded struct vb2_queue clear on queue_init()

2012-08-23 Thread Ezequiel Garcia
queue_init() is always called by v4l2_m2m_ctx_init(), which allocates a context struct v4l2_m2m_ctx with kzalloc. Therefore, there is no need to clear vb2_queue src/dst structs. Signed-off-by: Ezequiel Garcia --- drivers/media/platform/mem2mem_testdev.c |2 -- 1 files changed, 0 insertions(+

[PATCH 03/10] vivi: Remove unneeded struct vb2_queue clearing

2012-08-23 Thread Ezequiel Garcia
struct vb2_queue is allocated through kzalloc as part of a larger struct, there's no need to clear it. Signed-off-by: Ezequiel Garcia --- drivers/media/platform/vivi.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/media/platform/vivi.c b/drivers/media/platform/

[PATCH 02/10] pwc: Remove unneeded struct vb2_queue clearing

2012-08-23 Thread Ezequiel Garcia
struct vb2_queue is allocated through kzalloc as part of a larger struct, there's no need to clear it. Cc: Hans de Goede Signed-off-by: Ezequiel Garcia --- drivers/media/usb/pwc/pwc-if.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/media/usb/pwc/pwc-if.c b/dr

[PATCH 01/10] stk1160: Remove unneeded struct vb2_queue clearing

2012-08-23 Thread Ezequiel Garcia
struct vb2_queue is allocated through kzalloc as part of a larger struct, there's no need to clear it. Signed-off-by: Ezequiel Garcia --- drivers/media/usb/stk1160/stk1160-v4l.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/drivers/media/usb/stk1160/stk1160-v4l.c b/dr

Re: [PATCH 1/1] driver-core: Shut up dev_dbg_reatelimited() without DEBUG

2012-08-23 Thread Hin-Tak Leung
--- On Wed, 22/8/12, Antti Palosaari wrote: > On 08/22/2012 04:57 PM, Hin-Tak Leung > wrote: > > Antti Palosaari wrote: > >> Hello Hiroshi, > >> > >> On 08/21/2012 10:02 AM, Hiroshi Doyu wrote: > >>> Antti Palosaari > wrote @ Mon, 20 Aug 2012 23:29:34 +0200: > >>> > On 08/20/2012 02:14 PM,

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

2012-08-23 Thread Sakari Ailus
Hi Sylwester, Thanks for the patch. On Thu, Aug 23, 2012 at 11:51:26AM +0200, Sylwester Nawrocki wrote: > The V4L2_CID_FRAMESIZE control determines maximum number > of media bus samples transmitted within a single data frame. > It is useful for determining size of data buffer at the > receiver si

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

2012-08-23 Thread Jean Pihet
Hi Timo, On Wed, Aug 22, 2012 at 9:50 PM, Timo Kokkonen wrote: > The ir-rx51 driver calls omap_pm_set_max_mpu_wakeup_lat() in order to > avoid problems that occur when MPU goes to sleep in the middle of > sending an IR code. Without such calls it takes ridiculously long for > the MPU to wake up f

Re: RFC: Core + Radio profile

2012-08-23 Thread Andy Walls
On Wed, 2012-08-22 at 15:09 -0300, Mauro Carvalho Chehab wrote: > Em 22-08-2012 12:19, Mike Isely escreveu: > > On Wed, 22 Aug 2012, Mauro Carvalho Chehab wrote: > > > >> Em 22-08-2012 07:11, Hans Verkuil escreveu: > >>> I've added some more core profile requirements. > >> > > Streaming

Re: [media-ctl PATCH 1/1] Count users for entities

2012-08-23 Thread Laurent Pinchart
Hi Sakari, Thanks for the patch. On Monday 20 August 2012 18:32:15 Sakari Ailus wrote: > The subdev nodes used to be closed immediately on v4l2_subdev_close(), now > the subdev is only closed if there are no users left anymore. This changes > the API from immediate effect (close) to a reference c

Re: [PATCH] media/radio/shark2: Fix build error caused by missing dependencies

2012-08-23 Thread Hans de Goede
Hi, On 08/22/2012 08:57 PM, Arnd Bergmann wrote: On Wednesday 22 August 2012, Guenter Roeck wrote: On Wed, Aug 22, 2012 at 05:22:26PM +0200, Hans de Goede wrote: Hi, I've a better fix for this here: http://git.linuxtv.org/hgoede/gspca.git/shortlog/refs/heads/media-for_v3.6 I already send a p

[PATCH RFC 4/4] s5p-fimc: Add support for V4L2_PIX_FMT_S5C_UYVY_JPG fourcc

2012-08-23 Thread Sylwester Nawrocki
This patch adds support for the interleaved JPEG/UYVY V4L2_PIX_FMT_S5C_UYVY_JPG image format. To ensure the size of allocated buffers is correct for a subdev configuration during VIDIOC_STREAMON ioctl, an additional check is added the video at the data pipeline validation routine. Flag FMT_FLAGS_

[PATCH RFC 3/4] V4L: Add V4L2_PIX_FMT_S5C_UYVY_JPG fourcc definition

2012-08-23 Thread Sylwester Nawrocki
The V4L2_PIX_FMT_S5C_UYVY_JPG is a single-planar image format generated by S5C73M3 camera. The image data consist of interleaved JPEG compressed data and raw UYVY data and the meta data appended at the end. The meta-data is a list of offsets to UYVY data blocks of fixed size, which may be followed

[PATCH RFC 2/4] V4L: Add V4L2_MBUS_FMT_S5C_UYVY_JPEG_1X8 media bus format

2012-08-23 Thread Sylwester Nawrocki
This patch adds media bus pixel code for the interleaved JPEG/UYVY image format used by S5C73MX Samsung cameras. This interleaved image data is transferred on MIPI-CSI2 bus as User Defined Byte-based Data. It also defines an experimental vendor and device specific media bus formats section and add

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

2012-08-23 Thread Sylwester Nawrocki
The V4L2_CID_FRAMESIZE control determines maximum number of media bus samples transmitted within a single data frame. It is useful for determining size of data buffer at the receiver side. Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- Documentation/DocBook/media/v4l/control

[PATCH RFC 0/4] V4L2: Vendor specific media bus formats/ frame size control

2012-08-23 Thread Sylwester Nawrocki
This patch series introduces new image source class control - V4L2_CID_FRAMESIZE and vendor or device specific media bus format section. There was already a discussion WRT handling interleaved image data [1]. I'm not terribly happy with those vendor specific media bus formats but I couldn't find b

[PATCH v2] s5p-fimc: Add pipeline ops to separate FIMC-LITE module

2012-08-23 Thread Sylwester Nawrocki
In order to reuse the FIMC-LITE module on Exynos4 and Exynos5 SoC introduce a set of callbacks for the media pipeline control from within FIMC/FIMC-LITE video node. It lets us avoid symbol dependencies between FIMC-LITE and the whole media device driver, which simplifies the initialization sequence