On Sat, Aug 14, 2010 at 3:40 PM, Reinhard Meyer wrote: > Dear Mike Frysinger, >> On Sat, Aug 14, 2010 at 3:30 PM, Reinhard Meyer wrote: >>> #define DBU_ADDR 0xsomething (in a SoC header file) >>> >>> dbu_t *dbu = (dbu_t *)DBU_ADDR; (in a function) >> >> needs to be volatile ... > > Why? The elements are used as parameters to readl/writel functions: > status = readl(&dbu->sr);
if you use accessor functions and those guarantee volatility, then you dont need the markings on the pointer > I am quite sure we are NOT supposed to directly access hardware: > status = dbu->sr; it depends on the hardware -mike _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
