diff options
| author | Edward O'Callaghan <quasisec@google.com> | 2023-01-09 10:43:47 +1100 |
|---|---|---|
| committer | Edward O'Callaghan <quasisec@chromium.org> | 2023-01-11 22:51:35 +0000 |
| commit | a031c81f51eae94877a5076cd1a585d3cbf1bb5c (patch) | |
| tree | a73023d1aa7911313c4099c994f92c50a76e1ca1 /include | |
| parent | 4ae3ac3062230f3c7f3f16a5cdd4682b98bd7d2f (diff) | |
| download | flashrom-a031c81f51eae94877a5076cd1a585d3cbf1bb5c.tar.gz flashrom-a031c81f51eae94877a5076cd1a585d3cbf1bb5c.tar.bz2 flashrom-a031c81f51eae94877a5076cd1a585d3cbf1bb5c.zip | |
parallel: Drop explicit fallback_chip_X boilerplate
A NULL func pointer is necessary and sufficient for the
condition `NULL func pointer => fallback_chip_X' as to not
need this explicit specification.
Therefore drop the explicit need to specify these fallback
callback function pointer in the par_master struct.
This is a reasonable default for every driver in the tree.
Furthermore, move the 'fallback_chip_X()' func from the
generic programmer.c register logic into its relevant
home of parallel.c and make static local to clean up
link-time symbol space.
This simplifies the code and driver development.
Change-Id: If25c0048a07057aa72be6ffa8d8ad7f0a568dcf7
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/71745
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/programmer.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/include/programmer.h b/include/programmer.h index 67e7b4a9..281a4f69 100644 --- a/include/programmer.h +++ b/include/programmer.h @@ -442,12 +442,6 @@ struct par_master { int register_par_master(const struct par_master *mst, const enum chipbustype buses, void *data); /* programmer.c */ -void fallback_chip_writew(const struct flashctx *flash, uint16_t val, chipaddr addr); -void fallback_chip_writel(const struct flashctx *flash, uint32_t val, chipaddr addr); -void fallback_chip_writen(const struct flashctx *flash, const uint8_t *buf, chipaddr addr, size_t len); -uint16_t fallback_chip_readw(const struct flashctx *flash, const chipaddr addr); -uint32_t fallback_chip_readl(const struct flashctx *flash, const chipaddr addr); -void fallback_chip_readn(const struct flashctx *flash, uint8_t *buf, const chipaddr addr, size_t len); struct registered_master { enum chipbustype buses_supported; struct { |