Attention is currently required from: Paul Menzel, Ronak Kanabar, Subrata Banik.

View Change

2 comments:

• Commit Message:

  • Patch Set #2, Line 9: - Stack alignment:

    Made this to single CL as per : https://review.coreboot. […]

    Acknowledged

• File src/drivers/intel/fsp2_0/memory_init.c:

  • Patch Set #6, Line 37: static uint8_t temp_ram[CONFIG_FSP_TEMP_RAM_SIZE] __aligned(16);

    u need to maintain the compatibility with 32bit while adding support for 64bit. […]

    https://review.coreboot.org/c/coreboot/+/80277/comment/6deeca3f_9c40f470/
    Please refer above comment. Initially this was segregated then simplified to same alignment for both x32 and x64.

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