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 ... > -mike Why? The elements are used as parameters to readl/writel functions: status = readl(&dbu->sr);
I am quite sure we are NOT supposed to directly access hardware: status = dbu->sr; Best Regards, Reinhard _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot