Attention is currently required from: Edward O'Callaghan, Anastasia Klimchuk.

View Change

5 comments:

• Commit Message:

  • Patch Set #3, Line 7: out

    You can remove "out". Just "Mock open() for all chip tests". […]

    Ack

• Patchset:

  • Patch Set #6:

    Squashed into CB:62320

• File tests/chip.c:

  • Patch Set #3, Line 212: data

    Similarly as in previous patch, this can be `erase_chip_io_state` and same for the rest of the patch […]

    Ack

  • Patch Set #3, Line 213: "/run/lock/flashrom_lock"

    If this is exactly identical path for all the chip tests, can be a macro in this file?

    Done

  • Patch Set #3, Line 222: io_mock_register(&chip_io);

    You need to do `io_mock_register(NULL);` at the end of test, like verify tests do.

    Done

To view, visit change 62324. To unsubscribe, or for help writing mail filters, visit settings.