diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2022-08-12 13:28:46 +1000 |
|---|---|---|
| committer | Anastasia Klimchuk <aklm@chromium.org> | 2022-09-07 01:49:49 +0000 |
| commit | e316f1970d9c51ab167be0835a4cfbadf92e9653 (patch) | |
| tree | 3d021eb4f804b05ec57aad9c5e3cd46f1b7003b8 /realtek_mst_i2c_spi.c | |
| parent | 33af2e695d28a89e0eaf9b058310bc5bf0e6e80f (diff) | |
| download | flashrom-e316f1970d9c51ab167be0835a4cfbadf92e9653.tar.gz flashrom-e316f1970d9c51ab167be0835a4cfbadf92e9653.tar.bz2 flashrom-e316f1970d9c51ab167be0835a4cfbadf92e9653.zip | |
tree: Change signature of extract_programmer_param_str()
Results can be reproduced with the following invocation;
```
$ find -name '*.c' -exec sed -i 's/extract_programmer_param_str(/extract_programmer_param_str(NULL, /g' '{}' \;
```
This allows for a pointer to the actual programmer parameters
to be passed instead of a global.
Change-Id: I781a328fa280e0a9601050dd99a75af72c39c899
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/66654
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Diffstat (limited to 'realtek_mst_i2c_spi.c')
| -rw-r--r-- | realtek_mst_i2c_spi.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/realtek_mst_i2c_spi.c b/realtek_mst_i2c_spi.c index 8d4f86d63..6c2b86f11 100644 --- a/realtek_mst_i2c_spi.c +++ b/realtek_mst_i2c_spi.c @@ -450,7 +450,7 @@ static int get_params(bool *reset, bool *enter_isp, bool *allow_brick) int ret = 0; *allow_brick = false; /* Default behaviour is to bail. */ - param_str = extract_programmer_param_str("allow_brick"); + param_str = extract_programmer_param_str(NULL, "allow_brick"); if (param_str) { if (!strcmp(param_str, "yes")) { *allow_brick = true; @@ -462,7 +462,7 @@ static int get_params(bool *reset, bool *enter_isp, bool *allow_brick) free(param_str); *reset = false; /* Default behaviour is no MCU reset on tear-down. */ - param_str = extract_programmer_param_str("reset_mcu"); + param_str = extract_programmer_param_str(NULL, "reset_mcu"); if (param_str) { if (param_str[0] == '1') { *reset = true; @@ -476,7 +476,7 @@ static int get_params(bool *reset, bool *enter_isp, bool *allow_brick) free(param_str); *enter_isp = true; /* Default behaviour is enter ISP on setup. */ - param_str = extract_programmer_param_str("enter_isp"); + param_str = extract_programmer_param_str(NULL, "enter_isp"); if (param_str) { if (param_str[0] == '1') { *enter_isp = true; |