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

From: Andrew Lunn
Date: Wed Mar 18 2020 - 07:38:31 EST


On Wed, Mar 18, 2020 at 10:28:48AM +0100, Fuzzey, Martin wrote:
> Hi Andrew,
>
> On Wed, 18 Mar 2020 at 10:17, Andrew Lunn <andrew@xxxxxxx> wrote:
> >
> > 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.
> >
>
> If that's the consensus I can do it that way.
>
> But I should point out that there is already a precedent in mainline for this:
>
> Documentation/devicetree/bindings/net/can/fsl-flexcan.txt

Hi Martin

And there are probably hundreds of emails saying don't do this.

Andrew