diff options
| author | Lijuan Gao <lijuang@codeaurora.org> | 2021-06-09 18:31:03 +0800 |
|---|---|---|
| committer | Lijuan Gao <lijuang@codeaurora.org> | 2021-06-09 18:31:03 +0800 |
| commit | c2ddf115d591a3f4c3fadf096c838d28d04871a8 (patch) | |
| tree | b1a3a4580889f27fe7c7322053ae7e66d1e246fc | |
| parent | 070423b4b6f943f17e7a9c2e43594c663bac5d0c (diff) | |
QcomModulePkg: Return offset if ddr-regions already exist
Optimise for boot up time since it will search for special
node ddr-regions in apps boot loader. If put the node in
the beginning of the device tree can make it search faster,
so return the offset if ddr-regions already exist.
Change-Id: I8cc5c29147a0f0acb42ad4a6222dcff5a6c76f9d
| -rw-r--r-- | QcomModulePkg/Library/BootLib/UpdateDeviceTree.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/QcomModulePkg/Library/BootLib/UpdateDeviceTree.c b/QcomModulePkg/Library/BootLib/UpdateDeviceTree.c index efdfadd1f5..9438c45067 100644 --- a/QcomModulePkg/Library/BootLib/UpdateDeviceTree.c +++ b/QcomModulePkg/Library/BootLib/UpdateDeviceTree.c @@ -706,12 +706,11 @@ STATIC INT32 AddDDrRegionNode (VOID *Fdt) if (Offset < 0) { DEBUG ((EFI_D_ERROR, "Error adding ddr regions: %d\n", Offset)); } - return Offset; } else { - DEBUG ((EFI_D_ERROR, + DEBUG ((EFI_D_VERBOSE, "Attempted to create a ddr-regions node which already exists\n")); - return -1; } + return Offset; } STATIC INT32 AddDDrRegionNodeProp (struct ddr_regions_data_info *DdrRegionsInfo, |