Re: [PATCH v2 3/3] dt-bindings: net: Convert fsl,gianfar to YAML

[Rob Herring (Arm)]

On Fri, 28 Feb 2025 18:32:52 +0100, J. Neuschäfer wrote:
> Add a binding for the "Gianfar" ethernet controller, also known as
> TSEC/eTSEC.
> 
> Signed-off-by: J. Neuschäfer <j.ne@xxxxxxxxxx>
> ---
> 
> V2:
> - change MAC address in example to 00:00:00:00:00:00 instead of a real
>   Motorola MAC address (suggested by Andrew Lunn)
> - add constraints to #address/size-cells, fsl,num_tr/rx_queues
> - remove unnecessary type from dma-coherent
> - add minItems to interrupts
> - remove unnecessary #address/size-cells from queue-group@.*
> - describe interrupts of queue-group@.*
> - remove unnecessary bus in example
> - consistently use "type: boolean" for fsl,magic-packet,
>   instead of "$ref: /schemas/types.yaml#/definitions/flag"
> - fix name of rx-stash-idx property
> - fix type of rx-stash-len/idx properties
> - actually reference fsl,gianfar-mdio schema for mdio@.* subnodes
> - disambiguate compatible = "gianfar" schemas by using a "select:" schema
> ---
>  .../devicetree/bindings/net/fsl,gianfar.yaml       | 248 +++++++++++++++++++++
>  .../devicetree/bindings/net/fsl-tsec-phy.txt       |  39 +---
>  2 files changed, 249 insertions(+), 38 deletions(-)
> 

Reviewed-by: Rob Herring (Arm) <robh@xxxxxxxxxx>