Re: [PATCH 06/17] dt-bindings: net: mscc-miim: Add resets property

From: Andrew Lunn
Date: Tue Apr 30 2024 - 12:33:04 EST


> We have the same construction with the pinctrl driver used in the LAN966x
> Documentation/devicetree/bindings/pinctrl/mscc,ocelot-pinctrl.yaml
>
> The reset name is 'switch' in the pinctrl binding.
> I can use the same description here as the one present in the pinctrl binding:
> description: Optional shared switch reset.
> and keep 'switch' as reset name here (consistent with pinctrl reset name).
>
> What do you think about that ?

It would be good to document what it is shared with. So it seems to be
the switch itself, pinctl and MDIO? Anything else?

Andrew