From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dirk Behme Subject: Re: Boot failure on OMAP 3430 SDP Date: Sat, 21 Jun 2008 14:18:43 +0200 Message-ID: <485CF1A3.4070101@googlemail.com> References: <5A47E75E594F054BAF48C5E4FC4B92AB022BE46296@dbde02.ent.ti.com>,<485CA347.909@googlemail.com> <5A47E75E594F054BAF48C5E4FC4B92AB022BB66209@dbde02.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from fk-out-0910.google.com ([209.85.128.191]:53810 "EHLO fk-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750939AbYFUMSz (ORCPT ); Sat, 21 Jun 2008 08:18:55 -0400 Received: by fk-out-0910.google.com with SMTP id 18so1613915fkq.5 for ; Sat, 21 Jun 2008 05:18:53 -0700 (PDT) In-Reply-To: <5A47E75E594F054BAF48C5E4FC4B92AB022BB66209@dbde02.ent.ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Gadiyar, Anand" Cc: "linux-omap@vger.kernel.org" , "jouni.hogander@nokia.com" , Paul Walmsley Gadiyar, Anand wrote: >>>Hi all, >>> >>>Commit 5b36d70fa08f09de161487fe7f6e04200ffb8d71 breaks boot on OMAP3430 SDP. Reversing the patch allows the boot to work. >>> >>>Bootlog and the patch are shown below. >>> >>>Regards, >>>Anand >>> >>>================================================ >>> >>>commit 5b36d70fa08f09de161487fe7f6e04200ffb8d71 >>>Author: Jouni Hogander > >>>Date: Mon Jun 9 12:32:40 2008 +0300 >>> >>> PRCM: OMAP3: Fix to wrongly modified omap2_clk_wait_ready >>> >>> omap2_clk_wait_ready was wrongly modified to check >>> registers contents. This fix changes it back to check >>> addresses. >>> >>> Signed-off-by: Jouni Hogander > >>> Acked-by: Paul Walmsley > >>> Signed-off-by: Tony Lindgren > >>> >>>diff --git a/arch/arm/mach-omap2/clock.c b/arch/arm/mach-omap2/clock.c >>>index d3ab537..ed15868 100644 >>>--- a/arch/arm/mach-omap2/clock.c >>>+++ b/arch/arm/mach-omap2/clock.c >>>@@ -246,8 +246,8 @@ static void omap2_clk_wait_ready(struct clk *clk) >>> /* REVISIT: What are the appropriate exclusions for 34XX? */ >>> /* OMAP3: ignore DSS-mod clocks */ >>> if (cpu_is_omap34xx() && >>>- ((reg & ~0xff) == cm_read_mod_reg(OMAP3430_DSS_MOD, 0) || >>>- (((reg & ~0xff) == cm_read_mod_reg(CORE_MOD, 0)) && >>>+ ((reg & ~0xff) == (__force u32)OMAP34XX_CM_REGADDR(OMAP3430_DSS_MOD, 0) || >>>+ (((reg & ~0xff) == (__force u32)OMAP34XX_CM_REGADDR(CORE_MOD, 0)) && >>> clk-> >enable_bit == OMAP3430_EN_SSI_SHIFT))) >>> return; >> >>Hmm, looking at recent git, we already have OMAP34XX_CM_REGADDR >>instead of cm_read_mod_reg there >> >>http://source.mvista.com/git/?p=linux-omap-2.6.git;a=blob;f=arch/arm/mach-omap2/clock.c;h=ed1586847db4dd95cff8016e4259b533f1582737;hb=HEAD >> >>and it seems that this patch is already applied >> >>http://source.mvista.com/git/?p=linux-omap-2.6.git;a=commitdiff;h=5b36d70fa08f09de161487fe7f6e04200ffb8d71 >> >>? >> >>*BUT*: I still see below Unhandled fault at OMAP3 based Beagle board! >>Any idea? > > Oops. Sorry for this confusion. The patch inlined in the mail was the commit that > causes the boot failure - not the fix. I would rather have Jouni and Paul look at the > commit. I haven't looked too deeply at the patch to see why it causes the crash. > > Until this is fixed, you might just want to apply the patch in reverse. Ah, yes, sorry for my misunderstanding! And many thanks for finding the cause of this issue. Hopefully we will have a fix for this soon now. Sorry and thanks Dirk