RE: [PATCH 8/8] v4l: videobuf2: add CMA allocator

2010-12-08 Thread Sewoon Park
Hello Marek Szyprowski Wrote: > Sent: Monday, December 06, 2010 7:53 PM > To: linux-media@vger.kernel.org > Cc: m.szyprow...@samsung.com; pa...@osciak.com; kyungmin.p...@samsung.com; > andrze...@samsung.com > Subject: [PATCH 8/8] v4l: videobuf2: add CMA allocator > +struct vb2_cma_buf { > +

RE: [PATCH 2/8] v4l: videobuf2: add generic memory handling routines

2010-12-07 Thread Sewoon Park
Hello~ There are a lot of code changes compared with version5 patch. Some minor comments below. Marek Szyprowski Wrote : > Sent: Monday, December 06, 2010 7:53 PM > To: linux-media@vger.kernel.org > Cc: m.szyprow...@samsung.com; pa...@osciak.com; kyungmin.p...@samsung.com; > andrze...@samsung.com

RE: [PATCH 7/7] v4l: videobuf2: add CMA allocator

2010-11-24 Thread Sewoon Park
Hi~ Simply comments below Marek Szyprowski Saturday, November 20, 2010 12:56 AM Wrote: snip > +/* > + * videobuf2-cma.h - CMA memory allocator for videobuf2 > + * > + * Copyright (C) 2010 Samsung Electronics > + * > + * Author: Pawel Osciak > + * > + * This program is free software; you can redi

RE: [PATCH 1/7] v4l: add videobuf2 Video for Linux 2 driver framework

2010-11-22 Thread Sewoon Park
Hi~ Marek. Some comments in here. Saturday, November 20, 2010 12:56 AM Marek Szyprowski wrote : > To: linux-media@vger.kernel.org > Cc: m.szyprow...@samsung.com; pa...@osciak.com; kyungmin.p...@samsung.com > Subject: [PATCH 1/7] v4l: add videobuf2 Video for Linux 2 driver framework > > From: Paw

RE: [PATCH 4/7] v4l: videobuf2: add DMA coherent allocator

2010-11-17 Thread Sewoon Park
Hi Marek~ Marek Szyprowski wrote: > Sent: Wednesday, November 17, 2010 5:40 PM > To: linux-media@vger.kernel.org > Cc: m.szyprow...@samsung.com; pa...@osciak.com; kyungmin.p...@samsung.com > Subject: [PATCH 4/7] v4l: videobuf2: add DMA coherent allocator > > From: Pawel Osciak > > Add an imple

RE: [PATCH 5/6 v5] V4L/DVB: s5p-fimc: Add camera capture support

2010-10-21 Thread Sewoon Park
Latest your reply is easy to understand. And I send you another parts review comments. Tuesday, October 12, 2010 2:27, Sylwester Nawrocki wrote : > > Add a video device driver per each FIMC entity to support > the camera capture input mode. Video capture node is registered > only if CCD sensor da