From mboxrd@z Thu Jan 1 00:00:00 1970 From: YoungJun Cho Subject: Re: [RFC v2 PATCH v2 06/14] drm/exynos: support MIPI DSI command mode Date: Tue, 22 Apr 2014 10:06:13 +0900 Message-ID: <5355C085.8000703@samsung.com> References: <1398083321-8668-1-git-send-email-yj44.cho@samsung.com> <1398083321-8668-7-git-send-email-yj44.cho@samsung.com> <1460689.BVNsEv5Rlh@avalon> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mailout3.samsung.com (mailout3.samsung.com [203.254.224.33]) by gabe.freedesktop.org (Postfix) with ESMTP id 43ADF6E473 for ; Mon, 21 Apr 2014 18:06:18 -0700 (PDT) Received: from epcpsbgr5.samsung.com (u145.gpu120.samsung.co.kr [203.254.230.145]) by mailout3.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0N4E00DDAR2GWS00@mailout3.samsung.com> for dri-devel@lists.freedesktop.org; Tue, 22 Apr 2014 10:06:17 +0900 (KST) In-reply-to: <1460689.BVNsEv5Rlh@avalon> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Laurent Pinchart , dri-devel@lists.freedesktop.org Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, linux-samsung-soc@vger.kernel.org, pawel.moll@arm.com, ijc+devicetree@hellion.org.uk, sw0312.kim@samsung.com, a.hajda@samsung.com, sachin.kamat@linaro.org, kyungmin.park@samsung.com, robh+dt@kernel.org, galak@codeaurora.org, kgene.kim@samsung.com List-Id: dri-devel@lists.freedesktop.org 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 >> Acked-by: Inki Dae >> Acked-by: Kyungmin Park >> --- >> 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] >