From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH] mfd: twl4030-power: Fix poweroff with PM configuration enabled Date: Fri, 7 Nov 2014 11:25:49 +0000 Message-ID: <20141107112549.GA4047@n2100.arm.linux.org.uk> References: <20141102180756.GH31454@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20141102180756.GH31454@atomide.com> Sender: linux-kernel-owner@vger.kernel.org To: Tony Lindgren Cc: Lee Jones , Samuel Ortiz , linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org List-Id: linux-omap@vger.kernel.org On Sun, Nov 02, 2014 at 10:07:56AM -0800, Tony Lindgren wrote: > Commit e7cd1d1eb16f ("mfd: twl4030-power: Add generic reset > configuration") enabled configuring the PM features for twl4030. > > This caused poweroff command to fail on devices that have the > BCI charger on twl4030 wired, or have power wired for VBUS. > Instead of powering off, the device reboots. This is because > voltage is detected on charger or VBUS with the default bits > enabled for the power transition registers. > > To fix the issue, let's just clear VBUS and CHG bits as we want > poweroff command to keep the system powered off. > > Fixes: e7cd1d1eb16f ("mfd: twl4030-power: Add generic reset configuration") > Cc: stable@vger.kernel.org # v3.16+ > Reported-by: Russell King > Signed-off-by: Tony Lindgren This fixes the problem for me, thanks. This needs to go into -rc as it's a regression between booting non-DT and DT, and stands in the way of me switching from non-DT booting on OMAP3430. -- FTTC broadband for 0.8mile line: currently at 9.5Mbps down 400kbps up according to speedtest.net.