View Change

2 comments:

• File src/soc/amd/common/block/pi/agesawrapper.c:

  • Patch Set #5, Line 425:

    This section of code is needed, otherwise aip->AllocationMethod will always be ByHost and cause confusion inside AGESA (will assume heap is always ByHost) However, AGESA internals are different depending on ByHost, PreMemHeap and PostMemDram. I believe that it'll not boot without this code.

  • Patch Set #5, Line 430: static union AMD_MAX_ALLOC_PARAMS sp;

    I'm ok with making this static, but only p2, p11 and p12 are needed.

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