refactor(qemu): pass platform metadata as a function's argument

Based on the prototype modification of the event_log_measure_and_record
function in the previous patch, platform metadata was passed as an
argument.

Change-Id: I9d8316914c046f47cdc6875b16649479e82087aa
Signed-off-by: Manish V Badarkhe <Manish.Badarkhe@arm.com>

diff --git a/plat/qemu/qemu/platform.mk b/plat/qemu/qemu/platform.mk
index 6becc32fad5c1c80d36fc89bfd6784f3f78c6bb5..dfc5de2ce43a4bf0d6ab36c679b238c436a26b90 100644 (file)
--- a/plat/qemu/qemu/platform.mk
+++ b/plat/qemu/qemu/platform.mk
@@ -109,7 +109,6 @@ ifeq (${MEASURED_BOOT},1)
     endif
 
     BL2_SOURCES                +=      plat/qemu/qemu/qemu_measured_boot.c     \
-                               plat/qemu/qemu/qemu_common_measured_boot.c      \
                                plat/qemu/qemu/qemu_helpers.c           \
                                ${EVENT_LOG_SOURCES}
 

diff --git a/plat/qemu/qemu/qemu_common_measured_boot.c b/plat/qemu/qemu/qemu_common_measured_boot.c
deleted file mode 100644 (file)
index 41f7f87..0000000
--- a/plat/qemu/qemu/qemu_common_measured_boot.c
+++ /dev/null
@@ -1,34 +0,0 @@
-/*
- * Copyright (c) 2022, Linaro.
- *
- * SPDX-License-Identifier: BSD-3-Clause
- */
-
-#include <assert.h>
-#include <stdint.h>
-
-#include <common/desc_image_load.h>
-#include <drivers/measured_boot/event_log/event_log.h>
-#include <plat/common/platform.h>
-
-extern event_log_metadata_t qemu_event_log_metadata[];
-
-const event_log_metadata_t *plat_event_log_get_metadata(void)
-{
-       return qemu_event_log_metadata;
-}
-
-int plat_mboot_measure_image(unsigned int image_id, image_info_t *image_data)
-{
-       /* Calculate image hash and record data in Event Log */
-       int err = event_log_measure_and_record(image_data->image_base,
-                                              image_data->image_size,
-                                              image_id);
-       if (err != 0) {
-               ERROR("%s%s image id %u (%i)\n",
-                     "Failed to ", "record", image_id, err);
-               return err;
-       }
-
-       return 0;
-}

diff --git a/plat/qemu/qemu/qemu_measured_boot.c b/plat/qemu/qemu/qemu_measured_boot.c
index d9e475abe9c10166c6d46962f8c7e479cce3b5ce..122bb23b14d42147aac85b9cae7a88a85626e414 100644 (file)
--- a/plat/qemu/qemu/qemu_measured_boot.c
+++ b/plat/qemu/qemu/qemu_measured_boot.c
@@ -9,6 +9,7 @@
 
 #include <drivers/measured_boot/event_log/event_log.h>
 #include <plat/common/common_def.h>
+#include <plat/common/platform.h>
 #include <tools_share/tbbr_oid.h>
 
 #include "../common/qemu_private.h"
@@ -17,8 +18,8 @@
 static uint8_t event_log[PLAT_EVENT_LOG_MAX_SIZE];
 static uint64_t event_log_base;
 
-/* FVP table with platform specific image IDs, names and PCRs */
-const event_log_metadata_t qemu_event_log_metadata[] = {
+/* QEMU table with platform specific image IDs, names and PCRs */
+static const event_log_metadata_t qemu_event_log_metadata[] = {
        { BL31_IMAGE_ID, EVLOG_BL31_STRING, PCR_0 },
        { BL32_IMAGE_ID, EVLOG_BL32_STRING, PCR_0 },
        { BL32_EXTRA1_IMAGE_ID, EVLOG_BL32_EXTRA1_STRING, PCR_0 },
@@ -101,3 +102,19 @@ void bl2_plat_mboot_finish(void)
 
        dump_event_log((uint8_t *)event_log_base, event_log_cur_size);
 }
+
+int plat_mboot_measure_image(unsigned int image_id, image_info_t *image_data)
+{
+       /* Calculate image hash and record data in Event Log */
+       int err = event_log_measure_and_record(image_data->image_base,
+                                              image_data->image_size,
+                                              image_id,
+                                              qemu_event_log_metadata);
+       if (err != 0) {
+               ERROR("%s%s image id %u (%i)\n",
+                     "Failed to ", "record", image_id, err);
+               return err;
+       }
+
+       return 0;
+}