summaryrefslogtreecommitdiff
path: root/src/lib
diff options
context:
space:
mode:
authorEric Lai <eric_lai@quanta.corp-partner.google.com>2022-12-14 15:48:35 +0800
committerMartin L Roth <gaumless@gmail.com>2022-12-17 20:43:16 +0000
commit5b89bf4666424bb9b5c72d8fa2b30a124ac1026d (patch)
tree4170645a6ec2d2078a70e8e6f61df948cbd03f4b /src/lib
parentf49fcc6bf5ced406ce614ff8cc4dd7f172b38d65 (diff)
lib: Introduce fw_config_get_field
In some cases, fw_config is used for ids like sar_id, sku_id etc. To avoid calling fw_config_probe over and over, hence provide the method to return the value then caller can use the switch case instead of if else statement. TEST=get fw_config field value on nivviks. [INFO ] fw_config get field name=DB_USB, mask=0x3, shift=0, value =0x1 Signed-off-by: Eric Lai <eric_lai@quanta.corp-partner.google.com> Change-Id: Iae89668e8fe7322d5a4dcbf88a97d7ed36619af5 Reviewed-on: https://review.coreboot.org/c/coreboot/+/70745 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Julius Werner <jwerner@chromium.org> Reviewed-by: Tim Wawrzynczak <inforichland@gmail.com>
Diffstat (limited to 'src/lib')
-rw-r--r--src/lib/fw_config.c15
1 files changed, 15 insertions, 0 deletions
diff --git a/src/lib/fw_config.c b/src/lib/fw_config.c
index 72cf225cae..e1fbe74793 100644
--- a/src/lib/fw_config.c
+++ b/src/lib/fw_config.c
@@ -59,6 +59,21 @@ uint64_t fw_config_get(void)
return fw_config_value;
}
+uint64_t fw_config_get_field(const struct fw_config_field *field)
+{
+ /* If fw_config is not provisioned, then there is nothing to get. */
+ if (!fw_config_is_provisioned())
+ return UNDEFINED_FW_CONFIG;
+
+ int shift = __ffs64(field->mask);
+ const uint64_t value = (fw_config_get() & field->mask) >> shift;
+
+ printk(BIOS_INFO, "fw_config get field name=%s, mask=0x%" PRIx64 ", shift=%d, value=0x%"
+ PRIx64 "\n", field->field_name, field->mask, shift, value);
+
+ return value;
+}
+
bool fw_config_probe(const struct fw_config *match)
{
/* If fw_config is not provisioned, then there is nothing to match. */