Hi Albert, On Saturday 08 January 2011 12:06 PM, Albert ARIBAUD wrote: > Hi Aneesh, [snip ..] >> +/* some utility macros */ >> +#define mask(start, end) \ >> + (((1<< ((end) - (start) + 1)) - 1)<< (start)) >> + >> +#define mask_n_get(reg, start, end) \ >> + (((reg)& mask(start, end))>> (start)) > > Seeing as these functions are only used in the ARMv7 cache C file, they > should be moved there. >
I am working on v2 of this series. We had aligned on moving these macros to omap specific headers. But I now realize that I want to use them from cache_v7.c, so it can not be in omap header. I have used them in other places too in my recent work so it needs to be in a header file. I am putting these functions and some other utils I need in a new file "arch/arm/include/asm/utils.h" Is that fine? Best regards, Aneesh _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot