[U-Boot] [PATCH 4/6] davinci_emac: fix for running with dcache enabled

Anton Staaf robotboy at google.com
Thu Oct 13 22:40:07 CEST 2011


On Thu, Oct 13, 2011 at 1:31 PM, Wolfgang Denk <wd at denx.de> wrote:
> Dear Anton Staaf,
>
> In message <CAF6FioWnYJNBz0+4Af3-0vLCoGrgGgcN10z=K0df8Yv87GUVjw at mail.gmail.com> you wrote:
>>
>> Turns out I was able to define ARCH_DMA_MINALIGN for all U-Boot
>> architectures and use that
>> instead of the actual cache line size to generate correctly aligned
>> buffers.  My latest
>> patch set implements this.  So we shouldn't have any warnings being
>> generated and we should
>> have no need for the extra file (or addition to main.c) to generate
>> said warnings.  :)
>
> Great.  So this latest patch set is the final one?

I believe it is correct yes.  But it would be good to have each
architecture custodian
weigh in on the architectures I couldn't test directly.  Also, Mike
has submitted a
patch that adds the asm/cache.h file for blackfin from the Linux
kernel.  This patch
obsoletes one of the patches in my series.  My thinking is that we
should wait a bit to
hear from other custodians, and then I'll send a second version of the
patch set that
addresses any comments.  But I am also pretty confident that we could submit the
existing patch set (minus the blackfin asm/cache.h patch) and we would be safe.

Thanks,
    Anton

> Best regards,
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> Hokey religions and ancient weapons are  no  substitute  for  a  good
> blaster at your side.                                      - Han Solo
>


More information about the U-Boot mailing list