View Change

1 comment:

• File ichspi.c:

  • Patch Set #1, Line 2011: "for 100+ series PCH.\n");

    nit: mention Apollo Lake too?

    Isn't this all done already by the strings in chipset_enable.c ? I think this is the wrong place to be printing this stuff apart from the fact that we are taking the PCH100+ path here.

    snip from cros tree:
    ```
    chipset_enable.c: {0x8086, 0x5af0, B_FS, OK, "Intel", "Apollolake", enable_flash_apl},
    ```

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