Hi Wolfgang, 2011/4/26 Wolfgang Denk <w...@denx.de>: > Dear Macpaul Lin, >> +#define FTSDMC021_OFFSET_BANK7_BSR 0x2C /* External Bank Base/Size Reg >> 7 */ > > Lines too long. Please fix globally. > > I think it is generally wrong to manually define these offsets here. > You should use a C struct instead, and auto-generate the offsets if > needed using the asm-offsets approach (see top level Makefile for > details).
Since this is a patch send before in old times, I will fix it soon and send the next patch. >> +#define FTSDMC021_BANK_SIZE_128M 0x7 >> +#define FTSDMC021_BANK_SIZE_256M 0x8 >> +#define FTSDMC021_BANK_SIZE_512M 0x9 > > Why don't you use a generic macro here, like > > #define FTSDMC021_BANK_SIZE(sz) (ffs(x) - 21) Thanks for your suggestion. -- Best regards, Macpaul Lin _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot