[U-Boot] [PATCH 1/2] ppc: Include <asm/cache.h> in common.h
Anton Staaf
robotboy at chromium.org
Sun Oct 9 23:51:39 CEST 2011
On Sun, Oct 9, 2011 at 12:11 PM, Wolfgang Denk <wd at denx.de> wrote:
> Dear Anton Staaf,
>
> In message <CAF6FioV3YFk02Sgp0AasYGQJSNKv=cB-sSzCQr8F3S2esyAyRw at mail.gmail.com> you wrote:
>>
>> > This is needed for the patch "cache: add default setting for
>> > CONFIG_SYS_CACHELINE_SIZE" from Anton Staaf. As cache.h defines
>> > CONFIG_SYS_CACHELINE_SIZE for PPC targets.
>> >
>> > This will remove the following warnings/errors:
>> >
>> > include/common.h:819:2: warning: #warning CONFIG_SYS_CACHELINE_SIZE not d=
>> efined, using __BIGGEST_ALIGNMENT__
>> > cache.c:33: error: '__BIGGEST_ALIGNMENT__' undeclared (first use in this =
>> function)
> ...
>> It would be great if this could be enabled for all configs, not just
>> PPC. Would it make sense to remove this check and add #ifndef
>> __ASSEMBLY__ to the other cache.h files? The list is not too long
>> (m68k, sparc, sh, nios2, and arm). I'd be happy to look into that if
>> you'd rather get this in first.
>
> Agreed. But please let's do this in a second step (feel free to
> submit a patch :-)
Will do. :)
Thanks,
Anton
> I suggest to just add this under #ifndef __ASSEMBLY__
>
> 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
> Never put off until tomorrow what you can put off indefinitely.
>
More information about the U-Boot
mailing list