Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759440Ab2EUWKj (ORCPT ); Mon, 21 May 2012 18:10:39 -0400 Received: from mail-bk0-f46.google.com ([209.85.214.46]:48039 "EHLO mail-bk0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752749Ab2EUWKh (ORCPT ); Mon, 21 May 2012 18:10:37 -0400 Date: Tue, 22 May 2012 00:10:29 +0200 From: Andi Shyti To: NeilBrown Cc: Samuel Ortiz , Felipe Balbi , Anton Vorontsov , David Woodhouse , Grazvydas Ignotas , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org Subject: Re: [PATCH 4/4] usb: otg: twl4030-usb: Don't power down phy when it is in-use by charger. Message-ID: <20120521221029.GB4346@andi> Mail-Followup-To: NeilBrown , Samuel Ortiz , Felipe Balbi , Anton Vorontsov , David Woodhouse , Grazvydas Ignotas , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org References: <20120508213518.2844.95446.stgit@notabene.brown> <20120508214040.2844.81364.stgit@notabene.brown> <20120513181409.GB2641@andi> <20120518125010.582b191f@notabene.brown> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120518125010.582b191f@notabene.brown> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2084 Lines: 53 Hi, On Fri, May 18, 2012 at 12:50:10PM +1000, NeilBrown wrote: > On Sun, 13 May 2012 20:14:09 +0200 Andi Shyti wrote: > > On Wed, May 09, 2012 at 07:40:40AM +1000, NeilBrown wrote: > > > @@ -388,10 +388,16 @@ static void twl4030_phy_power(struct twl4030_usb *twl, int on) > > > (PHY_CLK_CTRL_CLOCKGATING_EN | > > > PHY_CLK_CTRL_CLK32K_EN)); > > > } else { > > > - __twl4030_phy_power(twl, 0); > > > regulator_disable(twl->usb1v5); > > > regulator_disable(twl->usb1v8); > > > regulator_disable(twl->usb3v1); > > > + if (!regulator_is_enabled(twl->usb3v1)) > > > + /* no-one else is requesting this > > > + * so it is OK to power-down the > > > + * phy. Sometimes a charger might > > > + * hold the regulator active. > > > + */ > > > + __twl4030_phy_power(twl, 0); > > > } > > > > Usually a regulator line is shared by more than one device and > > regulator_is_enable() returns true if at least one of these > > devices is holding the regulator. This means that here the check > > will not work if this is your case. > > > > Andi > > This regulator is inside an MFD and it only feeds a very limited number of > devices within that MFD. So I don't think there is much room for confusion. > > However is it a somewhat indirect method of signalling. I want the charger > to be able to tell the USB controller that it is using the PHY so please > don't turn it off. Doing that through the regulator seems simple and > effective. > Maybe there is a better way, but it isn't immediately clear what that would > be. > Suggestions welcome. When using regulators I would keep track of his status internally in the driver... 'true' if on or 'false' if of, in this way there is no room for confusion. But the mine is just a suggestion, you know the environment better Andi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/