diff options
| author | Anastasia Klimchuk <aklm@chromium.org> | 2021-05-21 09:40:58 +1000 |
|---|---|---|
| committer | Edward O'Callaghan <quasisec@chromium.org> | 2021-05-23 04:18:31 +0000 |
| commit | 6a5db26e686d32165621a6839843639192a33102 (patch) | |
| tree | 2566d9d58639d2115b9c9a2f26922cb6ed478880 /programmer.c | |
| parent | daa86b53012910428861330c8bf82ea3fe183796 (diff) | |
| download | flashrom-6a5db26e686d32165621a6839843639192a33102.tar.gz flashrom-6a5db26e686d32165621a6839843639192a33102.tar.bz2 flashrom-6a5db26e686d32165621a6839843639192a33102.zip | |
programmer: Smoothen register_par_master API
It was impossible to register a const struct par_master that would
point to dynamically allocated `data`. Fix that so that we won't
have to create more mutable globals.
BUG=b:185191942
TEST=builds
Change-Id: I95bc92f6c54c5bcdac1c522ca87054aaffed0f40
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/54169
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Edward O'Callaghan <quasisec@chromium.org>
Diffstat (limited to 'programmer.c')
| -rw-r--r-- | programmer.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/programmer.c b/programmer.c index 42ea2e3b..c276a69b 100644 --- a/programmer.c +++ b/programmer.c @@ -81,7 +81,8 @@ void fallback_chip_readn(const struct flashctx *flash, uint8_t *buf, } int register_par_master(const struct par_master *mst, - const enum chipbustype buses) + const enum chipbustype buses, + void *data) { struct registered_master rmst = {0}; @@ -96,6 +97,8 @@ int register_par_master(const struct par_master *mst, rmst.buses_supported = buses; rmst.par = *mst; + if (data) + rmst.par.data = data; return register_master(&rmst); } |