[U-Boot] [PATCH] Fix FEC compile warning message
Liew Tsi Chung-R5AAHP
Tsi-chung.Liew at freescale.com
Wed Jul 29 00:22:44 CEST 2009
Wolfgang,
Yes, I will do that for the next merging window. Ignore this patch.
Thanks!
Best Regards,
TsiChung
-----Original Message-----
From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de]
On Behalf Of Wolfgang Denk
Sent: Tuesday, July 28, 2009 4:59 PM
To: TsiChung Liew
Cc: U-Boot; Ben Warre7n
Subject: Re: [U-Boot] [PATCH] Fix FEC compile warning message
Dear TsiChung Liew,
In message <1248799132-29720-1-git-send-email-tsicliew at gmail.com> you
wrote:
> Provide function protocol for icache_invalid().
>
> Signed-off-by: TsiChung Liew <tsicliew at gmail.com>
> ---
> drivers/net/mcffec.c | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/mcffec.c b/drivers/net/mcffec.c index
> 64be5de..d3741a7 100644
> --- a/drivers/net/mcffec.c
> +++ b/drivers/net/mcffec.c
> @@ -95,6 +95,10 @@ struct fec_info_s fec_info[] = { #endif };
>
> +#ifdef CONFIG_SYS_UNIFY_CACHE
> +extern void icache_invalid(void);
> +#endif
Could you please rename this function into invalidate_icache(), and add
a prototype declaration to include/asm-?/cache.h, like it is for example
doene for PowerPC in include/asm-ppc/cache.h ?
I would like to see that all architectures use the same names for the
same functions...
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 If
it has syntax, it isn't user friendly.
_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
More information about the U-Boot
mailing list