Also, this will be not-for-merge for some time.
Patch set 2:Code-Review -2
3 comments:
File src/drivers/intel/fsp2_0/memory_init.c:
Patch Set #2, Line 402: if (!(CONFIG(FSP_M_XIP) && CONFIG(FSP_USES_CB_STACK))) {
Why is FSP_USES_CB_STACK added at all?
I could write this '!FSP_M_XIP || !FSP_USES_CB_STACK' if that is clearer.
I believe there will be request to split this entire function further, so this if very much WIP now.
Patch Set #2, Line 413: status = load_fspm_mem(&hdr, &file_data, &memmap);
memmap reference here
Patch Set #2, Line 424: do_fsp_memory_init(&hdr, s3wake, &memmap);
Another memmap reference here. But I believe amd/picasso, with DRAM already up, would not want to make this call. That patchtrain actually already has some comments about avoidind fsp_memory_init() call, but they still want FSP-M loader (afaics!).
To view, visit change 35289. To unsubscribe, or for help writing mail filters, visit settings.