Attention is currently required from: Wonkyu Kim, Subrata Banik, Anastasia Klimchuk.

Patch set 5:Code-Review +1

View Change

3 comments:

• File ich_descriptors.c:

  • Patch Set #5, Line 310: case CHIPSET_METEOR_LAKE:

    Is this correct?

• File ichspi.c:

  • Patch Set #5, Line 2062: ich_gen == CHIPSET_METEOR_LAKE)) {

    Please keep this aligned with the previous conditions. The indentation with 5 spaces is intentional, as it visually separates the condition from the body of the if-block.

• File programmer.h:

  • Patch Set #5, Line 357: CHIPSET_METEOR_LAKE,

    Any particular reason to put this here? I'd expect this to come after `CHIPSET_ELKHART_LAKE` to preserve chronological ordering and PCH/non-PCH grouping.

    Or is this placed after `CHIPSET_600_SERIES_ALDER_POINT`?

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