diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2022-08-11 22:02:26 +1000 |
|---|---|---|
| committer | Edward O'Callaghan <quasisec@chromium.org> | 2022-08-25 00:21:22 +0000 |
| commit | ac7f0c20b816784fca6b21e2a931f55b4089e219 (patch) | |
| tree | dbf5f7c9a27c2b8ed525e99e12f6c37faf64cf77 | |
| parent | 75b745544191b7c27db24563f54b64a0b6fdbf31 (diff) | |
| download | flashrom-ac7f0c20b816784fca6b21e2a931f55b4089e219.tar.gz flashrom-ac7f0c20b816784fca6b21e2a931f55b4089e219.tar.bz2 flashrom-ac7f0c20b816784fca6b21e2a931f55b4089e219.zip | |
flashrom.c: flatten out read_buf_from_include_args()
read_buf_from_include_args() does not need the whole flashctx,
rather it works with a fix layout and buffer. Just pass what
state is actually required.
BUG=b:242246291
TEST=builds
Change-Id: Ieadd68895c7ba05cc3d770d304351b02622f14d7
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/66633
Reviewed-by: Evan Benn <evanbenn@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Nikolai Artemiev <nartemiev@google.com>
Reviewed-by: Sam McNally <sammc@google.com>
Reviewed-by: Thomas Heijligen <src@posteo.de>
| -rw-r--r-- | cli_classic.c | 4 | ||||
| -rw-r--r-- | flashrom.c | 6 | ||||
| -rw-r--r-- | include/flash.h | 2 |
3 files changed, 5 insertions, 7 deletions
diff --git a/cli_classic.c b/cli_classic.c index 991804d1..d0712926 100644 --- a/cli_classic.c +++ b/cli_classic.c @@ -411,7 +411,7 @@ static int do_write(struct flashctx *const flash, const char *const filename, co * ... then update newcontents with contents from files provided to '-i' * args if needed. */ - if (read_buf_from_include_args(flash, newcontents)) + if (read_buf_from_include_args(get_layout(flash), newcontents)) goto _free_ret; if (referencefile) { @@ -445,7 +445,7 @@ static int do_verify(struct flashctx *const flash, const char *const filename) * ... then update newcontents with contents from files provided to '-i' * args if needed. */ - if (read_buf_from_include_args(flash, newcontents)) + if (read_buf_from_include_args(get_layout(flash), newcontents)) goto _free_ret; ret = flashrom_image_verify(flash, newcontents, flash_size); @@ -926,14 +926,12 @@ out: * then this will read file data into the corresponding region in the * supplied buffer. * - * @param flashctx Flash context to be used. + * @param layout The layout to be used. * @param buf Chip-sized buffer to write data to * @return 0 on success */ -int read_buf_from_include_args(const struct flashctx *const flash, - unsigned char *buf) +int read_buf_from_include_args(const struct flashrom_layout *const layout, unsigned char *buf) { - const struct flashrom_layout *const layout = get_layout(flash); const struct romentry *entry = NULL; /* diff --git a/include/flash.h b/include/flash.h index 449ed9d1..3fedb2b6 100644 --- a/include/flash.h +++ b/include/flash.h @@ -423,7 +423,7 @@ void print_banner(void); void list_programmers_linebreak(int startcol, int cols, int paren); int selfcheck(void); int read_buf_from_file(unsigned char *buf, unsigned long size, const char *filename); -int read_buf_from_include_args(const struct flashctx *const flash, unsigned char *buf); +int read_buf_from_include_args(const struct flashrom_layout *const layout, unsigned char *buf); int write_buf_to_file(const unsigned char *buf, unsigned long size, const char *filename); int write_buf_to_include_args(const struct flashctx *const flash, unsigned char *buf); int prepare_flash_access(struct flashctx *, bool read_it, bool write_it, bool erase_it, bool verify_it); |