From patchwork Sun Jun 12 22:31:23 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Fabio Estevam X-Patchwork-Id: 100120 X-Patchwork-Delegate: sbabic@denx.de Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from theia.denx.de (theia.denx.de [85.214.87.163]) by ozlabs.org (Postfix) with ESMTP id 85F88B704F for ; Mon, 13 Jun 2011 08:31:39 +1000 (EST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 77BD62818D; Mon, 13 Jun 2011 00:31:36 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 2qC8bK9+O5Qx; Mon, 13 Jun 2011 00:31:36 +0200 (CEST) Received: from theia.denx.de (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 02B452818E; Mon, 13 Jun 2011 00:31:33 +0200 (CEST) Received: from localhost (localhost [127.0.0.1]) by theia.denx.de (Postfix) with ESMTP id 9083B2818E for ; Mon, 13 Jun 2011 00:31:30 +0200 (CEST) X-Virus-Scanned: Debian amavisd-new at theia.denx.de Received: from theia.denx.de ([127.0.0.1]) by localhost (theia.denx.de [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id R6sdaH168Z8h for ; Mon, 13 Jun 2011 00:31:29 +0200 (CEST) X-policyd-weight: NOT_IN_SBL_XBL_SPAMHAUS=-1.5 NOT_IN_SPAMCOP=-1.5 NOT_IN_BL_NJABL=-1.5 (only DNSBL check requested) Received: from mail-pz0-f44.google.com (mail-pz0-f44.google.com [209.85.210.44]) by theia.denx.de (Postfix) with ESMTPS id 859F92818D for ; Mon, 13 Jun 2011 00:31:27 +0200 (CEST) Received: by pzk5 with SMTP id 5so1874040pzk.3 for ; Sun, 12 Jun 2011 15:31:24 -0700 (PDT) MIME-Version: 1.0 Received: by 10.68.8.105 with SMTP id q9mr1806883pba.3.1307917883910; Sun, 12 Jun 2011 15:31:23 -0700 (PDT) Received: by 10.68.42.73 with HTTP; Sun, 12 Jun 2011 15:31:23 -0700 (PDT) In-Reply-To: References: <4DEAAF27.8080406@embedded-sol.com> <4DEB3036.1010803@embedded-sol.com> Date: Sun, 12 Jun 2011 19:31:23 -0300 Message-ID: From: Fabio Estevam To: Felix Radensky Cc: fabio.estevam@freescale.com, U-Boot-Denx Subject: Re: [U-Boot] [PATCH v3] mx31ads: Use the new relocation scheme X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.9 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: u-boot-bounces@lists.denx.de Errors-To: u-boot-bounces@lists.denx.de On Sun, Jun 12, 2011 at 6:58 PM, Fabio Estevam wrote: > Hi Felix, > > On Sun, Jun 5, 2011 at 4:28 AM, Felix Radensky wrote: > ... >> Unrelated to your patch, u-boot prints >> >> CPU:   Freescale i.MX31 rev 2.0 unknown at 531 MHz.Reset cause: WDOG >> >> Do you think "unknown" can be replaced by something more meaningful ? > > I don't see the "unknown" string on my MX31PDK board. > > Can you try the patch below and let me know if it gets rid of it? > > diff --git a/arch/arm/cpu/arm1136/mx31/generic.c b/arch/arm/cpu/arm1136/mx31/gen > index 4ebf38d..f0cfa8a 100644 > --- a/arch/arm/cpu/arm1136/mx31/generic.c > +++ b/arch/arm/cpu/arm1136/mx31/generic.c > @@ -129,8 +129,9 @@ u32 get_cpu_rev(void) >        for (i = 0; i < ARRAY_SIZE(mx31_cpu_type); i++) >                if (srev == mx31_cpu_type[i].srev) >                        return mx31_cpu_type[i].v; > +       srev|= 0x8000; > > -       return srev | 0x8000; > +       return srev; >  } Please ignore my last message. I think you could try: Then please let us know what srev prints on the console. Regards, Fabio Estevam diff --git a/arch/arm/cpu/arm1136/mx31/generic.c b/arch/arm/cpu/arm1136/mx31/gen index 4ebf38d..36ceb02 100644 --- a/arch/arm/cpu/arm1136/mx31/generic.c +++ b/arch/arm/cpu/arm1136/mx31/generic.c @@ -159,6 +159,7 @@ static char *get_reset_cause(void) int print_cpuinfo (void) { u32 srev = get_cpu_rev(); + printf ("srev = %08x\n", srev); printf("CPU: Freescale i.MX31 rev %d.%d%s at %d MHz.", (srev & 0xF0) >> 4, (srev & 0x0F),