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 /util/flashrom_tester | |
| 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 'util/flashrom_tester')
0 files changed, 0 insertions, 0 deletions