[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 AM, Sylwester Nawrocki wr

[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 acc

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 for changes

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 AM, Sylwester Nawrocki wrote: This patch adds OF initialization

[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 v3 7/8] drm/exynos: fix incorrect interrupt induced by m2m operation.

2012-12-22 Thread Eunchul Kim
s stop at s/w reset to avoid 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/

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

2012-12-22 Thread Eunchul Kim
driver. Signed-off-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

[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 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 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 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 - 3

[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|8

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

2012-12-22 Thread Eunchul Kim
. * 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, remove it. Thank's BR Eunchul Kim Eunchul Kim (6

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

2012-12-22 Thread Eunchul Kim
Dear Sean Paul. Thank's for your comment. I am in charge of Display Part in Samsung TN. and I also use exynos drm and I contributed some part to dri-devel(e.g IPPs). Your comment is very helpful to me. I answer to your comment. please check my answer. Thank's BR Eunchul Kim On 12/22/2012 04

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

2012-12-22 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

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 12 drivers/gpu/drm/exynos/exynos_drm_gsc.c | 12 drivers/gpu/drm/exynos

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

2012-12-22 Thread Eunchul Kim
. * 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, remove it. Thank's BR Eunchul Kim Eunchul Kim (6

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos

[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 chulspro@samsung.com

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- drivers/gpu/drm/exynos/exynos_drm_gsc.c |2 +- drivers/gpu/drm/exynos

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

2012-12-22 Thread Eunchul Kim
. Signed-off-by: JoongMock Shin jmock.s...@samsung.com Signed-off-by: Eunchul Kim chulspro@samsung.com --- 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

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

2012-12-22 Thread Eunchul Kim
at s/w reset to avoid incorrect interrupt. Signed-off-by: Jinyoung Jeon jy0.j...@samsung.com Signed-off-by: Eunchul Kim chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/exynos

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/exynos

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

2012-12-21 Thread Eunchul Kim
. - 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 + drivers/gpu

[RFC] added hdcp driver in hdmi

2012-12-21 Thread Eunchul Kim
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 + drivers/gpu

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

2012-12-21 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

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

2012-12-21 Thread Eunchul Kim
g 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 index a99ceef..9f52b7f 100644 --- a/drivers/gpu/drm/

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

2012-12-21 Thread Eunchul Kim
From: JoongMock Shin <jmock.s...@samsung.com> 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

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

2012-12-21 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

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

2012-12-21 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 3/8] drm/exynos: fixed vflip, hflip case at the same time.

2012-12-21 Thread Eunchul Kim
in this 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 2/8] drm/exynos: removed needless error handling about property.

2012-12-21 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 |5

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

2012-12-21 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 0/8] drm/exynos: fixed and cleanup ipp subsystem drivers

2012-12-21 Thread Eunchul Kim
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/exynos: cleanup current command name. drm/exynos: removed

[RFC] added hdcp driver in hdmi

2012-12-21 Thread Eunchul Kim
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 + drivers/gpu

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

2012-12-21 Thread Eunchul Kim
. - 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 chulspro@samsung.com --- drivers/gpu/drm/exynos/Kconfig |6 + drivers/gpu/drm/exynos/Makefile

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 chulspro@samsung.com wrote: HDCP stands for High-bandwidth

[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.kim at samsung.com] >> Sent: Thursday, December 20, 2012

[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

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

2012-12-20 Thread Eunchul Kim
g 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 index 67e1b88..c9eaa81 100644 --- a/drivers/gpu/drm/

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

2012-12-20 Thread Eunchul Kim
From: JoongMock Shin <jmock.s...@samsung.com> 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

[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

[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 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 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 |5

[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 0/8] drm/exynos: fixed and cleanup ipp subsystem drivers

2012-12-20 Thread Eunchul Kim
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, removed it. Thank's BR Eunchul Kim Eunchul Kim

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c|8 drivers

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 12 drivers/gpu/drm/exynos/exynos_drm_gsc.c | 12 drivers/gpu/drm/exynos

[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 chulspro@samsung.com --- drivers/gpu

[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 chulspro@samsung.com --- 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

[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 chulspro@samsung.com --- 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

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

2012-12-20 Thread Eunchul Kim
From: JoongMock Shin jmock.s...@samsung.com 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 jmock.s...@samsung.com Signed-off-by: Eunchul Kim chulspro@samsung.com --- drivers/gpu

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

2012-12-20 Thread Eunchul Kim
...@samsung.com Signed-off-by: Eunchul Kim chulspro@samsung.com --- 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 index 67e1b88

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

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; inki

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

2012-12-20 Thread Eunchul Kim
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/exynos: cleanup current command name. drm/exynos: removed

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c|8 drivers

[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 chulspro@samsung.com --- 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

[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 chulspro@samsung.com --- 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

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c | 12 drivers/gpu/drm/exynos/exynos_drm_gsc.c | 12 drivers/gpu/drm/exynos

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

2012-12-20 Thread Eunchul Kim
From: JoongMock Shin jmock.s...@samsung.com 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 jmock.s...@samsung.com Signed-off-by: Eunchul Kim chulspro@samsung.com --- drivers/gpu

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

2012-12-20 Thread Eunchul Kim
...@samsung.com Signed-off-by: Eunchul Kim chulspro@samsung.com --- 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 index a99ceef

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

2012-12-20 Thread Eunchul Kim
in this case. so, fixed it. Signed-off-by: Eunchul Kim chulspro@samsung.com --- 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

[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 chulspro@samsung.com --- drivers/gpu/drm/exynos/exynos_drm_fimc.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

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

2012-12-14 Thread Eunchul Kim
Mixer. Changelog v5 - Changed return type - Define hard coding value - Changed to devm_ function 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

[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

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

2012-12-14 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 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 1/5] drm/exynos: add ipp subsystem

2012-12-14 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 v5 0/5] drm/exynos: add ipp subsystem and each ipp drivers

2012-12-14 Thread Eunchul Kim
= exynos_drm_ipp_queue_buf(fd, [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"); } /* Sto

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

2012-12-14 Thread Eunchul Kim
Thank's for your comment. please check my answer. BR Eunchul Kim On 12/13/2012 11:29 AM, Joonyoung Shim wrote: > Hi, > > I can't review about logic of driver because i don't know well but i > feel there are too many checking codes and unused or big size defines. - I think error h

[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 chulspro@samsung.com Signed-off-by: Jinyoung

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

2012-12-14 Thread Eunchul Kim
); } /* Stop */ ret = exynos_drm_ipp_cmd_ctrl(fd, cmd_ctrl, property.prop_id, IPP_CTRL_STOP); if (ret) fprintf(stderr, failed to ipp ctrl IPP_CMD_WB stop\n); return; } Eunchul Kim (5): drm/exynos: add ipp subsystem drm/exynos: add iommu support for ipp

[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 chulspro@samsung.com Signed

[RFC v4 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 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 v4 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 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 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 v4 0/5] drm/exynos: add ipp subsystem and each ipp drivers

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

[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.kim at samsung.com] >> Sent: Wednesday, December 12, 2012 6:26 PM &g

[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.dae at samsung.com] >> Sent: Wednesday, December 12, 2012 7:01 PM >> To: 'Inki Dae'; 'Eunchul Kim' >> Cc:

[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, 2012

[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, 2012

[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 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 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 0/5] drm/exynos: add ipp subsystem and each ipp drivers

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

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

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; jy0.j

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

2012-12-12 Thread Eunchul Kim
) fprintf(stderr, failed to ipp buf dst dequeue\n); } /* Stop */ ret = exynos_drm_ipp_cmd_ctrl(fd, cmd_ctrl, property.prop_id, IPP_CTRL_STOP); if (ret) fprintf(stderr, failed to ipp ctrl IPP_CMD_WB stop\n); return; } Eunchul

[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 chulspro@samsung.com Signed-off-by: Jinyoung

[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 chulspro@samsung.com Signed

[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 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 2:55 PM &g

[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/rotator/cr

[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, 2012 1

  1   2   >