GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

vc/amd/agesa/f14: Fix function return type 

F14GetNbCofVidUpdate() is declared elsewhere to be of type
F_CPU_IS_NBCOF_INIT_NEEDED, which is supposed to return a boolean value
(not an AGESA status). This is fixed in the corresponding f15tn and
f16kb code, so apply the same change here. This fixes a compiler error
when using LTO.

Change-Id: Ifc44e2c0467f8bd1f537b5a69c501ba51053d3d9
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39013
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Jacob Garber 2020-02-18 22:59:23 -07:00 committed by Angel Pons
parent 901cb9ca46
commit 365f52eb1f
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/vendorcode/amd/agesa/f14/Proc/CPU/Family/0x14/cpuF14Utilities.c
View File

@ -384,7 +384,7 @@ F14IsNbPstateEnabled (
*
* @retval AGESA_SUCCESS Always succeeds.
*/
AGESA_STATUS
BOOLEAN
F14GetNbCofVidUpdate (
IN CPU_SPECIFIC_SERVICES *FamilySpecificServices,
IN PCI_ADDR *PciAddress,
@ -393,7 +393,7 @@ F14GetNbCofVidUpdate (
)
{
*NbCofVidUpdateRequired = FALSE;
return (AGESA_SUCCESS);
return FALSE;
}
/*---------------------------------------------------------------------------------------*/

2
src/vendorcode/amd/agesa/f14/Proc/CPU/Family/0x14/cpuF14Utilities.h
View File

@ -99,7 +99,7 @@ F14GetCurrentNbFrequency (
IN AMD_CONFIG_PARAMS *StdHeader
);
AGESA_STATUS
BOOLEAN
F14GetNbCofVidUpdate (
IN CPU_SPECIFIC_SERVICES *FamilySpecificServices,
IN PCI_ADDR *PciAddress,