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.
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
(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,
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
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
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
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
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 -
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 -
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:
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
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
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
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
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(
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
__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
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
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
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
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
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
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
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
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
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 --
__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
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
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
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
-
--
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
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
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
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
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
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
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
-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
() 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
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
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
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:
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
() 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
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
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
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
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
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
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;
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
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
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
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
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
, 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
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
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
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
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
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
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
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
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
-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
= 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
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
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
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
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
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
-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
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
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
?
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
() 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
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
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
-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
-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
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
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
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 - 100 of 354 matches
Mail list logo