On 5/25/2017 10:39 AM, Martin Blumenstingl wrote: > Hi John, > > On Sat, May 6, 2017 at 7:37 PM, Martin Blumenstingl > <martin.blumenstingl@xxxxxxxxxxxxxx> wrote: >> USB support in the Meson8 SoCs is provided by a DWC2 controller which >> works with the same settings as Meson8b and GXBB. Using the generic >> "snps,dwc2" binding results in an endless stream of "Overcurrent change >> detected" messages. >> >> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@xxxxxxxxxxxxxx> > do you want me to re-send this with Rob's ACK? > also please let me know if something is still missing in this patch > >> --- >> Documentation/devicetree/bindings/usb/dwc2.txt | 1 + >> drivers/usb/dwc2/params.c | 2 ++ >> 2 files changed, 3 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/usb/dwc2.txt b/Documentation/devicetree/bindings/usb/dwc2.txt >> index 6c7c2bce6d0c..b55be381ae85 100644 >> --- a/Documentation/devicetree/bindings/usb/dwc2.txt >> +++ b/Documentation/devicetree/bindings/usb/dwc2.txt >> @@ -10,6 +10,7 @@ Required properties: >> - "rockchip,rk3288-usb", "rockchip,rk3066-usb", "snps,dwc2": for rk3288 Soc; >> - "lantiq,arx100-usb": The DWC2 USB controller instance in Lantiq ARX SoCs; >> - "lantiq,xrx200-usb": The DWC2 USB controller instance in Lantiq XRX SoCs; >> + - "amlogic,meson8-usb": The DWC2 USB controller instance in Amlogic Meson8 SoCs; >> - "amlogic,meson8b-usb": The DWC2 USB controller instance in Amlogic Meson8b SoCs; >> - "amlogic,meson-gxbb-usb": The DWC2 USB controller instance in Amlogic S905 SoCs; >> - "amcc,dwc-otg": The DWC2 USB controller instance in AMCC Canyonlands 460EX SoCs; >> diff --git a/drivers/usb/dwc2/params.c b/drivers/usb/dwc2/params.c >> index 2990c347289f..0d6290ed66ea 100644 >> --- a/drivers/usb/dwc2/params.c >> +++ b/drivers/usb/dwc2/params.c >> @@ -128,6 +128,8 @@ const struct of_device_id dwc2_of_match_table[] = { >> { .compatible = "lantiq,xrx200-usb", .data = dwc2_set_ltq_params }, >> { .compatible = "snps,dwc2" }, >> { .compatible = "samsung,s3c6400-hsotg" }, >> + { .compatible = "amlogic,meson8-usb", >> + .data = dwc2_set_amlogic_params }, >> { .compatible = "amlogic,meson8b-usb", >> .data = dwc2_set_amlogic_params }, >> { .compatible = "amlogic,meson-gxbb-usb", >> -- >> 2.12.2 >> > Adding Felipe Acked-by: John Youn <johnyoun@xxxxxxxxxxxx> John -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html