Hi Fabio,
Den søn. 14. apr. 2019 kl. 17.32 skrev Fabio Estevam <festevam@xxxxxxxxx>:
> > &wdog1 {
> > + /*
> > + * Errata e10574:
> > + * WDOG reset needs to run with WDOG_RESET_B signal enabled.
> > + * X1-51 (WDOG1#) signal needs carrier board handling to reset
> > + * TQMa7 on X1-22 (RESET_IN#).
> > + */
> > + fsl,ext-reset-output;
> > status = "okay";
>
> You should also add a pinctrl entry for the watchdog pin.
There is a pinctrl entry for the watchdog pin in imx7-mba7.dtsi file
almost identically to imx7s-warp.dts. There are technical 2 pins you can
use for the output, and this dtsi will be used for other product device
trees as well. Where the imx7-mba7.dtsi will only be used for the
development board.
I agree with you that the readability of this dtsi will improve,
and the next device tree I want to mainline also use this pin configuration.
Bruno
