[U-Boot] [PATCH 12/15] include: rk3399: Disable watchdog in TPL

Jagan Teki jagan at amarulasolutions.com
Sun Aug 25 20:14:16 UTC 2019


On Mon, Aug 5, 2019 at 6:13 PM Kever Yang <kever.yang at rock-chips.com> wrote:
>
>
> On 2019/7/29 下午3:47, Jagan Teki wrote:
> > Disable watchdog for TPL, since there is no support it.
> >
> > Signed-off-by: Jagan Teki <jagan at amarulasolutions.com>
> > ---
> >   include/configs/rk3399_common.h | 5 +++++
> >   1 file changed, 5 insertions(+)
> >
> > diff --git a/include/configs/rk3399_common.h b/include/configs/rk3399_common.h
> > index 8df0180284..67b48e03ba 100644
> > --- a/include/configs/rk3399_common.h
> > +++ b/include/configs/rk3399_common.h
> > @@ -46,6 +46,11 @@
> >   #define CONFIG_SYS_SDRAM_BASE               0
> >   #define SDRAM_MAX_SIZE                      0xf8000000
> >
> > +#ifdef CONFIG_TPL_BUILD
> > +# undef CONFIG_WDT
> > +# undef CONFIG_WATCHDOG
> > +#endif
>
>
> It will be better to use CONFIG_${SPL_TPL}WDT in watchdog Makefile,
>
> because all the SoC needs treat this driver separate in TPL/SPL/U-Boot
> proper.

Enabling TPL (via DM or so) cannot be an optimal solutions as of now,
since if it continuously resetting we need to take boot switch based
on bootcount, ie possible with SPL and U-Boot proper now. Making a
boot switch to TPL might increase code size, so will check this
decision later based on the requirement.


More information about the U-Boot mailing list