[U-Boot] [PATCH] Fix FEC compile warning message

Wolfgang Denk wd at denx.de
Tue Jul 28 23:59:13 CEST 2009


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.


More information about the U-Boot mailing list