Re: [PATCH v2 2/3] Documentation: devicetree: phy-tegra-usb: Add vbus-supply property for host mode PHYs

[Felipe Balbi <balbi-l0cyMroinI0@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 2541 bytes --]

On Tue, Jul 30, 2013 at 09:10:12AM +0300, Mikko Perttunen wrote:
> On 07/30/2013 12:23 AM, Felipe Balbi wrote:
> >* PGP Signed by an unknown key
> >
> >Hi,
> >
> >On Tue, Jul 30, 2013 at 12:19:32AM +0300, Felipe Balbi wrote:
> >>On Mon, Jul 29, 2013 at 09:24:46AM -0600, Stephen Warren wrote:
> >>>On 06/28/2013 06:33 AM, Mikko Perttunen wrote:
> >>>>Add vbus-supply as an optional property for host mode phy-tegra-usb PHYs.
> >>>
> >>>Felipe, I see that you didn't merge this one patch from this series. Was
> >>>there a reason for that? At this stage, I would consider it reasonable
> >>>for DT binding documentation changes to go through the same tree as
> >>>driver changes.
> >>
> >>no reason, just missed it. Will apply now.
> >
> >actually, patch _is_ in my tree:
> >
> >https://git.kernel.org/cgit/linux/kernel/git/balbi/usb.git/commit/?h=next&id=80bd8a94712a0a9f2baeb03ad0546dd70f27ac4e
> >
> >commit 80bd8a94712a0a9f2baeb03ad0546dd70f27ac4e
> >Author: Mikko Perttunen <mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >Date:   Wed Jul 17 10:37:50 2013 +0300
> >
> >     usb: tegra: Add vbus-supply property for host mode PHYs
> >
> >     Document vbus-supply as an optional property for host mode phy-tegra-usb PHYs.
> >
> >     Signed-off-by: Mikko Perttunen <mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >     Reviewed-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >     Tested-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >     Signed-off-by: Felipe Balbi <balbi-l0cyMroinI0@public.gmane.org>
> >
> >diff --git a/Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.txt b/Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.txt
> >index c4c9e9e..59c4854 100644
> >--- a/Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.txt
> >+++ b/Documentation/devicetree/bindings/usb/nvidia,tegra20-usb-phy.txt
> >@@ -48,5 +48,5 @@ Optional properties:
> >        peripheral means it is device controller
> >        otg means it can operate as either ("on the go")
> >
> >-Required properties for dr_mode == otg:
> >+VBUS control (required for dr_mode == host, optional for dr_mode == otg):
> >    - vbus-supply: regulator for VBUS
> >
> 
> Oops, looks like I've made a typo there while making the v2. This
> should be, optional for dr_mode == host, required for dr_mode == otg
> instead of the other way around. Can this fixup still be made?

send a patch on top of that, please.

-- 
balbi

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]