Hi Neil, On Tue, Mar 24, 2020 at 11:20 AM Neil Armstrong <narmstrong@xxxxxxxxxxxx> wrote: [...] > +static const char *meson_a1_phy_names[] = { > + "usb2-phy0", "usb2-phy1" > +}; my understanding is that the A1 SoC only has usb2-phy1. +Cc Hanjie Lin to confirm this apart from that: Reviewed-by: Martin Blumenstingl <martin.blumenstingl@xxxxxxxxxxxxxx> Martin