diff options
author | Anastasia Klimchuk <aklm@chromium.org> | 2022-10-17 11:59:09 +1100 |
---|---|---|
committer | Anastasia Klimchuk <aklm@chromium.org> | 2022-11-08 21:09:42 +0000 |
commit | af2ba601f4840172eb422a0f3efb5a740fb9698d (patch) | |
tree | bafcf8a04415c6833e099d1b932bb52a2dbfe550 /tests/linux_spi.c | |
parent | 0a896424abd490d1cfdd8a582ba17c25df75a0e0 (diff) | |
download | flashrom-af2ba601f4840172eb422a0f3efb5a740fb9698d.tar.gz flashrom-af2ba601f4840172eb422a0f3efb5a740fb9698d.tar.bz2 flashrom-af2ba601f4840172eb422a0f3efb5a740fb9698d.zip |
tests: Add prefix to io_mock functions not to clash with macros
Flashrom I/O mock functions need to be renamed so that they do not
have name clash with standard I/O, because the latter are allowed
to be macros. Adding a prefix to flashrom mock functions avoids
them being accidentally expanded. Standard I/O functions are
expanded and flashrom mocks stay as they are.
BUG=b:237606255
TEST=ninja test
1) gcc 12.2.0 on Debian
2) clang 15.0 on Chromium OS
Ticket: https://ticket.coreboot.org/issues/411
Change-Id: I7998a8fb1b9e65621e12adbfab5460a245d5606b
Signed-off-by: Anastasia Klimchuk <aklm@chromium.org>
Reviewed-on: https://review.coreboot.org/c/flashrom/+/68433
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Diffstat (limited to 'tests/linux_spi.c')
-rw-r--r-- | tests/linux_spi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/linux_spi.c b/tests/linux_spi.c index 2e3dc95f4..10096345f 100644 --- a/tests/linux_spi.c +++ b/tests/linux_spi.c @@ -60,8 +60,8 @@ void linux_spi_probe_lifecycle_test_success(void **state) .flags = { O_RDWR }, }; const struct io_mock linux_spi_io = { - .fgets = linux_spi_fgets, - .ioctl = linux_spi_ioctl, + .iom_fgets = linux_spi_fgets, + .iom_ioctl = linux_spi_ioctl, .fallback_open_state = &linux_spi_fallback_open_state, }; |