[U-Boot] [PATCH] spl: ram: Fix u_boot_pos calculation

[Simon Goldschmidt]

On 06.10.2018 17:29, Simon Goldschmidt wrote:
> On 04.10.2018 09:29, Michal Simek wrote:
>> The patch:
>> "spl: Weed out CONFIG_SYS_TEXT_BASE usage"
>> (sha1: 04ce5427bd3914cab8be78513275a20ab878520a)
>> introduced bug where assigning to u_boot_pos variable wasn't done
>> which end up in situation where SPL wasn't able to find out
>> image on the selected address.
>>
>> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
> I needed to apply this to boot my socfpga cyclone5 from FPGA (both SPL
> and U-Boot in FPGA onchip RAM).
>
> Tested-by: Simon Goldschmidt <simon.k.r.goldschmid at gmail.com>

There's a typo in my mail address :-(
Here's the correct tag:

Tested-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>


>
>> ---
>>
>> Lines are unfortunatelly longer then 80 char limits but
>> there is no nice way how to fix it.
>>
>> ---
>>    common/spl/spl_ram.c | 5 ++---
>>    1 file changed, 2 insertions(+), 3 deletions(-)
>>
>> diff --git a/common/spl/spl_ram.c b/common/spl/spl_ram.c
>> index 619b39a53748..5fcc3b1504bb 100644
>> --- a/common/spl/spl_ram.c
>> +++ b/common/spl/spl_ram.c
>> @@ -63,9 +63,8 @@ static int spl_ram_load_image(struct spl_image_info *spl_image,
>>    			 * No binman support or no information. For now, fix it
>>    			 * to the address pointed to by U-Boot.
>>    			 */
>> -			header = spl_get_load_buffer(-sizeof(*header),
>> -						     sizeof(*header));
>> -
>> +			u_boot_pos = (ulong)spl_get_load_buffer(-sizeof(*header),
>> +								sizeof(*header));
>>    		}
>>    		header = (struct image_header *)map_sysmem(u_boot_pos, 0);
>>    
>