* Tony Lindgren [080924 10:12]: > * Arun KS [080924 10:05]: > > On Wed, Sep 24, 2008 at 9:21 AM, Hiroshi DOYU wrote: > > > Hi, > > > > > > How about introducing a new typedef in order to accommodate the > > > different size of enable_regs without ifdefs? > > > > > > I think that this is apparently the case categorized into (a) in > > > 'CodyingStyle'. > > > > I am not sure which is the right way to do. Is this what you meant? > > No, let's not do this. It should be void __iomem * for sure. Most of > the omap1 clock registers looks just fine for void __iomem *, let's > figure out which registers are broken. I've pushed a fix for this. Turns out I missed some conversions to void __iomem * while merging code from arm-devel branch. Fix also attached. Can you please check that it works for you now? Tony