[U-Boot] [PATCH 2/3 v2] ARM: ARM1136 - Remove flush_cache from arch/arm/lib/cache.c
Wolfgang Denk
wd at denx.de
Thu Oct 6 23:50:21 CEST 2011
Dear Hong Xu,
In message <1313745795-1326-2-git-send-email-hong.xu at atmel.com> you wrote:
> arch/arm/lib/cache.c is cleaned and no real cache operation will be
> defined in this file. A new file arch/arm/cpu/arm1136/cache.c is
> created. This file will define the real cache operations.
>
> Signed-off-by: Hong Xu <hong.xu at atmel.com>
> Tested-by: Elen Song <elen.song at atmel.com>
> CC: Albert Aribaud <albert.u.boot at aribaud.net>
> ---
> Since V2
> Removed redundant ifdef for CONFIG_OMAP2420 || CONFIG_ARM1136
>
> arch/arm/cpu/arm1136/Makefile | 2 +-
> arch/arm/cpu/arm1136/cache.c | 31 +++++++++++++++++++++++++++++++
> 2 files changed, 32 insertions(+), 1 deletions(-)
> create mode 100644 arch/arm/cpu/arm1136/cache.c
Checkpatch says:
WARNING: externs should be avoided in .c files
#141: FILE: arch/arm/cpu/arm1136/cache.c:28:
+ void arm1136_cache_flush(void);
Please clean up and resubmit. Thanks.
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
I am a computer. I am dumber than any human and smarter than any ad-
ministrator.
More information about the U-Boot
mailing list