[PATCH v1 0/3] Update to test new ASPEED SDK version

[PATCH v1 0/3] Update to test new ASPEED SDK version

[Jamin Lin via]

v1:
  Update to test new ASPEED SDK version for AST2600, AST2500 ,and AST1030.

Jamin Lin (3):
  tests/functional/aspeed: Update test ASPEED SDK v09.06 for AST2500
  tests/functional/aspeed: Update test ASPEED SDK v09.06 for AST2500
  tests/functional/aspeed: Update test ASPEED SDK v03.00 for AST1030

 tests/functional/test_arm_aspeed_ast1030.py | 40 ++++-----------------
 tests/functional/test_arm_aspeed_ast2500.py |  8 ++---
 tests/functional/test_arm_aspeed_ast2600.py | 15 ++++----
 3 files changed, 19 insertions(+), 44 deletions(-)

-- 
2.43.0

[PATCH v1 1/3] tests/functional/aspeed: Update test ASPEED SDK v09.06 for AST2500

[Jamin Lin via]

Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
 tests/functional/test_arm_aspeed_ast2500.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/functional/test_arm_aspeed_ast2500.py b/tests/functional/test_arm_aspeed_ast2500.py
index 1ffba6c995..90f3940e2a 100755
--- a/tests/functional/test_arm_aspeed_ast2500.py
+++ b/tests/functional/test_arm_aspeed_ast2500.py
@@ -38,14 +38,14 @@ def test_arm_ast2500_evb_buildroot(self):
 
         self.do_test_arm_aspeed_buildroot_poweroff()
 
-    ASSET_SDK_V806_AST2500 = Asset(
-        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v08.06/ast2500-default-obmc.tar.gz',
-        'e1755f3cadff69190438c688d52dd0f0d399b70a1e14b1d3d5540fc4851d38ca')
+    ASSET_SDK_V906_AST2500 = Asset(
+        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.06/ast2500-default-obmc.tar.gz',
+        '542db84645b4efd8aed50385d7f4dd1caff379a987032311cfa7b563a3addb2a')
 
     def test_arm_ast2500_evb_sdk(self):
         self.set_machine('ast2500-evb')
 
-        self.archive_extract(self.ASSET_SDK_V806_AST2500)
+        self.archive_extract(self.ASSET_SDK_V906_AST2500)
 
         self.do_test_arm_aspeed_sdk_start(
             self.scratch_file("ast2500-default", "image-bmc"))
-- 
2.43.0

[PATCH v1 2/3] tests/functional/aspeed: Update test ASPEED SDK v09.06 for AST2500

[Jamin Lin via]

Update test for AST2600 production revision A3.

Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
 tests/functional/test_arm_aspeed_ast2600.py | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/tests/functional/test_arm_aspeed_ast2600.py b/tests/functional/test_arm_aspeed_ast2600.py
index 6ae4ed636a..c30397f60d 100755
--- a/tests/functional/test_arm_aspeed_ast2600.py
+++ b/tests/functional/test_arm_aspeed_ast2600.py
@@ -97,26 +97,27 @@ def test_arm_ast2600_evb_buildroot_tpm(self):
 
         self.do_test_arm_aspeed_buildroot_poweroff()
 
-    ASSET_SDK_V806_AST2600_A2 = Asset(
-        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v08.06/ast2600-a2-obmc.tar.gz',
-        '9083506135f622d5e7351fcf7d4e1c7125cee5ba16141220c0ba88931f3681a4')
+    ASSET_SDK_V906_AST2600 = Asset(
+        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.06/ast2600-default-obmc.tar.gz',
+        '768d76e247896ad78c154b9cff4f766da2ce65f217d620b286a4a03a8a4f68f5')
 
     def test_arm_ast2600_evb_sdk(self):
         self.set_machine('ast2600-evb')
 
-        self.archive_extract(self.ASSET_SDK_V806_AST2600_A2)
+        self.archive_extract(self.ASSET_SDK_V906_AST2600)
 
         self.vm.add_args('-device',
             'tmp105,bus=aspeed.i2c.bus.5,address=0x4d,id=tmp-test');
         self.vm.add_args('-device',
             'ds1338,bus=aspeed.i2c.bus.5,address=0x32');
         self.do_test_arm_aspeed_sdk_start(
-            self.scratch_file("ast2600-a2", "image-bmc"))
+            self.scratch_file("ast2600-default", "image-bmc"))
 
-        self.wait_for_console_pattern('ast2600-a2 login:')
+        self.wait_for_console_pattern('ast2600-default login:')
 
         exec_command_and_wait_for_pattern(self, 'root', 'Password:')
-        exec_command_and_wait_for_pattern(self, '0penBmc', 'root@ast2600-a2:~#')
+        exec_command_and_wait_for_pattern(self, '0penBmc',
+                                          'root@ast2600-default:~#')
 
         exec_command_and_wait_for_pattern(self,
             'echo lm75 0x4d > /sys/class/i2c-dev/i2c-5/device/new_device',
-- 
2.43.0

[PATCH v1 3/3] tests/functional/aspeed: Update test ASPEED SDK v03.00 for AST1030

[Jamin Lin via]

Updated test to target SDK version v03.00.
Removed redundant test case to simplify validation.
Cleaned up unsupported shell commands.

Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
---
 tests/functional/test_arm_aspeed_ast1030.py | 40 ++++-----------------
 1 file changed, 7 insertions(+), 33 deletions(-)

diff --git a/tests/functional/test_arm_aspeed_ast1030.py b/tests/functional/test_arm_aspeed_ast1030.py
index d45d9f7c1c..dafa1d24cb 100755
--- a/tests/functional/test_arm_aspeed_ast1030.py
+++ b/tests/functional/test_arm_aspeed_ast1030.py
@@ -12,36 +12,17 @@
 
 class AST1030Machine(LinuxKernelTest):
 
-    ASSET_ZEPHYR_1_04 = Asset(
+    ASSET_ZEPHYR_3_00 = Asset(
         ('https://github.com/AspeedTech-BMC'
-         '/zephyr/releases/download/v00.01.04/ast1030-evb-demo.zip'),
-        '4ac6210adcbc61294927918707c6762483fd844dde5e07f3ba834ad1f91434d3')
+         '/zephyr/releases/download/v00.03.00/ast1030-evb-demo.zip'),
+        '37fe3ecd4a1b9d620971a15b96492a81093435396eeac69b6f3e384262ff555f')
 
-    def test_ast1030_zephyros_1_04(self):
-        self.set_machine('ast1030-evb')
-
-        kernel_name = "ast1030-evb-demo/zephyr.elf"
-        kernel_file = self.archive_extract(
-            self.ASSET_ZEPHYR_1_04, member=kernel_name)
-
-        self.vm.set_console()
-        self.vm.add_args('-kernel', kernel_file, '-nographic')
-        self.vm.launch()
-        self.wait_for_console_pattern("Booting Zephyr OS")
-        exec_command_and_wait_for_pattern(self, "help",
-                                          "Available commands")
-
-    ASSET_ZEPHYR_1_07 = Asset(
-        ('https://github.com/AspeedTech-BMC'
-         '/zephyr/releases/download/v00.01.07/ast1030-evb-demo.zip'),
-        'ad52e27959746988afaed8429bf4e12ab988c05c4d07c9d90e13ec6f7be4574c')
-
-    def test_ast1030_zephyros_1_07(self):
+    def test_ast1030_zephyros_3_00(self):
         self.set_machine('ast1030-evb')
 
         kernel_name = "ast1030-evb-demo/zephyr.bin"
         kernel_file = self.archive_extract(
-            self.ASSET_ZEPHYR_1_07, member=kernel_name)
+            self.ASSET_ZEPHYR_3_00, member=kernel_name)
 
         self.vm.set_console()
         self.vm.add_args('-kernel', kernel_file, '-nographic')
@@ -49,17 +30,10 @@ def test_ast1030_zephyros_1_07(self):
         self.wait_for_console_pattern("Booting Zephyr OS")
         for shell_cmd in [
                 'kernel stacks',
-                'otp info conf',
-                'otp info scu',
                 'hwinfo devid',
                 'crypto aes256_cbc_vault',
-                'random get',
-                'jtag JTAG1 sw_xfer high TMS',
-                'adc ADC0 resolution 12',
-                'adc ADC0 read 42',
-                'adc ADC1 read 69',
-                'i2c scan I2C_0',
-                'i3c attach I3C_0',
+                'jtag jtag@7e6e4100 sw_xfer high TMS',
+                'iic scan i2c@7e7b0080',
                 'hash test',
                 'kernel uptime',
                 'kernel reboot warm',
-- 
2.43.0

Re: [PATCH v1 3/3] tests/functional/aspeed: Update test ASPEED SDK v03.00 for AST1030

[Philippe Mathieu-Daudé]

Hi Jamin,

On 22/4/25 11:15, Jamin Lin via wrote:
> Updated test to target SDK version v03.00.
> Removed redundant test case to simplify validation.

> Cleaned up unsupported shell commands.

Does that mean coverage is reduced?

> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
>   tests/functional/test_arm_aspeed_ast1030.py | 40 ++++-----------------
>   1 file changed, 7 insertions(+), 33 deletions(-)


> @@ -49,17 +30,10 @@ def test_ast1030_zephyros_1_07(self):
>           self.wait_for_console_pattern("Booting Zephyr OS")
>           for shell_cmd in [
>                   'kernel stacks',
> -                'otp info conf',
> -                'otp info scu',
>                   'hwinfo devid',
>                   'crypto aes256_cbc_vault',
> -                'random get',
> -                'jtag JTAG1 sw_xfer high TMS',
> -                'adc ADC0 resolution 12',
> -                'adc ADC0 read 42',
> -                'adc ADC1 read 69',
> -                'i2c scan I2C_0',
> -                'i3c attach I3C_0',
> +                'jtag jtag@7e6e4100 sw_xfer high TMS',
> +                'iic scan i2c@7e7b0080',
>                   'hash test',
>                   'kernel uptime',
>                   'kernel reboot warm',

Re: [PATCH v1 3/3] tests/functional/aspeed: Update test ASPEED SDK v03.00 for AST1030

[Cédric Le Goater]

On 4/22/25 11:15, Jamin Lin wrote:
> Updated test to target SDK version v03.00.
> Removed redundant test case to simplify validation.
> Cleaned up unsupported shell commands.

Were some commands removed from SDK v03.00 ? Testing different
zephyr versions might be interesting in that case, like it would
be for Linux too.

Thanks,

C.




> 
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
>   tests/functional/test_arm_aspeed_ast1030.py | 40 ++++-----------------
>   1 file changed, 7 insertions(+), 33 deletions(-)
> 
> diff --git a/tests/functional/test_arm_aspeed_ast1030.py b/tests/functional/test_arm_aspeed_ast1030.py
> index d45d9f7c1c..dafa1d24cb 100755
> --- a/tests/functional/test_arm_aspeed_ast1030.py
> +++ b/tests/functional/test_arm_aspeed_ast1030.py
> @@ -12,36 +12,17 @@
>   
>   class AST1030Machine(LinuxKernelTest):
>   
> -    ASSET_ZEPHYR_1_04 = Asset(
> +    ASSET_ZEPHYR_3_00 = Asset(
>           ('https://github.com/AspeedTech-BMC'
> -         '/zephyr/releases/download/v00.01.04/ast1030-evb-demo.zip'),
> -        '4ac6210adcbc61294927918707c6762483fd844dde5e07f3ba834ad1f91434d3')
> +         '/zephyr/releases/download/v00.03.00/ast1030-evb-demo.zip'),
> +        '37fe3ecd4a1b9d620971a15b96492a81093435396eeac69b6f3e384262ff555f')
>   
> -    def test_ast1030_zephyros_1_04(self):
> -        self.set_machine('ast1030-evb')
> -
> -        kernel_name = "ast1030-evb-demo/zephyr.elf"
> -        kernel_file = self.archive_extract(
> -            self.ASSET_ZEPHYR_1_04, member=kernel_name)
> -
> -        self.vm.set_console()
> -        self.vm.add_args('-kernel', kernel_file, '-nographic')
> -        self.vm.launch()
> -        self.wait_for_console_pattern("Booting Zephyr OS")
> -        exec_command_and_wait_for_pattern(self, "help",
> -                                          "Available commands")
> -
> -    ASSET_ZEPHYR_1_07 = Asset(
> -        ('https://github.com/AspeedTech-BMC'
> -         '/zephyr/releases/download/v00.01.07/ast1030-evb-demo.zip'),
> -        'ad52e27959746988afaed8429bf4e12ab988c05c4d07c9d90e13ec6f7be4574c')
> -
> -    def test_ast1030_zephyros_1_07(self):
> +    def test_ast1030_zephyros_3_00(self):
>           self.set_machine('ast1030-evb')
>   
>           kernel_name = "ast1030-evb-demo/zephyr.bin"
>           kernel_file = self.archive_extract(
> -            self.ASSET_ZEPHYR_1_07, member=kernel_name)
> +            self.ASSET_ZEPHYR_3_00, member=kernel_name)
>   
>           self.vm.set_console()
>           self.vm.add_args('-kernel', kernel_file, '-nographic')
> @@ -49,17 +30,10 @@ def test_ast1030_zephyros_1_07(self):
>           self.wait_for_console_pattern("Booting Zephyr OS")
>           for shell_cmd in [
>                   'kernel stacks',
> -                'otp info conf',
> -                'otp info scu',
>                   'hwinfo devid',
>                   'crypto aes256_cbc_vault',
> -                'random get',
> -                'jtag JTAG1 sw_xfer high TMS',
> -                'adc ADC0 resolution 12',
> -                'adc ADC0 read 42',
> -                'adc ADC1 read 69',
> -                'i2c scan I2C_0',
> -                'i3c attach I3C_0',
> +                'jtag jtag@7e6e4100 sw_xfer high TMS',
> +                'iic scan i2c@7e7b0080',
>                   'hash test',
>                   'kernel uptime',
>                   'kernel reboot warm',

Re: [PATCH v1 1/3] tests/functional/aspeed: Update test ASPEED SDK v09.06 for AST2500

[Cédric Le Goater]

On 4/22/25 11:15, Jamin Lin wrote:
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>


Reviewed-by: Cédric Le Goater <clg@redhat.com>

Thanks,

C.


> ---
>   tests/functional/test_arm_aspeed_ast2500.py | 8 ++++----
>   1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/functional/test_arm_aspeed_ast2500.py b/tests/functional/test_arm_aspeed_ast2500.py
> index 1ffba6c995..90f3940e2a 100755
> --- a/tests/functional/test_arm_aspeed_ast2500.py
> +++ b/tests/functional/test_arm_aspeed_ast2500.py
> @@ -38,14 +38,14 @@ def test_arm_ast2500_evb_buildroot(self):
>   
>           self.do_test_arm_aspeed_buildroot_poweroff()
>   
> -    ASSET_SDK_V806_AST2500 = Asset(
> -        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v08.06/ast2500-default-obmc.tar.gz',
> -        'e1755f3cadff69190438c688d52dd0f0d399b70a1e14b1d3d5540fc4851d38ca')
> +    ASSET_SDK_V906_AST2500 = Asset(
> +        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.06/ast2500-default-obmc.tar.gz',
> +        '542db84645b4efd8aed50385d7f4dd1caff379a987032311cfa7b563a3addb2a')
>   
>       def test_arm_ast2500_evb_sdk(self):
>           self.set_machine('ast2500-evb')
>   
> -        self.archive_extract(self.ASSET_SDK_V806_AST2500)
> +        self.archive_extract(self.ASSET_SDK_V906_AST2500)
>   
>           self.do_test_arm_aspeed_sdk_start(
>               self.scratch_file("ast2500-default", "image-bmc"))

Re: [PATCH v1 2/3] tests/functional/aspeed: Update test ASPEED SDK v09.06 for AST2500

[Cédric Le Goater]

On 4/22/25 11:15, Jamin Lin wrote:
> Update test for AST2600 production revision A3.

Reviewed-by: Cédric Le Goater <clg@redhat.com>

Thanks,

C.




> 
> Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> ---
>   tests/functional/test_arm_aspeed_ast2600.py | 15 ++++++++-------
>   1 file changed, 8 insertions(+), 7 deletions(-)
> 
> diff --git a/tests/functional/test_arm_aspeed_ast2600.py b/tests/functional/test_arm_aspeed_ast2600.py
> index 6ae4ed636a..c30397f60d 100755
> --- a/tests/functional/test_arm_aspeed_ast2600.py
> +++ b/tests/functional/test_arm_aspeed_ast2600.py
> @@ -97,26 +97,27 @@ def test_arm_ast2600_evb_buildroot_tpm(self):
>   
>           self.do_test_arm_aspeed_buildroot_poweroff()
>   
> -    ASSET_SDK_V806_AST2600_A2 = Asset(
> -        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v08.06/ast2600-a2-obmc.tar.gz',
> -        '9083506135f622d5e7351fcf7d4e1c7125cee5ba16141220c0ba88931f3681a4')
> +    ASSET_SDK_V906_AST2600 = Asset(
> +        'https://github.com/AspeedTech-BMC/openbmc/releases/download/v09.06/ast2600-default-obmc.tar.gz',
> +        '768d76e247896ad78c154b9cff4f766da2ce65f217d620b286a4a03a8a4f68f5')
>   
>       def test_arm_ast2600_evb_sdk(self):
>           self.set_machine('ast2600-evb')
>   
> -        self.archive_extract(self.ASSET_SDK_V806_AST2600_A2)
> +        self.archive_extract(self.ASSET_SDK_V906_AST2600)
>   
>           self.vm.add_args('-device',
>               'tmp105,bus=aspeed.i2c.bus.5,address=0x4d,id=tmp-test');
>           self.vm.add_args('-device',
>               'ds1338,bus=aspeed.i2c.bus.5,address=0x32');
>           self.do_test_arm_aspeed_sdk_start(
> -            self.scratch_file("ast2600-a2", "image-bmc"))
> +            self.scratch_file("ast2600-default", "image-bmc"))
>   
> -        self.wait_for_console_pattern('ast2600-a2 login:')
> +        self.wait_for_console_pattern('ast2600-default login:')
>   
>           exec_command_and_wait_for_pattern(self, 'root', 'Password:')
> -        exec_command_and_wait_for_pattern(self, '0penBmc', 'root@ast2600-a2:~#')
> +        exec_command_and_wait_for_pattern(self, '0penBmc',
> +                                          'root@ast2600-default:~#')
>   
>           exec_command_and_wait_for_pattern(self,
>               'echo lm75 0x4d > /sys/class/i2c-dev/i2c-5/device/new_device',

RE: [PATCH v1 3/3] tests/functional/aspeed: Update test ASPEED SDK v03.00 for AST1030

[Jamin Lin]

Hi Philip, Cedric, 

> 
> On 4/22/25 11:15, Jamin Lin wrote:
> > Updated test to target SDK version v03.00.
> > Removed redundant test case to simplify validation.
> > Cleaned up unsupported shell commands.
> 
> Were some commands removed from SDK v03.00 ? Testing different zephyr

Thanks for review and suggestion.
Yes, some commands have been removed in the pre-built image of SDK v03.00.
The changes in v1 reduce the overall test coverage.
I will send a v2 patch to retain the test_ast1030_zephyros_1_07 test case and 
update test_ast1030_zephyros_1_04 to use SDK v03.00.

Thanks-Jamin

> versions might be interesting in that case, like it would be for Linux too.
> 
> Thanks,
> 
> C.
> 
> 
> 
> 
> >
> > Signed-off-by: Jamin Lin <jamin_lin@aspeedtech.com>
> > ---
> >   tests/functional/test_arm_aspeed_ast1030.py | 40 ++++-----------------
> >   1 file changed, 7 insertions(+), 33 deletions(-)
> >
> > diff --git a/tests/functional/test_arm_aspeed_ast1030.py
> > b/tests/functional/test_arm_aspeed_ast1030.py
> > index d45d9f7c1c..dafa1d24cb 100755
> > --- a/tests/functional/test_arm_aspeed_ast1030.py
> > +++ b/tests/functional/test_arm_aspeed_ast1030.py
> > @@ -12,36 +12,17 @@
> >
> >   class AST1030Machine(LinuxKernelTest):
> >
> > -    ASSET_ZEPHYR_1_04 = Asset(
> > +    ASSET_ZEPHYR_3_00 = Asset(
> >           ('https://github.com/AspeedTech-BMC'
> > -         '/zephyr/releases/download/v00.01.04/ast1030-evb-demo.zip'),
> > -
> '4ac6210adcbc61294927918707c6762483fd844dde5e07f3ba834ad1f91434d3')
> > +         '/zephyr/releases/download/v00.03.00/ast1030-evb-demo.zip'),
> > +
> > +
> '37fe3ecd4a1b9d620971a15b96492a81093435396eeac69b6f3e384262ff555f')
> >
> > -    def test_ast1030_zephyros_1_04(self):
> > -        self.set_machine('ast1030-evb')
> > -
> > -        kernel_name = "ast1030-evb-demo/zephyr.elf"
> > -        kernel_file = self.archive_extract(
> > -            self.ASSET_ZEPHYR_1_04, member=kernel_name)
> > -
> > -        self.vm.set_console()
> > -        self.vm.add_args('-kernel', kernel_file, '-nographic')
> > -        self.vm.launch()
> > -        self.wait_for_console_pattern("Booting Zephyr OS")
> > -        exec_command_and_wait_for_pattern(self, "help",
> > -                                          "Available commands")
> > -
> > -    ASSET_ZEPHYR_1_07 = Asset(
> > -        ('https://github.com/AspeedTech-BMC'
> > -         '/zephyr/releases/download/v00.01.07/ast1030-evb-demo.zip'),
> > -
> 'ad52e27959746988afaed8429bf4e12ab988c05c4d07c9d90e13ec6f7be4574c')
> > -
> > -    def test_ast1030_zephyros_1_07(self):
> > +    def test_ast1030_zephyros_3_00(self):
> >           self.set_machine('ast1030-evb')
> >
> >           kernel_name = "ast1030-evb-demo/zephyr.bin"
> >           kernel_file = self.archive_extract(
> > -            self.ASSET_ZEPHYR_1_07, member=kernel_name)
> > +            self.ASSET_ZEPHYR_3_00, member=kernel_name)
> >
> >           self.vm.set_console()
> >           self.vm.add_args('-kernel', kernel_file, '-nographic') @@
> > -49,17 +30,10 @@ def test_ast1030_zephyros_1_07(self):
> >           self.wait_for_console_pattern("Booting Zephyr OS")
> >           for shell_cmd in [
> >                   'kernel stacks',
> > -                'otp info conf',
> > -                'otp info scu',
> >                   'hwinfo devid',
> >                   'crypto aes256_cbc_vault',
> > -                'random get',
> > -                'jtag JTAG1 sw_xfer high TMS',
> > -                'adc ADC0 resolution 12',
> > -                'adc ADC0 read 42',
> > -                'adc ADC1 read 69',
> > -                'i2c scan I2C_0',
> > -                'i3c attach I3C_0',
> > +                'jtag jtag@7e6e4100 sw_xfer high TMS',
> > +                'iic scan i2c@7e7b0080',
> >                   'hash test',
> >                   'kernel uptime',
> >                   'kernel reboot warm',