On Sat, Dec 21, 2013 at 12:59:41AM +0000, Russell King - ARM Linux wrote:
> On Fri, Dec 20, 2013 at 08:55:03AM -0800, Tony Lindgren wrote:
> > Yeah this seems to do the trick for me for the built-in DSS on LDP.
> >
> > Tony
> >
> > 8< -----------------------------------
> > From: Tony Lindgren <tony@xxxxxxxxxxx>
> > Date: Fri, 20 Dec 2013 08:53:27 -0800
> > Subject: [PATCH] ARM: OMAP2+: Fix LCD panel backlight regression for LDP legacy booting
> >
> > Looks like the LCD panel on LDP has been broken quite a while, and
> > recently got fixed. However, there's still an issue where the panel
> > backlight does not come on if the LCD drivers are built into the
> > kernel.
> >
> > Fix the issue by registering the DPI LCD panel only after the twl4030
> > GPIO has probed.
> >
> > Reported-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
> >
> > --- a/arch/arm/mach-omap2/board-ldp.c
> > +++ b/arch/arm/mach-omap2/board-ldp.c
> > @@ -248,7 +248,7 @@ static int ldp_twl_gpio_setup(struct device *dev, unsigned gpio, unsigned ngpio)
> > /* Backlight enable GPIO */
> > ldp_lcd_pdata.backlight_gpio = gpio + 15;
> >
> > - return 0;
> > + return platform_device_register(&ldp_lcd_device);
> > }
> >
> > static struct twl4030_gpio_platform_data ldp_gpio_data = {
> > @@ -346,7 +346,6 @@ static struct omap2_hsmmc_info mmc[] __initdata = {
> >
> > static struct platform_device *ldp_devices[] __initdata = {
> > &ldp_gpio_keys_device,
> > - &ldp_lcd_device,
> > };
> >
> > #ifdef CONFIG_OMAP_MUX
>
> I've added it to the test build for tonight (only as an uncommitted
> modification) so we'll see what happens tomorrow morning.
BTW, yes, this does result in the display working again on the LDP.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
