diff mbox series

[4/4] CI: Add m68k target

Message ID 20230319031454.282990-4-marek.vasut+renesas@mailbox.org
State Superseded
Delegated to: Tom Rini
Headers show
Series [1/4] arch: m68k: Use existing CONFIG_MCFTMR instead of CFG_MCFTMR | expand

Commit Message

Marek Vasut March 19, 2023, 3:14 a.m. UTC
Add M5208EVBE board to CI. This does not use default config due to
limitations of QEMU emulation, instead the timer is switched from
DMA timer to PIT timer and RAMBAR accesses are inhibited.

Local QEMU launch command is as follows:
$ qemu-system-m68k -nographic -machine mcf5208evb -cpu m5208 -bios u-boot.bin

Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
---
Cc: Angelo Dureghello <angelo@kernel-space.org>
Cc: Huan Wang <alison.wang@nxp.com>
Cc: Marek Vasut <marek.vasut+renesas@mailbox.org>
Cc: Simon Glass <sjg@chromium.org>
Cc: Stefan Roese <sr@denx.de>
Cc: Tom Rini <trini@konsulko.com>
---
 .azure-pipelines.yml | 5 +++++
 .gitlab-ci.yml       | 8 ++++++++
 2 files changed, 13 insertions(+)

Comments

Angelo Dureghello March 19, 2023, 7:27 p.m. UTC | #1
Hi Marek,

On 19/03/23 4:14 AM, Marek Vasut wrote:
> Add M5208EVBE board to CI. This does not use default config due to
> limitations of QEMU emulation, instead the timer is switched from
> DMA timer to PIT timer and RAMBAR accesses are inhibited.
> 
> Local QEMU launch command is as follows:
> $ qemu-system-m68k -nographic -machine mcf5208evb -cpu m5208 -bios u-boot.bin
> 
> Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
> ---
> Cc: Angelo Dureghello <angelo@kernel-space.org>
> Cc: Huan Wang <alison.wang@nxp.com>
> Cc: Marek Vasut <marek.vasut+renesas@mailbox.org>
> Cc: Simon Glass <sjg@chromium.org>
> Cc: Stefan Roese <sr@denx.de>
> Cc: Tom Rini <trini@konsulko.com>
> ---
>   .azure-pipelines.yml | 5 +++++
>   .gitlab-ci.yml       | 8 ++++++++
>   2 files changed, 13 insertions(+)
> 
> diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
> index 947c400f8d3..e4e0c4c4d0d 100644
> --- a/.azure-pipelines.yml
> +++ b/.azure-pipelines.yml
> @@ -268,6 +268,11 @@ stages:
>           qemu_arm64:
>             TEST_PY_BD: "qemu_arm64"
>             TEST_PY_TEST_SPEC: "not sleep"
> +        qemu_m68k:
> +          TEST_PY_BD: "M5208EVBE"
> +          TEST_PY_ID: "--id qemu"
> +          TEST_PY_TEST_SPEC: "not sleep and not efi"
> +          OVERRIDE: "-a CONFIG_M68K_QEMU=y -a CONFIG_MCFTMR=n"
>           qemu_malta:
>             TEST_PY_BD: "malta"
>             TEST_PY_ID: "--id qemu"
> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> index 272d69e2206..4c144ab938a 100644
> --- a/.gitlab-ci.yml
> +++ b/.gitlab-ci.yml
> @@ -322,6 +322,14 @@ qemu_arm64 test.py:
>       TEST_PY_TEST_SPEC: "not sleep"
>     <<: *buildman_and_testpy_dfn
>   
> +qemu_m68k test.py:
> +  variables:
> +    TEST_PY_BD: "M5208EVBE"
> +    TEST_PY_TEST_SPEC: "not sleep and not efi"
> +    TEST_PY_ID: "--id qemu"
> +    OVERRIDE: "-a CONFIG_M68K_QEMU=y -a CONFIG_MCFTMR=n"
> +  <<: *buildman_and_testpy_dfn
> +
>   qemu_malta test.py:
>     variables:
>       TEST_PY_BD: "malta"

i tested the patchset in u-boot-coldfire, i see also a pipeline error:

https://source.denx.de/u-boot/custodians/u-boot-coldfire/-/pipelines/15651/failures

regards,
angelo
Marek Vasut March 20, 2023, 1:36 a.m. UTC | #2
On 3/19/23 20:27, Angelo Dureghello wrote:

Hi,

[...]

>> +qemu_m68k test.py:
>> +  variables:
>> +    TEST_PY_BD: "M5208EVBE"
>> +    TEST_PY_TEST_SPEC: "not sleep and not efi"
>> +    TEST_PY_ID: "--id qemu"
>> +    OVERRIDE: "-a CONFIG_M68K_QEMU=y -a CONFIG_MCFTMR=n"
>> +  <<: *buildman_and_testpy_dfn
>> +
>>   qemu_malta test.py:
>>     variables:
>>       TEST_PY_BD: "malta"
> 
> i tested the patchset in u-boot-coldfire, i see also a pipeline error:
> 
> https://source.denx.de/u-boot/custodians/u-boot-coldfire/-/pipelines/15651/failures

That should be fixed here:

https://source.denx.de/u-boot/custodians/u-boot-sh/-/tree/m68k/wip

But I still suspect the qemu run will fail due to missing m68k qemu in 
the current docker image. Tom ?
diff mbox series

Patch

diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
index 947c400f8d3..e4e0c4c4d0d 100644
--- a/.azure-pipelines.yml
+++ b/.azure-pipelines.yml
@@ -268,6 +268,11 @@  stages:
         qemu_arm64:
           TEST_PY_BD: "qemu_arm64"
           TEST_PY_TEST_SPEC: "not sleep"
+        qemu_m68k:
+          TEST_PY_BD: "M5208EVBE"
+          TEST_PY_ID: "--id qemu"
+          TEST_PY_TEST_SPEC: "not sleep and not efi"
+          OVERRIDE: "-a CONFIG_M68K_QEMU=y -a CONFIG_MCFTMR=n"
         qemu_malta:
           TEST_PY_BD: "malta"
           TEST_PY_ID: "--id qemu"
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 272d69e2206..4c144ab938a 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -322,6 +322,14 @@  qemu_arm64 test.py:
     TEST_PY_TEST_SPEC: "not sleep"
   <<: *buildman_and_testpy_dfn
 
+qemu_m68k test.py:
+  variables:
+    TEST_PY_BD: "M5208EVBE"
+    TEST_PY_TEST_SPEC: "not sleep and not efi"
+    TEST_PY_ID: "--id qemu"
+    OVERRIDE: "-a CONFIG_M68K_QEMU=y -a CONFIG_MCFTMR=n"
+  <<: *buildman_and_testpy_dfn
+
 qemu_malta test.py:
   variables:
     TEST_PY_BD: "malta"