Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752661AbaBMXhb (ORCPT ); Thu, 13 Feb 2014 18:37:31 -0500 Received: from mail-out.m-online.net ([212.18.0.10]:54934 "EHLO mail-out.m-online.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751694AbaBMXh3 (ORCPT ); Thu, 13 Feb 2014 18:37:29 -0500 X-Auth-Info: ysJh/Cix06aDr7qU6MRdRwx5BgYyX2ZbDwrw8iU3k1A= Date: Fri, 14 Feb 2014 00:37:20 +0100 From: Anatolij Gustschin To: Heiko Schocher Cc: linux-arm-kernel@lists.infradead.org, Benoit Parrot , Rob Clark , David Airlie , Grant Likely , Rob Herring , Tomi Valkeinen , Sachin Kamat , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH] gpu:drm:tilcdc: backlight node never found Message-ID: <20140214003720.168c2d10@crub> In-Reply-To: <1392299258-16968-1-git-send-email-hs@denx.de> References: <1392299258-16968-1-git-send-email-hs@denx.de> X-Mailer: Claws Mail 3.8.0 (GTK+ 2.24.10; i686-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 13 Feb 2014 14:47:38 +0100 Heiko Schocher wrote: > In panel_probe() the backlight node is never found, correct this. > > Signed-off-by: Heiko Schocher Acked-by: Anatolij Gustschin > Cc: Anatolij Gustschin > Cc: Benoit Parrot > Cc: Rob Clark > Cc: David Airlie > Cc: Grant Likely > Cc: Rob Herring > Cc: Tomi Valkeinen > Cc: Sachin Kamat > Cc: dri-devel@lists.freedesktop.org > Cc: linux-kernel@vger.kernel.org > Cc: devicetree@vger.kernel.org > --- > drivers/gpu/drm/tilcdc/tilcdc_panel.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/tilcdc/tilcdc_panel.c b/drivers/gpu/drm/tilcdc/tilcdc_panel.c > index 86c6732..0b97cf4 100644 > --- a/drivers/gpu/drm/tilcdc/tilcdc_panel.c > +++ b/drivers/gpu/drm/tilcdc/tilcdc_panel.c > @@ -356,6 +356,7 @@ static struct of_device_id panel_of_match[]; > static int panel_probe(struct platform_device *pdev) > { > struct device_node *node = pdev->dev.of_node; > + struct device_node *backlight_node; > struct panel_module *panel_mod; > struct tilcdc_module *mod; > struct pinctrl *pinctrl; > @@ -395,9 +396,15 @@ static int panel_probe(struct platform_device *pdev) > > mod->preferred_bpp = panel_mod->info->bpp; > > - panel_mod->backlight = of_find_backlight_by_node(node); > - if (panel_mod->backlight) > - dev_info(&pdev->dev, "found backlight\n"); > + backlight_node = of_parse_phandle(pdev->dev.of_node, "backlight", 0); > + if (backlight_node) { > + panel_mod->backlight = > + of_find_backlight_by_node(backlight_node); > + if (panel_mod->backlight) > + dev_info(&pdev->dev, "found backlight\n"); > + } else { > + dev_warn(&pdev->dev, "backlight node missing...\n"); > + } > > return 0; > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/