Anastasia Klimchuk submitted this change.

View Change


Approvals: build bot (Jenkins): Verified Felix Singer: Looks good to me, approved Anastasia Klimchuk: Looks good to me, approved
dummyflasher.c: plumb programmer_cfg though get_params

Modify the type signature of the programmer get_param
helper functions to allow for the consumption of the
programmer parameterisation string data.

Change-Id: Ic5b35a7324b1c5a8ca9b8b16ee0b95036737ef47
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/66663
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
---
M dummyflasher.c
1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/dummyflasher.c b/dummyflasher.c
index f27ad46..bd97524 100644
--- a/dummyflasher.c
+++ b/dummyflasher.c
@@ -948,7 +948,8 @@
.erase = dummy_opaque_erase,
};

-static int init_data(struct emu_data *data, enum chipbustype *dummy_buses_supported)
+static int init_data(const struct programmer_cfg *cfg,
+ struct emu_data *data, enum chipbustype *dummy_buses_supported)
{
char *bustext = NULL;
char *tmp = NULL;
@@ -1354,7 +1355,7 @@
msg_pspew("%s\n", __func__);

enum chipbustype dummy_buses_supported;
- if (init_data(data, &dummy_buses_supported)) {
+ if (init_data(cfg, data, &dummy_buses_supported)) {
free(data);
return 1;
}

To view, visit change 66663. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic5b35a7324b1c5a8ca9b8b16ee0b95036737ef47
Gerrit-Change-Number: 66663
Gerrit-PatchSet: 9
Gerrit-Owner: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged