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

[Mikko Perttunen <mperttunen-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>]

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?

Thanks, Mikko