[PATCH] acpi: replace fwts_firmware_detect() by fw->firmware_type
Alex Hung
alex.hung at canonical.com
Thu Aug 26 17:51:16 UTC 2021
They are used interchangabley in fwts and let's stick with one for
readability and save some CPU cycles.
There are no functional changes.
Signed-off-by: Alex Hung <alex.hung at canonical.com>
---
src/acpi/boot/boot.c | 2 +-
src/lib/src/fwts_efi_module.c | 2 +-
src/opal/power_mgmt_info.c | 2 +-
src/opal/reserv_mem.c | 2 +-
src/uefi/csm/csm.c | 2 +-
src/uefi/esrt/esrt.c | 2 +-
src/uefi/esrtdump/esrtdump.c | 2 +-
src/uefi/uefibootpath/uefibootpath.c | 2 +-
src/uefi/uefidump/uefidump.c | 2 +-
9 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/acpi/boot/boot.c b/src/acpi/boot/boot.c
index cd41452f..ffbd2cd1 100644
--- a/src/acpi/boot/boot.c
+++ b/src/acpi/boot/boot.c
@@ -35,7 +35,7 @@ static int boot_init(fwts_framework *fw)
fwts_log_error(fw, "Cannot read ACPI tables.");
return FWTS_ERROR;
}
- if (fwts_firmware_detect() == FWTS_FIRMWARE_UEFI) {
+ if (fw->firmware_type == FWTS_FIRMWARE_UEFI) {
fwts_log_error(fw, "ACPI BOOT table is deprecated on UEFI firmware, skipping test");
return FWTS_SKIP;
}
diff --git a/src/lib/src/fwts_efi_module.c b/src/lib/src/fwts_efi_module.c
index 3756d614..c1bc5ea1 100644
--- a/src/lib/src/fwts_efi_module.c
+++ b/src/lib/src/fwts_efi_module.c
@@ -214,7 +214,7 @@ int fwts_lib_efi_runtime_kernel_lockdown(fwts_framework *fw)
int fwts_lib_efi_runtime_module_init(fwts_framework *fw, int *fd)
{
- if (fwts_firmware_detect() != FWTS_FIRMWARE_UEFI) {
+ if (fw->firmware_type != FWTS_FIRMWARE_UEFI) {
fwts_log_info(fw, "Cannot detect any UEFI firmware. Aborted.");
return FWTS_ABORTED;
}
diff --git a/src/opal/power_mgmt_info.c b/src/opal/power_mgmt_info.c
index f53525d2..dc0c79f6 100644
--- a/src/opal/power_mgmt_info.c
+++ b/src/opal/power_mgmt_info.c
@@ -84,7 +84,7 @@ static int power_mgmt_init(fwts_framework *fw)
{
int ret;
- if (fwts_firmware_detect() != FWTS_FIRMWARE_OPAL) {
+ if (fw->firmware_type != FWTS_FIRMWARE_OPAL) {
fwts_skipped(fw,
"The firmware type detected was non OPAL "
"so skipping the OPAL Power Management DT checks.");
diff --git a/src/opal/reserv_mem.c b/src/opal/reserv_mem.c
index 2195f2aa..53274951 100644
--- a/src/opal/reserv_mem.c
+++ b/src/opal/reserv_mem.c
@@ -105,7 +105,7 @@ static char *make_message(const char *fmt, ...)
static int reserv_mem_init(fwts_framework *fw)
{
- if (fwts_firmware_detect() != FWTS_FIRMWARE_OPAL) {
+ if (fw->firmware_type != FWTS_FIRMWARE_OPAL) {
fwts_skipped(fw,
"The firmware type detected was non OPAL "
"so skipping the OPAL Reserve Memory DT checks.");
diff --git a/src/uefi/csm/csm.c b/src/uefi/csm/csm.c
index 372b2919..b26e9c3c 100644
--- a/src/uefi/csm/csm.c
+++ b/src/uefi/csm/csm.c
@@ -45,7 +45,7 @@ static int csm_test1(fwts_framework *fw)
fwts_log_info(fw, "Checking for UEFI Compatibility Support Module (CSM)");
- if (fwts_firmware_detect() == FWTS_FIRMWARE_UEFI)
+ if (fw->firmware_type == FWTS_FIRMWARE_UEFI)
flag |= EFI_SUPPORT;
/* Get Int 10h vector from segment/offset realmode address */
diff --git a/src/uefi/esrt/esrt.c b/src/uefi/esrt/esrt.c
index 7193ef8c..8d613bcb 100644
--- a/src/uefi/esrt/esrt.c
+++ b/src/uefi/esrt/esrt.c
@@ -40,7 +40,7 @@
static int esrt_init(fwts_framework *fw)
{
- if (fwts_firmware_detect() != FWTS_FIRMWARE_UEFI) {
+ if (fw->firmware_type != FWTS_FIRMWARE_UEFI) {
fwts_log_info(fw, "Cannot detect any UEFI firmware. Aborted.");
return FWTS_ABORTED;
}
diff --git a/src/uefi/esrtdump/esrtdump.c b/src/uefi/esrtdump/esrtdump.c
index 68e5993b..ff4e40eb 100644
--- a/src/uefi/esrtdump/esrtdump.c
+++ b/src/uefi/esrtdump/esrtdump.c
@@ -32,7 +32,7 @@
static int esrtdump_init(fwts_framework *fw)
{
- if (fwts_firmware_detect() != FWTS_FIRMWARE_UEFI) {
+ if (fw->firmware_type != FWTS_FIRMWARE_UEFI) {
fwts_log_info(fw, "Cannot detect any UEFI firmware. Aborted.");
return FWTS_ABORTED;
}
diff --git a/src/uefi/uefibootpath/uefibootpath.c b/src/uefi/uefibootpath/uefibootpath.c
index 417d36c3..cb942020 100644
--- a/src/uefi/uefibootpath/uefibootpath.c
+++ b/src/uefi/uefibootpath/uefibootpath.c
@@ -35,7 +35,7 @@ static inline bool uefibootpath_check_pnpid(uint32_t id)
static int uefibootpath_init(fwts_framework *fw)
{
- if (fwts_firmware_detect() != FWTS_FIRMWARE_UEFI) {
+ if (fw->firmware_type != FWTS_FIRMWARE_UEFI) {
fwts_log_info(fw, "Cannot detect any UEFI firmware. Aborted.");
return FWTS_ABORTED;
}
diff --git a/src/uefi/uefidump/uefidump.c b/src/uefi/uefidump/uefidump.c
index 6cd4a05b..c584c50f 100644
--- a/src/uefi/uefidump/uefidump.c
+++ b/src/uefi/uefidump/uefidump.c
@@ -1523,7 +1523,7 @@ static void uefidump_var(fwts_framework *fw, fwts_uefi_var *var)
static int uefidump_init(fwts_framework *fw)
{
- if (fwts_firmware_detect() != FWTS_FIRMWARE_UEFI) {
+ if (fw->firmware_type != FWTS_FIRMWARE_UEFI) {
fwts_log_info(fw, "Cannot detect any UEFI firmware. Aborted.");
return FWTS_ABORTED;
}
--
2.33.0
More information about the fwts-devel
mailing list