[PATCH] cmd: go: Flush cache before jumping to app/image

Stefan sr at denx.de
Wed Feb 12 15:24:03 CET 2020


Hi Mauro,

On 12.02.20 14:50, Mauro Condarelli wrote:
> Hi Stefan,
> 
> On 2/12/20 2:22 PM, Stefan wrote:
>> Hi Mauro,
>>
>> On 12.02.20 14:09, Mauro Condarelli wrote:
>>> Hi Daniel,
>>> Hi Stefan,
>>>
>>> I'll test this ASAP, but, in the meantime, please have a look at the
>>> strange (and perhaps related) code in arch/mips/mach-mtmips/cpu.c
>>
>> You are most likely referring to the code in last_stage_init() that
>> also deals with a potential cache issue. Its still unresolved and
>> this "hack" is still needed for a working U-Boot image (e.g. no timeouts
>> in tftp).
> Yes, I was referring to that strange "D-cache" cleanup.
> Why not just a cache-flush()?

Frankly, I don't remember the exact details. I definitely tested also
with cache_flush() but either it introduced too much boot delay or it
simply didn't work. This d-cache issue was (is) pretty nasty.

>> Its only partly related to the current issue though as it also deals
>> with "cache" but addresses a different issue. Please test and report
>> back.
> Done.
> Confirmed.
> With new version in SPI NOR I could load (and execute successfully)
> all versions I had saved on my tftp server.

Good. :)

> If some further testing is useful don't hesitate to ask.

I'll send v2 in a minute. Please re-test it and send your tested-by, if
this also works for you.

Thanks,
Stefan


More information about the U-Boot mailing list