* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-25 13:55 ` Roger Quadros 0 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-11-25 13:55 UTC (permalink / raw) To: tony, bcousson Cc: nm, devicetree, linux-usb, linux-kernel, balbi, linux-omap, linux-arm-kernel, Roger Quadros Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply for the USB PHY. As the generic PHY driver can't handle more than one supply at the moment, we configure this supply to be always on. This will cause a very small power impact if the USB host subsystem is not in use, about 76.86 micro-W + LDO power. Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, so there won't be any functional impact on those boards other than some additional LDO power consumption. Reported-by: Nishanth Menon <nm@ti.com> Signed-off-by: Roger Quadros <rogerq@ti.com> --- arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ 2 files changed, 16 insertions(+) diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts index 31a632f..b39918e 100644 --- a/arch/arm/boot/dts/omap3-beagle-xm.dts +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts @@ -215,3 +215,11 @@ &usbhsehci { phys = <0 &hsusb2_phy>; }; + +&vaux2 { + regulator-name = "usb_1v8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; +}; + diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts index fa532aa..9764556 100644 --- a/arch/arm/boot/dts/omap3-beagle.dts +++ b/arch/arm/boot/dts/omap3-beagle.dts @@ -178,3 +178,11 @@ mode = <3>; power = <50>; }; + +&vaux2 { + regulator-name = "vdd_ehci"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; +}; + -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-25 13:55 ` Roger Quadros 0 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-11-25 13:55 UTC (permalink / raw) To: tony, bcousson Cc: balbi, nm, linux-omap, linux-usb, linux-arm-kernel, linux-kernel, devicetree, Roger Quadros Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply for the USB PHY. As the generic PHY driver can't handle more than one supply at the moment, we configure this supply to be always on. This will cause a very small power impact if the USB host subsystem is not in use, about 76.86 micro-W + LDO power. Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, so there won't be any functional impact on those boards other than some additional LDO power consumption. Reported-by: Nishanth Menon <nm@ti.com> Signed-off-by: Roger Quadros <rogerq@ti.com> --- arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ 2 files changed, 16 insertions(+) diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts index 31a632f..b39918e 100644 --- a/arch/arm/boot/dts/omap3-beagle-xm.dts +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts @@ -215,3 +215,11 @@ &usbhsehci { phys = <0 &hsusb2_phy>; }; + +&vaux2 { + regulator-name = "usb_1v8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; +}; + diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts index fa532aa..9764556 100644 --- a/arch/arm/boot/dts/omap3-beagle.dts +++ b/arch/arm/boot/dts/omap3-beagle.dts @@ -178,3 +178,11 @@ mode = <3>; power = <50>; }; + +&vaux2 { + regulator-name = "vdd_ehci"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; +}; + -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-25 13:55 ` Roger Quadros 0 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-11-25 13:55 UTC (permalink / raw) To: linux-arm-kernel Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply for the USB PHY. As the generic PHY driver can't handle more than one supply at the moment, we configure this supply to be always on. This will cause a very small power impact if the USB host subsystem is not in use, about 76.86 micro-W + LDO power. Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, so there won't be any functional impact on those boards other than some additional LDO power consumption. Reported-by: Nishanth Menon <nm@ti.com> Signed-off-by: Roger Quadros <rogerq@ti.com> --- arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ 2 files changed, 16 insertions(+) diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts index 31a632f..b39918e 100644 --- a/arch/arm/boot/dts/omap3-beagle-xm.dts +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts @@ -215,3 +215,11 @@ &usbhsehci { phys = <0 &hsusb2_phy>; }; + +&vaux2 { + regulator-name = "usb_1v8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; +}; + diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts index fa532aa..9764556 100644 --- a/arch/arm/boot/dts/omap3-beagle.dts +++ b/arch/arm/boot/dts/omap3-beagle.dts @@ -178,3 +178,11 @@ mode = <3>; power = <50>; }; + +&vaux2 { + regulator-name = "vdd_ehci"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; +}; + -- 1.8.3.2 ^ permalink raw reply related [flat|nested] 18+ messages in thread
[parent not found: <1385387745-7162-1-git-send-email-rogerq-l0cyMroinI0@public.gmane.org>]
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) 2013-11-25 13:55 ` Roger Quadros (?) @ 2013-11-25 15:50 ` Nishanth Menon -1 siblings, 0 replies; 18+ messages in thread From: Nishanth Menon @ 2013-11-25 15:50 UTC (permalink / raw) To: Roger Quadros Cc: Tony Lindgren, Benoit Cousson, dt list, linux-usb-u79uwXL29TY76Z2rM5mHXA, lkml, balbi-l0cyMroinI0, linux-omap, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org> wrote: > Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > for the USB PHY. > > As the generic PHY driver can't handle more than one supply > at the moment, we configure this supply to be always on. > This will cause a very small power impact if the USB host subsystem > is not in use, about 76.86 micro-W + LDO power. > > Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > so there won't be any functional impact on those boards other than > some additional LDO power consumption. > > Reported-by: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org> > Signed-off-by: Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org> Tested-by: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org> I might suggest though that the better alternative might be to get phy driver to handle multiple regulators considering that DT is supposed to represent the h/w topology. > --- > arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ > arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ > 2 files changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts > index 31a632f..b39918e 100644 > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts > @@ -215,3 +215,11 @@ > &usbhsehci { > phys = <0 &hsusb2_phy>; > }; > + > +&vaux2 { > + regulator-name = "usb_1v8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts > index fa532aa..9764556 100644 > --- a/arch/arm/boot/dts/omap3-beagle.dts > +++ b/arch/arm/boot/dts/omap3-beagle.dts > @@ -178,3 +178,11 @@ > mode = <3>; > power = <50>; > }; > + > +&vaux2 { > + regulator-name = "vdd_ehci"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > -- > 1.8.3.2 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-25 15:50 ` Nishanth Menon 0 siblings, 0 replies; 18+ messages in thread From: Nishanth Menon @ 2013-11-25 15:50 UTC (permalink / raw) To: Roger Quadros Cc: Tony Lindgren, Benoit Cousson, dt list, linux-usb, lkml, balbi, linux-omap, linux-arm-kernel@lists.infradead.org On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq@ti.com> wrote: > Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > for the USB PHY. > > As the generic PHY driver can't handle more than one supply > at the moment, we configure this supply to be always on. > This will cause a very small power impact if the USB host subsystem > is not in use, about 76.86 micro-W + LDO power. > > Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > so there won't be any functional impact on those boards other than > some additional LDO power consumption. > > Reported-by: Nishanth Menon <nm@ti.com> > Signed-off-by: Roger Quadros <rogerq@ti.com> Tested-by: Nishanth Menon <nm@ti.com> I might suggest though that the better alternative might be to get phy driver to handle multiple regulators considering that DT is supposed to represent the h/w topology. > --- > arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ > arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ > 2 files changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts > index 31a632f..b39918e 100644 > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts > @@ -215,3 +215,11 @@ > &usbhsehci { > phys = <0 &hsusb2_phy>; > }; > + > +&vaux2 { > + regulator-name = "usb_1v8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts > index fa532aa..9764556 100644 > --- a/arch/arm/boot/dts/omap3-beagle.dts > +++ b/arch/arm/boot/dts/omap3-beagle.dts > @@ -178,3 +178,11 @@ > mode = <3>; > power = <50>; > }; > + > +&vaux2 { > + regulator-name = "vdd_ehci"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > -- > 1.8.3.2 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-25 15:50 ` Nishanth Menon 0 siblings, 0 replies; 18+ messages in thread From: Nishanth Menon @ 2013-11-25 15:50 UTC (permalink / raw) To: linux-arm-kernel On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq@ti.com> wrote: > Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > for the USB PHY. > > As the generic PHY driver can't handle more than one supply > at the moment, we configure this supply to be always on. > This will cause a very small power impact if the USB host subsystem > is not in use, about 76.86 micro-W + LDO power. > > Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > so there won't be any functional impact on those boards other than > some additional LDO power consumption. > > Reported-by: Nishanth Menon <nm@ti.com> > Signed-off-by: Roger Quadros <rogerq@ti.com> Tested-by: Nishanth Menon <nm@ti.com> I might suggest though that the better alternative might be to get phy driver to handle multiple regulators considering that DT is supposed to represent the h/w topology. > --- > arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ > arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ > 2 files changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts > index 31a632f..b39918e 100644 > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts > @@ -215,3 +215,11 @@ > &usbhsehci { > phys = <0 &hsusb2_phy>; > }; > + > +&vaux2 { > + regulator-name = "usb_1v8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts > index fa532aa..9764556 100644 > --- a/arch/arm/boot/dts/omap3-beagle.dts > +++ b/arch/arm/boot/dts/omap3-beagle.dts > @@ -178,3 +178,11 @@ > mode = <3>; > power = <50>; > }; > + > +&vaux2 { > + regulator-name = "vdd_ehci"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > -- > 1.8.3.2 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel at lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
[parent not found: <CAGo_u6ryFJvBwd=S9A-nJw+eoGguPj0jMOtgMiZ7mXXAbh4i+w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) 2013-11-25 15:50 ` Nishanth Menon (?) @ 2013-11-26 8:56 ` Roger Quadros -1 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-11-26 8:56 UTC (permalink / raw) To: Nishanth Menon Cc: Tony Lindgren, Benoit Cousson, dt list, linux-usb-u79uwXL29TY76Z2rM5mHXA, lkml, balbi-l0cyMroinI0, linux-omap, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org On 11/25/2013 05:50 PM, Nishanth Menon wrote: > On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org> wrote: >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply >> for the USB PHY. >> >> As the generic PHY driver can't handle more than one supply >> at the moment, we configure this supply to be always on. >> This will cause a very small power impact if the USB host subsystem >> is not in use, about 76.86 micro-W + LDO power. > >> >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, >> so there won't be any functional impact on those boards other than >> some additional LDO power consumption. >> >> Reported-by: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org> >> Signed-off-by: Roger Quadros <rogerq-l0cyMroinI0@public.gmane.org> > > Tested-by: Nishanth Menon <nm-l0cyMroinI0@public.gmane.org> Thanks Nishant. > > I might suggest though that the better alternative might be to get phy > driver to handle multiple regulators considering that DT is supposed > to represent the h/w topology. Noted. I will work on this for 3.14. cheers, -roger > >> --- >> arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ >> arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ >> 2 files changed, 16 insertions(+) >> >> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts >> index 31a632f..b39918e 100644 >> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts >> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts >> @@ -215,3 +215,11 @@ >> &usbhsehci { >> phys = <0 &hsusb2_phy>; >> }; >> + >> +&vaux2 { >> + regulator-name = "usb_1v8"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + regulator-always-on; >> +}; >> + >> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts >> index fa532aa..9764556 100644 >> --- a/arch/arm/boot/dts/omap3-beagle.dts >> +++ b/arch/arm/boot/dts/omap3-beagle.dts >> @@ -178,3 +178,11 @@ >> mode = <3>; >> power = <50>; >> }; >> + >> +&vaux2 { >> + regulator-name = "vdd_ehci"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + regulator-always-on; >> +}; >> + >> -- >> 1.8.3.2 >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-26 8:56 ` Roger Quadros 0 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-11-26 8:56 UTC (permalink / raw) To: Nishanth Menon Cc: Tony Lindgren, Benoit Cousson, dt list, linux-usb, lkml, balbi, linux-omap, linux-arm-kernel@lists.infradead.org On 11/25/2013 05:50 PM, Nishanth Menon wrote: > On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq@ti.com> wrote: >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply >> for the USB PHY. >> >> As the generic PHY driver can't handle more than one supply >> at the moment, we configure this supply to be always on. >> This will cause a very small power impact if the USB host subsystem >> is not in use, about 76.86 micro-W + LDO power. > >> >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, >> so there won't be any functional impact on those boards other than >> some additional LDO power consumption. >> >> Reported-by: Nishanth Menon <nm@ti.com> >> Signed-off-by: Roger Quadros <rogerq@ti.com> > > Tested-by: Nishanth Menon <nm@ti.com> Thanks Nishant. > > I might suggest though that the better alternative might be to get phy > driver to handle multiple regulators considering that DT is supposed > to represent the h/w topology. Noted. I will work on this for 3.14. cheers, -roger > >> --- >> arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ >> arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ >> 2 files changed, 16 insertions(+) >> >> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts >> index 31a632f..b39918e 100644 >> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts >> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts >> @@ -215,3 +215,11 @@ >> &usbhsehci { >> phys = <0 &hsusb2_phy>; >> }; >> + >> +&vaux2 { >> + regulator-name = "usb_1v8"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + regulator-always-on; >> +}; >> + >> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts >> index fa532aa..9764556 100644 >> --- a/arch/arm/boot/dts/omap3-beagle.dts >> +++ b/arch/arm/boot/dts/omap3-beagle.dts >> @@ -178,3 +178,11 @@ >> mode = <3>; >> power = <50>; >> }; >> + >> +&vaux2 { >> + regulator-name = "vdd_ehci"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + regulator-always-on; >> +}; >> + >> -- >> 1.8.3.2 >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel@lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-26 8:56 ` Roger Quadros 0 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-11-26 8:56 UTC (permalink / raw) To: linux-arm-kernel On 11/25/2013 05:50 PM, Nishanth Menon wrote: > On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq@ti.com> wrote: >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply >> for the USB PHY. >> >> As the generic PHY driver can't handle more than one supply >> at the moment, we configure this supply to be always on. >> This will cause a very small power impact if the USB host subsystem >> is not in use, about 76.86 micro-W + LDO power. > >> >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, >> so there won't be any functional impact on those boards other than >> some additional LDO power consumption. >> >> Reported-by: Nishanth Menon <nm@ti.com> >> Signed-off-by: Roger Quadros <rogerq@ti.com> > > Tested-by: Nishanth Menon <nm@ti.com> Thanks Nishant. > > I might suggest though that the better alternative might be to get phy > driver to handle multiple regulators considering that DT is supposed > to represent the h/w topology. Noted. I will work on this for 3.14. cheers, -roger > >> --- >> arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ >> arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ >> 2 files changed, 16 insertions(+) >> >> diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts >> index 31a632f..b39918e 100644 >> --- a/arch/arm/boot/dts/omap3-beagle-xm.dts >> +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts >> @@ -215,3 +215,11 @@ >> &usbhsehci { >> phys = <0 &hsusb2_phy>; >> }; >> + >> +&vaux2 { >> + regulator-name = "usb_1v8"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + regulator-always-on; >> +}; >> + >> diff --git a/arch/arm/boot/dts/omap3-beagle.dts b/arch/arm/boot/dts/omap3-beagle.dts >> index fa532aa..9764556 100644 >> --- a/arch/arm/boot/dts/omap3-beagle.dts >> +++ b/arch/arm/boot/dts/omap3-beagle.dts >> @@ -178,3 +178,11 @@ >> mode = <3>; >> power = <50>; >> }; >> + >> +&vaux2 { >> + regulator-name = "vdd_ehci"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + regulator-always-on; >> +}; >> + >> -- >> 1.8.3.2 >> >> >> _______________________________________________ >> linux-arm-kernel mailing list >> linux-arm-kernel at lists.infradead.org >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) 2013-11-26 8:56 ` Roger Quadros @ 2013-11-26 22:27 ` Tony Lindgren -1 siblings, 0 replies; 18+ messages in thread From: Tony Lindgren @ 2013-11-26 22:27 UTC (permalink / raw) To: Roger Quadros Cc: Nishanth Menon, Benoit Cousson, dt list, linux-usb, lkml, balbi, linux-omap, linux-arm-kernel@lists.infradead.org * Roger Quadros <rogerq@ti.com> [131126 00:57]: > On 11/25/2013 05:50 PM, Nishanth Menon wrote: > > On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq@ti.com> wrote: > >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > >> for the USB PHY. > >> > >> As the generic PHY driver can't handle more than one supply > >> at the moment, we configure this supply to be always on. > >> This will cause a very small power impact if the USB host subsystem > >> is not in use, about 76.86 micro-W + LDO power. > > > >> > >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > >> so there won't be any functional impact on those boards other than > >> some additional LDO power consumption. > >> > >> Reported-by: Nishanth Menon <nm@ti.com> > >> Signed-off-by: Roger Quadros <rogerq@ti.com> > > > > Tested-by: Nishanth Menon <nm@ti.com> > > Thanks Nishant. Thanks, applying this into omap-for-v3.13/fixes-take4. Tony ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-11-26 22:27 ` Tony Lindgren 0 siblings, 0 replies; 18+ messages in thread From: Tony Lindgren @ 2013-11-26 22:27 UTC (permalink / raw) To: linux-arm-kernel * Roger Quadros <rogerq@ti.com> [131126 00:57]: > On 11/25/2013 05:50 PM, Nishanth Menon wrote: > > On Mon, Nov 25, 2013 at 7:55 AM, Roger Quadros <rogerq@ti.com> wrote: > >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > >> for the USB PHY. > >> > >> As the generic PHY driver can't handle more than one supply > >> at the moment, we configure this supply to be always on. > >> This will cause a very small power impact if the USB host subsystem > >> is not in use, about 76.86 micro-W + LDO power. > > > >> > >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > >> so there won't be any functional impact on those boards other than > >> some additional LDO power consumption. > >> > >> Reported-by: Nishanth Menon <nm@ti.com> > >> Signed-off-by: Roger Quadros <rogerq@ti.com> > > > > Tested-by: Nishanth Menon <nm@ti.com> > > Thanks Nishant. Thanks, applying this into omap-for-v3.13/fixes-take4. Tony ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) 2013-11-25 13:55 ` Roger Quadros @ 2013-12-03 3:54 ` Laurent Pinchart -1 siblings, 0 replies; 18+ messages in thread From: Laurent Pinchart @ 2013-12-03 3:54 UTC (permalink / raw) To: Roger Quadros Cc: tony, bcousson, balbi, nm, linux-omap, linux-usb, linux-arm-kernel, linux-kernel, devicetree Hi Roger, On Monday 25 November 2013 15:55:45 Roger Quadros wrote: > Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > for the USB PHY. > > As the generic PHY driver can't handle more than one supply > at the moment, we configure this supply to be always on. > This will cause a very small power impact if the USB host subsystem > is not in use, about 76.86 micro-W + LDO power. > > Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > so there won't be any functional impact on those boards other than > some additional LDO power consumption. Do I need any patch other than this one (on top of v3.13-rc1) to enable the ethernet port on a Beagleboard-xM rev B ? Here's what the kernel reports at boot (with ignore_loglevel set on the command line). [ 3.388305] ehci-omap 48064800.ehci: EHCI Host Controller [ 3.394165] ehci-omap 48064800.ehci: new USB bus registered, assigned bus number 2 [ 3.402252] ehci-omap 48064800.ehci: reset hcs_params 0x1313 dbg=0 cc=1 pcc=3 ordered ports=3 [ 3.411254] ehci-omap 48064800.ehci: reset hcc_params 0016 thresh 1 uframes 256/512/1024 park [ 3.422027] ehci-omap 48064800.ehci: park 0 [ 3.426544] ehci-omap 48064800.ehci: reset command 0080b02 park=3 ithresh=8 period=1024 Reset HALT [ 3.436248] ehci-omap 48064800.ehci: irq 93, io mem 0x48064800 [ 3.442443] ehci-omap 48064800.ehci: init command 0010005 (park)=0 ithresh=1 period=512 RUN [ 3.467407] ehci-omap 48064800.ehci: USB 2.0 started, EHCI 1.00 [ 3.475494] usb usb2: default language 0x0409 [ 3.480712] usb usb2: udev 1, busnum 2, minor = 128 [ 3.485809] usb usb2: New USB device found, idVendor=1d6b, idProduct=0002 [ 3.493011] usb usb2: New USB device strings: Mfr=3, Product=2, SerialNumber=1 [ 3.500640] usb usb2: Product: EHCI Host Controller [ 3.505737] usb usb2: Manufacturer: Linux 3.13.0-rc1-00022-g0c6e4db ehci_hcd [ 3.513183] usb usb2: SerialNumber: 48064800.ehci [ 3.521270] usb usb2: usb_probe_device [ 3.525207] usb usb2: configuration #1 chosen from 1 choice [ 3.531890] usb usb2: adding 2-0:1.0 (config #1, interface 0) [ 3.539855] hub 2-0:1.0: usb_probe_interface [ 3.544342] hub 2-0:1.0: usb_probe_interface - got id [ 3.549835] hub 2-0:1.0: USB hub found [ 3.554290] hub 2-0:1.0: 3 ports detected [ 3.558593] hub 2-0:1.0: standalone hub [ 3.562591] hub 2-0:1.0: individual port power switching [ 3.568237] hub 2-0:1.0: individual port over-current protection [ 3.574523] hub 2-0:1.0: power on to power good time: 20ms [ 3.581085] hub 2-0:1.0: local power source is good [ 3.587646] hub 2-0:1.0: enabling power on all ports [ 3.598693] of_get_named_gpiod_flags exited with status 0 [ 3.607208] input: gpio_keys.6 as /devices/gpio_keys.6/input/input0 [ 3.617370] twl_rtc rtc.10: setting system clock to 2000-01-01 00:37:16 UTC (946687036) [ 3.628295] omap_vout:Could not register Video driver [ 3.633575] sr_init: No PMIC hook to init smartreflex [ 3.641265] sr_init: platform driver register failed for SR [ 3.765655] ehci-omap 48064800.ehci: GetStatus port:2 status 001c03 0 ACK POWER sig=? CSC CONNECT [ 3.775329] hub 2-0:1.0: port 2: status 0501 change 0001 [ 3.877471] hub 2-0:1.0: state 7 ports 3 chg 0004 evt 0000 [ 3.883758] hub 2-0:1.0: port 2, status 0501, change 0000, 480 Mb/s [ 3.947357] ehci-omap 48064800.ehci: GetStatus port:2 status 001402 0 ACK POWER sig=k CSC [ 3.977264] hub 2-0:1.0: state 7 ports 3 chg 0000 evt 0004 [ 3.983062] ehci-omap 48064800.ehci: GetStatus port:2 status 003002 0 ACK POWER OWNER sig=se0 CSC [ 3.993041] hub 2-0:1.0: port 2, status 0100, change 0001, 12 Mb/s [ 4.157440] hub 2-0:1.0: debounce: port 2: total 100ms stable 100ms status 0x100 [ 4.165283] hub 2-0:1.0: hub_suspend [ 4.169311] usb usb2: bus auto-suspend, wakeup 1 [ 4.174133] ehci-omap 48064800.ehci: suspend root hub [ 15.878875] ALSA device list: [ 15.881988] No soundcards found. [ 15.888214] omap_uart 49020000.serial: no wakeirq for uart2 [ 15.894866] Waiting 10 sec before mounting root device... [ 120.959472] VFS: Unable to mount root fs via NFS, trying floppy. [ 120.968048] VFS: Cannot open root device "nfs" or unknown-block(2,0): error -6 [ 120.975585] Please append a correct "root=" boot option; here are the available partitions: [ 120.984558] b300 3941376 mmcblk0 driver: mmcblk [ 120.990203] b301 803249 mmcblk0p1 00000000-01 [ 120.995788] b302 3132675 mmcblk0p2 00000000-02 [ 121.001678] Kernel panic - not syncing: VFS: Unable to mount root fs on unknown-block(2,0) > Reported-by: Nishanth Menon <nm@ti.com> > Signed-off-by: Roger Quadros <rogerq@ti.com> > --- > arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ > arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ > 2 files changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts > b/arch/arm/boot/dts/omap3-beagle-xm.dts index 31a632f..b39918e 100644 > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts > @@ -215,3 +215,11 @@ > &usbhsehci { > phys = <0 &hsusb2_phy>; > }; > + > +&vaux2 { > + regulator-name = "usb_1v8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > diff --git a/arch/arm/boot/dts/omap3-beagle.dts > b/arch/arm/boot/dts/omap3-beagle.dts index fa532aa..9764556 100644 > --- a/arch/arm/boot/dts/omap3-beagle.dts > +++ b/arch/arm/boot/dts/omap3-beagle.dts > @@ -178,3 +178,11 @@ > mode = <3>; > power = <50>; > }; > + > +&vaux2 { > + regulator-name = "vdd_ehci"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-12-03 3:54 ` Laurent Pinchart 0 siblings, 0 replies; 18+ messages in thread From: Laurent Pinchart @ 2013-12-03 3:54 UTC (permalink / raw) To: linux-arm-kernel Hi Roger, On Monday 25 November 2013 15:55:45 Roger Quadros wrote: > Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > for the USB PHY. > > As the generic PHY driver can't handle more than one supply > at the moment, we configure this supply to be always on. > This will cause a very small power impact if the USB host subsystem > is not in use, about 76.86 micro-W + LDO power. > > Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > so there won't be any functional impact on those boards other than > some additional LDO power consumption. Do I need any patch other than this one (on top of v3.13-rc1) to enable the ethernet port on a Beagleboard-xM rev B ? Here's what the kernel reports at boot (with ignore_loglevel set on the command line). [ 3.388305] ehci-omap 48064800.ehci: EHCI Host Controller [ 3.394165] ehci-omap 48064800.ehci: new USB bus registered, assigned bus number 2 [ 3.402252] ehci-omap 48064800.ehci: reset hcs_params 0x1313 dbg=0 cc=1 pcc=3 ordered ports=3 [ 3.411254] ehci-omap 48064800.ehci: reset hcc_params 0016 thresh 1 uframes 256/512/1024 park [ 3.422027] ehci-omap 48064800.ehci: park 0 [ 3.426544] ehci-omap 48064800.ehci: reset command 0080b02 park=3 ithresh=8 period=1024 Reset HALT [ 3.436248] ehci-omap 48064800.ehci: irq 93, io mem 0x48064800 [ 3.442443] ehci-omap 48064800.ehci: init command 0010005 (park)=0 ithresh=1 period=512 RUN [ 3.467407] ehci-omap 48064800.ehci: USB 2.0 started, EHCI 1.00 [ 3.475494] usb usb2: default language 0x0409 [ 3.480712] usb usb2: udev 1, busnum 2, minor = 128 [ 3.485809] usb usb2: New USB device found, idVendor=1d6b, idProduct=0002 [ 3.493011] usb usb2: New USB device strings: Mfr=3, Product=2, SerialNumber=1 [ 3.500640] usb usb2: Product: EHCI Host Controller [ 3.505737] usb usb2: Manufacturer: Linux 3.13.0-rc1-00022-g0c6e4db ehci_hcd [ 3.513183] usb usb2: SerialNumber: 48064800.ehci [ 3.521270] usb usb2: usb_probe_device [ 3.525207] usb usb2: configuration #1 chosen from 1 choice [ 3.531890] usb usb2: adding 2-0:1.0 (config #1, interface 0) [ 3.539855] hub 2-0:1.0: usb_probe_interface [ 3.544342] hub 2-0:1.0: usb_probe_interface - got id [ 3.549835] hub 2-0:1.0: USB hub found [ 3.554290] hub 2-0:1.0: 3 ports detected [ 3.558593] hub 2-0:1.0: standalone hub [ 3.562591] hub 2-0:1.0: individual port power switching [ 3.568237] hub 2-0:1.0: individual port over-current protection [ 3.574523] hub 2-0:1.0: power on to power good time: 20ms [ 3.581085] hub 2-0:1.0: local power source is good [ 3.587646] hub 2-0:1.0: enabling power on all ports [ 3.598693] of_get_named_gpiod_flags exited with status 0 [ 3.607208] input: gpio_keys.6 as /devices/gpio_keys.6/input/input0 [ 3.617370] twl_rtc rtc.10: setting system clock to 2000-01-01 00:37:16 UTC (946687036) [ 3.628295] omap_vout:Could not register Video driver [ 3.633575] sr_init: No PMIC hook to init smartreflex [ 3.641265] sr_init: platform driver register failed for SR [ 3.765655] ehci-omap 48064800.ehci: GetStatus port:2 status 001c03 0 ACK POWER sig=? CSC CONNECT [ 3.775329] hub 2-0:1.0: port 2: status 0501 change 0001 [ 3.877471] hub 2-0:1.0: state 7 ports 3 chg 0004 evt 0000 [ 3.883758] hub 2-0:1.0: port 2, status 0501, change 0000, 480 Mb/s [ 3.947357] ehci-omap 48064800.ehci: GetStatus port:2 status 001402 0 ACK POWER sig=k CSC [ 3.977264] hub 2-0:1.0: state 7 ports 3 chg 0000 evt 0004 [ 3.983062] ehci-omap 48064800.ehci: GetStatus port:2 status 003002 0 ACK POWER OWNER sig=se0 CSC [ 3.993041] hub 2-0:1.0: port 2, status 0100, change 0001, 12 Mb/s [ 4.157440] hub 2-0:1.0: debounce: port 2: total 100ms stable 100ms status 0x100 [ 4.165283] hub 2-0:1.0: hub_suspend [ 4.169311] usb usb2: bus auto-suspend, wakeup 1 [ 4.174133] ehci-omap 48064800.ehci: suspend root hub [ 15.878875] ALSA device list: [ 15.881988] No soundcards found. [ 15.888214] omap_uart 49020000.serial: no wakeirq for uart2 [ 15.894866] Waiting 10 sec before mounting root device... [ 120.959472] VFS: Unable to mount root fs via NFS, trying floppy. [ 120.968048] VFS: Cannot open root device "nfs" or unknown-block(2,0): error -6 [ 120.975585] Please append a correct "root=" boot option; here are the available partitions: [ 120.984558] b300 3941376 mmcblk0 driver: mmcblk [ 120.990203] b301 803249 mmcblk0p1 00000000-01 [ 120.995788] b302 3132675 mmcblk0p2 00000000-02 [ 121.001678] Kernel panic - not syncing: VFS: Unable to mount root fs on unknown-block(2,0) > Reported-by: Nishanth Menon <nm@ti.com> > Signed-off-by: Roger Quadros <rogerq@ti.com> > --- > arch/arm/boot/dts/omap3-beagle-xm.dts | 8 ++++++++ > arch/arm/boot/dts/omap3-beagle.dts | 8 ++++++++ > 2 files changed, 16 insertions(+) > > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts > b/arch/arm/boot/dts/omap3-beagle-xm.dts index 31a632f..b39918e 100644 > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts > @@ -215,3 +215,11 @@ > &usbhsehci { > phys = <0 &hsusb2_phy>; > }; > + > +&vaux2 { > + regulator-name = "usb_1v8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + > diff --git a/arch/arm/boot/dts/omap3-beagle.dts > b/arch/arm/boot/dts/omap3-beagle.dts index fa532aa..9764556 100644 > --- a/arch/arm/boot/dts/omap3-beagle.dts > +++ b/arch/arm/boot/dts/omap3-beagle.dts > @@ -178,3 +178,11 @@ > mode = <3>; > power = <50>; > }; > + > +&vaux2 { > + regulator-name = "vdd_ehci"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > +}; > + -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) 2013-12-03 3:54 ` Laurent Pinchart (?) @ 2013-12-03 9:36 ` Roger Quadros -1 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-12-03 9:36 UTC (permalink / raw) To: Laurent Pinchart Cc: tony, bcousson, balbi, nm, linux-omap, linux-usb, linux-arm-kernel, linux-kernel, devicetree Hi Laurent, On 12/03/2013 05:54 AM, Laurent Pinchart wrote: > Hi Roger, > > On Monday 25 November 2013 15:55:45 Roger Quadros wrote: >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply >> for the USB PHY. >> >> As the generic PHY driver can't handle more than one supply >> at the moment, we configure this supply to be always on. >> This will cause a very small power impact if the USB host subsystem >> is not in use, about 76.86 micro-W + LDO power. >> >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, >> so there won't be any functional impact on those boards other than >> some additional LDO power consumption. > > Do I need any patch other than this one (on top of v3.13-rc1) to enable the > ethernet port on a Beagleboard-xM rev B ? Here's what the kernel reports at > boot (with ignore_loglevel set on the command line). > It seems on Rev A/B, the power enable line for the USB hub has reversed polarity than Rev C. Does the below patch work for you? If yes, how do we account for it? Do we add a new file "omap3-beagle-xm-ab.dts" for rev A/B boards? cheers, -roger diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts index b39918e..434d903 100644 --- a/arch/arm/boot/dts/omap3-beagle-xm.dts +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts @@ -78,6 +78,7 @@ regulator-max-microvolt = <3300000>; gpio = <&twl_gpio 18 0>; /* GPIO LEDA */ startup-delay-us = <70000>; + enable-active-high; }; /* HS USB Host PHY on PORT 2 */ ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-12-03 9:36 ` Roger Quadros 0 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-12-03 9:36 UTC (permalink / raw) To: Laurent Pinchart Cc: tony, bcousson, balbi, nm, linux-omap, linux-usb, linux-arm-kernel, linux-kernel, devicetree Hi Laurent, On 12/03/2013 05:54 AM, Laurent Pinchart wrote: > Hi Roger, > > On Monday 25 November 2013 15:55:45 Roger Quadros wrote: >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply >> for the USB PHY. >> >> As the generic PHY driver can't handle more than one supply >> at the moment, we configure this supply to be always on. >> This will cause a very small power impact if the USB host subsystem >> is not in use, about 76.86 micro-W + LDO power. >> >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, >> so there won't be any functional impact on those boards other than >> some additional LDO power consumption. > > Do I need any patch other than this one (on top of v3.13-rc1) to enable the > ethernet port on a Beagleboard-xM rev B ? Here's what the kernel reports at > boot (with ignore_loglevel set on the command line). > It seems on Rev A/B, the power enable line for the USB hub has reversed polarity than Rev C. Does the below patch work for you? If yes, how do we account for it? Do we add a new file "omap3-beagle-xm-ab.dts" for rev A/B boards? cheers, -roger diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts index b39918e..434d903 100644 --- a/arch/arm/boot/dts/omap3-beagle-xm.dts +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts @@ -78,6 +78,7 @@ regulator-max-microvolt = <3300000>; gpio = <&twl_gpio 18 0>; /* GPIO LEDA */ startup-delay-us = <70000>; + enable-active-high; }; /* HS USB Host PHY on PORT 2 */ ^ permalink raw reply related [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-12-03 9:36 ` Roger Quadros 0 siblings, 0 replies; 18+ messages in thread From: Roger Quadros @ 2013-12-03 9:36 UTC (permalink / raw) To: linux-arm-kernel Hi Laurent, On 12/03/2013 05:54 AM, Laurent Pinchart wrote: > Hi Roger, > > On Monday 25 November 2013 15:55:45 Roger Quadros wrote: >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply >> for the USB PHY. >> >> As the generic PHY driver can't handle more than one supply >> at the moment, we configure this supply to be always on. >> This will cause a very small power impact if the USB host subsystem >> is not in use, about 76.86 micro-W + LDO power. >> >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, >> so there won't be any functional impact on those boards other than >> some additional LDO power consumption. > > Do I need any patch other than this one (on top of v3.13-rc1) to enable the > ethernet port on a Beagleboard-xM rev B ? Here's what the kernel reports at > boot (with ignore_loglevel set on the command line). > It seems on Rev A/B, the power enable line for the USB hub has reversed polarity than Rev C. Does the below patch work for you? If yes, how do we account for it? Do we add a new file "omap3-beagle-xm-ab.dts" for rev A/B boards? cheers, -roger diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts b/arch/arm/boot/dts/omap3-beagle-xm.dts index b39918e..434d903 100644 --- a/arch/arm/boot/dts/omap3-beagle-xm.dts +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts @@ -78,6 +78,7 @@ regulator-max-microvolt = <3300000>; gpio = <&twl_gpio 18 0>; /* GPIO LEDA */ startup-delay-us = <70000>; + enable-active-high; }; /* HS USB Host PHY on PORT 2 */ ^ permalink raw reply related [flat|nested] 18+ messages in thread
* Re: [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) 2013-12-03 9:36 ` Roger Quadros @ 2013-12-03 11:10 ` Laurent Pinchart -1 siblings, 0 replies; 18+ messages in thread From: Laurent Pinchart @ 2013-12-03 11:10 UTC (permalink / raw) To: Roger Quadros Cc: tony, bcousson, balbi, nm, linux-omap, linux-usb, linux-arm-kernel, linux-kernel, devicetree Hi Roger, On Tuesday 03 December 2013 11:36:57 Roger Quadros wrote: > On 12/03/2013 05:54 AM, Laurent Pinchart wrote: > > Hi Roger, > > > > On Monday 25 November 2013 15:55:45 Roger Quadros wrote: > >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > >> for the USB PHY. > >> > >> As the generic PHY driver can't handle more than one supply > >> at the moment, we configure this supply to be always on. > >> This will cause a very small power impact if the USB host subsystem > >> is not in use, about 76.86 micro-W + LDO power. > >> > >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > >> so there won't be any functional impact on those boards other than > >> some additional LDO power consumption. > > > > Do I need any patch other than this one (on top of v3.13-rc1) to enable > > the ethernet port on a Beagleboard-xM rev B ? Here's what the kernel > > reports at boot (with ignore_loglevel set on the command line). > > It seems on Rev A/B, the power enable line for the USB hub has reversed > polarity than Rev C. > > Does the below patch work for you? It does, thank you. > If yes, how do we account for it? Do we add a new file > "omap3-beagle-xm-ab.dts" for rev A/B boards? Unless we want to add board code back with a runtime check, which I doubt would be regarded as a good idea, I don't see any other easy solution. > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts > b/arch/arm/boot/dts/omap3-beagle-xm.dts index b39918e..434d903 100644 > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts > @@ -78,6 +78,7 @@ > regulator-max-microvolt = <3300000>; > gpio = <&twl_gpio 18 0>; /* GPIO LEDA */ > startup-delay-us = <70000>; > + enable-active-high; > }; > > /* HS USB Host PHY on PORT 2 */ -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 18+ messages in thread
* [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) @ 2013-12-03 11:10 ` Laurent Pinchart 0 siblings, 0 replies; 18+ messages in thread From: Laurent Pinchart @ 2013-12-03 11:10 UTC (permalink / raw) To: linux-arm-kernel Hi Roger, On Tuesday 03 December 2013 11:36:57 Roger Quadros wrote: > On 12/03/2013 05:54 AM, Laurent Pinchart wrote: > > Hi Roger, > > > > On Monday 25 November 2013 15:55:45 Roger Quadros wrote: > >> Beagle (rev. C4) and Beagle-XM (all revs) need VAUX2 1.8V supply > >> for the USB PHY. > >> > >> As the generic PHY driver can't handle more than one supply > >> at the moment, we configure this supply to be always on. > >> This will cause a very small power impact if the USB host subsystem > >> is not in use, about 76.86 micro-W + LDO power. > >> > >> Older Beagle boards (prior to C4) don't have VAUX2 connected anywhere, > >> so there won't be any functional impact on those boards other than > >> some additional LDO power consumption. > > > > Do I need any patch other than this one (on top of v3.13-rc1) to enable > > the ethernet port on a Beagleboard-xM rev B ? Here's what the kernel > > reports at boot (with ignore_loglevel set on the command line). > > It seems on Rev A/B, the power enable line for the USB hub has reversed > polarity than Rev C. > > Does the below patch work for you? It does, thank you. > If yes, how do we account for it? Do we add a new file > "omap3-beagle-xm-ab.dts" for rev A/B boards? Unless we want to add board code back with a runtime check, which I doubt would be regarded as a good idea, I don't see any other easy solution. > diff --git a/arch/arm/boot/dts/omap3-beagle-xm.dts > b/arch/arm/boot/dts/omap3-beagle-xm.dts index b39918e..434d903 100644 > --- a/arch/arm/boot/dts/omap3-beagle-xm.dts > +++ b/arch/arm/boot/dts/omap3-beagle-xm.dts > @@ -78,6 +78,7 @@ > regulator-max-microvolt = <3300000>; > gpio = <&twl_gpio 18 0>; /* GPIO LEDA */ > startup-delay-us = <70000>; > + enable-active-high; > }; > > /* HS USB Host PHY on PORT 2 */ -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 18+ messages in thread
end of thread, other threads:[~2013-12-03 11:10 UTC | newest]
Thread overview: 18+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-25 13:55 [PATCH] ARM: dts: omap3-beagle: Fix USB host on beagle boards (for 3.13) Roger Quadros
2013-11-25 13:55 ` Roger Quadros
2013-11-25 13:55 ` Roger Quadros
[not found] ` <1385387745-7162-1-git-send-email-rogerq-l0cyMroinI0@public.gmane.org>
2013-11-25 15:50 ` Nishanth Menon
2013-11-25 15:50 ` Nishanth Menon
2013-11-25 15:50 ` Nishanth Menon
[not found] ` <CAGo_u6ryFJvBwd=S9A-nJw+eoGguPj0jMOtgMiZ7mXXAbh4i+w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-11-26 8:56 ` Roger Quadros
2013-11-26 8:56 ` Roger Quadros
2013-11-26 8:56 ` Roger Quadros
2013-11-26 22:27 ` Tony Lindgren
2013-11-26 22:27 ` Tony Lindgren
2013-12-03 3:54 ` Laurent Pinchart
2013-12-03 3:54 ` Laurent Pinchart
2013-12-03 9:36 ` Roger Quadros
2013-12-03 9:36 ` Roger Quadros
2013-12-03 9:36 ` Roger Quadros
2013-12-03 11:10 ` Laurent Pinchart
2013-12-03 11:10 ` Laurent Pinchart
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.