| Message ID | 1359351936-20618-2-git-send-email-vikas.sajjan@linaro.org (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from <linux-media-owner@vger.kernel.org>) id 1TzhXT-0004WV-Dx; Mon, 28 Jan 2013 06:45:51 +0100 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.75/mailfrontend-4) with esmtp id 1TzhXS-0005mi-B2; Mon, 28 Jan 2013 06:45:51 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751644Ab3A1Fpt (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Mon, 28 Jan 2013 00:45:49 -0500 Received: from mail-pa0-f42.google.com ([209.85.220.42]:42487 "EHLO mail-pa0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751436Ab3A1Fps (ORCPT <rfc822;linux-media@vger.kernel.org>); Mon, 28 Jan 2013 00:45:48 -0500 Received: by mail-pa0-f42.google.com with SMTP id rl6so1310247pac.15 for <linux-media@vger.kernel.org>; Sun, 27 Jan 2013 21:45:48 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references:x-gm-message-state; bh=x2iAgXUDo9nFjao90FpQDnPVqaoxsY5v8PwTQ2a9fPo=; b=TogyPou3WzZaj0c4kbGbJy/PJY+Ely/6ZPDZOjj3VPDwCf62oOlTR/7c4Okx3HIZe/ XvYvKgJd8K844WSwyK0e3WcH/TNpSj/+mXMfibNHqgk+ltvvWgYeBYa1SZSqD7tsmVdv cTe45IKQ5HZ5BNWVcEfObrNSvWtDpavsCGTbVhxjbvQuD7y0l9T4ToJmDwEk6Y+cMVwr Ry1w6zen/jUgMCw0Ev3yaJx+cihibU6Tdg3ja40/iaDhva2lOC75EtTJrSCQEWp6p4Lx a7uKVjLCCPNbJCqOTcA8Kysvsp+yzvR5F1T366ZnvnMuZq9JKUCpbV4GtEu5yHxz3Ug4 MVSw== X-Received: by 10.68.220.161 with SMTP id px1mr35270383pbc.167.1359351948062; Sun, 27 Jan 2013 21:45:48 -0800 (PST) Received: from localhost.localdomain ([115.113.119.130]) by mx.google.com with ESMTPS id ho4sm5634695pbc.54.2013.01.27.21.45.44 (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sun, 27 Jan 2013 21:45:47 -0800 (PST) From: Vikas Sajjan <vikas.sajjan@linaro.org> To: dri-devel@lists.freedesktop.org Cc: linux-media@vger.kernel.org, kgene.kim@samsung.com, s.trumtrar@pengutronix.de, inki.dae@samsung.com, l.krishna@samsung.com Subject: [PATCH] video: drm: exynos: Adds display-timing node parsing using video helper function Date: Mon, 28 Jan 2013 11:15:36 +0530 Message-Id: <1359351936-20618-2-git-send-email-vikas.sajjan@linaro.org> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1359351936-20618-1-git-send-email-vikas.sajjan@linaro.org> References: <1359351936-20618-1-git-send-email-vikas.sajjan@linaro.org> X-Gm-Message-State: ALoCoQkrrEpn1j0tNjc6X9hVkIluRhbPss3xT0exVaztOQlimujKm8Ba5Y73eHAAQ8sZqhYq3Axh Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 5.6.1.2065439, Antispam-Engine: 2.7.2.376379, Antispam-Data: 2013.1.28.53617 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' MULTIPLE_RCPTS 0.1, HTML_00_01 0.05, HTML_00_10 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_2000_2999 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __CP_MEDIA_BODY 0, __CP_URI_IN_BODY 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILER 0, __HAS_X_MAILING_LIST 0, __MIME_TEXT_ONLY 0, __MULTIPLE_RCPTS_CC_X2 0, __SANE_MSGID 0, __SUBJ_ALPHA_END 0, __TO_MALFORMED_2 0, __TO_NO_NAME 0, __URI_NO_WWW 0, __URI_NS , __YOUTUBE_RCVD 0' |
Commit Message
Vikas Sajjan
Jan. 28, 2013, 5:45 a.m. UTC
This patch adds display-timing node parsing using video helper function Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com> Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> --- drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35 ++++++++++++++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-)
Comments
Hello Vikas, On Mon, Jan 28, 2013 at 11:15 AM, Vikas Sajjan <vikas.sajjan@linaro.org> wrote: > This patch adds display-timing node parsing using video helper function > > Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com> > Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> > --- > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35 ++++++++++++++++++++++++++++-- > 1 file changed, 33 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > index bf0d9ba..975e7f7 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > @@ -19,6 +19,7 @@ > #include <linux/clk.h> > #include <linux/of_device.h> > #include <linux/pm_runtime.h> > +#include <linux/pinctrl/consumer.h> > > #include <video/samsung_fimd.h> > #include <drm/exynos_drm.h> > @@ -903,21 +904,51 @@ static int __devinit fimd_probe(struct platform_device *pdev) > struct device *dev = &pdev->dev; > struct fimd_context *ctx; > struct exynos_drm_subdrv *subdrv; > - struct exynos_drm_fimd_pdata *pdata; > + struct exynos_drm_fimd_pdata *pdata = pdev->dev.platform_data; > struct exynos_drm_panel_info *panel; > + struct fb_videomode *fbmode; > + struct device *disp_dev = &pdev->dev; > + struct pinctrl *pctrl; > struct resource *res; > int win; > int ret = -EINVAL; > > DRM_DEBUG_KMS("%s\n", __FILE__); > > - pdata = pdev->dev.platform_data; > + if (pdev->dev.of_node) { > + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); > + if (!pdata) { > + dev_err(dev, "memory allocation for pdata failed\n"); > + return -ENOMEM; > + } > + > + fbmode = devm_kzalloc(dev, sizeof(*fbmode), GFP_KERNEL); > + if (!fbmode) { > + dev_err(dev, "memory allocation for fbmode failed\n"); > + return -ENOMEM; > + } > + > + ret = of_get_fb_videomode(disp_dev->of_node, fbmode, -1); > + if (ret) { > + dev_err(dev, "failed to get fb_videomode\n"); > + return -EINVAL; > + } > + pdata->panel.timing = (struct fb_videomode) *fbmode; > + } > + > if (!pdata) { > dev_err(dev, "no platform data specified\n"); > return -EINVAL; > } > > + pctrl = devm_pinctrl_get_select_default(dev); Where this "pctrl" variable is being used? > + if (IS_ERR(pctrl)) { > + dev_err(dev, "no pinctrl data provided.\n"); > + return -EINVAL; > + } > + > panel = &pdata->panel; > + > if (!panel) { > dev_err(dev, "panel is null.\n"); > return -EINVAL; > -- > 1.7.9.5 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Jan 28, 2013 at 12:45 AM, Vikas Sajjan <vikas.sajjan@linaro.org> wrote: > This patch adds display-timing node parsing using video helper function > > Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com> > Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> > --- > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35 ++++++++++++++++++++++++++++-- > 1 file changed, 33 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > index bf0d9ba..975e7f7 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > @@ -19,6 +19,7 @@ > #include <linux/clk.h> > #include <linux/of_device.h> > #include <linux/pm_runtime.h> > +#include <linux/pinctrl/consumer.h> > > #include <video/samsung_fimd.h> > #include <drm/exynos_drm.h> > @@ -903,21 +904,51 @@ static int __devinit fimd_probe(struct platform_device *pdev) > struct device *dev = &pdev->dev; > struct fimd_context *ctx; > struct exynos_drm_subdrv *subdrv; > - struct exynos_drm_fimd_pdata *pdata; > + struct exynos_drm_fimd_pdata *pdata = pdev->dev.platform_data; > struct exynos_drm_panel_info *panel; > + struct fb_videomode *fbmode; > + struct device *disp_dev = &pdev->dev; > + struct pinctrl *pctrl; > struct resource *res; > int win; > int ret = -EINVAL; > > DRM_DEBUG_KMS("%s\n", __FILE__); > > - pdata = pdev->dev.platform_data; > + if (pdev->dev.of_node) { > + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); > + if (!pdata) { > + dev_err(dev, "memory allocation for pdata failed\n"); > + return -ENOMEM; > + } > + > + fbmode = devm_kzalloc(dev, sizeof(*fbmode), GFP_KERNEL); > + if (!fbmode) { > + dev_err(dev, "memory allocation for fbmode failed\n"); Why dev_err instead of DRM_ERROR? > + return -ENOMEM; > + } > + > + ret = of_get_fb_videomode(disp_dev->of_node, fbmode, -1); > + if (ret) { > + dev_err(dev, "failed to get fb_videomode\n"); > + return -EINVAL; > + } > + pdata->panel.timing = (struct fb_videomode) *fbmode; > + } > + > if (!pdata) { This condition is kind of weird, in that it's really checking if (!pdev->dev.of_node) implicitly (since you already check the allocation of pdata above). Seems like you could make this more clear and save a level of indentation by doing the following above: if (!pdev->dev.of_node) { DRM_ERROR("Device tree node was not found\n"); return -EINVAL; } Then just get rid of this check and the one wrapping the allocations above. Sean > dev_err(dev, "no platform data specified\n"); > return -EINVAL; > } > > + pctrl = devm_pinctrl_get_select_default(dev); > + if (IS_ERR(pctrl)) { > + dev_err(dev, "no pinctrl data provided.\n"); > + return -EINVAL; > + } > + > panel = &pdata->panel; > + > if (!panel) { > dev_err(dev, "panel is null.\n"); > return -EINVAL; > -- > 1.7.9.5 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Jan 28, 2013 at 9:24 PM, Sean Paul <seanpaul@chromium.org> wrote: > > On Mon, Jan 28, 2013 at 12:45 AM, Vikas Sajjan <vikas.sajjan@linaro.org> > wrote: > > This patch adds display-timing node parsing using video helper function > > > > Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com> > > Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> > > --- > > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35 > > ++++++++++++++++++++++++++++-- > > 1 file changed, 33 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c > > b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > > index bf0d9ba..975e7f7 100644 > > --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c > > +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c > > @@ -19,6 +19,7 @@ > > #include <linux/clk.h> > > #include <linux/of_device.h> > > #include <linux/pm_runtime.h> > > +#include <linux/pinctrl/consumer.h> > > > > #include <video/samsung_fimd.h> > > #include <drm/exynos_drm.h> > > @@ -903,21 +904,51 @@ static int __devinit fimd_probe(struct > > platform_device *pdev) > > struct device *dev = &pdev->dev; > > struct fimd_context *ctx; > > struct exynos_drm_subdrv *subdrv; > > - struct exynos_drm_fimd_pdata *pdata; > > + struct exynos_drm_fimd_pdata *pdata = pdev->dev.platform_data; > > struct exynos_drm_panel_info *panel; > > + struct fb_videomode *fbmode; > > + struct device *disp_dev = &pdev->dev; > > + struct pinctrl *pctrl; > > struct resource *res; > > int win; > > int ret = -EINVAL; > > > > DRM_DEBUG_KMS("%s\n", __FILE__); > > > > - pdata = pdev->dev.platform_data; > > + if (pdev->dev.of_node) { > > + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); > > + if (!pdata) { > > + dev_err(dev, "memory allocation for pdata > > failed\n"); > > + return -ENOMEM; > > + } > > + > > + fbmode = devm_kzalloc(dev, sizeof(*fbmode), GFP_KERNEL); > > + if (!fbmode) { > > + dev_err(dev, "memory allocation for fbmode > > failed\n"); > > Why dev_err instead of DRM_ERROR? > Will change it to DRM_ERROR > > + return -ENOMEM; > > + } > > + > > + ret = of_get_fb_videomode(disp_dev->of_node, fbmode, > > -1); > > + if (ret) { > > + dev_err(dev, "failed to get fb_videomode\n"); > > + return -EINVAL; > > + } > > + pdata->panel.timing = (struct fb_videomode) *fbmode; > > + } > > + > > if (!pdata) { > > This condition is kind of weird, in that it's really checking if > (!pdev->dev.of_node) implicitly (since you already check the > allocation of pdata above). > Even I thought the same. But kept this check because If DT node is not available and driver still gets plat data from machine file then the if (pdev->dev.of_node){} block will be skipped and plat data should be checked. > Seems like you could make this more clear and save a level of > indentation by doing the following above: > > if (!pdev->dev.of_node) { > DRM_ERROR("Device tree node was not found\n"); > return -EINVAL; > } > If I return -EINVAL here then this driver will become Full DT based one. and probe will be failed if DT node is not present. Will correct the if check and post the next version. /Leela Krishna Amudala > Then just get rid of this check and the one wrapping the allocations > above. > > Sean > > > dev_err(dev, "no platform data specified\n"); > > return -EINVAL; > > } > > > > + pctrl = devm_pinctrl_get_select_default(dev); > > + if (IS_ERR(pctrl)) { > > + dev_err(dev, "no pinctrl data provided.\n"); > > + return -EINVAL; > > + } > > + > > panel = &pdata->panel; > > + > > if (!panel) { > > dev_err(dev, "panel is null.\n"); > > return -EINVAL; > > -- > > 1.7.9.5 > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@lists.freedesktop.org > > http://lists.freedesktop.org/mailman/listinfo/dri-devel > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Jan 28, 2013 at 12:02 PM, Leela Krishna Amudala <l.krishna@samsung.com> wrote: > On Mon, Jan 28, 2013 at 9:24 PM, Sean Paul <seanpaul@chromium.org> wrote: >> >> On Mon, Jan 28, 2013 at 12:45 AM, Vikas Sajjan <vikas.sajjan@linaro.org> >> wrote: >> > This patch adds display-timing node parsing using video helper function >> > >> > Signed-off-by: Leela Krishna Amudala <l.krishna@samsung.com> >> > Signed-off-by: Vikas Sajjan <vikas.sajjan@linaro.org> >> > --- >> > drivers/gpu/drm/exynos/exynos_drm_fimd.c | 35 >> > ++++++++++++++++++++++++++++-- >> > 1 file changed, 33 insertions(+), 2 deletions(-) >> > >> > diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c >> > b/drivers/gpu/drm/exynos/exynos_drm_fimd.c >> > index bf0d9ba..975e7f7 100644 >> > --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c >> > +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c >> > @@ -19,6 +19,7 @@ >> > #include <linux/clk.h> >> > #include <linux/of_device.h> >> > #include <linux/pm_runtime.h> >> > +#include <linux/pinctrl/consumer.h> >> > >> > #include <video/samsung_fimd.h> >> > #include <drm/exynos_drm.h> >> > @@ -903,21 +904,51 @@ static int __devinit fimd_probe(struct >> > platform_device *pdev) >> > struct device *dev = &pdev->dev; >> > struct fimd_context *ctx; >> > struct exynos_drm_subdrv *subdrv; >> > - struct exynos_drm_fimd_pdata *pdata; >> > + struct exynos_drm_fimd_pdata *pdata = pdev->dev.platform_data; >> > struct exynos_drm_panel_info *panel; >> > + struct fb_videomode *fbmode; >> > + struct device *disp_dev = &pdev->dev; >> > + struct pinctrl *pctrl; >> > struct resource *res; >> > int win; >> > int ret = -EINVAL; >> > >> > DRM_DEBUG_KMS("%s\n", __FILE__); >> > >> > - pdata = pdev->dev.platform_data; >> > + if (pdev->dev.of_node) { >> > + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); >> > + if (!pdata) { >> > + dev_err(dev, "memory allocation for pdata >> > failed\n"); >> > + return -ENOMEM; >> > + } >> > + >> > + fbmode = devm_kzalloc(dev, sizeof(*fbmode), GFP_KERNEL); >> > + if (!fbmode) { >> > + dev_err(dev, "memory allocation for fbmode >> > failed\n"); >> >> Why dev_err instead of DRM_ERROR? >> > > Will change it to DRM_ERROR > >> > + return -ENOMEM; >> > + } >> > + >> > + ret = of_get_fb_videomode(disp_dev->of_node, fbmode, >> > -1); >> > + if (ret) { >> > + dev_err(dev, "failed to get fb_videomode\n"); >> > + return -EINVAL; >> > + } >> > + pdata->panel.timing = (struct fb_videomode) *fbmode; >> > + } >> > + >> > if (!pdata) { >> >> This condition is kind of weird, in that it's really checking if >> (!pdev->dev.of_node) implicitly (since you already check the >> allocation of pdata above). >> > Even I thought the same. But kept this check because If DT node is > not available and driver still gets plat data from machine file then the > if (pdev->dev.of_node){} block will be skipped and plat data should be checked. > In that case, you should probably add that assignment of pdata = pdev->dev.platform_data back ;) Sean >> Seems like you could make this more clear and save a level of >> indentation by doing the following above: >> >> if (!pdev->dev.of_node) { >> DRM_ERROR("Device tree node was not found\n"); >> return -EINVAL; >> } >> > If I return -EINVAL here then this driver will become Full DT based one. > and probe will be failed if DT node is not present. > > Will correct the if check and post the next version. > > /Leela Krishna Amudala >> Then just get rid of this check and the one wrapping the allocations >> above. >> >> Sean >> >> > dev_err(dev, "no platform data specified\n"); >> > return -EINVAL; >> > } >> > >> > + pctrl = devm_pinctrl_get_select_default(dev); >> > + if (IS_ERR(pctrl)) { >> > + dev_err(dev, "no pinctrl data provided.\n"); >> > + return -EINVAL; >> > + } >> > + >> > panel = &pdata->panel; >> > + >> > if (!panel) { >> > dev_err(dev, "panel is null.\n"); >> > return -EINVAL; >> > -- >> > 1.7.9.5 >> > >> > _______________________________________________ >> > dri-devel mailing list >> > dri-devel@lists.freedesktop.org >> > http://lists.freedesktop.org/mailman/listinfo/dri-devel >> _______________________________________________ >> dri-devel mailing list >> dri-devel@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/dri-devel -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/gpu/drm/exynos/exynos_drm_fimd.c b/drivers/gpu/drm/exynos/exynos_drm_fimd.c index bf0d9ba..975e7f7 100644 --- a/drivers/gpu/drm/exynos/exynos_drm_fimd.c +++ b/drivers/gpu/drm/exynos/exynos_drm_fimd.c @@ -19,6 +19,7 @@ #include <linux/clk.h> #include <linux/of_device.h> #include <linux/pm_runtime.h> +#include <linux/pinctrl/consumer.h> #include <video/samsung_fimd.h> #include <drm/exynos_drm.h> @@ -903,21 +904,51 @@ static int __devinit fimd_probe(struct platform_device *pdev) struct device *dev = &pdev->dev; struct fimd_context *ctx; struct exynos_drm_subdrv *subdrv; - struct exynos_drm_fimd_pdata *pdata; + struct exynos_drm_fimd_pdata *pdata = pdev->dev.platform_data; struct exynos_drm_panel_info *panel; + struct fb_videomode *fbmode; + struct device *disp_dev = &pdev->dev; + struct pinctrl *pctrl; struct resource *res; int win; int ret = -EINVAL; DRM_DEBUG_KMS("%s\n", __FILE__); - pdata = pdev->dev.platform_data; + if (pdev->dev.of_node) { + pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL); + if (!pdata) { + dev_err(dev, "memory allocation for pdata failed\n"); + return -ENOMEM; + } + + fbmode = devm_kzalloc(dev, sizeof(*fbmode), GFP_KERNEL); + if (!fbmode) { + dev_err(dev, "memory allocation for fbmode failed\n"); + return -ENOMEM; + } + + ret = of_get_fb_videomode(disp_dev->of_node, fbmode, -1); + if (ret) { + dev_err(dev, "failed to get fb_videomode\n"); + return -EINVAL; + } + pdata->panel.timing = (struct fb_videomode) *fbmode; + } + if (!pdata) { dev_err(dev, "no platform data specified\n"); return -EINVAL; } + pctrl = devm_pinctrl_get_select_default(dev); + if (IS_ERR(pctrl)) { + dev_err(dev, "no pinctrl data provided.\n"); + return -EINVAL; + } + panel = &pdata->panel; + if (!panel) { dev_err(dev, "panel is null.\n"); return -EINVAL;