Rajendra Nayak <[email protected]> writes: > Do not skip the sysc programming in the hmwod framework based > on the cached value alone, since at times the module might have lost > context (due to the Powerdomain in which the module belongs > transitions to either Open Switch RET or OFF).
Shouldn't the driver for each IP be responsible for restoring it's register contents after context loss, including it's SYSC? Seems to me that if SYSC is lost, it means the driver's save/restore is buggy. Kevin > Signed-off-by: Rajendra Nayak <[email protected]> > Cc: Paul Walmsley <[email protected]> > Cc: Benoit Cousson <[email protected]> > Cc: Santosh Shilimkar <[email protected]> > Cc: Kevin Hilman <[email protected]> > --- > arch/arm/mach-omap2/omap_hwmod.c | 5 ++++- > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-omap2/omap_hwmod.c > b/arch/arm/mach-omap2/omap_hwmod.c > index cb911d7..e57fe21 100644 > --- a/arch/arm/mach-omap2/omap_hwmod.c > +++ b/arch/arm/mach-omap2/omap_hwmod.c > @@ -115,7 +115,10 @@ static void _write_sysconfig(u32 v, struct omap_hwmod > *oh) > > /* XXX ensure module interface clock is up */ > > - if (oh->_sysc_cache != v) { > + if ((oh->_sysc_cache != v) || > + /* Did the module go to off and loose context? */ > + (oh->_sysc_cache != > + omap_hwmod_readl(oh, oh->class->sysc->sysc_offs))) { > oh->_sysc_cache = v; > omap_hwmod_writel(v, oh, oh->class->sysc->sysc_offs); > } -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
