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
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