On Fri, Nov 25, 2011 at 8:18 AM, Jason Liu <jason.hui at linaro.org> wrote: > +static char *get_reset_cause(void) > +{ > + u32 cause; > + struct src *src_regs = (struct src *)SRC_BASE_ADDR; > + > + cause = readl(&src_regs->srsr); > + writel(cause, &src_regs->srsr); Isn´t this writel unneeded? Regards, Fabio Estevam