[U-Boot] [PATCH v2 07/41] board: ge: bx50v3: use imx wdt
Robert Beckett
bob.beckett at collabora.com
Fri Oct 25 17:42:14 UTC 2019
On Wed, 2019-10-23 at 17:33 -0300, Fabio Estevam wrote:
> On Wed, Oct 23, 2019 at 3:22 PM Robert Beckett
> <bob.beckett at collabora.com> wrote:
> > Enable DM imx WDT
> > Enable SYSRESET_WATCHDOG to maintain WDT based reset ability
> >
> > Signed-off-by: Robert Beckett <bob.beckett at collabora.com>
> > ---
> > arch/arm/dts/imx6q-bx50v3.dts | 4 ++++
> > configs/ge_bx50v3_defconfig | 3 +++
> > 2 files changed, 7 insertions(+)
> >
> > diff --git a/arch/arm/dts/imx6q-bx50v3.dts b/arch/arm/dts/imx6q-
> > bx50v3.dts
> > index 28a35394f0..2e2e3bbc24 100644
> > --- a/arch/arm/dts/imx6q-bx50v3.dts
> > +++ b/arch/arm/dts/imx6q-bx50v3.dts
> > @@ -18,6 +18,10 @@
> > ethernet0 = &fec;
> > };
> >
> > + wdt-reboot {
> > + compatible = "wdt-reboot";
> > + wdt = <&wdog1>;
> > + };
> > };
>
> Looking at arch/arm/boot/dts/imx6q-bx50v3.dtsi from Linux, which
> includes arch/arm/boot/dts/imx6q-bx50v3.dtsi, the wdog definition is
> present:
>
> &wdog1 {
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_wdog>;
> fsl,ext-reset-output;
> };
>
> So it would be better just use the same bindings in U-Boot without
> the
> need of deviating the device tree.
wdt-reboot is a node to represent the wdt based sysreset driver.
It references the existing wdog1 via a phandle.
See drivers/sysreset/sysreset_watchdog.c
More information about the U-Boot
mailing list