vboot: collect auxfw/EC-related stubs in vboot2 namespace
Collect all auxfw and EC-related stubs in the vboot2
namespace within 2lib/2stub.c.
BUG=b:124141368, chromium:968464
TEST=make clean && make runtests
BRANCH=none
Signed-off-by: Joel Kitching <kitching@google.com>
Change-Id: If09a13df10aa78deee696f4422615b635238f3cf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2635675
Tested-by: Joel Kitching <kitching@chromium.org>
Commit-Queue: Joel Kitching <kitching@chromium.org>
Reviewed-by: Yu-Ping Wu <yupingso@chromium.org>
diff --git a/firmware/2lib/2stub.c b/firmware/2lib/2stub.c
index c5f7bbf..1454f77 100644
--- a/firmware/2lib/2stub.c
+++ b/firmware/2lib/2stub.c
@@ -26,6 +26,13 @@
}
__attribute__((weak))
+void vb2ex_abort(void)
+{
+ /* Stub simply exits. */
+ abort();
+}
+
+__attribute__((weak))
vb2_error_t vb2ex_tpm_clear_owner(struct vb2_context *ctx)
{
fprintf(stderr, "%s: function not implemented\n", __func__);
@@ -48,6 +55,80 @@
return VB2_ERROR_EX_UNIMPLEMENTED;
}
+/*****************************************************************************/
+/* auxfw and EC-related stubs */
+
+__attribute__((weak))
+int vb2ex_ec_trusted(void)
+{
+ return 1;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_running_rw(int *in_rw)
+{
+ *in_rw = 0;
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_jump_to_rw(void)
+{
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_disable_jump(void)
+{
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_hash_image(enum vb2_firmware_selection select,
+ const uint8_t **hash, int *hash_size)
+{
+ static const uint8_t fake_hash[32] = {1, 2, 3, 4};
+
+ *hash = fake_hash;
+ *hash_size = sizeof(fake_hash);
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_get_expected_image_hash(enum vb2_firmware_selection select,
+ const uint8_t **hash, int *hash_size)
+{
+ static const uint8_t fake_hash[32] = {1, 2, 3, 4};
+
+ *hash = fake_hash;
+ *hash_size = sizeof(fake_hash);
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_update_image(enum vb2_firmware_selection select)
+{
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_protect(enum vb2_firmware_selection select)
+{
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_vboot_done(struct vb2_context *ctx)
+{
+ return VB2_SUCCESS;
+}
+
+__attribute__((weak))
+vb2_error_t vb2ex_ec_battery_cutoff(void)
+{
+ return VB2_SUCCESS;
+}
+
__attribute__((weak))
vb2_error_t vb2ex_auxfw_check(enum vb2_auxfw_update_severity *severity)
{
@@ -67,13 +148,6 @@
return VB2_SUCCESS;
}
-__attribute__((weak))
-void vb2ex_abort(void)
-{
- /* Stub simply exits. */
- abort();
-}
-
/*****************************************************************************/
/* UI-related stubs */
diff --git a/firmware/stub/vboot_api_stub.c b/firmware/stub/vboot_api_stub.c
index 521ca4e..a0da9ae 100644
--- a/firmware/stub/vboot_api_stub.c
+++ b/firmware/stub/vboot_api_stub.c
@@ -37,90 +37,6 @@
}
__attribute__((weak))
-int vb2ex_ec_trusted(void)
-{
- return 1;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_running_rw(int *in_rw)
-{
- *in_rw = 0;
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_jump_to_rw(void)
-{
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_disable_jump(void)
-{
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_hash_image(enum vb2_firmware_selection select,
- const uint8_t **hash, int *hash_size)
-{
- static const uint8_t fake_hash[32] = {1, 2, 3, 4};
-
- *hash = fake_hash;
- *hash_size = sizeof(fake_hash);
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_get_expected_image_hash(enum vb2_firmware_selection select,
- const uint8_t **hash, int *hash_size)
-{
- static const uint8_t fake_hash[32] = {1, 2, 3, 4};
-
- *hash = fake_hash;
- *hash_size = sizeof(fake_hash);
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_update_image(enum vb2_firmware_selection select)
-{
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_protect(enum vb2_firmware_selection select)
-{
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_vboot_done(struct vb2_context *ctx)
-{
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_ec_battery_cutoff(void)
-{
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_auxfw_check(enum vb2_auxfw_update_severity *severity)
-{
- *severity = VB2_AUXFW_NO_UPDATE;
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
-vb2_error_t vb2ex_auxfw_update(void)
-{
- return VB2_SUCCESS;
-}
-
-__attribute__((weak))
vb2_error_t VbExLegacy(enum VbAltFwIndex_t altfw_num)
{
return VB2_SUCCESS;