On Fri, Aug 9, 2019 at 9:00 AM Neil Armstrong <narmstrong@xxxxxxxxxxxx> wrote: > > 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. sorry, I missed that my Reviewed-by still stands - thanks for pointing it out (again)