On Thursday, September 08, 2011 18:20:16 Simon Glass wrote:
> --- a/board/nvidia/common/board.c
> +++ b/board/nvidia/common/board.c
> @@ -307,3 +307,11 @@ int board_mmc_getcd(u8 *cd, struct mmc *mmc)
>       return 0;
>  }
>  #endif
> +
> +#ifndef CONFIG_SYS_DCACHE_OFF
> +void enable_caches(void)
> +{
> +     /* Enable D-cache. I-cache is already enabled in start.S */
> +     dcache_enable();
> +}
> +#endif

not specific to your patch, but this seems kind of dumb to copy & paste the 
same thing between all the arm sub arches/boards.  why cant the default 
enable_caches() look like this for arm:
void enable_caches(void)
{
#ifndef CONFIG_SYS_DCACHE_OFF
        dcache_enable();
#endif
#ifndef CONFIG_SYS_ICACHE_OFF
        icache_enable();
#endif
}
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to