Hi Anand, On 13/01/2019 16:30, Anand Moon wrote: > Add missing vin-supply node 5.0V regulator setting to power > to usb ports, changes help fix the usb reset warning. > > [ 795.380156] usb 1-1.2: reset high-speed USB device number 3 using dwc2 > [ 798.356073] usb 1-1.2: reset high-speed USB device number 3 using dwc2 > [ 801.331999] usb 1-1.2: reset high-speed USB device number 3 using dwc2 > [ 804.307919] usb 1-1.2: reset high-speed USB device number 3 using dwc2 > [ 807.283844] usb 1-1.2: reset high-speed USB device number 3 using dwc2 > > Fixes: 5a0803bd5ae (ARM64: dts: meson-gxbb-odroidc2: Enable USB Nodes) > Tested-by: Kevin Hilman <khil...@baylibre.com> > Cc: Martin Blumenstingl <martin.blumensti...@googlemail.com> > Cc: Jerome Brunet <jbru...@baylibre.com> > Cc: Neil Armstrong <narmstr...@baylibre.com> > Signed-off-by: Anand Moon <linux.am...@gmail.com> > --- > Rebased on Kevin's amlogic/v5.0/fixes branch > Changes from patch v1 [0] https://lore.kernel.org/patchwork/patch/1031243/ > - Split the patch into two part: new patch > - Fix the commit message > - Added Tested by Kevin Hilman > --- > arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts > b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts > index 25a36d117183..a947b46bfaa8 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts > @@ -28,6 +28,13 @@ > reg = <0x0 0x0 0x0 0x80000000>; > }; > > + p5v0: regulator-p5v0 { > + compatible = "regulator-fixed"; > + regulator-name = "P5V0"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + }; > + > usb_otg_pwr: regulator-usb-pwrs { > compatible = "regulator-fixed"; > > @@ -36,6 +43,8 @@ > regulator-min-microvolt = <5000000>; > regulator-max-microvolt = <5000000>; > > + vin-supply = <&p5v0>; > + > gpio = <&gpio_ao GPIOAO_5 GPIO_ACTIVE_HIGH>; > enable-active-high; > }; >
Reviewed-by: Neil Armstrong <narmstr...@baylibre.com> PS: Please add a cover-letter next time ! Neil
