Attention is currently required from: Edward O'Callaghan, Angel Pons, Anastasia Klimchuk. Hello build bot (Jenkins), Nico Huber, Edward O'Callaghan, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/flashrom/+/57437
to look at the new patch set (#4).
Change subject: tests: Revise mock chip definition and usage ......................................................................
tests: Revise mock chip definition and usage
This patch is doing few things:
1) Makes chip definitions static global so that they can be reused between test functions.
2) Promotes existing mock chip from 8KiB to 8MiB and eraseblocks are expanded accordingly. Old value of 8KiB was very small and it was confusing. Mock chip looks more realistic now.
3) Uses KiB and MiB macros from flash.h for mock chip definition
4) Renames CHIP_TOTAL_SIZE to MOCK_CHIP_SIZE to avoid confusion (there is also a W25Q128.V chip in the tests)
5) Makes chip definitions const so that every test can work on a fresh copy on the stack.
BUG=b:181803212 TEST=builds and ninja test
Change-Id: Ia9b5fc71e30610684e68e9aca9fb1970da8f840a Signed-off-by: Anastasia Klimchuk aklm@chromium.org --- M tests/chip.c 1 file changed, 60 insertions(+), 117 deletions(-)
git pull ssh://review.coreboot.org:29418/flashrom refs/changes/37/57437/4