Chris, On Wed, Oct 15, 2014 at 1:48 PM, <zyw@xxxxxxxxxxxxxx> wrote: > From: Chris Zhong <zyw@xxxxxxxxxxxxxx> > > save and restore some clks, which might be changed in suspend. > > Signed-off-by: Tony Xie <xxx@xxxxxxxxxxxxxx> > Signed-off-by: Chris Zhong <zyw@xxxxxxxxxxxxxx> > --- > > drivers/clk/rockchip/clk-rk3288.c | 61 +++++++++++++++++++++++++++++++++++++ > 1 file changed, 61 insertions(+) Put this as patch set #1? It's safe to do this first and the other patch won't work without it (right?) > +static int rk3288_clk_suspend(void) > +{ > + int i, reg_id; > + > + for (i = 0; i < ARRAY_SIZE(rk3288_saved_cru_reg_ids); i++) { > + reg_id = rk3288_saved_cru_reg_ids[i]; > + > + rk3288_saved_cru_regs[i] = > + __raw_readl(rk3288_cru_base + reg_id); As talked about off-list, should probably not be the __raw version here and elsewhere in this file. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html