On 06/02/2020 22:55, Rob Herring wrote: > On Thu, 6 Feb 2020 07:10:50 +0100, Yuti Amonkar wrote: >> - Add Cadence MHDP PHY bindings in YAML format. >> - Add Torrent PHY reference clock bindings. >> - Add sub-node bindings for each group of PHY lanes based on PHY type. >> Each sub-node includes properties such as master lane number, link reset, >> phy type, number of lanes etc. >> - Add reset support including PHY reset and individual lane reset. >> - Add a new compatible string used for TI SoCs using Torrent PHY. >> This will not affect ABI as the driver has never been functional, >> and therefore do not exist in any active use case. >> >> Signed-off-by: Yuti Amonkar <yamonkar@xxxxxxxxxxx> >> --- >> .../bindings/phy/phy-cadence-torrent.yaml | 143 ++++++++++++++++++ >> 1 file changed, 143 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/phy/phy-cadence-torrent.yaml >> > > My bot found errors running 'make dt_binding_check' on your patch: > > Documentation/devicetree/bindings/display/simple-framebuffer.example.dts:21.16-37.11: Warning (chosen_node_is_root): /example-0/chosen: chosen node must be at root node > Error: Documentation/devicetree/bindings/phy/phy-cadence-torrent.example.dts:33.42-43 syntax error > FATAL ERROR: Unable to parse input tree > scripts/Makefile.lib:300: recipe for target 'Documentation/devicetree/bindings/phy/phy-cadence-torrent.example.dt.yaml' failed > make[1]: *** [Documentation/devicetree/bindings/phy/phy-cadence-torrent.example.dt.yaml] Error 1 > Makefile:1263: recipe for target 'dt_binding_check' failed > make: *** [dt_binding_check] Error 2 > > See https://patchwork.ozlabs.org/patch/1234141 > Please check and re-submit. > This should work now, that 8a79db5e83a5d52c74e6f3c40d6f312cf899213e is in, if applied again on top of recently tagged v5.6-rc1 . Best regards, Jyri -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki