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