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;

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

2011-12-29 Thread HeungJun, Kim
Sent: Friday, 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_BAL

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...@samsu

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;

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

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;

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;

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...

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-med

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

2011-12-28 Thread HeungJun, Kim
mber 28, 2011 10:35 PM > 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_BALAN

[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 Signed-off-by: Kyungmin Park --- Documentation/DocBook/media/v4l/controls.xml | 11 +++ drivers/media/video

[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 Signed-off-by: Kyungmin Park --- Documentation/DocBook/media/v4l/controls.xml | 12 drivers

[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 Signed-off-by: Kyungmin Park --- Documentation/DocBook/media/v4l/controls.xml | 88 ++ drivers/media/video/v4l2-ctrls.c | 21 ++ include/linux

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

2011-12-27 Thread HeungJun, Kim
: HeungJun, Kim Signed-off-by: Kyungmin Park --- Documentation/DocBook/media/v4l/controls.xml | 38 ++ drivers/media/video/v4l2-ctrls.c | 12 include/linux/videodev2.h|9 ++ 3 files changed, 59 insertions(+), 0 deletions

[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

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.infrad

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

2011-12-12 Thread HeungJun, Kim
e name "object recognition", not the "face detection", for any other device or driver. In a few days, I'll share the CIDs I have thought for m5mols driver. And, I hope to discuss about this with OMAP FDIF. Thank you. Regards, Heungjun Kim > -Original Message

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: R

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: [P

Re: [RFC] SUBDEV_S/G_SELECTION IOCTLs

2011-11-11 Thread HeungJun Kim
dle the size more easily? Regards, Heungjun Kim 2011. 11. 9. 오전 6:55 Sakari Ailus 작성: > 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 discu

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 >

[PATCH] MAINTAINERS: Add m5mols driver maintainers

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

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

2011-10-26 Thread HeungJun, Kim
Hello Mauro, the following changes since commit 35a912455ff5640dc410e91279b03e04045265b2: merge branch 'v4l_for_linus' into staging/for_v3.2 (2011-10-19 12:41:18 -0200) are available in the git repository at: git://git.infradead.org/users/kmpark/linux-2.6-samsung m5mols heungj

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

2011-10-21 Thread HeungJun, Kim
off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols.h |7 +-- drivers/media/video/m5mols/m5mols_capture.c | 34 ++- drivers/media/video/m5mols/m5mols_core.c| 60 +++ 3 files changed, 32 insertions(+), 69 del

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

2011-10-21 Thread HeungJun, Kim
: HeungJun, Kim Signed-off-by: Kyungmin Park --- 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/video/m5mols/m5mols.h b/drivers/media/video/m5mols

[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 Signed-off-by: Kyungmin Park --- drivers/media

[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 Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols

[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 Signed-off-by: Kyungmin Park --- 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 a/drivers/media/video/m5mols/m5mols_core

[PATCH v2 1/4] m5mols: Fix capture image size register definition

2011-05-31 Thread HeungJun, Kim
The main capture and the thumbnail image size registers were erroneously defined to have 1 byte width, resulting in wrong reported image size. Fix this by changing the registers width to correct value. Reported-by: Sylwester Nawrocki Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park

[PATCH v2 4/4] m5mols: add parenthesis <> for the head and back of email address

2011-05-31 Thread HeungJun, Kim
Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols.h |4 ++-- drivers/media/video/m5mols/m5mols_capture.c |4 ++-- drivers/media/video/m5mols/m5mols_controls.c |4 ++-- drivers/media/video/m5mols/m5mols_core.c |4

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

2011-05-31 Thread HeungJun, Kim
Remove union version in the m5mols_get_version(), and read version information directly. Also remove VERSION_SIZE. Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols.h |1 - drivers/media/video/m5mols/m5mols_core.c | 42

[PATCH v2 2/4] m5mols: add m5mols_read_u8/u16/u32() according to I2C byte width

2011-05-31 Thread HeungJun, Kim
byte width like u8, u16, u32 as same reason. Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols.h | 52 +++--- drivers/media/video/m5mols/m5mols_capture.c | 18 +++--- drivers/media/video/m5mols/m5mols_controls.c |2

[PATCH v2 0/4] Fix micellaneous issues for M-5MOLS driver

2011-05-31 Thread HeungJun, Kim
to discard 4/5 patch for keeping the previous one. Thanks, and any comments welcome. Regards, Heungjun 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-info.html

[PATCH 4/5] m5mols: rename m5mols_capture_error_handler() to proper name

2011-05-27 Thread HeungJun, Kim
The m5mols_capture_post_work() is collecting works after capture. The order should be kept, and it's safe to say success of capture when all this conditions are fine. Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols_capture.c |

[PATCH 5/5] m5mols: add parenthesis <> for the head and back of email address

2011-05-27 Thread HeungJun, Kim
Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols.h |4 ++-- drivers/media/video/m5mols/m5mols_capture.c |4 ++-- drivers/media/video/m5mols/m5mols_controls.c |4 ++-- drivers/media/video/m5mols/m5mols_core.c |4

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

2011-05-27 Thread HeungJun, Kim
Remove union version in the m5mols_get_version(), and read version information directly. Also remove VERSION_SIZE. Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols.h |1 - drivers/media/video/m5mols/m5mols_core.c | 42

[PATCH 2/5] m5mols: add m5mols_read_u8/u16/u32() according to I2C byte width

2011-05-27 Thread HeungJun, Kim
byte width like u8, u16, u32 as same reason. Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols.h | 52 +++--- drivers/media/video/m5mols/m5mols_capture.c | 18 +++--- drivers/media/video/m5mols/m5mols_controls.c |2

[PATCH 1/5] m5mols: fix reading wrong size of captured main/thumb image

2011-05-27 Thread HeungJun, Kim
Signed-off-by: HeungJun, Kim Signed-off-by: Kyungmin Park --- drivers/media/video/m5mols/m5mols_reg.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/video/m5mols/m5mols_reg.h b/drivers/media/video/m5mols/m5mols_reg.h index b83e36f..8260f50 100644

[PATCH 0/5] Fix micellaneous issues for M-5MOLS driver

2011-05-27 Thread HeungJun, Kim
nts welcome. Regards, Heungjun 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-info.html

[PATCH v4] Add support for M5MO-LS 8 Mega Pixel camera

2011-02-07 Thread Heungjun Kim
Add I2C/V4L2 subdev driver for M5MO-LS camera sensor with integrated image processor. Signed-off-by: Heungjun Kim Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- Hello, This is fourth version of M5MOLS 8 Mega Pixel camera sensor. The fourth version changes are below: 1

[PATCH v3] Add support for M5MO-LS 8 Mega Pixel camera

2011-01-24 Thread Heungjun Kim
Add I2C/V4L2 subdev driver for M5MO-LS camera sensor with integrated image processor. Signed-off-by: Heungjun Kim Signed-off-by: Sylwester Nawrocki Signed-off-by: Kyungmin Park --- Hello, This is third version of M5MOLS 8 Mega Pixel camera sensor. The first version patch is here: http