On 15/09/2020 21:59, Martin Blumenstingl wrote: > Hi Neil, > > On Wed, Sep 9, 2020 at 6:04 PM Neil Armstrong <narmstrong@xxxxxxxxxxxx> wrote: >> >> The Amlogic AXG is close from the GXL Glue but with a single OTG PHY. > s/close from/close to/ > > [...] >> +static struct dwc3_meson_g12a_drvdata axg_drvdata = { >> + .otg_switch_supported = true, >> + .clks = meson_gxl_clocks, >> + .num_clks = ARRAY_SIZE(meson_gxl_clocks), >> + .phy_names = meson_a1_phy_names, >> + .num_phys = ARRAY_SIZE(meson_a1_phy_names), > I wonder if we should also update the comment for meson_a1_phy_names > to understand that part I had to go back to the dt-bindings patch Sure, Thanks, Neil > > > Best regards, > Martin >