[U-Boot] [PATCH 3/6] SPL: Port SPL framework to powerpc

[Tom Rini <trini@ti.com>]

On 08/23/2012 11:16 AM, Stefan Roese wrote:
> On 08/23/2012 07:10 PM, Tom Rini wrote:
>>> +#ifdef CONFIG_ARM
>>>  /* Define global data structure pointer to it*/
>>>  gd_t gdata __attribute__ ((section(".data")));
>>> +#endif
>>
>> So you handle cleaning up the BSS differently, interesting.  I'm going
>> to see if that would work for ARM too..
> 
> Yes. Might be that I missed something though. I'll re-check tomorrow.
> 
>> [snip]
>>> @@ -89,7 +106,11 @@ void spl_parse_image_header(const struct image_header *header)
>>>  		spl_image.size = __be32_to_cpu(header->ih_size) + header_size;
>>>  		spl_image.entry_point = __be32_to_cpu(header->ih_load);
>>>  		/* Load including the header */
>>> +#ifdef CONFIG_ARM
>>>  		spl_image.load_addr = spl_image.entry_point - header_size;
>>> +#else
>>> +		spl_image.load_addr = __be32_to_cpu(header->ih_load);
>>> +#endif
>>
>> This isn't an ARM-ism but is instead because spl_nor.c isn't offsetting
>> where the header is like mmc/nand/ymodem do, yes?  Would it be possible
>> to make spl_nor.c behave like the others?  One of the reasons I ask is
>> I'm looking at a NOR chip on my desk...
> 
> I was wondering about this line as well. Please explain: Why can't ARM
> just use header->ih_load as load_addr?

Off the top of my head, I believe what goes on is that we read things
into SDRAM such that the header is taken into account and we don't need
to relocate the payload (U-Boot or Linux).

-- 
Tom