diff options
author | Eric Lai <eric_lai@quanta.corp-partner.google.com> | 2022-12-14 15:48:35 +0800 |
---|---|---|
committer | Martin L Roth <gaumless@gmail.com> | 2022-12-17 20:43:16 +0000 |
commit | 5b89bf4666424bb9b5c72d8fa2b30a124ac1026d (patch) | |
tree | 4170645a6ec2d2078a70e8e6f61df948cbd03f4b /src/lib | |
parent | f49fcc6bf5ced406ce614ff8cc4dd7f172b38d65 (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.c | 15 |
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. */ |