[U-Boot] [PATCH] nios2: set default cache configuration in start.S
Marek Vasut
marex at denx.de
Fri Oct 9 16:39:10 CEST 2015
On Friday, October 09, 2015 at 11:03:36 AM, Thomas Chou wrote:
> Hi Marek,
Hi,
> On 10/09/2015 05:14 AM, Marek Vasut wrote:
> > Did I miss the DT part somewhere or is that not part of this patch ?
>
> As we are just barely starting from reset, there is no luxury of DT.
> After this, DT will be used with the cache flush patch.
That's right.
> > Another question I have is -- why don't we set CONFIG_SYS_{I,D}CACHE_SIZE
> > to maximum value? This should work for all nios2 cache configurations
> > and it should also be correct, no ? Speed might be a bit of an issue
> > here, but I doubt the impact would be that large.
>
> I am doing all the best to get rid of the CONFIG_SYS_{I,D}CACHE_SIZE...
> So I avoid to use the same macro names. Here I use the maximum value in
> this startup only. The speed penalty is only once here. After we get DT
> initialized, we have the correct values of cache configuration.
Ah, right. That's good.
> It might be awful to use the maximum value all the time. See, we flush
> cache for every eth packet!
You should only flush the part where the packet is located ;-) I finally
have the MAX10 kit, so I can check the ethernet at least :)
> Thanks a lot for your review.
Thanks for keeping up with my weird questions ;-)
Best regards,
Marek Vasut
More information about the U-Boot
mailing list