Re: [PATCH net-next 1/4] dt-bindings: net: dsa: add Mediatek MT7530 binding

From: Andrew Lunn
Date: Mon Mar 13 2017 - 12:43:19 EST


> +- mediatek,reset-pin: Phandle to the pinctrl node used for the reset. Which
> + must be required if the property mediatek,mcm of specified as
> + "disabled". See
> + Documentation/devicetree/bindings/pinctrl/pinctrl-mt65xx.txt for
> + the mediatek pintcrl setting for the details.

Hi Sean

This appears to be a plan GPIO line. Marvell has the same. It would be
nice to be consistent with the naming. From
Documentation/devicetree/bindings/net/dsa/marvell.txt

Optional properties:

- reset-gpios : Should be a gpio specifier for a reset line

Andrew