> +static inline u32 ioread(u32 reg) > +{ > + return ioread32(base + reg); > +} > + > +static inline void iowrite(u32 val, u32 reg) > +{ > + iowrite32(val, base + reg); > +} Why not just use ioread32/iowrite32 directly ? Otherwise this looks way way better. Alan - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html