Hi Joel,

On 02.09.22 08:00, Joel Stanley wrote:
On Fri, 2 Sept 2022 at 04:09, Joel Stanley <j...@jms.id.au> wrote:

On Mon, 29 Aug 2022 at 06:23, Stefan Roese <s...@denx.de> wrote:

The evb-ast2600 target always runs into an timeout error when run via
Azure CI. For test purpose only, this patch removes this CI build
target so that the world build can be run.

Joel, Chia-Wei, could you perhaps take a look at this and help to fix
the problems that I'm seeing with this build target here?

Where does it hang? Can you attach the logs?

The hang appears to be in the timer code, where the spl hangs due to a
div by zero. When inspecting, gd->arch.timer_rate_hz is zero at this
point.

However I also noticed the cycling code introduces a symbol that ends
up in BSS. If I give cyclic_running a non-zero value it moves back
into the data section, and the SPL and u-boot proper make it all the
way through.

I might rework the code a bit, so that cyclic does not introduce new
static variables but uses a new data pointer in struct global_data
instead.

This suggests there's still something wrong with the BSS handling for
the 2600 SPL (not resolved by f6810b749f2e ("aspeed/ast2600: Fix SPL
linker script") or the follow up fix). If I use the common arm spl
linker script, the system boots.

Thanks for looking into this.

Chai Wei, I think we need a different solution for placing the BSS
outside of the flash.

Thanks,
Stefan

Cheers,

Joel


What kind of debugging have you tried to date?

Cheers,

Joel


Signed-off-by: Stefan Roese <s...@denx.de>
Cc: Joel Stanley <j...@jms.id.au>
Cc: Chia-Wei Wang <chiawei_w...@aspeedtech.com>
---
  .azure-pipelines.yml | 3 ---
  1 file changed, 3 deletions(-)

diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
index 0fa92479b4c4..e48f1667f8de 100644
--- a/.azure-pipelines.yml
+++ b/.azure-pipelines.yml
@@ -261,9 +261,6 @@ stages:
          evb_ast2500:
            TEST_PY_BD: "evb-ast2500"
            TEST_PY_ID: "--id qemu"
-        evb_ast2600:
-          TEST_PY_BD: "evb-ast2600"
-          TEST_PY_ID: "--id qemu"
          vexpress_ca9x4:
            TEST_PY_BD: "vexpress_ca9x4"
            TEST_PY_ID: "--id qemu"
--
2.37.2


Viele Grüße,
Stefan Roese

--
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de

Reply via email to