[PATCH] tester/zynqmp: Simpify the reasons to reset

[Kinsey Moore]

Comment below.

On 9/6/2021 23:38, chrisj at rtems.org wrote:
> From: Chris Johns <chrisj at rtems.org>
>
> - Only reset if the bootloader error is hardware related.
> ---
>   tester/rtems/testing/bsps/xilinx_zynqmp_ilp32_zu3eg.ini | 2 +-
>   tester/rtems/testing/bsps/xilinx_zynqmp_lp64_zu3eg.ini  | 2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tester/rtems/testing/bsps/xilinx_zynqmp_ilp32_zu3eg.ini b/tester/rtems/testing/bsps/xilinx_zynqmp_ilp32_zu3eg.ini
> index 5fd2039..8fd2aae 100644
> --- a/tester/rtems/testing/bsps/xilinx_zynqmp_ilp32_zu3eg.ini
> +++ b/tester/rtems/testing/bsps/xilinx_zynqmp_ilp32_zu3eg.ini
> @@ -36,6 +36,6 @@ bsp                = xilinx_zynqmp_ilp32_zu3eg
>   arch               = aarch64
>   tester             = %{_rtscripts}/tftp.cfg
>   jobs               = 1
> -target_reset_regex = ^No ethernet found.*|^BOOTP broadcast 6.*|^.+complete\.+ TIMEOUT.*|^.*Retry count exceeded.*|^.*: No link\..*|^.*TFTP server died; starting again.*
> +target_reset_regex = ^No ethernet found.*|^.+complete\.+ TIMEOUT.*|^.*: No link\..*
Is there a reason this is different from the LP64 restart regex below?
>   target_start_regex = ^.*Xilinx Zynq MP First Stage Boot Loader.*
>   requires           = target_on_command, target_off_command, target_reset_command, bsp_tty_dev
> diff --git a/tester/rtems/testing/bsps/xilinx_zynqmp_lp64_zu3eg.ini b/tester/rtems/testing/bsps/xilinx_zynqmp_lp64_zu3eg.ini
> index dff7d86..663e5de 100644
> --- a/tester/rtems/testing/bsps/xilinx_zynqmp_lp64_zu3eg.ini
> +++ b/tester/rtems/testing/bsps/xilinx_zynqmp_lp64_zu3eg.ini
> @@ -37,6 +37,6 @@ arch               = aarch64
>   tester             = %{_rtscripts}/tftp.cfg
>   jobs               = 1
>   test_restarts      = 3
> -target_reset_regex = ^No ethernet found.*|^BOOTP broadcast 6.*|^.+complete\.+ TIMEOUT.*|^.*Retry count exceeded.*|^.*: No link\..*|^.*TFTP server died; starting again.*
> +target_reset_regex = ^No ethernet found.*|^.*: No link\..*
>   target_start_regex = ^.*Xilinx Zynq MP First Stage Boot Loader.*
>   requires           = target_on_command, target_off_command, target_reset_command, bsp_tty_dev