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 Regards, Martin