Hi Laurent,

Thank you for the comment.

On 04/22/2014 07:52 AM, Laurent Pinchart wrote:
Hi YoungJun,

Thank you for the patch.

On Monday 21 April 2014 21:28:33 YoungJun Cho wrote:
This patch adds I80 interface for FIMD to support command mode panel.

For this, the below features are added:
- Sets display interface mode relevant registers properly according to the
   interface type from DT
- Adds drm_panel_cpu_timings structure
  . The command mode panel sets them as the private attributes in struct
    drm_display_mode and FIMD gets them by fimd_mode_set().
- Adds TE interrupt handler
   . FIMD driver should know TE signal from lcd panel to avoid tearing issue.
- Adds trigger feature
   . In case of command mode panel, FIMD should set trigger bit,
     so that image data has to be transferred to display bus or lcd panel.

Changelog v2:
- Moves CPU timings relevant properties to panel DT
   (commented by Laurent Pinchart, Andrzej Hajda)

Signed-off-by: YoungJun Cho <yj44....@samsung.com>
Acked-by: Inki Dae <inki....@samsung.com>
Acked-by: Kyungmin Park <kyungmin.p...@samsung.com>
---
  drivers/gpu/drm/exynos/Kconfig           |    1 +
  drivers/gpu/drm/exynos/exynos_drm_crtc.c |   11 ++
  drivers/gpu/drm/exynos/exynos_drm_crtc.h |    2 +
  drivers/gpu/drm/exynos/exynos_drm_drv.h  |    2 +
  drivers/gpu/drm/exynos/exynos_drm_dsi.c  |   13 ++
  drivers/gpu/drm/exynos/exynos_drm_fimd.c |  280 ++++++++++++++++++++++-----
  include/drm/drm_mipi_dsi.h               |    2 +
  include/drm/drm_panel.h                  |    7 +

Could you please split the DRM core changes into two separate standalone
patches (as they're unrelated to each other) ?

Ok, it could be split into core and driver ones.
The DRM core changes are required to link with TE irq handler callback.

I'll do it next patch set.

Thank you.
Best regards YJ


  include/video/samsung_fimd.h             |    3 +-
  9 files changed, 277 insertions(+), 44 deletions(-)

[snip]


--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to