On 12/04/15 11:13, Shannon Zhao wrote: > From: Shannon Zhao <[email protected]> > > If there are more than one memory nodes in FDT, currently UEFI will > assert. These two patches firstly let UEFI find the memory node with > lowest base address and check if the address is what we expected and set > PcdSystemMemorySize as the size of this node. Then add other memory > spaces through gDS->AddMemorySpace() when it parses FDT. > > Changes since v2: > * Move the new added function to new driver > * call gDS->SetMemorySpaceAttributes to set the attribute > > Changes since v1: > * Fix commit message and add reviewed-by tag from Laszlo (PATCH 1/2) > * Factor the codes out as a new function, fix coding style (PATCH 2/2) > > Shannon Zhao (2): > ArmVirtPkg: Find the lowest memory node > ArmVirtPkg: Add memory space for the high memory nodes > > ArmVirtPkg/ArmVirtQemu.dsc | 1 + > ArmVirtPkg/ArmVirtQemu.fdf | 1 + > ArmVirtPkg/HighMemDxe/HighMemDxe.c | 105 > +++++++++++++++++++++++++++ > ArmVirtPkg/HighMemDxe/HighMemDxe.inf | 51 +++++++++++++ > ArmVirtPkg/Library/ArmVirtPlatformLib/Virt.c | 30 ++++---- > 5 files changed, 175 insertions(+), 13 deletions(-) > create mode 100644 ArmVirtPkg/HighMemDxe/HighMemDxe.c > create mode 100644 ArmVirtPkg/HighMemDxe/HighMemDxe.inf >
(4) I also intend to fix up the commit subjects as: - ArmVirtPkg: ArmVirtPlatformLib: Find the lowest memory node - ArmVirtPkg: HighMemDxe: Add memory space for the high memory nodes (That is, the module should also be named on the subject line. Sorry for not pointing this out earlier.) Thanks Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
