From mboxrd@z Thu Jan 1 00:00:00 1970 From: l.fu@pengutronix.de (Luotao Fu) Date: Thu, 17 Jun 2010 21:22:57 +0200 Subject: [PATCH] video/imxfb: fix suspend when screen is blanked In-Reply-To: <1276801147-4103-5-git-send-email-eric@eukrea.com> References: <1276801147-4103-1-git-send-email-eric@eukrea.com> <1276801147-4103-2-git-send-email-eric@eukrea.com> <1276801147-4103-3-git-send-email-eric@eukrea.com> <1276801147-4103-4-git-send-email-eric@eukrea.com> <1276801147-4103-5-git-send-email-eric@eukrea.com> Message-ID: <20100617192257.GA18802@pengutronix.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On Thu, Jun 17, 2010 at 08:59:05PM +0200, Eric B?nard wrote: > if the screen is blanked when suspending, the clock is already disabled, > leading to the following log : > WARNING: at arch/arm/plat-mxc/clock.c:59 __clk_disable+0x68/0x74() > Modules linked in: > [] (unwind_backtrace+0x0/0xf8) from [] (warn_slowpath_common+0x48/0x78) > [] (warn_slowpath_common+0x48/0x78) from [] (__clk_disable+0x68/0x74) > [] (__clk_disable+0x68/0x74) from [] (__clk_disable+0x1c/0x74) > [] (__clk_disable+0x1c/0x74) from [] (clk_disable+0x24/0x34) > [] (clk_disable+0x24/0x34) from [] (imxfb_disable_controller+0x38/0x48) > [] (imxfb_disable_controller+0x38/0x48) from [] (imxfb_suspend+0x14/0x1c) > > Signed-off-by: Eric B?nard > Cc: kernel at pengutronix.de > Cc: u.kleine-koenig at pengutronix.de > Cc: linux-fbdev at vger.kernel.org > Cc: linux-arm-kernel at lists.infradead.org > --- > drivers/video/imxfb.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/video/imxfb.c b/drivers/video/imxfb.c > index b4b6dec..6cbfbff 100644 > --- a/drivers/video/imxfb.c > +++ b/drivers/video/imxfb.c > @@ -597,8 +597,8 @@ static int imxfb_suspend(struct platform_device *dev, pm_message_t state) > struct imxfb_info *fbi = info->par; > > pr_debug("%s\n", __func__); > - > - imxfb_disable_controller(fbi); > + if (readl(fbi->regs + LCDC_RMCR)) > + imxfb_disable_controller(fbi); > return 0; > } I've already submitted a patch to fix this issue: http://lkml.org/lkml/2010/6/3/187 cheers Luotao Fu -- Pengutronix e.K. | Dipl.-Ing. Luotao Fu | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: Digital signature URL: