[U-Boot] [PATCH v3] mx31ads: Use the new relocation scheme

[Fabio Estevam]

On Sun, Jun 12, 2011 at 6:58 PM, Fabio Estevam <festevam at gmail.com> wrote:
> Hi Felix,
>
> On Sun, Jun 5, 2011 at 4:28 AM, Felix Radensky <felix at embedded-sol.com> 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:

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),

Then please let us know what srev prints on the console.

Regards,

Fabio Estevam