diff options
author | Aarya Chaumal <aarya.chaumal@gmail.com> | 2022-07-04 18:21:50 +0530 |
---|---|---|
committer | Thomas Heijligen <src@posteo.de> | 2022-07-11 12:15:13 +0000 |
commit | edcea80d68e0f029b79bc273ba622dc4a3e6cb2b (patch) | |
tree | e2ca6c352d2b94f209ec995a573c55e205373ac5 /spi.c | |
parent | d0ae8686b1dc642575055c6d0b7e2825f1e9ebbb (diff) | |
download | flashrom-edcea80d68e0f029b79bc273ba622dc4a3e6cb2b.tar.gz flashrom-edcea80d68e0f029b79bc273ba622dc4a3e6cb2b.tar.bz2 flashrom-edcea80d68e0f029b79bc273ba622dc4a3e6cb2b.zip |
spi: Add function to probe erase command opcode for all spi_master
Add a field, probe_opcode, to struct spi_master which points to a
function returning a bool by checking if a given command is supported by
the programmer in use. This is used for getting a whitelist of commands
supported by the programmer, as some programmers like ichspi don't
support all opcodes.
Most programmers use the default function, which just returns true.
ICHSPI and dummyflasher use their specialized function.
Change-Id: I6852ef92788221f471a859c879f8aff42558d36d
Signed-off-by: Aarya Chaumal <aarya.chaumal@gmail.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/65183
Reviewed-by: Thomas Heijligen <src@posteo.de>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'spi.c')
-rw-r--r-- | spi.c | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -134,6 +134,11 @@ int spi_aai_write(struct flashctx *flash, const uint8_t *buf, unsigned int start return flash->mst->spi.write_aai(flash, buf, start, len); } +bool default_spi_probe_opcode(struct flashctx *flash, uint8_t opcode) +{ + return true; +} + int register_spi_master(const struct spi_master *mst, void *data) { struct registered_master rmst = {0}; @@ -146,7 +151,7 @@ int register_spi_master(const struct spi_master *mst, void *data) } if (!mst->write_aai || !mst->write_256 || !mst->read || !mst->command || - !mst->multicommand || + !mst->multicommand || !mst->probe_opcode || ((mst->command == default_spi_send_command) && (mst->multicommand == default_spi_send_multicommand))) { msg_perr("%s called with incomplete master definition. " |