- Aug 12, 2019
-
-
Michael D Kinney authored
VS2017 builds for X64 are failing due to the use of the memset() intrinsic. The local variable GopVersion is an array that is initialed in the declaration. Update to use ZeroMem() to initialize GopVersion. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <Zailiang.sun@intel.com> Reviewed-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
The following commit removed the call to SetupInfo() from SystemConfigExtractConfig(). This caused the BIOS, Processor, Platform, and Memory information strings from being filled in with their proper values. https://github.com/tianocore/edk2-platforms/commit/6738c654c8eddccc2a1230a4541bc2cb74c10ea5 The call to SetupInfo() is added back. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <Zailiang.sun@intel.com> Tested-by:
Gary Lin <glin@suse.com>
-
- Aug 11, 2019
-
-
Nate DeSimone authored
The build for KabylakeOpenBoardPkg is presently broken due to WORKSPACE_PLATFORM_BIN being defined to a marco of the same name in KabylakeOpenBoardPkg/KabylakeRvp3/build_config.cfg Setting this macro to an empty string resolves the issue. Cc: Chasel Chiu <chasel.chiu@intel.com> Cc: Michael Kubacki <michael.a.kubacki@intel.com> Signed-off-by:
Nate DeSimone <nathaniel.l.desimone@intel.com> Reviewed-by:
Michael Kubacki <michael.a.kubacki@intel.com>
-
- Aug 09, 2019
-
-
devel@edk2.groups.io authored
LVFS assumes the GUIDs in the metainfo.xml are lowercase, so fwupdmgr may fail to match the devices. Convert the GUIDs to lowercase to be compatible with LVFS. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Signed-off-by:
Gary Lin <glin@suse.com> Reviewed-by:
Michael D Kinney <michael.d.kinney@intel.com>
-
Gary Lin authored
fwupd 1.2.10 checks the bit 3 of BIOS Characteristics Extension Byte 2, i.e. UefiSpecificationSupported, to determine if the system supports UEFI or not. Since the bit was missing, fwupd failed to load uefi plugin. This commit adds 3 bits of BIOS Characteristics Extension Byte 2: TargetContentDistributionEnabled, UefiSpecificationSupported, and VirtualMachineSupported to complete the table. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Signed-off-by:
Gary Lin <glin@suse.com> Reviewed-by:
Michael D Kinney <michael.d.kinney@intel.com>
-
Gary Lin authored
In PlatformBootOption.c: * mContinueBoot and mBootMenuBoot were assigned as "!EnterSetup" before being used. However, EnterSetup is always "FALSE", so it's meaningless to keep those variables. * mPxeBoot is defined but never used. * DetectKeypressCallback() is never used. So we can remove it altogether with mHotKeypressed and HotKeyEvent. * RegisterBootOptionHotkey() only uses EFI_INPUT_KEY from EFI_KEY_DATA, so we don't need to use EFI_KEY_DATA for F2 and F7 in RegisterStaticHotkey(). v2: Update the description for the removal of EFI_KEY_DATA Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Signed-off-by:
Gary Lin <glin@suse.com> Reviewed-by:
Michael D Kinney <michael.d.kinney@intel.com>
-
devel@edk2.groups.io authored
* PlatformBootManagerAfterConsole() always sorted the boot options at the end, and this would invalidate the BootOrder created by the user. Besides, the current implementation would sort the boot options twice, and this is not efficient. We should just sort the boot options once at the end of the function only for the very first boot. * Update the string in IsNeedSortBootOption() to match the description of BootManagerMenuApp. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Signed-off-by:
Gary Lin <glin@suse.com> Reviewed-by:
Michael D Kinney <michael.d.kinney@intel.com>
-
Gary Lin authored
* BootManagerMenuApp wasn't imported so the Boot Device Menu never showed correctly. * Also fix the description of UiApp. In the "Rule.Common.UEFI_APPLICATION.UI" section, it intends to change the description of UiApp to "Enter Setup" to make the boot option self-explaining in the Boot Menu. But this didn't happen since the rule wasn't assigned to UiApp correctly, so the Boot Menu only showed "UiApp" which is not understandable to users. This patch adds "RuleOverride = UI" to UiApp.inf to change the description. v2: Update the description Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Signed-off-by:
Gary Lin <glin@suse.com> Reviewed-by:
Michael D Kinney <michael.d.kinney@intel.com>
-
Gary Lin authored
In BootOptionPriority(), the higher number implies the lower boot priority. This commit lowers the priority of network booting and raises the priority of the local devices. Besides, SD and EMMC are also added to BootOptionPriority() since Minnowboard comes with a SD card slot. v2: Also match MSG_URI_DP to sort HTTP Boot options Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Signed-off-by:
Gary Lin <glin@suse.com> Reviewed-by:
Michael D Kinney <michael.d.kinney@intel.com>
-
Michael Brown authored
Most functions in FdtDxe currently return VOID and report errors using only DEBUG. Update functions to return EFI_STATUS and report errors using Print() so that errors are at least visible in non-debug builds. Signed-off-by:
Michael Brown <mbrown@fensystems.co.uk> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org>
-
- Aug 08, 2019
-
-
Michael Kubacki authored
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2063 Assigns a unique GUID to PLATFORM_GUID. Cc: Chasel Chiu <chasel.chiu@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by:
Michael Kubacki <michael.a.kubacki@intel.com> Reviewed-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Nate DeSimone <nathaniel.l.desimone@intel.com> Reviewed-by:
Liming Gao <liming.gao@intel.com>
-
Michael Kubacki authored
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2064 Assigns a unique GUID to PLATFORM_GUID. Cc: Ankit Sinha <ankit.sinha@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by:
Michael Kubacki <michael.a.kubacki@intel.com> Reviewed-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Nate DeSimone <nathaniel.l.desimone@intel.com> Reviewed-by:
Liming Gao <liming.gao@intel.com>
-
Michael Kubacki authored
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=1983 PeiLib.inf and TestPointCheckLibNull.inf currently have the same GUID. This patch assigns a unique GUID to each library INF file. Cc: Chasel Chiu <chasel.chiu@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by:
Michael Kubacki <michael.a.kubacki@intel.com> Reviewed-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Liming Gao <liming.gao@intel.com>
-
Satoru Okamoto authored
The latest NetsecDxe requires issueing phy reset at the last stage of initialization to safely exit loopback mode. However, as a result, it takes a couple of seconds for link state to get stable, which could cause auto-chosen pxeboot to fail due to MediaPresent check error. This patch adds link state check with 5s timeout in NetsecDxe initialization. The timeout value can be adjustable via configuration file. Signed-off-by:
Masahisa Kojima <masahisa.kojima@linaro.org> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org>
-
Satoru Okamoto authored
NETSEC hardware requires stable RXCLK input upon initialization triggered with DISCORE = 0. However, RXCLK input could be unstable depending on phy chipset and deployed network environment, which could cause NETSEC to hang up during initialization. We solve this platform/environment dependent issue by temporarily putting phy in loopback mode, then we can expect the stable RXCLK input. Signed-off-by:
Masahisa Kojima <masahisa.kojima@linaro.org> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org>
-
Satoru Okamoto authored
This is a refactoring of phy address handling in Netsec driver. NETSEC SDK, low level driver for NetsecDxe, did not store phy address. User should specify the phy address as an argument to the SDK public functions. It prevented NETSEC SDK from internally controlling phy, and it also bothers user application with phy address management. With that, we encapsulate the phy address into NETSEC SDK. Signed-off-by:
Masahisa Kojima <masahisa.kojima@linaro.org> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org>
-
Pete Batard authored
This documents the installation of vanilla Debian 10.0 ARM64 (netinst), which we validated for both Model B and Model B+. Also fix an erroneous reference in an RPi3.fdf comment. Signed-off-by:
Pete Batard <pete@akeo.ie> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org>
-
Marcin Wojtas authored
On another SoC revision, the capability register marks HS400 support as enabled. However in case the interface itself is powered with 3.3V this flag must be unset by the SdMmcOverride protocol callback - otherwise the generic EmmcSwitchToHS400 () would be executed with a failure. Ensure that in case of SlowMode or 3.3V operation, the HS400 capability will be disabled in the SdMmc driver, along with other highest-speed modes. Signed-off-by:
Marcin Wojtas <mw@semihalf.com> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org>
-
- Jul 31, 2019
-
-
Chasel Chiu authored
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1863 PcdFsp*BaseAddress now will be updated in FDF basing on flash map. DSC will only define types of those PCDs and always having 0 as default. Test: internal platform booted with this patch. Cc: Michael Kubacki <michael.a.kubacki@intel.com> Cc: Ankit Sinha <ankit.sinha@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Nate DeSimone <nathaniel.l.desimone@intel.com>
-
Chasel Chiu authored
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1863 PcdFsp*BaseAddress now will be updated in FDF basing on flash map. DSC will only define types of those PCDs and always having 0 as default. Test: internal platform booted with this patch. Cc: Michael Kubacki <michael.a.kubacki@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Nate DeSimone <nathaniel.l.desimone@intel.com>
-
Chasel Chiu authored
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1863 Consume RebaseFspBinBaseAddress.py which will only rebase FSP binary without patching platform DSC. Cc: Michael Kubacki <michael.a.kubacki@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Nate DeSimone <nathaniel.l.desimone@intel.com>
-
Chasel Chiu authored
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1863 Add python script which will rebase FSP binary without patching platform DSC for Fsp*BaseAddress PCDs. Those base address PCD will be updated in FDF basing on flash map. Cc: Michael Kubacki <michael.a.kubacki@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Liming Gao <liming.gao@intel.com> Signed-off-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Nate DeSimone <nathaniel.l.desimone@intel.com>
-
- Jul 30, 2019
-
-
Agyeman, Prince authored
Added vbt and configured gpio to enable N1xxxW display. Cc: Liming Gao <liming.gao@intel.com> Cc: Ankit Sinha <ankit.sinha@intel.com> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com> Cc: Michael Kubacki <michael.a.kubacki@intel.com> Cc: Chasel Chiu <chasel.chiu@intel.com> Signed-off-by:
Prince Agyeman <prince.agyeman@intel.com> Reviewed-by:
Chasel Chiu <chasel.chiu@intel.com> Reviewed-by:
Nate DeSimone <nathaniel.l.desimone@intel.com>
-
- Jul 29, 2019
-
-
Michael D Kinney authored
https://bugzilla.tianocore.org/show_bug.cgi?id=1373 Replace BSD 2-Clause License with BSD+Patent License. This change is based on the following emails: https://lists.01.org/pipermail/edk2-devel/2019-February/036260.html https://lists.01.org/pipermail/edk2-devel/2018-October/030385.html RFCs with detailed process for the license change: V3: https://lists.01.org/pipermail/edk2-devel/2019-March/038116.html V2: https://lists.01.org/pipermail/edk2-devel/2019-March/037669.html V1: https://lists.01.org/pipermail/edk2-devel/2019-March/037500.html Email thread with approval from Marvell: https://edk2.groups.io/g/devel/message/41507 Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Cc: Marcin Wojtas <mw@semihalf.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org> Reviewed-by:
Marcin Wojtas <mw@semihalf.com>
-
Michael D Kinney authored
https://bugzilla.tianocore.org/show_bug.cgi?id=1373 Replace BSD 2-Clause License with BSD+Patent License. This change is based on the following emails: https://lists.01.org/pipermail/edk2-devel/2019-February/036260.html https://lists.01.org/pipermail/edk2-devel/2018-October/030385.html RFCs with detailed process for the license change: V3: https://lists.01.org/pipermail/edk2-devel/2019-March/038116.html V2: https://lists.01.org/pipermail/edk2-devel/2019-March/037669.html V1: https://lists.01.org/pipermail/edk2-devel/2019-March/037500.html Email thread with approval from Marvell: https://edk2.groups.io/g/devel/message/41507 Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Cc: Marcin Wojtas <mw@semihalf.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org> Reviewed-by:
Marcin Wojtas <mw@semihalf.com>
-
Michael D Kinney authored
https://bugzilla.tianocore.org/show_bug.cgi?id=1373 Replace BSD 2-Clause License with BSD+Patent License. This change is based on the following emails: https://lists.01.org/pipermail/edk2-devel/2019-February/036260.html https://lists.01.org/pipermail/edk2-devel/2018-October/030385.html RFCs with detailed process for the license change: V3: https://lists.01.org/pipermail/edk2-devel/2019-March/038116.html V2: https://lists.01.org/pipermail/edk2-devel/2019-March/037669.html V1: https://lists.01.org/pipermail/edk2-devel/2019-March/037500.html Email thread with approval from Marvell: https://edk2.groups.io/g/devel/message/41507 Cc: Leif Lindholm <leif.lindholm@linaro.org> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org> Cc: Marcin Wojtas <mw@semihalf.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org> Reviewed-by:
Marcin Wojtas <mw@semihalf.com>
-
- Jul 25, 2019
-
-
Michael Brown authored
Older device trees tend to use the alias "ethernet". Newer device trees tend to use "ethernet0" since older versions of U-Boot would skip aliases that do not include an index number. See, for example, Linux kernel commit 10b6c0c ("ARM: dts: bcm2835: add index to the ethernet alias") and U-Boot commit f8e57c6 ("fdt_support: Fixup 'ethernet' aliases not ending in digits"). Ensure that both "ethernet" and "ethernet0" aliases are present within the device tree, to provide compatibility with operating systems that expect either alias, and with device trees that contain either (or both) aliases. Signed-off-by:
Michael Brown <mbrown@fensystems.co.uk> Reviewed-by:
Leif Lindholm <leif.lindholm@linaro.org>
-
- Jul 24, 2019
-
-
Liming Gao authored
C tools are compiled first. Their generation files .o/.obj/.d/.pdb should be ignored. Signed-off-by:
Liming Gao <liming.gao@intel.com> Cc: Bob Feng <bob.c.feng@intel.com> Reviewed-by:
Bob Feng <bob.c.feng@intel.com>
-
- Jul 23, 2019
-
-
Michael D Kinney authored
Update to use S3BootScriptWidth enum values instead of EfiBootScriptWidth enum values when calling S3BootScriptLib services. This fixes an XCODE5 build failure. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Reviewed-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Remove #ifdefs for __GNUC__, so the builds are the same for GCC, VS20xx, and XCODE5. This resolves an XCODE5 build break for an unsupported pragma. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Tested-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
* Remove extra parenthesis in if statements * Remove local variable structure initialization that adds use of memcpy() on XCODE X64 builds. Move per-initialize structure to a module global. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Tested-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Tested-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Add missing #if for SENSOR_INFO_VAR_SUPPORT around the function InitializeSensorInfoVariable(). This function is referencing 2 global variables that are not defined, and this breaks VS2017 for undefined global variables. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Tested-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Update calls to S3BootScriptxxx() services to use type S3_BOOT_SCRIPT_LIB_WIDTH instead of EFI_BOOT_SCRIPT_WIDTH. Fixes XCODE5 build failures. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Reviewed-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Remove non-ASCII characters from comments in source files. These are preventing the build tool from generating report files on Linux systems. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Reviewed-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Convert Build_IFWI and bld_vlv BAT/sh scripts to OS independent python script PreBuild.py. This script generates the BiosId file. Standard EDK II build commands are used to build FW images and capsules. * Sample VS2015x86 commands for IA32/X64 and DEBUG/RELEASE build -a IA32 -a X64 -t VS2015x86 -p Vlv2TbltDevicePkg\PlatformPkgX64.dsc build -a IA32 -a X64 -t VS2015x86 -p Vlv2TbltDevicePkg\PlatformPkgX64.dsc build -a IA32 -t VS2015x86 -p Vlv2TbltDevicePkg\PlatformPkgIA32.dsc build -a IA32 -t VS2015x86 -p Vlv2TbltDevicePkg\PlatformPkgIA32.dsc * Sample GCC5 commands for IA32/X64 and DEBUG/RELEASE build -a IA32 -a X64 -n 5 -t GCC5 -p Vlv2TbltDevicePkg/PlatformPkgX64.dsc build -a IA32 -a X64 -n 5 -t GCC5 -p Vlv2TbltDevicePkg/PlatformPkgX64.dsc build -a IA32 -n 5 -t GCC5 -p Vlv2TbltDevicePkg/PlatformPkgIA32.dsc build -a IA32 -n 5 -t GCC5 -p Vlv2TbltDevicePkg/PlatformPkgIA32.dsc Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Reviewed-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Convert all the BAT/sh files used to generate capsules to OS independent Python script. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Reviewed-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
* Consolidate DSC/FDF files to remove Linux/GCC specific ones. Since the FCE tool is no longer required, the FDF files for building under all supported Host OS environments are now the same. * Update the IA32 and X64 build output directories to use a consistent naming convention. * Make small adjustment to FV layout so everything still fits when SOURCE_DEBUG_ENABLE is TRUE. * Remove unused directories that are not referenced by any DSC/FDF files. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Reviewed-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Update this platform to not use the FCE tool to init the platform setup variable with default values from HII forms. Instead, use a Structured PCD with field default values extracted from the HII. The default value for the Structured PCD is stored in the VPD region of the FLASH, so it is always available. The platform setup variable is set to the value from the VPD region if the variable does not exist or is the wrong size. This change also removes the need for Linux/GCC binary files use to initialize the NV store. * Update PlatformSetupoDxe VFR to declare the platform setup structure as an efivarstore instead of varstore. * Update PlatformSetupDxe to return EFI_UNSUPPORTED from the HII_CONFIG_ACCESS_PROTOCOL RouteConfig() and ExtractConfig() services. * Fix a use after free issue in PlatformSetupDxe that could store a random value into the "OsSelection" when the Load Default and Exit setup menu item is used. * Add PlatformSetupDefault.dsc file that contains the default values for all setup questions. * Update DSC files to declare VPD_TOOL_GUID which is required to generate the binary file used to populate the VPD region * Update DSC files to declare Default Stores for both STANDARD and MANUFACTURING mode. Only STANDARD mode is used right now. * Update FDF files to add a VPD region * Update GCC FDF file to initialize the NvStorageVariable regions the same as the non GCC FDF file. * Update DEC file with platform setup Structured PCD called PcdSetup that is associated with the SYSTEM_CONFIGURATION C structure. * Update PlatformEarlyInit module to detect if the platform setup variable exists or is corrupt. If it does not exist or is corrupt, then set PcdSetNvStoreDefaultId to select STANDARD defaults. This initializes a HOB that contains the platform setup defaults and this HOB is used by the Variable PEIM to return the platform setup variable value. * Remove FCE.exe and BfmLib.exe files Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Tested-by:
Gary Lin <glin@suse.com>
-
Michael D Kinney authored
Add boot mode detection back into PlatformPei that was inadvertently removed in the following commit: https://github.com/tianocore/edk2-platforms/commit/d6211390793fbd0a89b14001c43e0ef942c85425 Boot mode detection at this point is required to detect the boot mode of BOOT_ON_FLASH_UPDATE that is required to detect, coalesce, and process UEFI Capsules. Cc: Zailiang Sun <zailiang.sun@intel.com> Cc: Yi Qian <yi.qian@intel.com> Cc: Gary Lin <glin@suse.com> Signed-off-by:
Michael D Kinney <michael.d.kinney@intel.com> Reviewed-by:
Zailiang Sun <zailiang.sun@intel.com> Tested-by:
Gary Lin <glin@suse.com>
-