diff --git a/sw/device/silicon_creator/rom_ext/e2e/boot_svc/BUILD b/sw/device/silicon_creator/rom_ext/e2e/boot_svc/BUILD index c68d755f346ee..6f5c31115c5fb 100644 --- a/sw/device/silicon_creator/rom_ext/e2e/boot_svc/BUILD +++ b/sw/device/silicon_creator/rom_ext/e2e/boot_svc/BUILD @@ -32,7 +32,7 @@ opentitan_test( name = "boot_svc_empty_test", srcs = ["boot_svc_empty_test.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {firmware}@0x10000 {firmware}@0x90000", @@ -57,7 +57,7 @@ opentitan_test( name = "boot_svc_wakeup_test", srcs = ["boot_svc_wakeup_test.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {firmware}@0x10000", @@ -85,7 +85,7 @@ opentitan_test( name = "boot_svc_next_test", srcs = ["boot_svc_next_test.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {firmware}@0x10000 {firmware}@0x90000", @@ -111,7 +111,7 @@ opentitan_test( name = "boot_svc_primary_test", srcs = ["boot_svc_primary_test.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {firmware}@0x10000 {firmware}@0x90000", @@ -145,7 +145,7 @@ opentitan_test( name = "boot_svc_bad_next_test", srcs = ["boot_svc_bad_next_test.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {firmware}@0x10000 {firmware}@0x90000", @@ -173,26 +173,27 @@ opentitan_test( "//sw/device/silicon_creator/rom_ext/e2e/verified_boot:boot_test", ], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( data = [ "//sw/device/silicon_creator/lib/ownership/keys/fake:no_owner_recovery_key", ], exit_failure = "(PASS|FAIL|FAULT).*\n", - # This test requires serial break support which is not available in CI yet. - tags = ["broken"], test_cmd = """ --exec="transport init" --exec="fpga clear-bitstream" --exec="fpga load-bitstream {bitstream}" --exec="bootstrap --clear-uart=true {firmware}" - --exec="console --non-interactive --exit-success='ownership_state: \\x00\\x00\\x00\\x00\r\n' --exit-failure='{exit_failure}'" + --exec="console --non-interactive --exit-success='ownership_state = OWND\r\n' --exit-failure='{exit_failure}'" --exec="rescue boot-svc ownership-unlock \ --mode Any \ --nonce 0 \ --sign $(location //sw/device/silicon_creator/lib/ownership/keys/fake:no_owner_recovery_key)" --exec="console --non-interactive --exit-success='ownership_state = UANY\r\n' --exit-failure='{exit_failure}'" + + # Since we've altered the ownership state, clear the bitstream to not affect later tests. + --exec="fpga clear-bitstream" no-op """, ), @@ -215,7 +216,7 @@ opentitan_binary( testonly = True, srcs = ["boot_svc_min_sec_ver_test.c"], exec_env = [ - "//hw/top_earlgrey:fpga_cw310_rom_ext", + "//hw/top_earlgrey:fpga_hyper310_rom_ext", ], linker_script = "//sw/device/lib/testing/test_framework:ottf_ld_silicon_owner_slot_virtual", manifest = ":manifest_version_4", @@ -237,7 +238,7 @@ opentitan_test( name = "boot_svc_min_sec_ver_test", srcs = ["boot_svc_min_sec_ver_test.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {firmware}@0x10000 {min_sec_ver_4:signed_bin}@0x90000", diff --git a/sw/device/silicon_creator/rom_ext/e2e/handoff/BUILD b/sw/device/silicon_creator/rom_ext/e2e/handoff/BUILD index 62c61017e5750..ad984a00a61d8 100644 --- a/sw/device/silicon_creator/rom_ext/e2e/handoff/BUILD +++ b/sw/device/silicon_creator/rom_ext/e2e/handoff/BUILD @@ -59,7 +59,7 @@ _FAULT_TEST_CASES = { ], defines = test_data["defines"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( exit_success = test_data["exit_success"], diff --git a/sw/device/silicon_creator/rom_ext/e2e/lockdown/BUILD b/sw/device/silicon_creator/rom_ext/e2e/lockdown/BUILD index 383b836e8fca4..723c6e032bbd3 100644 --- a/sw/device/silicon_creator/rom_ext/e2e/lockdown/BUILD +++ b/sw/device/silicon_creator/rom_ext/e2e/lockdown/BUILD @@ -14,7 +14,7 @@ opentitan_test( name = "otp_creator_lockdown", srcs = ["otp_creator_lockdown.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( exit_failure = "PASS|FAIL|BFV:.*\r\n", @@ -36,7 +36,7 @@ opentitan_test( name = "otp_dai_lockdown", srcs = ["otp_dai_lockdown.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( exit_failure = "PASS|FAIL|BFV:.*\r\n", @@ -60,7 +60,7 @@ opentitan_test( name = "sram_lockdown", srcs = ["sram_lockdown.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, deps = [ "//hw/ip/sram_ctrl/data:sram_ctrl_regs", @@ -74,7 +74,7 @@ opentitan_test( name = "epmp_rlb_lockdown", srcs = ["epmp_rlb_lockdown.c"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, deps = [ "//hw/top_earlgrey/sw/autogen:top_earlgrey", diff --git a/sw/device/silicon_creator/rom_ext/e2e/rescue/BUILD b/sw/device/silicon_creator/rom_ext/e2e/rescue/BUILD index 1eca887cbb426..44362216faf5d 100644 --- a/sw/device/silicon_creator/rom_ext/e2e/rescue/BUILD +++ b/sw/device/silicon_creator/rom_ext/e2e/rescue/BUILD @@ -35,7 +35,7 @@ _POSITIONS = { "//sw/device/silicon_creator/rom_ext/e2e/verified_boot:boot_test", ], exec_env = [ - "//hw/top_earlgrey:fpga_cw310_rom_ext", + "//hw/top_earlgrey:fpga_hyper310_rom_ext", ], linker_script = position["linker_script"], deps = [ @@ -52,7 +52,7 @@ _POSITIONS = { opentitan_test( name = "rescue_firmware_{}".format(name), exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "", @@ -60,7 +60,6 @@ _POSITIONS = { ":boot_test_{}".format(name): "payload", }, slot = position["slot"], - tags = ["broken"], test_cmd = """ --exec="transport init" --exec="fpga load-bitstream {bitstream}" @@ -81,7 +80,7 @@ _POSITIONS = { opentitan_test( name = "next_slot", exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {slot_a:signed_bin}@0x10000 {slot_b:signed_bin}@0x90000", @@ -89,8 +88,6 @@ opentitan_test( ":boot_test_slot_a": "slot_a", ":boot_test_slot_b": "slot_b", }, - # This test requires serial break support which is not available in CI yet. - tags = ["broken"], test_cmd = """ --exec="transport init" --exec="fpga load-bitstream {bitstream}" @@ -111,7 +108,7 @@ opentitan_test( opentitan_test( name = "primary_slot", exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{rom_ext}@0 {slot_a:signed_bin}@0x10000 {slot_b:signed_bin}@0x90000", @@ -119,8 +116,6 @@ opentitan_test( ":boot_test_slot_a": "slot_a", ":boot_test_slot_b": "slot_b", }, - # This test requires serial break support which is not available in CI yet. - tags = ["broken"], test_cmd = """ --exec="transport init" --exec="fpga load-bitstream {bitstream}" diff --git a/sw/device/silicon_creator/rom_ext/e2e/verified_boot/BUILD b/sw/device/silicon_creator/rom_ext/e2e/verified_boot/BUILD index 144a098ee7931..1717a5eb4fd94 100644 --- a/sw/device/silicon_creator/rom_ext/e2e/verified_boot/BUILD +++ b/sw/device/silicon_creator/rom_ext/e2e/verified_boot/BUILD @@ -84,7 +84,7 @@ _POSITIONS = { name = "position_{}".format(name), srcs = [":boot_test"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( assemble = "{romext}@{romext_offset} {firmware}@{owner_offset}", @@ -124,7 +124,7 @@ opentitan_test( name = "bad_manifest_test", srcs = [":boot_test"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( exit_failure = "PASS|FAIL|FAULT|BFV:.{8}", @@ -169,7 +169,7 @@ _KEYS = { name = "key_{}".format(name), srcs = [":boot_test"], exec_env = { - "//hw/top_earlgrey:fpga_cw310_rom_ext": None, + "//hw/top_earlgrey:fpga_hyper310_rom_ext": None, }, fpga = fpga_params( exit_failure = keyinfo["exit_failure"],