From ca5d3e3b2b81c6fd970e7dc7884d633b49648f18 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Fri, 23 Apr 2021 14:34:49 +0200 Subject: drivers/pc80/rtc/option.c: Constrain API to integer values None of the options accessed within coreboot is a string, and there are no guarantees that the code works as intended with them. Given that the current option API only supports integers for now, do not try to access options whose type is 's' (string). Change-Id: Ib67b126d972c6d55b77ea5ecfb862b4e9c766fe5 Signed-off-by: Angel Pons Reviewed-on: https://review.coreboot.org/c/coreboot/+/52637 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber Reviewed-by: Patrick Rudolph --- src/include/option.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/include') diff --git a/src/include/option.h b/src/include/option.h index e7db02e88a..8316bd842c 100644 --- a/src/include/option.h +++ b/src/include/option.h @@ -7,13 +7,13 @@ void sanitize_cmos(void); -enum cb_err cmos_set_option(const char *name, void *val); -enum cb_err cmos_get_option(void *dest, const char *name); +enum cb_err cmos_set_uint_option(const char *name, unsigned int *value); +enum cb_err cmos_get_uint_option(unsigned int *dest, const char *name); static inline enum cb_err set_uint_option(const char *name, unsigned int value) { if (CONFIG(USE_OPTION_TABLE)) - return cmos_set_option(name, &value); + return cmos_set_uint_option(name, &value); return CB_CMOS_OTABLE_DISABLED; } @@ -22,7 +22,7 @@ static inline int get_uint_option(const char *name, const unsigned int fallback) { if (CONFIG(USE_OPTION_TABLE)) { unsigned int value = 0; - if (cmos_get_option(&value, name) == CB_SUCCESS) + if (cmos_get_uint_option(&value, name) == CB_SUCCESS) return value; } return fallback; -- cgit v1.2.3