Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 9, 2015 at 3:08 PM, Rob Herring wrote: > On Wed, Dec 9, 2015 at 1:59 PM, Bjorn Andersson > wrote: >> On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: >> >>> On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >>> > On 12/08/2015 08:11 PM, Rob Herring wrote: >>> >> On Tue, Dec 08, 2015

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 09, 2015 at 11:20:45PM +0100, Sebastian Reichel wrote: > On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > > There are other switches in the charger block that are not > > exposed yet. This one handles the the OTG (vbus) charge pathway. > > Others handle other charge

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 09, 2015 at 11:20:45PM +0100, Sebastian Reichel wrote: > On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > > There are other switches in the charger block that are not > > exposed yet. This one handles the the OTG (vbus) charge pathway. > > Others handle other charge

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 9, 2015 at 3:08 PM, Rob Herring wrote: > On Wed, Dec 9, 2015 at 1:59 PM, Bjorn Andersson > wrote: >> On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: >> >>> On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >>>

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

Hi, On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > Or maybe our power maintainers will chime in with some wisdom. I > can't be the first person to be adding a charge pathway switch to > mainline, so I'm open to doing it the "right way". :-) I don't think there is a standard way for

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 9, 2015 at 1:59 PM, Bjorn Andersson wrote: > On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: > >> On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >> > On 12/08/2015 08:11 PM, Rob Herring wrote: >> >> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: >> >>> Add a binding for

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: > On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: > > On 12/08/2015 08:11 PM, Rob Herring wrote: > >> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: > >>> Add a binding for the regulator which controls the OTG chargepath switch. > >>>

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > On 12/09/2015 06:36 AM, Rob Herring wrote: > > You need a regulator-name property. Also, should should define valid > > values for regulator-min-microvolt and regulator-max-microvolt. > All of those are listed as optional properties in

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On 12/09/2015 06:36 AM, Rob Herring wrote: > On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >> On 12/08/2015 08:11 PM, Rob Herring wrote: >>> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: Add a binding for the regulator which controls the OTG chargepath switch. The OTG

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: > On 12/08/2015 08:11 PM, Rob Herring wrote: >> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: >>> Add a binding for the regulator which controls the OTG chargepath switch. >>> The OTG switch gets its power from pm8941_5vs1, and that

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On 12/08/2015 08:11 PM, Rob Herring wrote: > On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: >> Add a binding for the regulator which controls the OTG chargepath switch. >> The OTG switch gets its power from pm8941_5vs1, and that should be >> expressed as a usb-otg-in-supply property in

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On 12/08/2015 08:11 PM, Rob Herring wrote: > On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: >> Add a binding for the regulator which controls the OTG chargepath switch. >> The OTG switch gets its power from pm8941_5vs1, and that should be >> expressed as a usb-otg-in-supply property in

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: > On 12/08/2015 08:11 PM, Rob Herring wrote: >> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: >>> Add a binding for the regulator which controls the OTG chargepath switch. >>> The OTG switch gets its power from

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On 12/09/2015 06:36 AM, Rob Herring wrote: > On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >> On 12/08/2015 08:11 PM, Rob Herring wrote: >>> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: Add a binding for the regulator which controls the OTG

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > On 12/09/2015 06:36 AM, Rob Herring wrote: > > You need a regulator-name property. Also, should should define valid > > values for regulator-min-microvolt and regulator-max-microvolt. > All of those are listed as optional properties in

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed, Dec 9, 2015 at 1:59 PM, Bjorn Andersson wrote: > On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: > >> On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >> > On 12/08/2015 08:11 PM, Rob Herring wrote: >> >> On Tue, Dec 08, 2015 at

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

Hi, On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > Or maybe our power maintainers will chime in with some wisdom. I > can't be the first person to be adding a charge pathway switch to > mainline, so I'm open to doing it the "right way". :-) I don't think there is a standard way for

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: > On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: > > On 12/08/2015 08:11 PM, Rob Herring wrote: > >> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: > >>> Add a binding for the regulator which controls the OTG

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: > Add a binding for the regulator which controls the OTG chargepath switch. > The OTG switch gets its power from pm8941_5vs1, and that should be > expressed as a usb-otg-in-supply property in the DT node for the > charger driver. The

[PATCH v2 1/3] dt-binding: power: Add otg regulator binding

Add a binding for the regulator which controls the OTG chargepath switch. The OTG switch gets its power from pm8941_5vs1, and that should be expressed as a usb-otg-in-supply property in the DT node for the charger driver. The regulator name is "otg". Signed-off-by: Tim Bird --- Changes since v1

[PATCH v2 1/3] dt-binding: power: Add otg regulator binding

Add a binding for the regulator which controls the OTG chargepath switch. The OTG switch gets its power from pm8941_5vs1, and that should be expressed as a usb-otg-in-supply property in the DT node for the charger driver. The regulator name is "otg". Signed-off-by: Tim Bird

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: > Add a binding for the regulator which controls the OTG chargepath switch. > The OTG switch gets its power from pm8941_5vs1, and that should be > expressed as a usb-otg-in-supply property in the DT node for the > charger driver. The