URGENT RESPONSE NEEDED

2018-10-12 Thread Sean Kim.
Hello my dear. Did you receive my email message to you? Please, get back to me ASAP as the matter is becoming late. Expecting your urgent response. Sean.

Re: [Xen-devel][RFC 3/3] xen/gntdev: Add support for Linux dma buffers

2018-05-21 Thread Dongwon Kim
Still need more time to review the whole code changes but I noticed one thing. We've been using the term "hyper_dmabuf" for hypervisor-agnostic linux dmabuf solution and we are planning to call any of our future solution for other hypervisors the same name. So having same name for this

buffer sharing across VMs - xen-zcopy and hyper_dmabuf discussion

2018-04-12 Thread Dongwon Kim
(changed subject and decoupling from udmabuf thread) On Wed, Apr 11, 2018 at 08:59:32AM +0300, Oleksandr Andrushchenko wrote: > On 04/10/2018 08:26 PM, Dongwon Kim wrote: > >On Tue, Apr 10, 2018 at 09:37:53AM +0300, Oleksandr Andrushchenko wrote: > >>On 04/06/2018 09:57 PM,

Re: [RfC PATCH] Add udmabuf misc device

2018-04-10 Thread Dongwon Kim
On Tue, Apr 10, 2018 at 09:37:53AM +0300, Oleksandr Andrushchenko wrote: > On 04/06/2018 09:57 PM, Dongwon Kim wrote: > >On Fri, Apr 06, 2018 at 03:36:03PM +0300, Oleksandr Andrushchenko wrote: > >>On 04/06/2018 02:57 PM, Gerd Hoffmann wrote: > >>> Hi, > &g

Re: [RfC PATCH] Add udmabuf misc device

2018-04-06 Thread Dongwon Kim
On Fri, Apr 06, 2018 at 03:36:03PM +0300, Oleksandr Andrushchenko wrote: > On 04/06/2018 02:57 PM, Gerd Hoffmann wrote: > > Hi, > > > >>>I fail to see any common ground for xen-zcopy and udmabuf ... > >>Does the above mean you can assume that xen-zcopy and udmabuf > >>can co-exist as two

Re: [PATCH v3 1/2] staging: media: davinci_vpfe: add error handling on kmalloc failure

2018-03-26 Thread Ji-Hun Kim
On Wed, Mar 21, 2018 at 01:39:09PM +0900, Ji-Hun Kim wrote: > There is no failure checking on the param value which will be allocated > memory by kmalloc. Add a null pointer checking statement. Then goto error: > and return -ENOMEM error code when kmalloc is failed. > > Signed-off

[PATCH v3 2/2] staging: media: davinci_vpfe: add kfree() on goto err statement

2018-03-20 Thread Ji-Hun Kim
It needs to free of allocated params value in the goto error statement. Signed-off-by: Ji-Hun Kim <ji_hun@samsung.com> --- Changes since v2: - add kfree(params) on the error case of the function - rename unclear goto statement name - declare the params value at start of the functi

[PATCH v3 1/2] staging: media: davinci_vpfe: add error handling on kmalloc failure

2018-03-20 Thread Ji-Hun Kim
There is no failure checking on the param value which will be allocated memory by kmalloc. Add a null pointer checking statement. Then goto error: and return -ENOMEM error code when kmalloc is failed. Signed-off-by: Ji-Hun Kim <ji_hun@samsung.com> --- Changes since v1: - Return with -

[PATCH v2] staging: media: davinci_vpfe: add error handling on kmalloc failure

2018-03-18 Thread Ji-Hun Kim
There is no failure checking on the param value which will be allocated memory by kmalloc. Add a null pointer checking statement. Then goto error: and return -ENOMEM error code when kmalloc is failed. Signed-off-by: Ji-Hun Kim <ji_hun@samsung.com> --- Changes since v1: - Return with -

Re: Re: [PATCH] staging: media: davinci_vpfe: add error handling on kmalloc failure

2018-03-18 Thread Ji-Hun Kim
On Fri, Mar 16, 2018 at 11:32:34AM +0300, Dan Carpenter wrote: > On Fri, Mar 16, 2018 at 01:58:23PM +0900, Ji-Hun Kim wrote: > > There is no failure checking on the param value which will be allocated > > memory by kmalloc. Add a null pointer checking statement. Then goto error:

[PATCH] staging: media: davinci_vpfe: add error handling on kmalloc failure

2018-03-15 Thread Ji-Hun Kim
There is no failure checking on the param value which will be allocated memory by kmalloc. Add a null pointer checking statement. Then goto error: and return -ENOMEM error code when kmalloc is failed. Signed-off-by: Ji-Hun Kim <ji_hun@samsung.com> --- drivers/staging/media/davinc

Re: [PATCH 1/2] media: usb: uvc: remove duplicate & operation

2017-11-05 Thread Jaejoong Kim
Hi, Laurent Could you please review this patch? Thanks, jaejoong 2017-10-20 16:25 GMT+09:00 Jaejoong Kim <climbbb@gmail.com>: > usb_endpoint_maxp() has an inline keyword and searches for bits[10:0] > by & operation with 0x7ff. So, we can remove the duplicate & op

Re: [PATCH] media: av7110: switch to useing timer_setup()

2017-10-24 Thread Jaejoong Kim
Hi, [PATCH] media: av7110: switch to useing timer_setup() ^^^ typo error. 2017-10-25 9:40 GMT+09:00 Dmitry Torokhov : > In preparation for unconditionally passing the struct timer_list pointer to > all timer

[PATCH 1/2] media: usb: uvc: remove duplicate & operation

2017-10-20 Thread Jaejoong Kim
usb_endpoint_maxp() has an inline keyword and searches for bits[10:0] by & operation with 0x7ff. So, we can remove the duplicate & operation with 0x7ff. Signed-off-by: Jaejoong Kim <climbbb@gmail.com> --- drivers/media/usb/uvc/uvc_video.c | 4 ++-- 1 file changed, 2 in

[PATCH 2/2] media: usb: usbtv: remove duplicate & operation

2017-10-20 Thread Jaejoong Kim
usb_endpoint_maxp() has an inline keyword and searches for bits[10:0] by & operation with 0x7ff. So, we can remove the duplicate & operation with 0x7ff. Signed-off-by: Jaejoong Kim <climbbb@gmail.com> --- drivers/media/usb/usbtv/usbtv-core.c | 2 +- 1 file changed, 1 insertion(

[PATCH 0/2] media: usb: remove duplicate & operation

2017-10-20 Thread Jaejoong Kim
usb_endpoint_maxp() has an inline keyword and searches for bits[10:0] by & operation with 0x7ff. So, we can remove the duplicate & operation with 0x7ff. Jaejoong Kim (2): media: usb: uvc: remove duplicate & operation media: usb: usbtv: remove duplicate & operation drivers

[PATCH] [media] uvcvideo: change result code of debugfs_init to void

2017-01-11 Thread Jaejoong Kim
The device driver should keep going even if debugfs initialization fails. So, change the return type to void. Signed-off-by: Jaejoong Kim <climbbb@gmail.com> --- drivers/media/usb/uvc/uvc_debugfs.c | 15 ++- drivers/media/usb/uvc/uvcvideo.h| 4 ++-- 2 files chan

[PATCH 1/2] s5p-mfc: fix spelling errors

2015-10-02 Thread Ingi Kim
This patch fixes spelling errors in mfc encoder. inavild -> invaild Signed-off-by: Ingi Kim <ingi2@samsung.com> --- drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c

Re: [PATCH 4/4] ARM64: dts: exynos5433: add jpeg node

2015-09-30 Thread Kukjin Kim
On 09/28/15 21:02, Krzysztof Kozlowski wrote: > W dniu 28.09.2015 o 20:54, Bartlomiej Zolnierkiewicz pisze: >> >> Hi, >> >> On Monday, September 28, 2015 05:52:13 PM Krzysztof Kozlowski wrote: >>> W dniu 21.09.2015 o 18:59, Andrzej Pietrasiewicz pisze: Hi Hans, W dniu 21.09.2015 o

[PATCH] s5p-mfc: fix state check from encoder queue_setup

2015-05-13 Thread Seung-Woo Kim
from queue_setup. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/platform/s5p-mfc/s5p_mfc_enc.c |9 + 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c

Re: ioremap_uc() followed by set_memory_wc() - burrying MTRR

2015-04-17 Thread Hyong-Youb Kim
On Thu, Apr 16, 2015 at 08:54:26PM +0200, Luis R. Rodriguez wrote: Without WC, descriptors would end up as multiple 4B or 8B MWr packets to the NIC, which has a pretty big performance impact on this particular NIC. How big are the descriptors? Some are 64B (a batch of eight 8B

Re: ioremap_uc() followed by set_memory_wc() - burrying MTRR

2015-04-15 Thread Hyong-Youb Kim
On Thu, Apr 16, 2015 at 01:58:16AM +0200, Luis R. Rodriguez wrote: An alternative... is to just ioremap_wc() the entire region, including MMIO registers for these old devices. I see one ethernet driver that does this, myri10ge, and am curious how and why they ended up deciding this and if

RE: [PATCH] Documentation/dma-buf-sharing.txt: update API descriptions

2014-05-13 Thread Gioh Kim
this be the same as the name and email used in the Signed-off-by line below. update some descriptions for API arguments and descriptions. Nit: Update since it's the beginning of a sentence. Signed-off-by: Gioh Kim gioh@lge.com --- Documentation/dma-buf-sharing.txt | 10 ++ 1

[PATCH] Documentation/dma-buf-sharing.txt: update API descriptions

2014-05-13 Thread Gioh Kim
Update some descriptions for API arguments and descriptions. Signed-off-by: Gioh Kim gioh@lge.com --- Documentation/dma-buf-sharing.txt | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/Documentation/dma-buf-sharing.txt b/Documentation/dma-buf-sharing.txt

RE: [PATCH 1/5] ARM: S5PV210: Remove camera support from mach-goni.c

2014-05-02 Thread Kukjin Kim
is applied first. Signed-off-by: Sylwester Nawrocki s.nawro...@samsung.com Acked-by: Kyungmin Park kyungmin.p...@samsung.com Hi Sylwester, Cleanup is always welcome ;-) I think, when this series is ready for mainline, this will be handled in media tree. So, Acked-by: Kukjin Kim kgene

Re: [Linaro-mm-sig] [PATCH] dma-buf: add meta data attachment

2014-03-25 Thread Seung-Woo Kim
really be added as part of the dma_buf_attachment, and not separately. This is basic concept of my RFC patch. http://www.kernelhub.org/?p=2msg=268056 Best Regards, - Seung-Woo Kim The meta-data here can be any kind of data related to the dmabuf, , it's specific for each dmabuf. For example

[PATCH] [media] s5-mfc: remove meaningless assignment

2014-03-05 Thread Seung-Woo Kim
Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c b/drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c index 2475a3c..ee05f2d

[PATCH] [media] s5-mfc: remove meaningless memory bank assignment

2014-03-05 Thread Seung-Woo Kim
There was assignment of memory bank with dma address converted from physical address. But allocation has been changed with dma function, so the assignment is not necessary. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- change from v1 - fixes subject and adds proper description

Re: [PATCH] [media] s5-mfc: remove meaningless memory bank assignment

2014-03-05 Thread Seung-Woo Kim
Hello Sachin, On 2014년 03월 05일 20:42, Sachin Kamat wrote: On 5 March 2014 16:38, Seung-Woo Kim sw0312@samsung.com wrote: (...) - dev-bank1 = dev-bank1; Are you sure this isn't some kind of typo? If not then your commit description is too verbose to actually say that the code

Re: [PATCH] [media] s5-mfc: remove meaningless memory bank assignment

2014-03-05 Thread Seung-Woo Kim
Hello Sachin, On 2014년 03월 06일 12:28, Sachin Kamat wrote: Hi Seung-Woo, On 6 March 2014 07:43, Seung-Woo Kim sw0312@samsung.com wrote: Hello Sachin, On 2014년 03월 05일 20:42, Sachin Kamat wrote: On 5 March 2014 16:38, Seung-Woo Kim sw0312@samsung.com wrote: (...) - dev

I HAVE A PROPOSAL FOR YOU!

2014-01-28 Thread Yung kim
and you are not expected to invest in Iraq If yes, let me know and we will discuss this project proper. yung kim. yung@qq.com -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http

Re: DMABUF doesn't work when frame size not equal to the size of GPU bo

2014-01-07 Thread Seung-Woo Kim
queue_setup in the __qbuf_dmabuf() like __vb2_buf_mem_alloc() or vb2_mmap(). Regards, - Seung-Woo Kim The fd in v4l2_buffer structure is fd of gem object created by DRM_IOCTL_MODE_CREATE_DUMB. I've upload the program that can reproduce this issue on intel platform. You just need to clone

Exportation

2013-12-13 Thread Kim
and you are not expected to invest in Iraq If yes, let me know and we will discuss this project proper. Kim. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo

[PATCH] [media] videobuf2: Add log for size checking error in __qbuf_dmabuf

2013-11-28 Thread Seung-Woo Kim
__qbuf_dmabuf checks whether size of provided dmabuf is large enough, and it returns error without any log. So this patch adds error log in the case. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/v4l2-core/videobuf2-core.c |2 ++ 1 files changed, 2 insertions(+), 0

Re: [media-workshop] V2: Agenda for the Edinburgh mini-summit

2013-10-16 Thread Milo Kim
Hi Bryan, On 10/17/2013 02:17 AM, Bryan Wu wrote: On Tue, Oct 15, 2013 at 6:49 PM, Milo Kim milo@ti.com wrote: Hi Bryan, On 10/16/2013 03:37 AM, Bryan Wu wrote: On Fri, Oct 11, 2013 at 12:38 AM, Laurent Pinchart laurent.pinch...@ideasonboard.com wrote: Hi Bryan, On Thursday 10

Re: [media-workshop] V2: Agenda for the Edinburgh mini-summit

2013-10-15 Thread Milo Kim
by Milo Kim. This trigger will provide flashing and strobing control of a LED device and for sure the LED device driver like drivers/leds/leds-lm355x.c. So we basically can do this: 1. add V4L2 Flash subdev into ledtrig-camera.c. So this trigger driver can provide trigger API to kernel drivers

Re: [PATCH] s5p-jpeg: fix uninitialized use in hdr parse

2013-10-13 Thread Seung-Woo Kim
Hi Sylwester, Thanks for you comment. On 2013년 10월 12일 18:39, Sylwester Nawrocki wrote: Hi Seung-Woo, On 10/10/2013 09:06 AM, Seung-Woo Kim wrote: For hdr parse error, it can return false without any assignments which cause build warning. Signed-off-by: Seung-Woo Kimsw0312

[PATCH v2] s5p-jpeg: fix uninitialized use in hdr parse

2013-10-13 Thread Seung-Woo Kim
drivers/media/platform/s5p-jpeg/jpeg-core.c:433: warning: 'height' may be used uninitialized in this function drivers/media/platform/s5p-jpeg/jpeg-core.c:433: warning: 'width' may be used uninitialized in this function Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- change from v1 - add

[PATCH] s5p-jpeg: fix uninitialized use in hdr parse

2013-10-10 Thread Seung-Woo Kim
For hdr parse error, it can return false without any assignments which cause build warning. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/platform/s5p-jpeg/jpeg-core.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/s5p

[PATCH] s5p-jpeg: fix encoder and decoder video dev names

2013-10-10 Thread Seung-Woo Kim
It is hard to distinguish between decoder and encoder video device because their names are same. So this patch fixes the names. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/platform/s5p-jpeg/jpeg-core.c |8 1 files changed, 4 insertions(+), 4 deletions

Re: [PATCH V5 1/5] ARM: dts: Add MIPI PHY node to exynos4.dtsi

2013-10-05 Thread Kukjin Kim
On 10/02/13 00:26, Kishon Vijay Abraham I wrote: On Wednesday 02 October 2013 02:43 AM, Sylwester Nawrocki wrote: On 10/01/2013 07:28 AM, Kishon Vijay Abraham I wrote: On Sunday 29 September 2013 12:57 AM, Sylwester Nawrocki wrote: Add PHY provider node for the MIPI CSIS and MIPI DSIM

Re: [PATCH V5 1/5] ARM: dts: Add MIPI PHY node to exynos4.dtsi

2013-10-05 Thread Kukjin Kim
On 10/06/13 09:09, Kukjin Kim wrote: On 10/02/13 00:26, Kishon Vijay Abraham I wrote: On Wednesday 02 October 2013 02:43 AM, Sylwester Nawrocki wrote: On 10/01/2013 07:28 AM, Kishon Vijay Abraham I wrote: On Sunday 29 September 2013 12:57 AM, Sylwester Nawrocki wrote: Add PHY provider node

RE: [PATCH 1/5] ARM: Kconfig: Move if ARCH_S3C64XX statement to mach-s3c64xx/Kconfig

2013-09-30 Thread Kukjin Kim
Tomasz Figa wrote: All other platforms have this condition checked inside their own Kconfig files, so for consistency this patch makes it this way for mach-s3c64xx as well. Signed-off-by: Tomasz Figa tomasz.f...@gmail.com --- arch/arm/Kconfig | 2 --

[PATCH] media: vb2: add log for size checking error in __qbuf_userptr

2013-08-20 Thread Seung-Woo Kim
__qbuf_userptr checks whether provided buffer is large enough, and it returns error without any log. Signed-off-by: Seung-Woo Kim sw0312@samsung.com Signed-off-by: Heejin Woo heejin@samsung.com --- drivers/media/v4l2-core/videobuf2-core.c |4 1 files changed, 4 insertions(+), 0

RE: [PATCH V6 1/4] ARM: dts: Add DP PHY node to exynos5250.dtsi

2013-08-14 Thread Kukjin Kim
Jingoo Han wrote: Add PHY provider node for the DP PHY. Signed-off-by: Jingoo Han jg1@samsung.com Reviewed-by: Tomasz Figa t.f...@samsung.com Acked-by: Felipe Balbi ba...@ti.com Acked-by: Kukjin Kim kgene@samsung.com Thanks, Kukjin --- arch/arm/boot/dts/exynos5250.dtsi | 13

Re: [PATCH 2/2] media: s5p-mfc: remove DT hacks and simplify initialization code

2013-08-06 Thread Kukjin Kim
On 08/06/13 19:22, Kamil Debski wrote: Hi Kukjin, This patch looks good. Best wishes, Kamil Debski From: Marek Szyprowski [mailto:m.szyprow...@samsung.com] Sent: Monday, August 05, 2013 2:27 PM This patch removes custom initialization of reserved memory regions from s5p-mfc driver. Memory

Re: [PATCH 0/2] Exynos: MFC: clean up device tree bindings

2013-08-05 Thread Kukjin Kim
On 08/05/13 21:26, Marek Szyprowski wrote: Hello, Hi, This patch series are an attempt to cleanup the reserved memory device tree bindings for MFC device. MFC device has two memory ports (AXI masters), which are used to do DMA. Usually separate memory regions are being defined for each of

RE: [PATCH v3 5/5] ARM: Samsung: Remove the MIPI PHY setup code

2013-06-27 Thread Kukjin Kim
- -- 5 files changed, 75 deletions(-) delete mode 100644 arch/arm/plat-samsung/setup-mipiphy.c Looks good, please feel free to add my ack on this, Acked-by: Kukjin Kim kgene@samsung.com Thanks, - Kukjin -- To unsubscribe from this list: send the line unsubscribe linux-media in the body

RE: [PATCH v2 32/38] [media] exynos4-is: Remove check for SOC_EXYNOS4412

2013-06-18 Thread Kukjin Kim
Tomasz Figa wrote: Since SOC_EXYNOS4412 Kconfig symbol has been removed, it is enough to check for SOC_EXYNOS4212 for both SoCs from Exynos4x12 series. Cc: linux-media@vger.kernel.org Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: Sylwester Nawrocki s.nawro...@samsung.com

Re: [RFC PATCH 5/5] ARM: Samsung: Remove MIPI PHY setup code

2013-06-16 Thread Kukjin Kim
On 06/15/13 02:45, Sylwester Nawrocki wrote: Generic PHY drivers are used to handle the MIPI CSIS and MIPI DSIM DPHYs so we can remove now unused code at arch/arm/plat-samsung. If so, sounds good :) In case there is any board file for S5PV210 platforms using MIPI CSIS/DSIM (not any upstream

RE: [PATCH 27/28] ARM: EXYNOS: Remove CONFIG_SOC_EXYNOS4412

2013-06-14 Thread Kukjin Kim
Tomasz Figa wrote: Exynos4212 and Exynos4412 SoCs differ only in number of ARM cores and there is no need to have separate Kconfig options for them, since they use the same code. This patch removes CONFIG_SOC_EXYNOS4412, leaving CONFIG_SOC_EXYNOS4212 as the one supporting both SoCs from

PROTECTED PROJECT

2013-06-11 Thread Kim
client and you are not expected to invest in Iraq If yes, let me know and we will discuss this project proper. Email:k...@attorneykim.nazuka.net Kim. -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord...@vger.kernel.org More majordomo info

[RFC][PATCH 0/2] dma-buf: add importer private data for reimporting

2013-05-31 Thread Seung-Woo Kim
importer private data in dma-buf attachment can be used by importer to reimport same dma-buf. Seung-Woo Kim (2): dma-buf: add importer private data to attachment drm/prime: find gem object from the reimported dma-buf drivers/base/dma-buf.c | 31

[RFC][PATCH 2/2] drm/prime: find gem object from the reimported dma-buf

2013-05-31 Thread Seung-Woo Kim
be used at reimport time if it is assigned with drm gem object at first import. This can also remove remapping dma address for the hardware having its own iommu. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/gpu/drm/drm_prime.c| 19 ++- drivers/gpu

[RFC][PATCH 1/2] dma-buf: add importer private data to attachment

2013-05-31 Thread Seung-Woo Kim
-by: Seung-Woo Kim sw0312@samsung.com --- drivers/base/dma-buf.c | 31 +++ include/linux/dma-buf.h |4 2 files changed, 35 insertions(+), 0 deletions(-) diff --git a/drivers/base/dma-buf.c b/drivers/base/dma-buf.c index 08fe897..a1eaaf2 100644

[RESEND][PATCH 0/2] media: fix polling not to wait if a buffer is available

2013-05-20 Thread Seung-Woo Kim
() or queue is cancelled. So I add check routine for done_list before calling poll_wait(). But I'm not sure that locking for done_lock of queue is also needed in this case or not because done_list of queue is checked without locking in some other parts of vb2. Seung-Woo Kim (2): media: vb2: return

[RESEND][PATCH 1/2] media: vb2: return for polling if a buffer is available

2013-05-20 Thread Seung-Woo Kim
The vb2_poll() does not need to wait next vb_buffer_done() if there is already a buffer in done_list of queue, but current vb2_poll() always waits. So done_list is checked before calling poll_wait(). Signed-off-by: Seung-Woo Kim sw0312@samsung.com Acked-by: Marek Szyprowski m.szyprow

[RESEND][PATCH 2/2] media: v4l2-mem2mem: return for polling if a buffer is available

2013-05-20 Thread Seung-Woo Kim
The v4l2_m2m_poll() does not need to wait if there is already a buffer in done_list of source and destination queues, but current v4l2_m2m_poll() always waits. So done_list of each queue is checked before calling poll_wait(). Signed-off-by: Seung-Woo Kim sw0312@samsung.com Acked-by: Marek

RE: [RFC 0/2] V4L2 API for exposing flash subdevs as LED class device

2013-05-06 Thread Kim, Milo
Hi Andrzej, -Original Message- From: linux-leds-ow...@vger.kernel.org [mailto:linux-leds- ow...@vger.kernel.org] On Behalf Of Andrzej Hajda Sent: Monday, May 06, 2013 6:34 PM To: linux-media@vger.kernel.org; linux-l...@vger.kernel.org; devicetree-disc...@lists.ozlabs.org Cc:

[PATCH] media: vb2: add length check for mmap

2013-04-11 Thread Seung-Woo Kim
The length of mmap() can be bigger than length of vb2 buffer, so it should be checked. Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/v4l2-core/videobuf2-core.c |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/drivers/media/v4l2-core/videobuf2

[RFC][PATCH 0/2] media: fix polling not to wait if a buffer is available

2013-04-01 Thread Seung-Woo Kim
() or queue is cancelled. So I add check routine for done_list before calling poll_wait(). But I'm not sure that locking for done_lock of queue is also needed in this case or not because done_list of queue is checked without locking in some other parts of vb2. Seung-Woo Kim (2): media: vb2: return

[RFC][PATCH 1/2] media: vb2: return for polling if a buffer is available

2013-04-01 Thread Seung-Woo Kim
The vb2_poll() does not need to wait next vb_buffer_done() if there is already a buffer in done_list of queue, but current vb2_poll() always waits. So done_list is checked before calling poll_wait(). Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media/v4l2-core/videobuf2-core.c

[RFC][PATCH 2/2] media: v4l2-mem2mem: return for polling if a buffer is available

2013-04-01 Thread Seung-Woo Kim
The v4l2_m2m_poll() does not need to wait if there is already a buffer in done_list of source and destination queues, but current v4l2_m2m_poll() always waits. So done_list of each queue is checked before calling poll_wait(). Signed-off-by: Seung-Woo Kim sw0312@samsung.com --- drivers/media

RE: [PATCH 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-02-01 Thread Kukjin Kim
Sylwester Nawrocki wrote: On 02/01/2013 09:33 AM, Sachin Kamat wrote: On 1 February 2013 06:57, Inki Dae inki@samsung.com wrote: For example, If compatible = samsung,g2d-3.0 is added to exynos4210.dtsi, it'd be reasonable. But what if that compatible string is added to

RE: [PATCH 5/5] s5p-fimc: Redefine platform data structure for fimc-is

2013-02-01 Thread Kukjin Kim
enumeration is extended to include both FIMC Writeback input types. The bus_type enumeration and the data structure name in the board files are modified according to the above changes. Cc: Kukjin Kim kgene@samsung.com Cc: linux-samsung-...@vger.kernel.org Signed-off

RE: [PATCH 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-02-01 Thread Kukjin Kim
Kukjin Kim wrote: Oops, I'm re-sending due to my e-mail client problem :-( Sylwester Nawrocki wrote: On 02/01/2013 09:33 AM, Sachin Kamat wrote: On 1 February 2013 06:57, Inki Dae inki@samsung.com wrote: For example, If compatible = samsung,g2d-3.0 is added to exynos4210

RE: [PATCH 2/2] drm/exynos: Add device tree based discovery support for G2D

2013-01-31 Thread Kukjin Kim
Sylwester Nawrocki wrote: Hi Inki, Hi Sylwester and Inki, On 01/31/2013 02:30 AM, Inki Dae wrote: -Original Message- From: Sylwester Nawrocki [mailto:sylvester.nawro...@gmail.com] Sent: Thursday, January 31, 2013 5:51 AM To: Inki Dae Cc: Sachin Kamat;

Re: [PATCH] drivers: cma: don't fail if migration returns -EAGAIN

2012-05-10 Thread Minchan Kim
On Thu, May 10, 2012 at 03:19:35PM +0200, Marek Szyprowski wrote: alloc_contig_range() function might return -EAGAIN if migrate_pages() call migrate_page never return -EAGAIN and I can't find any -EAGAIN return in alloc_contig_range. Am I seeing different tree? fails for some temporarily

Re: [RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control

2012-01-01 Thread Kim, Heungjun
On 2011년 12월 30일 19:30, Sylwester Nawrocki wrote: Hi HeungJun, On 12/29/2011 06:08 AM, HeungJun, Kim wrote: -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Wednesday, December 28, 2011 10:52 PM

Re: [RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control

2011-12-29 Thread Kim, Heungjun
Hi, On 2011년 12월 30일 08:58, Laurent Pinchart wrote: Hi, On Thursday 29 December 2011 06:08:07 HeungJun, Kim wrote: On Wednesday, December 28, 2011 10:52 PM Laurent Pinchart wrote: On Wednesday 28 December 2011 14:35:00 Sylwester Nawrocki wrote: On 12/28/2011 07:23 AM, HeungJun, Kim wrote

RE: [RFC PATCH 2/4] v4l: Add V4L2_CID_SCENEMODE menu control

2011-12-29 Thread HeungJun, Kim
Hi, -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Friday, December 30, 2011 9:11 AM To: HeungJun, Kim Cc: linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl; sakari.ai

RE: [RFC PATCH 3/4] v4l: Add V4L2_CID_WDR button control

2011-12-29 Thread HeungJun, Kim
Hi, -Original Message- From: Laurent Pinchart [mailto:laurent.pinch...@ideasonboard.com] Sent: Friday, December 30, 2011 9:13 AM To: HeungJun, Kim Cc: linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl; sakari.ai...@iki.fi; s.nawro...@samsung.com; kyungmin.p

RE: [RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control

2011-12-29 Thread HeungJun, Kim
, December 30, 2011 8:34 AM To: Laurent Pinchart Cc: Sylwester Nawrocki; HeungJun, Kim; linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl; kyungmin.p...@samsung.com; Hans de Goede Subject: Re: [RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control Hi Laurent

RE: [RFC PATCH 0/4] Add some new camera controls

2011-12-29 Thread HeungJun, Kim
Hi, -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Friday, December 30, 2011 9:16 AM To: HeungJun, Kim Cc: linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl; sakari.ai

RE: [RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control

2011-12-28 Thread HeungJun, Kim
To: HeungJun, Kim Cc: linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl; sakari.ai...@iki.fi; laurent.pinch...@ideasonboard.com; kyungmin.p...@samsung.com; Hans de Goede Subject: Re: [RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control Hi, On 12/28/2011 07:23

RE: [RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control

2011-12-28 Thread HeungJun, Kim
Hi Laurent and Sylwester, -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Wednesday, December 28, 2011 10:52 PM To: Sylwester Nawrocki Cc: HeungJun, Kim; linux-media@vger.kernel.org; mche

RE: [RFC PATCH 2/4] v4l: Add V4L2_CID_SCENEMODE menu control

2011-12-28 Thread HeungJun, Kim
Hi Laurent, -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Wednesday, December 28, 2011 10:56 PM To: HeungJun, Kim Cc: linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl

RE: [RFC PATCH 3/4] v4l: Add V4L2_CID_WDR button control

2011-12-28 Thread HeungJun, Kim
Hi, -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Wednesday, December 28, 2011 10:57 PM To: HeungJun, Kim Cc: linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl; sakari.ai

RE: [RFC PATCH 4/4] v4l: Add V4L2_CID_ANTISHAKE button control

2011-12-28 Thread HeungJun, Kim
Hi, -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Wednesday, December 28, 2011 10:58 PM To: HeungJun, Kim Cc: linux-media@vger.kernel.org; mche...@redhat.com; hverk...@xs4all.nl; sakari.ai

RE: [RFC PATCH 0/4] Add some new camera controls

2011-12-28 Thread HeungJun, Kim
Hi Larent, Thanks for the comments very well, and I replied the other each mails. -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Laurent Pinchart Sent: Wednesday, December 28, 2011 11:01 PM To: HeungJun, Kim Cc

[RFC PATCH 0/4] Add some new camera controls

2011-12-27 Thread HeungJun, Kim
Hi everyone, This RFC patch series include new 4 controls ID for digital camera. I about to suggest these controls by the necessity enabling the M-5MOLS sensor's function, and I hope to discuss this in here. Any opinions and thoughts are very welcome! It's good to connect Sylwester's suggestion

[RFC PATCH 1/4] v4l: Add V4L2_CID_PRESET_WHITE_BALANCE menu control

2011-12-27 Thread HeungJun, Kim
-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- Documentation/DocBook/media/v4l/controls.xml | 38 ++ drivers/media/video/v4l2-ctrls.c | 12 include/linux/videodev2.h|9

[RFC PATCH 2/4] v4l: Add V4L2_CID_SCENEMODE menu control

2011-12-27 Thread HeungJun, Kim
= 12, V4L2_SCENEMODE_TEXT = 13, V4L2_SCENEMODE_CANDLE = 14, }; Signed-off-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- Documentation/DocBook/media/v4l/controls.xml | 88 ++ drivers/media/video/v4l2

[RFC PATCH 3/4] v4l: Add V4L2_CID_WDR button control

2011-12-27 Thread HeungJun, Kim
It adds the new CID for setting White Balance Preset. This CID is provided as button type. This can commands only if the camera turn on/off this function. Signed-off-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- Documentation/DocBook/media

[RFC PATCH 4/4] v4l: Add V4L2_CID_ANTISHAKE button control

2011-12-27 Thread HeungJun, Kim
It adds the new CID for setting Anti-shake. This CID is provided as button type. This can commands only if the camera turn on/off this function. Signed-off-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- Documentation/DocBook/media/v4l

RE: [RFC PATCH v1 6/7] media: video: introduce face detection driver module

2011-12-12 Thread HeungJun, Kim
this with OMAP FDIF. Thank you. Regards, Heungjun Kim -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Ming Lei Sent: Monday, December 12, 2011 6:50 PM To: Sylwester Nawrocki Cc: linux-o...@vger.kernel.org; linux-arm

RE: [RFC PATCH v1 6/7] media: video: introduce face detection driver module

2011-12-12 Thread HeungJun, Kim
Hi Ming and Sylwester, Thanks for the reply. -Original Message- From: Ming Lei [mailto:ming@canonical.com] Sent: Tuesday, December 13, 2011 1:02 PM To: HeungJun, Kim Cc: Sylwester Nawrocki; linux-o...@vger.kernel.org; linux-arm- ker...@lists.infradead.org; linux-ker

RE: [PATCH 4/5] m5mols: Add boot flag for not showing the msg of I2C error

2011-11-14 Thread HeungJun, Kim
Hi Sylwester, -Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Sylwester Nawrocki Sent: Tuesday, November 15, 2011 7:35 AM To: Heungjun Kim Cc: linux-media; Kyungmin Park Subject: Re: [PATCH 4/5] m5mols: Add boot

RE: [PATCH 2/5] m5mols: Replace IRQ workqueue to waitqueue only

2011-11-14 Thread HeungJun, Kim
-Original Message- From: linux-media-ow...@vger.kernel.org [mailto:linux-media- ow...@vger.kernel.org] On Behalf Of Sylwester Nawrocki Sent: Tuesday, November 15, 2011 8:04 AM To: HeungJun, Kim Cc: linux-media@vger.kernel.org; Kyungmin Park Subject: Re: [PATCH 2/5] m5mols: Replace

[PATCH] MAINTAINERS: Add m5mols driver maintainers

2011-11-11 Thread HeungJun, Kim
Add the maintainers for the m5mols driver Signed-off-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- MAINTAINERS |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 5e587fc..91c5511

RE: [GIT PULL FOR v3.2] m5mols misc fixes about booting time

2011-11-11 Thread HeungJun, Kim
Hello Mauro, Please, check this patch series and let me know how's about submission. Is needed more any Ack?? Regards, Heungjun Kim -Original Message- From: HeungJun, Kim [mailto:riverful@samsung.com] Sent: Wednesday, October 26, 2011 7:37 PM To: linux-media@vger.kernel.org

Re: [RFC] SUBDEV_S/G_SELECTION IOCTLs

2011-11-11 Thread HeungJun Kim
? Regards, Heungjun Kim 2011. 11. 9. 오전 6:55 Sakari Ailus sakari.ai...@iki.fi 작성: Hi all, This RFC discusses the SUBDEV_S_SELECTION/SUBDEV_G_SELECTION API which is intended to amend and replace the existing SUBDEV_[SG]_CROP API. These IOCTLs have previously been discussed in the Cambridge V4L2

[PATCH 3/5] m5mols: Support for interrupt in the sensor's booting time

2011-10-21 Thread HeungJun, Kim
() for stabilization. Signed-off-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/media/video/m5mols/m5mols_core.c | 16 ++-- drivers/media/video/m5mols/m5mols_reg.h |3 ++- 2 files changed, 12 insertions(+), 7 deletions(-) diff --git

[PATCH 1/5] m5mols: Add more functions to check busy status

2011-10-21 Thread HeungJun, Kim
This patch add 3 type of checking busy status functions. 1) Keep busy-loop ignoring the error of I2C negative return, 2) Provide the read status value from I2C to the caller, 3) Compare masked value with desire value. Signed-off-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin

[PATCH 5/5] m5mols: Relocation the order and count for CAPTURE interrupt

2011-10-21 Thread HeungJun, Kim
The double enabling CAPTURE interrupt is not needed in m5mols_start_capture(), so remove these, and add one at the only booting time once. Also, fix the order of CAPTURE sequence to the right way. Signed-off-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p

[PATCH 4/5] m5mols: Add boot flag for not showing the msg of I2C error

2011-10-21 Thread HeungJun, Kim
-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/media/video/m5mols/m5mols.h |2 ++ drivers/media/video/m5mols/m5mols_core.c | 17 + 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/drivers/media

[PATCH 2/5] m5mols: Replace IRQ workqueue to waitqueue only

2011-10-21 Thread HeungJun, Kim
-by: HeungJun, Kim riverful@samsung.com Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- drivers/media/video/m5mols/m5mols.h |7 +-- drivers/media/video/m5mols/m5mols_capture.c | 34 ++- drivers/media/video/m5mols/m5mols_core.c| 60 +++ 3

Re: [RFCv2 PATCH 08/11] v4l2-ctrls: simplify event subscription.

2011-06-08 Thread Kim, HeungJun
the other one like (!flag) not ret, then it looks better to return immediately, or the other handling methods, not using continueing to check error at the end. Regards, Heungjun Kim -- To unsubscribe from this list: send the line unsubscribe linux-media in the body of a message to majord

Re: [PATCH v9] Add support for M-5MOLS 8 Mega Pixel camera ISP

2011-06-06 Thread Kim HeungJun
Hi Sakari, 2011. 6. 5., 오후 8:55, Sakari Ailus 작성: On Thu, May 26, 2011 at 04:12:47PM +0900, Kim, HeungJun wrote: Hi Sakari, Hi HeungJun, 2011-05-25 ?? 10:54, Sakari Ailus ??? ???: Hi HeungJun, Thanks for the patch! Also, thanks for the interests of this driver! [snip

Re: [PATCH v2 3/4] m5mols: remove union in the m5mols_get_version(), and VERSION_SIZE

2011-06-06 Thread Kim HeungJun
2011. 6. 5., 오후 9:11, Sakari Ailus 작성: On Sun, Jun 05, 2011 at 03:03:47PM +0300, Sakari Ailus wrote: [clip] - /* store version information swapped for being readable */ - info-ver = version.ver; info-ver.fw= be16_to_cpu(info-ver.fw); info-ver.hw=

  1   2   3   4   >