summaryrefslogtreecommitdiffstats
path: root/IntelFsp2WrapperPkg
diff options
context:
space:
mode:
authorLaszlo Ersek <lersek@redhat.com>2019-09-17 17:44:30 +0200
committerLaszlo Ersek <lersek@redhat.com>2019-10-09 09:40:10 +0200
commit04a0c3e8e1384fc054243b61712a4bd6e454e7df (patch)
tree2155f006d0bc4f689744765546d8775e509a645c /IntelFsp2WrapperPkg
parent7f72ec0b15fdc30899452279c988bf5dfb311951 (diff)
downloadedk2-04a0c3e8e1384fc054243b61712a4bd6e454e7df.tar.gz
edk2-04a0c3e8e1384fc054243b61712a4bd6e454e7df.tar.bz2
edk2-04a0c3e8e1384fc054243b61712a4bd6e454e7df.zip
UefiPayloadPkg/BlSupportPei: fix MMCONFIG assignment from XSDT
(This patch is unrelated to the rest of this series; its purpose is to enable building the UefiPayloadPkg DSC files with GCC.) When building "UefiPayloadPkg/UefiPayloadPkgIa32.dsc" with GCC48 for the DEBUG target, the compiler reports that "Entry32" may be used uninitialized in ParseAcpiInfo(), in the XSDT branch. Code inspection proves the compiler right. In the XSDT branch, the code from the RSDT branch must have been duplicated, and "Entry32" references were replaced with "Entry64" -- except where "MmCfgHdr" is assigned. Fix this bug by introducing a helper variable called "Signature", so that we have to refer to "Entry32" or "Entry64" only once per loop body. Cc: Benjamin You <benjamin.you@intel.com> Cc: Guo Dong <guo.dong@intel.com> Cc: Maurice Ma <maurice.ma@intel.com> Signed-off-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Guo Dong <guo.dong@intel.com> Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
Diffstat (limited to 'IntelFsp2WrapperPkg')
0 files changed, 0 insertions, 0 deletions