On Fri Jul 10 2020, Rob Herring wrote: > On Fri, 10 Jul 2020 11:06:18 +0200, Kurt Kanzenbach wrote: >> For future DSA drivers it makes sense to add a generic DSA yaml binding which >> can be used then. This was created using the properties from dsa.txt. It >> includes the ports and the dsa,member property. >> >> Suggested-by: Florian Fainelli >> Signed-off-by: Kurt Kanzenbach >> --- >> .../devicetree/bindings/net/dsa/dsa.yaml | 80 +++++++++++++++++++ >> 1 file changed, 80 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/net/dsa/dsa.yaml >> > > > My bot found errors running 'make dt_binding_check' on your patch: > > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/ti,cpsw-switch.example.dt.yaml: switch@0: 'ports' is a required property > /builds/robherring/linux-dt-review/Documentation/devicetree/bindings/net/qcom,ipq8064-mdio.example.dt.yaml: switch@10: 'ports' is a required property Okay, the requirement for 'ports' has be to removed. Thanks, Kurt