Re: [PATCH] Temporary fix to boot OMAP1 based boards

[Tony Lindgren <tony@atomide.com>]

* Paul Walmsley <paul@booyaka.com> [080924 11:03]:
> On Wed, 24 Sep 2008, Tony Lindgren wrote:
> 
> > * Tony Lindgren <tony@atomide.com> [080924 10:12]:
> > > * Arun KS <arunks@mistralsolutions.com> [080924 10:05]:
> > > > On Wed, Sep 24, 2008 at 9:21 AM, Hiroshi DOYU <Hiroshi.DOYU@nokia.com> 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.
> 
> Those registers should be u16 for OMAP2/3.  OMAP1 needs something similar, 
> which will also get rid of those casts to void __iomem * - hacking on 
> something now ...

Argh, sorry. Yeah it's an offset for omap2/3.. Will revert immediately.

Tony