On Fri, Apr 17, 2015 at 06:45:59PM +0530, Vinod Koul wrote: > +u8 ssth_readb_traced(struct ssth_lib *dsp, u32 offset) > +{ > + u8 val; > + > + val = readb((dsp)->mmio_base + offset); > + dev_dbg(dsp->dev, "readb offset=%#x val=%#x\n", offset, val); > + > + return val; > +} Much of this looks like a MMIO regmap, there's the memcpy() bits too which aren't but otherwise...