Attention is currently required from: Patrick Georgi, Rizwan Qureshi, Stefan Reinauer, Angel Pons, Sridhar Siricilla, Alex Levin, YH Lin, Nico Huber, Martin Roth - Personal, Caveh Jalali, David Hendricks, Tim Wawrzynczak, Nick Vaccaro, Boris Mittelberg.

View Change

3 comments:

• Commit Message:

  • Patch Set #7, Line 17:

    Without this synchronisation being implemented, flashrom is running
    into below error:
    
    Erasing and writing flash chip... Timeout error between offset
    0x0061c000 and 0x0061c03f (= 0x0061c000 + 63)! FAILED!
    Uh oh. Erase/write failed. Checking if anything has changed.
    

    Please let me know if anything we need to help for moving this CL?

    What was said in the first message of the comment thread.

    I have reattempted to update the commit msg with setup details and replication steps.

    Please suggest if you need better wordings.

  • Patch Set #7, Line 25: TEST=Concurrent flashrom access is not throwing timeout.

    Please let me know if anything we need to help for moving this CL?

    What was said in the first message of the comment thread.

    I'm able to verify this change on eve device. Updated the same in the commit msg.

• File ichspi.c:

  • Patch Set #12, Line 1412: }

    Please move out of the read-modify-write. […]

    Ack

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