[U-Boot] env_mmc: Fix crashing bug encountered after enabling ARM relocation
Steve Sakoman
sakoman at gmail.com
Sun Oct 10 17:47:55 CEST 2010
On Sun, Oct 10, 2010 at 7:52 AM, Sergei Shtylyov <sshtylyov at mvista.com> wrote:
> Hello.
>
> On 10-10-2010 17:42, Steve Sakoman wrote:
>
>>>> The crash was occuring in env_relocate because it was being called prior
>>>> to mmc_initialize. This patch moves the MMC initialization earlier in
>>>> the init process.
>
>>>> This patch also cleans up the env_relocate_spec code in env_mmc.c
>
>>>> Developed jointly with Stefano Babic
>
>>>> Signed-off-by: Steve Sakoman<steve.sakoman at linaro.org>
>>>> ---
>
>>>> diff --git a/arch/arm/lib/board.c b/arch/arm/lib/board.c
>>>> index 5f2dfd0..0e2f129 100644
>>>> --- a/arch/arm/lib/board.c
>>>> +++ b/arch/arm/lib/board.c
>>>> @@ -779,6 +779,11 @@ void board_init_r (gd_t *id, ulong dest_addr)
>>>> onenand_init();
>>>> #endif
>>>>
>>>> +#ifdef CONFIG_GENERIC_MMC
>>>> + puts ("MMC: ");
>>>> + mmc_initialize (bd);
>>>
>>> This would cause checkpatch.pl to complain -- no spaces allowed before
>>> (.
>
>> I will resubmit a V2 with that change as well as Stefano's Signed-off-by
>
> You've posted V2 with Stefano's ACK... and this line unchanged. :-)
???
+#ifdef CONFIG_GENERIC_MMC
+ puts("MMC: ");
+ mmc_initialize (bd);
+#endif
+
It looks changed to me!
Steve
More information about the U-Boot
mailing list