| Message ID | 20200917065949.3476-1-narmstrong@baylibre.com |
|---|---|
| Headers | show |
| Series | usb: dwc-meson-g12a: Add support for USB on S400 board | expand |
Hi Felipe, Is there anything to change in this serie ? Thanks, Neil On 17/09/2020 08:59, Neil Armstrong wrote: > The Amlogic AXG is close to the GXL Glue but with a single OTG PHY. > > It needs the same init sequence as GXL & GXM, but it seems it doesn't need > the host disconnect bit. > > The Glue driver reuses the already implemented GXL & GXM work. > > The USB2 PHY driver needs a slight tweak to keep the OTG detection working. > > Changes since v1 at [1]: > - s/close from/close to/g > - collected review tags > - added small comment about phy management in patch 3 > - removed status = "okay" in patch 4 > - removed invalid phy-supply of phy1 in patch 5 > > [1] http://lore.kernel.org/r/20200909160409.8678-1-narmstrong@baylibre.com > > Neil Armstrong (5): > phy: amlogic: phy-meson-gxl-usb2: keep ID pull-up even in Host mode > dt-bindings: usb: amlogic,meson-g12a-usb-ctrl: add the Amlogic AXG > Families USB Glue Bindings > usb: dwc-meson-g12a: Add support for USB on AXG SoCs > arm64: dts: meson-axg: add USB nodes > arm64: dts: meson-axg-s400: enable USB OTG > > .../usb/amlogic,meson-g12a-usb-ctrl.yaml | 22 +++++++- > .../arm64/boot/dts/amlogic/meson-axg-s400.dts | 6 +++ > arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 50 +++++++++++++++++++ > drivers/phy/amlogic/phy-meson-gxl-usb2.c | 3 +- > drivers/usb/dwc3/dwc3-meson-g12a.c | 18 +++++++ > 5 files changed, 97 insertions(+), 2 deletions(-) >
Neil Armstrong <narmstrong@baylibre.com> writes: > Hi Felipe, > > Is there anything to change in this serie ? I've been waiting for Kishon's review of drivers/phy parts. I can take the rest, but without Kishon's ack, drivers/phy will be left out.
On 29/09/2020 12:33, Felipe Balbi wrote: > Neil Armstrong <narmstrong@baylibre.com> writes: > >> Hi Felipe, >> >> Is there anything to change in this serie ? > > I've been waiting for Kishon's review of drivers/phy parts. I can take > the rest, but without Kishon's ack, drivers/phy will be left out. > No problem, please go ahead and take the rest, I'll re-submit the phy change again if Kishon wants some changes. Thanks, Neil
Hi Kevin, On 17/09/2020 08:59, Neil Armstrong wrote: > The Amlogic AXG is close to the GXL Glue but with a single OTG PHY. > > It needs the same init sequence as GXL & GXM, but it seems it doesn't need > the host disconnect bit. > > The Glue driver reuses the already implemented GXL & GXM work. > > The USB2 PHY driver needs a slight tweak to keep the OTG detection working. > > Changes since v1 at [1]: > - s/close from/close to/g > - collected review tags > - added small comment about phy management in patch 3 > - removed status = "okay" in patch 4 > - removed invalid phy-supply of phy1 in patch 5 > > [1] http://lore.kernel.org/r/20200909160409.8678-1-narmstrong@baylibre.com > > Neil Armstrong (5): > phy: amlogic: phy-meson-gxl-usb2: keep ID pull-up even in Host mode > dt-bindings: usb: amlogic,meson-g12a-usb-ctrl: add the Amlogic AXG > Families USB Glue Bindings > usb: dwc-meson-g12a: Add support for USB on AXG SoCs > arm64: dts: meson-axg: add USB nodes > arm64: dts: meson-axg-s400: enable USB OTG Can you pick the DT patches ? Thanks, Neil > > .../usb/amlogic,meson-g12a-usb-ctrl.yaml | 22 +++++++- > .../arm64/boot/dts/amlogic/meson-axg-s400.dts | 6 +++ > arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 50 +++++++++++++++++++ > drivers/phy/amlogic/phy-meson-gxl-usb2.c | 3 +- > drivers/usb/dwc3/dwc3-meson-g12a.c | 18 +++++++ > 5 files changed, 97 insertions(+), 2 deletions(-) >
Neil Armstrong <narmstrong@baylibre.com> writes: > Hi Kevin, > > On 17/09/2020 08:59, Neil Armstrong wrote: >> The Amlogic AXG is close to the GXL Glue but with a single OTG PHY. >> >> It needs the same init sequence as GXL & GXM, but it seems it doesn't need >> the host disconnect bit. >> >> The Glue driver reuses the already implemented GXL & GXM work. >> >> The USB2 PHY driver needs a slight tweak to keep the OTG detection working. >> >> Changes since v1 at [1]: >> - s/close from/close to/g >> - collected review tags >> - added small comment about phy management in patch 3 >> - removed status = "okay" in patch 4 >> - removed invalid phy-supply of phy1 in patch 5 >> >> [1] http://lore.kernel.org/r/20200909160409.8678-1-narmstrong@baylibre.com >> >> Neil Armstrong (5): >> phy: amlogic: phy-meson-gxl-usb2: keep ID pull-up even in Host mode >> dt-bindings: usb: amlogic,meson-g12a-usb-ctrl: add the Amlogic AXG >> Families USB Glue Bindings >> usb: dwc-meson-g12a: Add support for USB on AXG SoCs >> arm64: dts: meson-axg: add USB nodes >> arm64: dts: meson-axg-s400: enable USB OTG > > Can you pick the DT patches ? Yes, queued up. Since we're already at -rc7, this may not make it until v5.11. Kevin
The Amlogic AXG is close to the GXL Glue but with a single OTG PHY. It needs the same init sequence as GXL & GXM, but it seems it doesn't need the host disconnect bit. The Glue driver reuses the already implemented GXL & GXM work. The USB2 PHY driver needs a slight tweak to keep the OTG detection working. Changes since v1 at [1]: - s/close from/close to/g - collected review tags - added small comment about phy management in patch 3 - removed status = "okay" in patch 4 - removed invalid phy-supply of phy1 in patch 5 [1] http://lore.kernel.org/r/20200909160409.8678-1-narmstrong@baylibre.com Neil Armstrong (5): phy: amlogic: phy-meson-gxl-usb2: keep ID pull-up even in Host mode dt-bindings: usb: amlogic,meson-g12a-usb-ctrl: add the Amlogic AXG Families USB Glue Bindings usb: dwc-meson-g12a: Add support for USB on AXG SoCs arm64: dts: meson-axg: add USB nodes arm64: dts: meson-axg-s400: enable USB OTG .../usb/amlogic,meson-g12a-usb-ctrl.yaml | 22 +++++++- .../arm64/boot/dts/amlogic/meson-axg-s400.dts | 6 +++ arch/arm64/boot/dts/amlogic/meson-axg.dtsi | 50 +++++++++++++++++++ drivers/phy/amlogic/phy-meson-gxl-usb2.c | 3 +- drivers/usb/dwc3/dwc3-meson-g12a.c | 18 +++++++ 5 files changed, 97 insertions(+), 2 deletions(-)