From: Tony Lindgren <tony-4v6yS6AI5VpBDgjK7y7TUQ@public.gmane.org>
To: Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org>
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org,
linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] ARM: dts: omap3-beagle-xm: Add USB Host support for Rev Ax/Bx
Date: Tue, 16 Apr 2013 17:52:32 -0700 [thread overview]
Message-ID: <20130417005232.GV10155@atomide.com> (raw)
In-Reply-To: <516BF4F3.5060904-l0cyMroinI0@public.gmane.org>
* Roger Quadros <rogerq@ti.com> [130415 05:44]:
> On 04/15/2013 03:35 PM, Roger Quadros wrote:
> > Provide RESET and Power regulators for the USB PHY,
> > the USB Host port mode and the PHY device.
> >
> > Also provide pin multiplexer information for USB host
> > pins.
> >
> > This will not work for Rev Cx boards because of reversed logic
> > for USB_POWER_Enable.
> >
> > CC: Benoît Cousson <b-cousson@ti.com>
> > Signed-off-by: Roger Quadros <rogerq@ti.com>
> > ---
> > arch/arm/boot/dts/omap3-beagle-xm.dts | 62 +++++++++++++++++++++++++++++++++
> > 1 files changed, 62 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> > index 5a31964..d394c51 100644
> > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> > @@ -57,6 +57,60 @@
> > ti,mcbsp = <&mcbsp2>;
> > ti,codec = <&twl_audio>;
> > };
> > +
> > + /* 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";
> > + regulator-name = "hsusb2_vbus";
> > + regulator-min-microvolt = <3300000>;
> > + regulator-max-microvolt = <3300000>;
> > + gpio = <&twl_gpio 18 0>; /* GPIO LEDA */
> > + startup-delay-us = <70000>;
> > + enable-active-high; /* FIXME: active-low for Rev. C */
>
> Benoit & Tony,
>
> Any ideas how to tackle the reversed logic for Rev. C boards?
Sounds like we need a shared omap3-beage.dtsi, then omap3-beagle-xm.dts
and omap3-beagle-rev-c.dts. Then xm and rev-c can both include the
common .dtsi.
Regards,
Tony
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: omap3-beagle-xm: Add USB Host support for Rev Ax/Bx
Date: Tue, 16 Apr 2013 17:52:32 -0700 [thread overview]
Message-ID: <20130417005232.GV10155@atomide.com> (raw)
In-Reply-To: <516BF4F3.5060904@ti.com>
* Roger Quadros <rogerq@ti.com> [130415 05:44]:
> On 04/15/2013 03:35 PM, Roger Quadros wrote:
> > Provide RESET and Power regulators for the USB PHY,
> > the USB Host port mode and the PHY device.
> >
> > Also provide pin multiplexer information for USB host
> > pins.
> >
> > This will not work for Rev Cx boards because of reversed logic
> > for USB_POWER_Enable.
> >
> > CC: Beno?t Cousson <b-cousson@ti.com>
> > Signed-off-by: Roger Quadros <rogerq@ti.com>
> > ---
> > arch/arm/boot/dts/omap3-beagle-xm.dts | 62 +++++++++++++++++++++++++++++++++
> > 1 files changed, 62 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> > index 5a31964..d394c51 100644
> > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> > @@ -57,6 +57,60 @@
> > ti,mcbsp = <&mcbsp2>;
> > ti,codec = <&twl_audio>;
> > };
> > +
> > + /* 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";
> > + regulator-name = "hsusb2_vbus";
> > + regulator-min-microvolt = <3300000>;
> > + regulator-max-microvolt = <3300000>;
> > + gpio = <&twl_gpio 18 0>; /* GPIO LEDA */
> > + startup-delay-us = <70000>;
> > + enable-active-high; /* FIXME: active-low for Rev. C */
>
> Benoit & Tony,
>
> Any ideas how to tackle the reversed logic for Rev. C boards?
Sounds like we need a shared omap3-beage.dtsi, then omap3-beagle-xm.dts
and omap3-beagle-rev-c.dts. Then xm and rev-c can both include the
common .dtsi.
Regards,
Tony
WARNING: multiple messages have this Message-ID (diff)
From: Tony Lindgren <tony@atomide.com>
To: Roger Quadros <rogerq@ti.com>
Cc: b-cousson@ti.com, linux-omap@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
devicetree-discuss@lists.ozlabs.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ARM: dts: omap3-beagle-xm: Add USB Host support for Rev Ax/Bx
Date: Tue, 16 Apr 2013 17:52:32 -0700 [thread overview]
Message-ID: <20130417005232.GV10155@atomide.com> (raw)
In-Reply-To: <516BF4F3.5060904@ti.com>
* Roger Quadros <rogerq@ti.com> [130415 05:44]:
> On 04/15/2013 03:35 PM, Roger Quadros wrote:
> > Provide RESET and Power regulators for the USB PHY,
> > the USB Host port mode and the PHY device.
> >
> > Also provide pin multiplexer information for USB host
> > pins.
> >
> > This will not work for Rev Cx boards because of reversed logic
> > for USB_POWER_Enable.
> >
> > CC: Benoît Cousson <b-cousson@ti.com>
> > Signed-off-by: Roger Quadros <rogerq@ti.com>
> > ---
> > arch/arm/boot/dts/omap3-beagle-xm.dts | 62 +++++++++++++++++++++++++++++++++
> > 1 files changed, 62 insertions(+), 0 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts
> > index 5a31964..d394c51 100644
> > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts
> > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts
> > @@ -57,6 +57,60 @@
> > ti,mcbsp = <&mcbsp2>;
> > ti,codec = <&twl_audio>;
> > };
> > +
> > + /* 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";
> > + regulator-name = "hsusb2_vbus";
> > + regulator-min-microvolt = <3300000>;
> > + regulator-max-microvolt = <3300000>;
> > + gpio = <&twl_gpio 18 0>; /* GPIO LEDA */
> > + startup-delay-us = <70000>;
> > + enable-active-high; /* FIXME: active-low for Rev. C */
>
> Benoit & Tony,
>
> Any ideas how to tackle the reversed logic for Rev. C boards?
Sounds like we need a shared omap3-beage.dtsi, then omap3-beagle-xm.dts
and omap3-beagle-rev-c.dts. Then xm and rev-c can both include the
common .dtsi.
Regards,
Tony
next prev parent reply other threads:[~2013-04-17 0:52 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-15 12:35 [PATCH] ARM: dts: omap3-beagle-xm: Add USB Host support for Rev Ax/Bx Roger Quadros
2013-04-15 12:35 ` Roger Quadros
2013-04-15 12:35 ` Roger Quadros
2013-04-15 12:39 ` Roger Quadros
2013-04-15 12:39 ` Roger Quadros
2013-04-15 12:39 ` Roger Quadros
[not found] ` <516BF4F3.5060904-l0cyMroinI0@public.gmane.org>
2013-04-17 0:52 ` Tony Lindgren [this message]
2013-04-17 0:52 ` Tony Lindgren
2013-04-17 0:52 ` Tony Lindgren
2013-04-17 1:30 ` Robert Nelson
2013-04-17 1:30 ` Robert Nelson
2013-04-17 7:56 ` Igor Grinberg
2013-04-17 7:56 ` Igor Grinberg
2013-04-17 8:38 ` Roger Quadros
2013-04-17 8:38 ` Roger Quadros
2013-04-17 8:38 ` Roger Quadros
2013-04-17 12:26 ` Igor Grinberg
2013-04-17 12:26 ` Igor Grinberg
2013-04-17 12:26 ` Igor Grinberg
2013-04-17 17:11 ` Tony Lindgren
2013-04-17 17:11 ` Tony Lindgren
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130417005232.GV10155@atomide.com \
--to=tony-4v6ys6ai5vpbdgjk7y7tuq@public.gmane.org \
--cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=rogerq-l0cyMroinI0@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.