Patch set 6:Code-Review -1

View Change

1 comment:

• File src/vendorcode/intel/fsp/fsp2_0/cooperlake_sp/FspmUpd.h:

  • Patch Set #1, Line 498:

    #if CONFIG_HAVE_FSPM_DIMM_WATERMARK_OPTION
    /** Offset 0x00C4 - Usage type for Memory Refresh Watermark
      Select Memory Refresh Watermark, 0: Auto (default), 1: High, 2: Low
    **/
      UINT8                       MemRefreshWaterMark;
    #endif
    

    Any reason to put this here and not in one of the UnusedUpdSpace areas where you would get around needing a config guard to protect against incompatibilities between FSPs?

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