[PATCH V4 2/2] configs: andes: add watchdog support fot andes ae350

Wed Jan 31 07:30:41 CET 2024

On Wed, Jan 24, 2024 at 09:37:41AM +0100, Stefan Roese wrote:
Hi Stefan,
Thank you for testing and reviewing.
I have checked the error message and the relationship to my patch.
The reason for the failure has nothing to do with my patch.
Could you kindly help me run the CI test again?

Randolph
Sincerely
> Hi Randolph,
> 
> On 1/24/24 07:21, Randolph wrote:
> > It adds the ATCWDT200 support for Andes AE350 platform.
> > It also enables wdt command support.
> > 
> > Signed-off-by: CL Wang <cl634 at andestech.com>
> > Signed-off-by: Randolph <randolph at andestech.com>
> > Reviewed-by: Leo Yu-Chi Liang <ycliang at andestech.com>
> 
> This fails in world CI build in "test.py for sandbox":
> 
> https://dev.azure.com/sr0718/0cded7c3-6e6a-4b57-8d0f-65c99496c42f/_apis/build/builds/338/logs/181
> 
> Not sure how this is related. Could you please take a look?
> 
> Thanks,
> Stefan
> 
> > ---
> >   configs/ae350_rv32_defconfig         | 4 ++++
> >   configs/ae350_rv32_spl_defconfig     | 4 ++++
> >   configs/ae350_rv32_spl_xip_defconfig | 4 ++++
> >   configs/ae350_rv32_xip_defconfig     | 4 ++++
> >   configs/ae350_rv64_defconfig         | 4 ++++
> >   configs/ae350_rv64_spl_defconfig     | 4 ++++
> >   configs/ae350_rv64_spl_xip_defconfig | 4 ++++
> >   configs/ae350_rv64_xip_defconfig     | 4 ++++
> >   8 files changed, 32 insertions(+)
> > 
> > diff --git a/configs/ae350_rv32_defconfig b/configs/ae350_rv32_defconfig
> > index 3bfa3e9f8e..35ad62c845 100644
> > --- a/configs/ae350_rv32_defconfig
> > +++ b/configs/ae350_rv32_defconfig
> > @@ -23,6 +23,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -49,3 +50,6 @@ CONFIG_BAUDRATE=38400
> >   CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> > diff --git a/configs/ae350_rv32_spl_defconfig b/configs/ae350_rv32_spl_defconfig
> > index aeb50206d2..41cd457bc3 100644
> > --- a/configs/ae350_rv32_spl_defconfig
> > +++ b/configs/ae350_rv32_spl_defconfig
> > @@ -33,6 +33,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -58,3 +59,6 @@ CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> >   # CONFIG_BINMAN_FDT is not set
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> > diff --git a/configs/ae350_rv32_spl_xip_defconfig b/configs/ae350_rv32_spl_xip_defconfig
> > index f15ec301ce..954e2f2de7 100644
> > --- a/configs/ae350_rv32_spl_xip_defconfig
> > +++ b/configs/ae350_rv32_spl_xip_defconfig
> > @@ -34,6 +34,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -59,3 +60,6 @@ CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> >   # CONFIG_BINMAN_FDT is not set
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> > diff --git a/configs/ae350_rv32_xip_defconfig b/configs/ae350_rv32_xip_defconfig
> > index c40eb043c5..95863595d2 100644
> > --- a/configs/ae350_rv32_xip_defconfig
> > +++ b/configs/ae350_rv32_xip_defconfig
> > @@ -24,6 +24,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -50,3 +51,6 @@ CONFIG_BAUDRATE=38400
> >   CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> > diff --git a/configs/ae350_rv64_defconfig b/configs/ae350_rv64_defconfig
> > index 7ae938aeb2..988214285a 100644
> > --- a/configs/ae350_rv64_defconfig
> > +++ b/configs/ae350_rv64_defconfig
> > @@ -23,6 +23,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -49,3 +50,6 @@ CONFIG_BAUDRATE=38400
> >   CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> > diff --git a/configs/ae350_rv64_spl_defconfig b/configs/ae350_rv64_spl_defconfig
> > index 68ac4325ab..e929320433 100644
> > --- a/configs/ae350_rv64_spl_defconfig
> > +++ b/configs/ae350_rv64_spl_defconfig
> > @@ -33,6 +33,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -58,3 +59,6 @@ CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> >   # CONFIG_BINMAN_FDT is not set
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> > diff --git a/configs/ae350_rv64_spl_xip_defconfig b/configs/ae350_rv64_spl_xip_defconfig
> > index 839ca335d4..3f92643b07 100644
> > --- a/configs/ae350_rv64_spl_xip_defconfig
> > +++ b/configs/ae350_rv64_spl_xip_defconfig
> > @@ -34,6 +34,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -59,3 +60,6 @@ CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> >   # CONFIG_BINMAN_FDT is not set
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> > diff --git a/configs/ae350_rv64_xip_defconfig b/configs/ae350_rv64_xip_defconfig
> > index 5432b6d6d7..835f020990 100644
> > --- a/configs/ae350_rv64_xip_defconfig
> > +++ b/configs/ae350_rv64_xip_defconfig
> > @@ -24,6 +24,7 @@ CONFIG_SYS_PROMPT="RISC-V # "
> >   CONFIG_CMD_IMLS=y
> >   CONFIG_CMD_MMC=y
> >   CONFIG_CMD_SF_TEST=y
> > +CONFIG_CMD_WDT=y
> >   # CONFIG_CMD_SETEXPR is not set
> >   CONFIG_BOOTP_PREFER_SERVERIP=y
> >   CONFIG_CMD_CACHE=y
> > @@ -50,3 +51,6 @@ CONFIG_BAUDRATE=38400
> >   CONFIG_SYS_NS16550=y
> >   CONFIG_SPI=y
> >   CONFIG_ATCSPI200_SPI=y
> > +# CONFIG_WATCHDOG_AUTOSTART is not set
> > +CONFIG_WDT=y
> > +CONFIG_WDT_ATCWDT200=y
> 
> Viele Grüße,
> Stefan Roese
> 
> -- 
> DENX Software Engineering GmbH,      Managing Director: Erika Unter
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr at denx.de