diff options
author | Edward O'Callaghan <quasisec@google.com> | 2023-03-01 10:50:37 +1100 |
---|---|---|
committer | Thomas Heijligen <src@posteo.de> | 2023-03-03 17:33:06 +0000 |
commit | c66d2bd1ca384dd673b24420e11865fde86060fc (patch) | |
tree | 59904dcc86a95c34a8ddae5880e08c3c47f894ca /include/hwaccess_x86_msr.h | |
parent | 29a3a09f9155e5de69d6d2a4bf582a9e09b8e0a4 (diff) | |
download | flashrom-c66d2bd1ca384dd673b24420e11865fde86060fc.tar.gz flashrom-c66d2bd1ca384dd673b24420e11865fde86060fc.tar.bz2 flashrom-c66d2bd1ca384dd673b24420e11865fde86060fc.zip |
spi: Make default cmd helpers static internal
Avoid these leaking into driver implementations as a NULL
field now implies their implementation. This removes one
source of a driver bug where both `mst->command` AND
`mst->multicommand` are set to default implementations
which is actually a cyclical control flow condition.
The driver however must still have either `mst->command` OR
`mst->multicommand` defined and so both cannot be NULL.
This simplifies the code and driver development.
Change-Id: I4ef95846c2f005cf4aa727f31548c6877d2d4801
Signed-off-by: Edward O'Callaghan <quasisec@google.com>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/73337
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Thomas Heijligen <src@posteo.de>
Reviewed-by: Anastasia Klimchuk <aklm@chromium.org>
Diffstat (limited to 'include/hwaccess_x86_msr.h')
0 files changed, 0 insertions, 0 deletions