Hi,
I guess you had some of the stuff I commented on already fixed earlier,
looks like I missed your integration patch.
* Kalle Jokiniemi <ext-kalle.jokiniemi@xxxxxxxxx> [080611 03:47]:
> Hi,
>
> Good comments. I'll tighten the patch-set into only two patches: regdefs
> and implementation. Some other comments below.
>
> On ti, 2008-06-10 at 15:20 -0700, ext Tony Lindgren wrote:
<snip>
> >
> > > +static inline void sr_write_reg(struct omap_sr *sr, int offset, u32 value)
> > > +{
> > > + omap_writel(value, sr->srbase_addr + offset);
> > > +}
> > > +
> > > +static inline void sr_modify_reg(struct omap_sr *sr, int offset, u32 mask,
> > > + u32 value)
> > > +{
> > > + u32 reg_val;
> > > +
> > > + reg_val = omap_readl(sr->srbase_addr + offset);
> > > + reg_val &= ~mask;
> > > + reg_val |= value;
> > > +
> > > + omap_writel(reg_val, sr->srbase_addr + offset);
> > > +}
> > > +
> > > +static inline u32 sr_read_reg(struct omap_sr *sr, int offset)
> > > +{
> > > + return omap_readl(sr->srbase_addr + offset);
> > > +}
> > > +
> > > +
> >
> > The read and write registers are better done using __raw_readl/writel()
> > instead. Unless the register address is static, the function adds the
> > IO offset every time unnecessarily.
> >
> > So please set the sr->srbase_addr = io_v2p(SOME_PHYS_ADDR) during init
> > and change the read and write to use __raw_readl/writel() instead.
>
> did you mean "sr->srbase_addr = io_p2v(SOME_PHYS_ADDR)" ?
Yeah, sorry that's a pretty confusing typo there :)
Regards,
Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
