Hi, On 08/08/2019 21:50, Martin Blumenstingl wrote: > On Thu, Aug 8, 2019 at 10:54 AM Neil Armstrong <narmstrong@xxxxxxxxxxxx> wrote: >> >> Now that we have the DT validation in place, let's convert the device tree >> bindings for the Amlogic G12A USB3 + PCIE Combo PHY over to a YAML schemas. >> >> While the original phy bindings specifies phy-supply as required, >> the examples and implementations makes it optional, thus phy-supply >> is not present in the properties and required lists. > nit-pick: the original bindings didn't mention the phy-supply property at all > I'm not sure if you have to re-send it, maybe this can be fixed up > while applying? It didn't, but the phy.txt made it mandatory, but was optional in all examples and implementation, thus rob asked me to precise it in the commit log. Neil > >> Reviewed-by: Rob Herring <robh@xxxxxxxxxx> >> Signed-off-by: Neil Armstrong <narmstrong@xxxxxxxxxxxx> > Reviewed-by: Martin Blumenstingl <martin.blumenstingl@xxxxxxxxxxxxxx> >