* Kevin Hilman <khilman@xxxxxxxxxx> [141024 11:04]: > Nishanth Menon <nm@xxxxxx> writes: > > > CPU logic power state is never programmed in either the initialization > > or the suspend/resume logic, instead, we depend on mpuss to program this > > properly. However, this leaves CPU logic power state indeterminate and > > most probably in reset configuration (If bootloader or other similar > > software have'nt monkeyed with the register). This can make powerstate= > > RET be either programmed for CSWR (logic=ret) or OSWR(logic = OFF) and > > in OSWR, there can be context loss when the code does not expect it. > > > > To prevent all these confusions, just support clearly ON, INA, CSWR, > > OFF which is the intent of the existing code by explicitly programming > > logic state. > > > > NOTE: since this is a hot path (using in cpuidle), the exit path just > > programs powerstate (logic state is immaterial when powerstate is ON). > > > > Without doing this, we end up with lockups when CPUs enter OSWR and > > multiple blocks loose context, when we expect them to hit CSWR. > > > > Signed-off-by: Nishanth Menon <nm@xxxxxx> > > Acked-by: Kevin Hilman <khilman@xxxxxxxxxx> Applying into omap-for-v3.19/soc thanks. Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
