From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joonyoung Shim Subject: Re: [PATCH v7 0/2] Add display-timing node parsing to exynos drm fimd Date: Thu, 21 Feb 2013 15:57:56 +0900 Message-ID: <5125C574.5060307@samsung.com> References: <1361423512-2882-1-git-send-email-vikas.sajjan@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-reply-to: <1361423512-2882-1-git-send-email-vikas.sajjan@linaro.org> Sender: linux-media-owner@vger.kernel.org To: Vikas Sajjan Cc: dri-devel@lists.freedesktop.org, l.krishna@samsung.com, kgene.kim@samsung.com, linux-media@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org Hi, Please refer my comments about v6 patch. On 02/21/2013 02:11 PM, Vikas Sajjan wrote: > Add display-timing node parsing to drm fimd and depends on > the display helper patchset at > http://lists.freedesktop.org/archives/dri-devel/2013-January/033998.html > > It also adds pinctrl support for drm fimd. > > changes since v6: > addressed comments from Inki Dae to > separated out the pinctrl functionality and made a separate patch. > > changes since v5: > - addressed comments from Inki Dae , > to remove the allocation of 'fbmode' and replaced > '-1'in "of_get_fb_videomode(dev->of_node, fbmode, -1)" with > OF_USE_NATIVE_MODE. > > changes since v4: > - addressed comments from Paul Menzel > , to modify the commit message > > changes since v3: > - addressed comments from Sean Paul , to modify > the return values and print messages. > > changes since v2: > - moved 'devm_pinctrl_get_select_default' function call under > 'if (pdev->dev.of_node)', this makes NON-DT code unchanged. > (reported by: Rahul Sharma ) > > changes since v1: > - addressed comments from Sean Paul > > > Vikas Sajjan (2): > video: drm: exynos: Add display-timing node parsing using video > helper function > video: drm: exynos: Add pinctrl support to fimd > > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 36 ++++++++++++++++++++++++++---- > 1 file changed, 32 insertions(+), 4 deletions(-) >