[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