[RFC 2/4] drm/exynos: add fimc ipp driver

2012-11-22 Thread Eunchul Kim
ved with mergeing similar part. > > Some comments are below. > > On 2012? 10? 29? 22:10, Eunchul Kim wrote: >> FIMC is stand for Fully Interfactive Mobile Camera and >> supports image scaler/rotator/crop/flip/csc and input/output DMA operations. >> input DMA reads image da

[PATCH 1/4] drm/exynos: add ipp subsystem

2012-10-18 Thread Eunchul Kim
driver for each hardware. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 24 + drivers/gpu/drm/exynos/exynos_drm_drv.h |7 + drivers/gpu

[PATCH 4/4] drm/exynos: add gsc ipp driver

2012-10-18 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

[PATCH 0/4] add ipp subsystem and each ipp drivers

2012-10-18 Thread Eunchul Kim
format, position, rotation, flip about source/destination. DRM_EXYNOS_IPP_QUEUE_BUF : enqueue/dequeue buffer and make event list. DRM_EXYNOS_IPP_CMD_CTRL : play/stop/pause/resume control. - event DRM_EXYNOS_IPP_EVENT : event to give notification completion of buffer DMA with buffer list Eunchul Kim (4

[PATCH 2/4] drm/exynos: add fimc ipp driver

2012-10-18 Thread Eunchul Kim
and display output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1

[PATCH 3/4] drm/exynos: add rotator ipp driver

2012-10-18 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[PATCH 0/4] add ipp subsystem and each ipp drivers

2012-10-18 Thread Eunchul Kim
format, position, rotation, flip about source/destination. DRM_EXYNOS_IPP_QUEUE_BUF : enqueue/dequeue buffer and make event list. DRM_EXYNOS_IPP_CMD_CTRL : play/stop/pause/resume control. - event DRM_EXYNOS_IPP_EVENT : event to give notification completion of buffer DMA with buffer list Eunchul Kim (4

[PATCH 2/4] drm/exynos: add fimc ipp driver

2012-10-18 Thread Eunchul Kim
and display output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1

[PATCH 3/4] drm/exynos: add rotator ipp driver

2012-10-18 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[PATCH 4/4] drm/exynos: add gsc ipp driver

2012-10-18 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

[PATCH 1/4] drm/exynos: add ipp subsystem

2012-10-18 Thread Eunchul Kim
driver for each hardware. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 24 + drivers/gpu/drm/exynos/exynos_drm_drv.h |7 + drivers/gpu

[PATCH 0/4] add ipp subsystem and each ipp drivers

2012-10-29 Thread Eunchul Kim
Thank's comment and I will modify description of Exynos5 series. and than I will send it. On 10/26/2012 11:43 PM, Inki Dae wrote: > 2012/10/18 Eunchul Kim : >> Hi All. >> >> I am responsible for a display part from Samsung Electronics >> Telecommunication Divisio

[PATCH 1/4] drm/exynos: add ipp subsystem

2012-10-29 Thread Eunchul Kim
Thank's quick review I have some question about your some comment's please check my comment and send one more time. On 10/27/2012 12:47 AM, Inki Dae wrote: > below is quick review. > > 2012/10/18 Eunchul Kim : >> IPP stand for Image Post Processing and supports image sca

[RFC 0/4] drm/exynos: add ipp subsystem and each ipp drivers

2012-10-29 Thread Eunchul Kim
/* For destination buffer dequeue to IPP */ for (i = 0; i < MAX_BUF; i++) { ret = exynos_drm_ipp_queue_buf(fd, &qbuf2[i], EXYNOS_DRM_OPS_DST, IPP_BUF_DEQUEUE, property.prop_id, i, gem2[i].handle);

[RFC 1/4] drm/exynos: add ipp subsystem

2012-10-29 Thread Eunchul Kim
driver for each hardware. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 24 + drivers/gpu/drm/exynos/exynos_drm_drv.h |7 + drivers/gpu

[RFC 2/4] drm/exynos: add fimc ipp driver

2012-10-29 Thread Eunchul Kim
and display output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1

[RFC 3/4] drm/exynos: add rotator ipp driver

2012-10-29 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[RFC 4/4] drm/exynos: add gsc ipp driver

2012-10-29 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

Re: [PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-05 Thread Eunchul Kim
ake up queue. */ + atomic_set(&ctx->wait_vsync_event, 0); + DRM_WAKEUP(&ctx->wait_vsync_queue); + } I lost your PATCH mail. so, I send my comment using this body mail. Thank's BR Eunchul Kim. On 12/05/2012 04:27 PM, Inki Dae wrote: 2012/12/5 Inki Dae

[RFC v2 2/5] drm/exynos: add iommu support for ipp

2012-12-10 Thread Eunchul Kim
This patch adds iommu support for ipp. For this, it adds subdrv_probe/remove callback to enable or disable ipp iommu. we can get or put device address to a gem handle from user through exynos_drm_gem_get/put_dma_addr(). Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu

[RFC v2 0/5] drm/exynos: add ipp subsystem and each ipp drivers

2012-12-10 Thread Eunchul Kim
fprintf(stderr, "failed to ipp buf dst dequeue\n"); } /* For destination buffer dequeue to IPP */ for (i = 0; i < MAX_BUF; i++) { ret = exynos_drm_ipp_queue_buf(fd, &qbuf2[i], EXYNOS_DRM_OPS_DST, IPP_BUF_DEQUEUE

[RFC v2 4/5] drm/exynos: add rotator ipp driver

2012-12-10 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[RFC v2 5/5] drm/exynos: add gsc ipp driver

2012-12-10 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |5 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

[RFC v3 0/5] drm/exynos: add ipp subsystem and each ipp drivers

2012-12-11 Thread Eunchul Kim
IPP */ for (i = 0; i < MAX_BUF; i++) { ret = exynos_drm_ipp_queue_buf(fd, &qbuf2[i], EXYNOS_DRM_OPS_DST, IPP_BUF_DEQUEUE, property.prop_id, i, gem2[i].handle); if (ret < 0) fprintf(std

[RFC v3 2/5] drm/exynos: add iommu support for ipp

2012-12-11 Thread Eunchul Kim
This patch adds iommu support for ipp. For this, it adds subdrv_probe/remove callback to enable or disable ipp iommu. we can get or put device address to a gem handle from user through exynos_drm_gem_get/put_dma_addr(). Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu

[RFC v3 4/5] drm/exynos: add rotator ipp driver

2012-12-11 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

Re: [RFC v3 4/5] drm/exynos: add rotator ipp driver

2012-12-12 Thread Eunchul Kim
Thank's your comment. I answer your comment. please check that. Thank's BR Eunchul Kim On 12/12/2012 05:29 PM, Inki Dae wrote: -Original Message----- From: Eunchul Kim [mailto:chulspro@samsung.com] Sent: Wednesday, December 12, 2012 4:35 PM To: dri-devel@lists.freed

Re: [RFC v3 4/5] drm/exynos: add rotator ipp driver

2012-12-12 Thread Eunchul Kim
Thank's your comment BR Eunchul Kim On 12/12/2012 07:39 PM, Inki Dae wrote: -Original Message- From: Inki Dae [mailto:inki@samsung.com] Sent: Wednesday, December 12, 2012 7:01 PM To: 'Inki Dae'; 'Eunchul Kim' Cc: jy0.j...@samsung.com; sw0312

Re: [RFC v3 4/5] drm/exynos: add rotator ipp driver

2012-12-12 Thread Eunchul Kim
Thank's for your comment. BR Eunchul Kim. On 12/12/2012 06:46 PM, Inki Dae wrote: One more comment. -Original Message- From: Eunchul Kim [mailto:chulspro@samsung.com] Sent: Wednesday, December 12, 2012 6:26 PM To: Inki Dae Cc: dri-devel@lists.freedesktop.org;

[RFC v4 0/5] drm/exynos: add ipp subsystem and each ipp drivers

2012-12-12 Thread Eunchul Kim
e_buf(fd, &qbuf2[i], EXYNOS_DRM_OPS_DST, IPP_BUF_DEQUEUE, property.prop_id, i, gem2[i].handle); if (ret < 0) fprintf(stderr, "failed to ipp buf dst dequeue\n"); } /* Stop */

[RFC v4 2/5] drm/exynos: add iommu support for ipp

2012-12-12 Thread Eunchul Kim
This patch adds iommu support for ipp. For this, it adds subdrv_probe/remove callback to enable or disable ipp iommu. we can get or put device address to a gem handle from user through exynos_drm_gem_get/put_dma_addr(). Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu

[RFC v4 4/5] drm/exynos: add rotator ipp driver

2012-12-12 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[RFC v5 2/5] drm/exynos: add iommu support for ipp

2012-12-14 Thread Eunchul Kim
This patch adds iommu support for ipp. For this, it adds subdrv_probe/remove callback to enable or disable ipp iommu. we can get or put device address to a gem handle from user through exynos_drm_gem_get/put_dma_addr(). Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu

[RFC v5 0/5] drm/exynos: add ipp subsystem and each ipp drivers

2012-12-14 Thread Eunchul Kim
IPP_BUF_DEQUEUE, property.prop_id, i, gem2[i].handle); if (ret < 0) fprintf(stderr, "failed to ipp buf dst dequeue\n"); } /* Stop */ ret = exynos_drm_ipp_cmd_ctrl(fd, &cmd_ctrl, property.prop_id, IPP_CT

[RFC v5 4/5] drm/exynos: add rotator ipp driver

2012-12-14 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[PATCH 0/8] drm/exynos: fixed and cleanup ipp subsystem drivers

2012-12-20 Thread Eunchul Kim
be NULL. so, we removed error handling about NULL. * cleanup comment of abbreviation : we changed comments for right grammar. * cleanup needless parenthesis : we missed changing of fimc side. so, I added it. * removed color bar pattern display : we don't use color display at writeback. so

[PATCH 1/8] drm/exynos: cleanup current command name.

2012-12-20 Thread Eunchul Kim
This patch changed current command name from cmd to c_node. because we already use cmd for command control ioctl in another structure. so, this name make some confusion. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c|8 drivers/gpu/drm/exynos

[PATCH 2/8] drm/exynos: removed needless error handling about property.

2012-12-20 Thread Eunchul Kim
This patch removed property error handling. property couldn't be NULL. so, I removed it. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 12 drivers/gpu/drm/exynos/exynos_drm_gsc.c | 12 drivers/gpu/drm/exynos/exynos_drm_ipp.c |

[PATCH 3/8] drm/exynos: fixed vflip, hflip case at the same time.

2012-12-20 Thread Eunchul Kim
This patch fixed vflip, hflip at the same time. If we want to change 180 degree about buffer, then we can use h,vflip or 180 degree. we supports 180 degree using h,vflip. but we make error handling in this case. so, fixed it. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos

[PATCH 4/8] drm/exynos: fixed warnning in GSC build.

2012-12-20 Thread Eunchul Kim
This patch fixed warnning in GSC build. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_gsc.c b/drivers/gpu/drm/exynos/exynos_drm_gsc.c index ba5fefd..4546833

[PATCH 5/8] drm/exynos: cleanup comment of abbreviation.

2012-12-20 Thread Eunchul Kim
This patch cleanup comment of abbreviation. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_ipp.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a

[PATCH 6/8] drm/exynos: removed color bar pattern display.

2012-12-20 Thread Eunchul Kim
From: JoongMock Shin This patch removed color bar pattern register. we not use color bar any more. and don't support color bar at writeback operation. Signed-off-by: JoongMock Shin Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |9 +++-- 1 files chang

[PATCH 7/8] drm/exynos: fixed unnormal interrupt in m2m operation.

2012-12-20 Thread Eunchul Kim
From: Jinyoung Jeon This patch fixed unnormal interrupt at m2m operation sometimes. m2m operation called s/w reset during every frame. but m2m occured interrupt after s/w reset sometimes. so, we cleared dma operation and capture operation. Signed-off-by: Jinyoung Jeon Signed-off-by: Eunchul

[PATCH 8/8] drm/exynos: cleanup needless parenthesis.

2012-12-20 Thread Eunchul Kim
This patch cleanup needless parenthesis. we got the comment from GSC. but we missed fimc side. so, we cleanup the code. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/exynos

Re: [PATCH 3/8] drm/exynos: fixed vflip, hflip case at the same time.

2012-12-20 Thread Eunchul Kim
Thank's for comment. Oops, sorry that is my fault. I will resend it. BR Eunchul Kim On 12/20/2012 06:48 PM, Inki Dae wrote: -Original Message- From: Eunchul Kim [mailto:chulspro@samsung.com] Sent: Thursday, December 20, 2012 6:32 PM To: dri-devel@lists.freedesktop.org;

[PATCH v2 0/8] drm/exynos: fixed and cleanup ipp subsystem drivers

2012-12-20 Thread Eunchul Kim
ent of abbreviation : we changed comments for right grammar. * cleanup needless parenthesis : we missed changing of fimc side. so, I added it. * removed color bar pattern display : we don't use color display at writeback. so, removed it. Thank's BR Eunchul Kim Eunchul Kim (6): drm/exyn

[PATCH v2 1/8] drm/exynos: cleanup current command name.

2012-12-20 Thread Eunchul Kim
This patch changed current command name from cmd to c_node. because we already use cmd for command control ioctl in another structure. so, this name make some confusion. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c|8 drivers/gpu/drm/exynos

[PATCH v2 4/8] drm/exynos: fixed warnning in GSC build.

2012-12-20 Thread Eunchul Kim
This patch fixed warnning in GSC build. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_gsc.c b/drivers/gpu/drm/exynos/exynos_drm_gsc.c index 3e5b456..410175a

[PATCH v2 5/8] drm/exynos: cleanup comment of abbreviation.

2012-12-20 Thread Eunchul Kim
This patch cleanup comment of abbreviation. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_ipp.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a

[PATCH v2 2/8] drm/exynos: removed needless error handling about property.

2012-12-20 Thread Eunchul Kim
This patch removed property error handling. property couldn't be NULL. so, I removed it. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 12 drivers/gpu/drm/exynos/exynos_drm_gsc.c | 12 drivers/gpu/drm/exynos/exynos_drm_ipp.c |

[PATCH v2 6/8] drm/exynos: removed color bar pattern display.

2012-12-20 Thread Eunchul Kim
From: JoongMock Shin This patch removed color bar pattern register. we not use color bar any more. and don't support color bar at writeback operation. Signed-off-by: JoongMock Shin Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |9 +++-- 1 files chang

[PATCH v2 7/8] drm/exynos: fixed unnormal interrupt in m2m operation.

2012-12-20 Thread Eunchul Kim
From: Jinyoung Jeon This patch fixed unnormal interrupt at m2m operation sometimes. m2m operation called s/w reset during every frame. but m2m occured interrupt after s/w reset sometimes. so, we cleared dma operation and capture operation. Signed-off-by: Jinyoung Jeon Signed-off-by: Eunchul

[PATCH v2 3/8] drm/exynos: fixed vflip, hflip case at the same time.

2012-12-20 Thread Eunchul Kim
case. so, fixed it. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c|1 + drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_rotator.c |1 + include/uapi/drm/exynos_drm.h |2 ++ 4 files changed, 5

[PATCH v2 8/8] drm/exynos: cleanup needless parenthesis.

2012-12-20 Thread Eunchul Kim
This patch cleanup needless parenthesis. we got the comment from GSC. but we missed fimc side. so, we cleanup the code. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/exynos

[RFC] added hdcp driver in hdmi

2012-12-21 Thread Eunchul Kim
's BR Eunchul Kim Eunchul Kim (1): drm/exynos: added hdcp driver for contents protection. drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 12 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + d

[RFC] drm/exynos: added hdcp driver for contents protection.

2012-12-21 Thread Eunchul Kim
evice. - compare Ri,Rj. If same and not repeater, then start encryption. - If not same, retry. If repeater, then start second authentication. - stop encryption. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + driver

Re: [RFC] drm/exynos: added hdcp driver for contents protection.

2012-12-21 Thread Eunchul Kim
Thank's for your advice. Your opinion is very helpful to me. I will consider your comments, and then I resend it. Thank's BR Eunchul Kim On 12/21/2012 08:53 PM, Alan Cox wrote: On Fri, 21 Dec 2012 18:47:57 +0900 Eunchul Kim wrote: HDCP stands for High-bandwidth Digital Content

[PATCH v3 2/8] drm/exynos: remove needless error handling to property.

2012-12-22 Thread Eunchul Kim
This patch removes property error handling. because property couldn't be NULL. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 12 drivers/gpu/drm/exynos/exynos_drm_gsc.c | 12 drivers/gpu/drm/exynos/exynos_drm_ipp.c |5 ---

[PATCH v3 0/8] drm/exynos: fix and cleanup ipp subsystem drivers

2012-12-22 Thread Eunchul Kim
ULL. so, we remove error handling about NULL. * correct some comments to abbreviation : we change comments for right grammar. * remove needless parenthesis : we misse changing of fimc side. so, I added it. * remove color bar pattern operation : we don't use color display at writeback. so,

[PATCH v3 1/8] drm/exynos: change member variable name.

2012-12-22 Thread Eunchul Kim
This patch change current command name from cmd to c_node. because we are using the member name, 'cmd', for command control ioctl in another structure. so, this patch changes it to c_node to avoid such confusing. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c

[PATCH v3 3/8] drm/exynos: consider both case of vflip and hflip.

2012-12-22 Thread Eunchul Kim
This patch considers both case of vflip and hflip. If we want that the contents in buffer to be rotated to 180 degree, then we can use h,vflip or 180 degree. Changelog v2: - added EXYNOS_DRM_FLIP_BOTH enum value to avoid build warnning. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos

[PATCH v3 4/8] drm/exynos: fix build warning.

2012-12-22 Thread Eunchul Kim
This patch fixes erroneous register read. reall function needs register base address + offset but exynos_drm_gsc module used only offset to read a register. so this patch uses gsc_read function instead of readl. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- 1

[PATCH v3 5/8] drm/exynos: correct some comments to abbreviation.

2012-12-22 Thread Eunchul Kim
This patch cleanup corrects some comments to abbreviation. We would like to prevent it stands misunderstood. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_ipp.c |2

[PATCH v3 6/8] drm/exynos: remove color bar pattern operation.

2012-12-22 Thread Eunchul Kim
f-by: JoongMock Shin Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |9 +++-- 1 files changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.c b/drivers/gpu/drm/exynos/exynos_drm_fimc.c index 8cfc5a9..a99ceef 100644 --- a/

[PATCH v3 7/8] drm/exynos: fix incorrect interrupt induced by m2m operation.

2012-12-22 Thread Eunchul Kim
incorrect interrupt. Signed-off-by: Jinyoung Jeon Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.c b/drivers/gpu/drm/exynos/exynos_drm_fimc.c

[PATCH v3 8/8] drm/exynos: remove needless parenthesis.

2012-12-22 Thread Eunchul Kim
This patch removes needless parenthesis. This was pointed out but in case of fimc side. we missed it. Signed-off-by: Eunchul Kim --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimc.c b

[PATCH 0/4] add ipp subsystem and each ipp drivers

2012-10-18 Thread Eunchul Kim
format, position, rotation, flip about source/destination. DRM_EXYNOS_IPP_QUEUE_BUF : enqueue/dequeue buffer and make event list. DRM_EXYNOS_IPP_CMD_CTRL : play/stop/pause/resume control. - event DRM_EXYNOS_IPP_EVENT : event to give notification completion of buffer DMA with buffer list Eunchul Kim (4

[PATCH 4/4] drm/exynos: add gsc ipp driver

2012-10-18 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

[PATCH 3/4] drm/exynos: add rotator ipp driver

2012-10-18 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[PATCH 0/4] add ipp subsystem and each ipp drivers

2012-10-18 Thread Eunchul Kim
format, position, rotation, flip about source/destination. DRM_EXYNOS_IPP_QUEUE_BUF : enqueue/dequeue buffer and make event list. DRM_EXYNOS_IPP_CMD_CTRL : play/stop/pause/resume control. - event DRM_EXYNOS_IPP_EVENT : event to give notification completion of buffer DMA with buffer list Eunchul Kim (4

[PATCH 3/4] drm/exynos: add rotator ipp driver

2012-10-18 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[PATCH 4/4] drm/exynos: add gsc ipp driver

2012-10-18 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

Re: [PATCH 0/4] add ipp subsystem and each ipp drivers

2012-10-29 Thread Eunchul Kim
Thank's comment and I will modify description of Exynos5 series. and than I will send it. On 10/26/2012 11:43 PM, Inki Dae wrote: 2012/10/18 Eunchul Kim : Hi All. I am responsible for a display part from Samsung Electronics Telecommunication Division. and I am going to add post-proce

Re: [PATCH 1/4] drm/exynos: add ipp subsystem

2012-10-29 Thread Eunchul Kim
Thank's quick review I have some question about your some comment's please check my comment and send one more time. On 10/27/2012 12:47 AM, Inki Dae wrote: below is quick review. 2012/10/18 Eunchul Kim : IPP stand for Image Post Processing and supports image scaler/rotator /cro

[RFC 0/4] drm/exynos: add ipp subsystem and each ipp drivers

2012-10-29 Thread Eunchul Kim
t;); } /* For destination buffer dequeue to IPP */ for (i = 0; i < MAX_BUF; i++) { ret = exynos_drm_ipp_queue_buf(fd, &qbuf2[i], EXYNOS_DRM_OPS_DST, IPP_BUF_DEQUEUE, property.prop_id, i, gem2[i].ha

[RFC 3/4] drm/exynos: add rotator ipp driver

2012-10-29 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[RFC 4/4] drm/exynos: add gsc ipp driver

2012-10-29 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

Re: [PATCH v2 2/3] drm/exynos: Rework fimc clocks handling

2013-04-19 Thread Eunchul Kim
Dear Sylwester Nawrocki Thank you for your update. I have some question of your patch. please give your information to me. Thank's BR Eunchul Kim. On 04/17/2013 06:53 PM, Sylwester Nawrocki wrote: The clocks handling is refactored and a "mux" clock handling is added to account

Re: [PATCH 3/3] drm/exynos: Add device tree support for fimc ipp driver

2013-04-19 Thread Eunchul Kim
Dear Sylwester Nawrocki. Sorry. I didn't check your third patch. I understand the "mux", "parent" meaning. please ignore my comment of "mux", "parent" and please check below comments. Thank's BR Eunchul Kim. On 04/17/2013 02:31

[RFC 1/4] drm/exynos: add ipp subsystem

2012-12-05 Thread Eunchul Kim
base them to be > based on exynos-drm-next. > > Below is a little bit comments. Please, describe your codes as much as > possible. > > Thanks, > Inki Dae > >> -Original Message- >> From: Eunchul Kim [mailto:chulspro.kim at samsung.com] >> Sent: Monday

[PATCH 0/3] drm/exynos: modify usage of wait for vblank

2012-12-05 Thread Eunchul Kim
ake up queue. */ + atomic_set(&ctx->wait_vsync_event, 0); + DRM_WAKEUP(&ctx->wait_vsync_queue); + } I lost your PATCH mail. so, I send my comment using this body mail. Thank's BR Eunchul Kim. On 12/05/2012 04:27 PM, Inki Dae wrote: > 2012/12/5 Inki

[RFC v2 2/5] drm/exynos: add iommu support for ipp

2012-12-10 Thread Eunchul Kim
This patch adds iommu support for ipp. For this, it adds subdrv_probe/remove callback to enable or disable ipp iommu. we can get or put device address to a gem handle from user through exynos_drm_gem_get/put_dma_addr(). Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu

[RFC v2 0/5] drm/exynos: add ipp subsystem and each ipp drivers

2012-12-10 Thread Eunchul Kim
f dst dequeue\n"); } /* For destination buffer dequeue to IPP */ for (i = 0; i < MAX_BUF; i++) { ret = exynos_drm_ipp_queue_buf(fd, &qbuf2[i], EXYNOS_DRM_OPS_DST, IPP_BUF_DEQUEUE, property.prop_id,

[RFC v2 4/5] drm/exynos: add rotator ipp driver

2012-12-10 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[RFC v2 3/5] drm/exynos: add fimc ipp driver

2012-12-10 Thread Eunchul Kim
and display output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1

[RFC v2 5/5] drm/exynos: add gsc ipp driver

2012-12-10 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |5 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

[RFC v2 1/5] drm/exynos: add ipp subsystem

2012-12-10 Thread Eunchul Kim
driver for each hardware. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 24 + drivers/gpu/drm/exynos/exynos_drm_drv.h |7 + drivers/gpu

[RFC v2 1/5] drm/exynos: add ipp subsystem

2012-12-11 Thread Eunchul Kim
Thank's your comments. I answer your requestion. please check that. Thank's BR Eunchul Kim On 12/11/2012 01:37 PM, Inki Dae wrote: > > >> -Original Message----- >> From: Eunchul Kim [mailto:chulspro.kim at samsung.com] >> Sent: Monday, December 10,

[RFC v2 3/5] drm/exynos: add fimc ipp driver

2012-12-11 Thread Eunchul Kim
Thank's your comments. I answer your comments. please check that. Thank's BR Eunchul Kim On 12/11/2012 02:16 PM, Inki Dae wrote: > > >> -Original Message----- >> From: Eunchul Kim [mailto:chulspro.kim at samsung.com] >> Sent: Monday, December 10,

[RFC v2 5/5] drm/exynos: add gsc ipp driver

2012-12-11 Thread Eunchul Kim
Thank's your comments I answer your comments. please check that. Thank's BR Eunchul Kim On 12/11/2012 02:49 PM, Joonyoung Shim wrote: > Hi, > > On 12/10/2012 10:45 PM, Eunchul Kim wrote: >> GSC is stand for General SCaler and supports supports >> image scaler/

[RFC v2 1/5] drm/exynos: add ipp subsystem

2012-12-11 Thread Eunchul Kim
Thank's for comments. I answer your comment, please check that. Thank's BR Eunchul Kim. On 12/11/2012 04:17 PM, Inki Dae wrote: > > >> -Original Message----- >> From: Eunchul Kim [mailto:chulspro.kim at samsung.com] >> Sent: Tuesday, December 11, 2012

[RFC v2 5/5] drm/exynos: add gsc ipp driver

2012-12-11 Thread Eunchul Kim
OK, I know what you mean. I talk about GSC feature with Inki.Dae. I will re-send our final source instead of first commit. please one more time review about next RFC v3. Thank's BR Eunchul Kim On 12/11/2012 05:31 PM, Joonyoung Shim wrote: > Hi, > > I want to see only just curr

[RFC v3 0/5] drm/exynos: add ipp subsystem and each ipp drivers

2012-12-12 Thread Eunchul Kim
< MAX_BUF; i++) { ret = exynos_drm_ipp_queue_buf(fd, &qbuf2[i], EXYNOS_DRM_OPS_DST, IPP_BUF_DEQUEUE, property.prop_id, i, gem2[i].handle); if (ret < 0) fprintf(stderr, "failed

[RFC v3 2/5] drm/exynos: add iommu support for ipp

2012-12-12 Thread Eunchul Kim
This patch adds iommu support for ipp. For this, it adds subdrv_probe/remove callback to enable or disable ipp iommu. we can get or put device address to a gem handle from user through exynos_drm_gem_get/put_dma_addr(). Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu

[RFC v3 1/5] drm/exynos: add ipp subsystem

2012-12-12 Thread Eunchul Kim
driver for each hardware. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 24 + drivers/gpu/drm/exynos/exynos_drm_drv.h |7 + drivers/gpu

[RFC v3 4/5] drm/exynos: add rotator ipp driver

2012-12-12 Thread Eunchul Kim
Rotator supports rotation/crop/flip and input/output DMA operations Rotator ipp driver supports 90,180,270 degree rotaion and vertical, horizontal flip. and has some limitations(source and destination format have to be same, no scaler) Signed-off-by: Eunchul Kim Signed-off-by: Youngjun Cho

[RFC v3 3/5] drm/exynos: add fimc ipp driver

2012-12-12 Thread Eunchul Kim
and display output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1

[RFC v3 5/5] drm/exynos: add gsc ipp driver

2012-12-12 Thread Eunchul Kim
output operations. Signed-off-by: Eunchul Kim Signed-off-by: Jinyoung Jeon --- drivers/gpu/drm/exynos/Kconfig |5 + drivers/gpu/drm/exynos/Makefile |1 + drivers/gpu/drm/exynos/exynos_drm_drv.c | 15 + drivers/gpu/drm/exynos/exynos_drm_drv.h |1 + drivers/gpu/drm

[RFC v3 1/5] drm/exynos: add ipp subsystem

2012-12-12 Thread Eunchul Kim
Thank's your comment. I anwer your comment, please check that. Thank's BR Eunchul Kim. On 12/12/2012 05:18 PM, Inki Dae wrote: > > >> -Original Message----- >> From: Eunchul Kim [mailto:chulspro.kim at samsung.com] >> Sent: Wednesday, December 12

[RFC v3 4/5] drm/exynos: add rotator ipp driver

2012-12-12 Thread Eunchul Kim
Thank's your comment. I answer your comment. please check that. Thank's BR Eunchul Kim On 12/12/2012 05:29 PM, Inki Dae wrote: > > >> -Original Message----- >> From: Eunchul Kim [mailto:chulspro.kim at samsung.com] >> Sent: Wednesday, December 12

  1   2   >