Attention is currently required from: Nico Huber, Edward O'Callaghan.

View Change

6 comments:

• File ichspi.c:

  • Patch Set #12, Line 1500: flash->chip->tested = TEST_OK_PREWB;

    What if ich_hwseq_get_flash_id() bailed?

    Done

  • Patch Set #12, Line 135: #define HSFC_CYCLE_RDID HSFC_FCYCLE_MASK(6)

    My dyslexic brain can hardly keep track of all this pre-processor nonsense going on here but this al […]

    Unresolving so I don't forget this.

  • Patch Set #12, Line 1404: if ((chip->manufacture_id == mfg_id) && (chip->model_id == model_id))

    Should check if it's a SPI flash and if the database numbers are for RDID.

    Done

  • Patch Set #12, Line 1466: const int len = sizeof(data);

    Why 4?

    Done

  • Patch Set #12, Line 1474: if (ich_exec_sync_hwseq_xfer(flash, HSFC_CYCLE_RDID, 1, len, ich_generation,

    Should check if RDID cycle is supported (since PCH100).

    Unresolving so I don't forget this.

  • Patch Set #12, Line 1494: return -1;

    Does this break normal read/erase/write operation if there is no matching entry?

    Done

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