Re: [PATCH 1/7] mfd: twl4030-power: Fix hang on reboot if sleep configuration was loaded earlier

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> > > Looks like we can still hit the issue of wrong load order of
> > > twl4030 configuration. If we have a sleep configuration loaded,
> > > and do a warm reset, the device can hang while initializing the
> > > wakeup12 sequence. We do have a warning message about wrong order
> > > of twl4030 configuration, but in this case it does not help as
> > > the sleep configuration was loaded during the previous boot and
> > > the state of twl4030 is maintained throughout the warm reset.
> > > 
> > > Fix the issue by clearing any existing sleep configuration
> > > before we load the warm reset configuration.
> > > 
> > > Cc: Peter Ujfalusi <peter.ujfalusi@xxxxxx>
> > > Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>
> > > ---
> > >  drivers/mfd/twl4030-power.c | 6 ++++++
> > >  1 file changed, 6 insertions(+)
> > 
> > Can't vouch for the semantics of the patch, but the code looks good.
> > 
> >   Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>
> > 
> > Can I just apply this, or would you like Peter to review it first?
> 
> As far as I'm concerned you can go ahead an apply, it's been
> floating on the list for a while and Peter is probably busy
> with other things.

No problem.  Just a couple of things for you to look at in the other
patches, then we can go ahead and apply for v3.16.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux