From 5c710ea54afabf8c0e5f3c8ada45c4a5f9f0781b Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Fri, 12 Aug 2022 13:47:29 +1000 Subject: tree: Port programmers to pass programmer_cfg to extractors Ran; ``` $ find -name '*.c' -exec sed -i 's/extract_programmer_param_str(NULL/extract_programmer_param_str(cfg/g' '{}' \; ``` Manually fix i2c_helper_linux.c and other cases after. Treat cases of; - pcidev.c , and - usb_device.c as exceptional to be dealt with in later patches. Change-Id: If7b7987e803d35582dda219652a6fc3ed5729b47 Signed-off-by: Edward O'Callaghan Reviewed-on: https://review.coreboot.org/c/flashrom/+/66656 Tested-by: build bot (Jenkins) Reviewed-by: Anastasia Klimchuk Reviewed-by: Felix Singer --- sb600spi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'sb600spi.c') diff --git a/sb600spi.c b/sb600spi.c index b5d10bc3..40a6dbbf 100644 --- a/sb600spi.c +++ b/sb600spi.c @@ -416,7 +416,7 @@ static int handle_speed(const struct programmer_cfg *cfg, int16_t spireadmode_idx = -1; char *param_str; - param_str = extract_programmer_param_str(NULL, "spispeed"); + param_str = extract_programmer_param_str(cfg, "spispeed"); if (param_str != NULL) { unsigned int i; for (i = 0; i < ARRAY_SIZE(spispeeds); i++) { @@ -440,7 +440,7 @@ static int handle_speed(const struct programmer_cfg *cfg, free(param_str); } - param_str = extract_programmer_param_str(NULL, "spireadmode"); + param_str = extract_programmer_param_str(cfg, "spireadmode"); if (param_str != NULL) { unsigned int i; for (i = 0; i < ARRAY_SIZE(spireadmodes); i++) { @@ -529,7 +529,7 @@ static int handle_imc(const struct programmer_cfg *cfg, struct pci_dev *dev, enu return 0; bool amd_imc_force = false; - char *param_value = extract_programmer_param_str(NULL, "amd_imc_force"); + char *param_value = extract_programmer_param_str(cfg, "amd_imc_force"); if (param_value && !strcmp(param_value, "yes")) { amd_imc_force = true; msg_pspew("amd_imc_force enabled.\n"); -- cgit v1.2.3