Dear Hong Xu,

In message <[email protected]> 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 <[email protected]>
> Tested-by: Elen Song <[email protected]>
> CC: Albert Aribaud <[email protected]>
> ---
> 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: [email protected]
I am a computer. I am dumber than any human and smarter than any  ad-
ministrator.
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to