From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] ARM: OMAP: sram: fix BUG in dpll code for !PM case Date: Tue, 17 Apr 2012 10:22:27 -0700 Message-ID: <20120417172227.GB21106@atomide.com> References: <1333749201-23255-1-git-send-email-notasas@gmail.com> <20120409224946.GC6487@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-ewr.mailhop.org ([204.13.248.71]:29401 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753330Ab2DQRWb (ORCPT ); Tue, 17 Apr 2012 13:22:31 -0400 Content-Disposition: inline In-Reply-To: <20120409224946.GC6487@atomide.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Grazvydas Ignotas Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org * Tony Lindgren [120409 15:52]: > * Grazvydas Ignotas [120406 14:56]: > > _omap3_sram_configure_core_dpll is called when SDRC is reprogrammed, > > which is done regardless of CONFIG_PM setting, so we always need it's > > setup code too. Without this, we hit a BUG() on OMAP3 when kernel is > > built without CONFIG_PM: > > > > Reprogramming SDRC clock to 332000000 Hz > > ------------[ cut here ]------------ > > kernel BUG at arch/arm/plat-omap/sram.c:342! > > Internal error: Oops - BUG: 0 [#1] ARM > > ... > > [] (omap3_configure_core_dpll+0x68/0x6c) from [] (omap3_core_dpll_m2_set_rate+0x1) > > [] (omap3_core_dpll_m2_set_rate+0x138/0x1b0) from [] (omap2_clk_set_rate+0x14/0x2) > > [] (omap2_clk_set_rate+0x14/0x20) from [] (clk_set_rate+0x54/0x74) > > [] (clk_set_rate+0x54/0x74) from [] (omap_sdrc_init+0x70/0x90) > > [] (omap_sdrc_init+0x70/0x90) from [] (omap3pandora_init+0x11c/0x164) > > [] (omap3pandora_init+0x11c/0x164) from [] (customize_machine+0x20/0x28) > > [] (customize_machine+0x20/0x28) from [] (do_one_initcall+0xa0/0x16c) > > [] (do_one_initcall+0xa0/0x16c) from [] (kernel_init+0x104/0x1ac) > > [] (kernel_init+0x104/0x1ac) from [] (kernel_thread_exit+0x0/0x8) > > Looks like this was caused by the added ifdef CONFIG_PM from > commit 3231fc88 (OMAP3: PM: restore SRAM functions after off-mode). Applying this into fixes. Tony From mboxrd@z Thu Jan 1 00:00:00 1970 From: tony@atomide.com (Tony Lindgren) Date: Tue, 17 Apr 2012 10:22:27 -0700 Subject: [PATCH] ARM: OMAP: sram: fix BUG in dpll code for !PM case In-Reply-To: <20120409224946.GC6487@atomide.com> References: <1333749201-23255-1-git-send-email-notasas@gmail.com> <20120409224946.GC6487@atomide.com> Message-ID: <20120417172227.GB21106@atomide.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org * Tony Lindgren [120409 15:52]: > * Grazvydas Ignotas [120406 14:56]: > > _omap3_sram_configure_core_dpll is called when SDRC is reprogrammed, > > which is done regardless of CONFIG_PM setting, so we always need it's > > setup code too. Without this, we hit a BUG() on OMAP3 when kernel is > > built without CONFIG_PM: > > > > Reprogramming SDRC clock to 332000000 Hz > > ------------[ cut here ]------------ > > kernel BUG at arch/arm/plat-omap/sram.c:342! > > Internal error: Oops - BUG: 0 [#1] ARM > > ... > > [] (omap3_configure_core_dpll+0x68/0x6c) from [] (omap3_core_dpll_m2_set_rate+0x1) > > [] (omap3_core_dpll_m2_set_rate+0x138/0x1b0) from [] (omap2_clk_set_rate+0x14/0x2) > > [] (omap2_clk_set_rate+0x14/0x20) from [] (clk_set_rate+0x54/0x74) > > [] (clk_set_rate+0x54/0x74) from [] (omap_sdrc_init+0x70/0x90) > > [] (omap_sdrc_init+0x70/0x90) from [] (omap3pandora_init+0x11c/0x164) > > [] (omap3pandora_init+0x11c/0x164) from [] (customize_machine+0x20/0x28) > > [] (customize_machine+0x20/0x28) from [] (do_one_initcall+0xa0/0x16c) > > [] (do_one_initcall+0xa0/0x16c) from [] (kernel_init+0x104/0x1ac) > > [] (kernel_init+0x104/0x1ac) from [] (kernel_thread_exit+0x0/0x8) > > Looks like this was caused by the added ifdef CONFIG_PM from > commit 3231fc88 (OMAP3: PM: restore SRAM functions after off-mode). Applying this into fixes. Tony