From mboxrd@z Thu Jan  1 00:00:00 1970
From: Fabio Estevam <fabioestevam@yahoo.com>
Date: Thu, 10 Mar 2011 05:47:15 -0800 (PST)
Subject: [U-Boot] [PATCH 1/2] ARM: MX31: print WRSR to indicate the
	source of the last reset
In-Reply-To: <1299754388-31648-1-git-send-email-agust@denx.de>
Message-ID: <324735.19801.qm@web39408.mail.mud.yahoo.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Anatolij,

--- On Thu, 3/10/11, Anatolij Gustschin <agust@denx.de> wrote:
....?? ???
> mx31_get_mcu_main_clk() / 1000000);
> +??? struct wdog_regs *wdog = (struct> wdog_regs *)WDOG_BASE;
> +
> +??? printf("CPU:???Freescale i.MX31 at %d MHz (WRSR=0x%04x)\n",
> +??
> mx31_get_mcu_main_clk() / 1000000, wdog->wrsr);
>  ??? return 0;

Wouldn?t it be better to show a string for the reset source, rather than a pure register value?

I sent a similar patch yesterday for MX31PDK: http://permalink.gmane.org/gmane.comp.boot-loaders.u-boot/95615

,but I agree that moving it to the generic code is a better idea.

Regards,

Fabio Estevam