Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754246Ab3JCN7Y (ORCPT ); Thu, 3 Oct 2013 09:59:24 -0400 Received: from bear.ext.ti.com ([192.94.94.41]:35358 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753667Ab3JCN7W (ORCPT ); Thu, 3 Oct 2013 09:59:22 -0400 Message-ID: <524D780C.1090708@ti.com> Date: Thu, 3 Oct 2013 16:58:36 +0300 From: Roger Quadros User-Agent: Mozilla/5.0 (X11; Linux i686; rv:16.0) Gecko/20121011 Thunderbird/16.0.1 MIME-Version: 1.0 To: Benoit Cousson CC: , , , , , , , , Subject: Re: [PATCH v3 05/10] ARM: dts: omap3-beagle: Use reset-gpios for hsusb2_reset References: <1380012836-29900-1-git-send-email-rogerq@ti.com> <1380012836-29900-6-git-send-email-rogerq@ti.com> <524D483D.9040907@ti.com> <524D5D88.9020308@baylibre.com> <524D74A5.7010604@baylibre.com> In-Reply-To: <524D74A5.7010604@baylibre.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2618 Lines: 84 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 >>>> --- >>>> 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/