diff options
author | Anastasia Klimchuk <aklm@chromium.org> | 2021-08-27 15:47:46 +1000 |
---|---|---|
committer | Edward O'Callaghan <quasisec@chromium.org> | 2021-08-30 02:43:20 +0000 |
commit | 099e3785126437abafb4f4784f05ae469deb2d1d (patch) | |
tree | 36049a66ce0b7011da5852f6cf48b9e7c0d167b7 /atapromise.c | |
parent | 31b283bd21ea1f0e9fc99f2ee17dfce64b3938e2 (diff) | |
download | flashrom-099e3785126437abafb4f4784f05ae469deb2d1d.tar.gz flashrom-099e3785126437abafb4f4784f05ae469deb2d1d.tar.bz2 flashrom-099e3785126437abafb4f4784f05ae469deb2d1d.zip |
par_master: Fix propagation of register_par_master() return values
This patch checks return value of register_par_master()
so that in case of an error this error is not ignored anymore.
BUG=b:185191942
TEST=builds and ninja test
Change-Id: I377afae41708c7433a56615e2f096bce9c5349f1
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/57192
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'atapromise.c')
-rw-r--r-- | atapromise.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/atapromise.c b/atapromise.c index 171b22f56..e3009e423 100644 --- a/atapromise.c +++ b/atapromise.c @@ -152,14 +152,13 @@ static int atapromise_init(void) } max_rom_decode.parallel = rom_size; - register_par_master(&par_master_atapromise, BUS_PARALLEL, NULL); msg_pwarn("Do not use this device as a generic programmer. It will leave anything outside\n" "the first %zu kB of the flash chip in an undefined state. It works fine for the\n" "purpose of updating the firmware of this device (padding may necessary).\n", rom_size / 1024); - return 0; + return register_par_master(&par_master_atapromise, BUS_PARALLEL, NULL); } const struct programmer_entry programmer_atapromise = { |