Re: [PATCH 3/4] dt-bindings: fec: document the new fsl,stop-mode property

From: Andrew Lunn
Date: Wed Mar 18 2020 - 05:17:46 EST


On Tue, Mar 17, 2020 at 05:50:05PM +0100, Martin Fuzzey wrote:
> This property allows the appropriate GPR register bit to be set
> for wake on lan support.
>
> Signed-off-by: Martin Fuzzey <martin.fuzzey@xxxxxxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/net/fsl-fec.txt | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt
> index 5b88fae0..bd0ef5e 100644
> --- a/Documentation/devicetree/bindings/net/fsl-fec.txt
> +++ b/Documentation/devicetree/bindings/net/fsl-fec.txt
> @@ -19,6 +19,11 @@ Optional properties:
> number to 1.
> - fsl,magic-packet : If present, indicates that the hardware supports waking
> up via magic packet.
> +- fsl,stop-mode: register bits of stop mode control, the format is
> + <&gpr reg bit>.
> + gpr is the phandle to general purpose register node.
> + reg is the gpr register offset for the stop request.
> + bit is the bit offset for the stop request.

Hi Martin

You should not be putting registers and values into device tree.

The regmap is fine. But could you add the register and the bit to
fec_devtype[IMX6SX_FEC], fec_devtype[IMX6UL_FEC], etc.

Andrew