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 /internal.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 'internal.c')
-rw-r--r-- | internal.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -128,7 +128,7 @@ static int get_params(int *boardenable, int *boardmismatch, *board_vendor = NULL; *board_model = NULL; - arg = extract_programmer_param_str("boardenable"); + arg = extract_programmer_param_str(NULL, "boardenable"); if (arg && !strcmp(arg,"force")) { *boardenable = 1; } else if (arg && !strlen(arg)) { @@ -142,7 +142,7 @@ static int get_params(int *boardenable, int *boardmismatch, } free(arg); - arg = extract_programmer_param_str("boardmismatch"); + arg = extract_programmer_param_str(NULL, "boardmismatch"); if (arg && !strcmp(arg,"force")) { *boardmismatch = 1; } else if (arg && !strlen(arg)) { @@ -156,7 +156,7 @@ static int get_params(int *boardenable, int *boardmismatch, } free(arg); - arg = extract_programmer_param_str("laptop"); + arg = extract_programmer_param_str(NULL, "laptop"); if (arg && !strcmp(arg, "force_I_want_a_brick")) *force_laptop = 1; else if (arg && !strcmp(arg, "this_is_not_a_laptop")) @@ -172,7 +172,7 @@ static int get_params(int *boardenable, int *boardmismatch, } free(arg); - arg = extract_programmer_param_str("mainboard"); + arg = extract_programmer_param_str(NULL, "mainboard"); if (arg && strlen(arg)) { if (board_parse_parameter(arg, board_vendor, board_model)) { free(arg); |