linux-kernel - Re: [PATCH v3 05/10] ARM: dts: omap3-beagle: Use reset-gpios for hsusb2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <524D780C.1090708@ti.com>
Date:	Thu, 3 Oct 2013 16:58:36 +0300
From:	Roger Quadros <rogerq@...com>
To:	Benoit Cousson <bcousson@...libre.com>
CC:	<balbi@...com>, <tony@...mide.com>, <p.zabel@...gutronix.de>,
	<broonie@...nel.org>, <linux-omap@...r.kernel.org>,
	<linux-arm-kernel@...ts.infradead.org>,
	<linux-usb@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
	<devicetree@...r.kernel.org>
Subject: Re: [PATCH v3 05/10] ARM: dts: omap3-beagle: Use reset-gpios for
 hsusb2_reset

Hi Benoit,

On 10/03/2013 04:44 PM, Benoit Cousson wrote:
> On 03/10/2013 14:05, Benoit Cousson wrote:
>> Hi Roger,
>>
>> Yes, I will. I've been waiting for these ones for so long :-)
> 
> In fact it does not apply correctly on my for_3.13/dts branch :-(
> 
> error: arch/arm/boot/dts/omap3-beagle.dts: patch does not apply
> Patch failed at 0004 ARM: dts: omap3-beagle: Make USB host pin naming consistent
> 
> Could you rebase it?

Looks like it was already applied before. Could you please skip that and use the rest?
I've checked that the remaining patches apply fine on top of your for_3.13/dts
branch. 

cheers,
-roger

>>
>> On 03/10/2013 12:34, Roger Quadros wrote:
>>> Hi Benoit,
>>>
>>> Could you please take the device tree related patches [5 to 10] in
>>> this series?
>>> Thanks.
>>>
>>> cheers,
>>> -roger
>>>
>>> On 09/24/2013 11:53 AM, Roger Quadros wrote:
>>>> We no longer need to model the RESET line as a regulator since
>>>> the USB phy-nop driver accepts "reset-gpios" property.
>>>>
>>>> Signed-off-by: Roger Quadros <rogerq@...com>
>>>> ---
>>>>   arch/arm/boot/dts/omap3-beagle.dts |   13 +------------
>>>>   1 files changed, 1 insertions(+), 12 deletions(-)
>>>>
>>>> diff --git a/arch/arm/boot/dts/omap3-beagle.dts
>>>> b/arch/arm/boot/dts/omap3-beagle.dts
>>>> index dfd8310..71bde47 100644
>>>> --- a/arch/arm/boot/dts/omap3-beagle.dts
>>>> +++ b/arch/arm/boot/dts/omap3-beagle.dts
>>>> @@ -44,17 +44,6 @@
>>>>           };
>>>>       };
>>>>
>>>> -    /* HS USB Port 2 RESET */
>>>> -    hsusb2_reset: hsusb2_reset_reg {
>>>> -        compatible = "regulator-fixed";
>>>> -        regulator-name = "hsusb2_reset";
>>>> -        regulator-min-microvolt = <3300000>;
>>>> -        regulator-max-microvolt = <3300000>;
>>>> -        gpio = <&gpio5 19 0>;    /* gpio_147 */
>>>> -        startup-delay-us = <70000>;
>>>> -        enable-active-high;
>>>> -    };
>>>> -
>>>>       /* HS USB Port 2 Power */
>>>>       hsusb2_power: hsusb2_power_reg {
>>>>           compatible = "regulator-fixed";
>>>> @@ -68,7 +57,7 @@
>>>>       /* HS USB Host PHY on PORT 2 */
>>>>       hsusb2_phy: hsusb2_phy {
>>>>           compatible = "usb-nop-xceiv";
>>>> -        reset-supply = <&hsusb2_reset>;
>>>> +        reset-gpios = <&gpio5 19 GPIO_ACTIVE_LOW>;    /* gpio_147 */
>>>>           vcc-supply = <&hsusb2_power>;
>>>>       };
>>>>
>>>>
>>>
>>
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/