AGESA: Move heap_status_name() implementation
Place it within class libagesa to avoid including AGESA internal header heapManager.h in coreboot proper build CPPFLAGS. Change-Id: Iae86d6631d7a6ba6ea2588a53b292b435dfd7861 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/31511 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
		
				
					committed by
					
						 Patrick Georgi
						Patrick Georgi
					
				
			
			
				
	
			
			
			
						parent
						
							35f9507b08
						
					
				
				
					commit
					c6918f99d7
				
			| @@ -21,8 +21,6 @@ | ||||
| #include <AGESA.h> | ||||
| #include <AMD.h> | ||||
|  | ||||
| #include <heapManager.h> | ||||
|  | ||||
| static const char undefined[] = "undefined"; | ||||
|  | ||||
| /* Match order of enum AGESA_STRUCT_NAME. */ | ||||
| @@ -34,11 +32,6 @@ static const char *AgesaFunctionNameStr[] = { | ||||
| 	"Amd2dDataEye", "AmdS3FinalRestore", "AmdInitRtb" | ||||
| }; | ||||
|  | ||||
| /* heapManager.h */ | ||||
| static const char *HeapStatusStr[] = { | ||||
| 	"DoNotExistYet", "LocalCache", "TempMem", "SystemMem", "DoNotExistAnymore","S3Resume" | ||||
| }; | ||||
|  | ||||
| /* This function has to match with enumeration of AGESA_STRUCT_NAME defined | ||||
|  * inside AMD.h header file. Unfortunately those are different across | ||||
|  * different vendorcode subtrees. | ||||
| @@ -64,15 +57,6 @@ const char *agesa_struct_name(int state) | ||||
| 	return AgesaFunctionNameStr[index]; | ||||
| } | ||||
|  | ||||
| const char *heap_status_name(int status) | ||||
| { | ||||
| 	if ((status < HEAP_DO_NOT_EXIST_YET) || (status > HEAP_S3_RESUME)) | ||||
| 		return undefined; | ||||
|  | ||||
| 	int index = status - HEAP_DO_NOT_EXIST_YET; | ||||
| 	return HeapStatusStr[index]; | ||||
| } | ||||
|  | ||||
| /* | ||||
|  * Possible AGESA_STATUS values: | ||||
|  * | ||||
|   | ||||
| @@ -25,6 +25,8 @@ | ||||
| #include <northbridge/amd/agesa/agesa_helper.h> | ||||
| #include <northbridge/amd/agesa/BiosCallOuts.h> | ||||
| #include <amdlib.h> | ||||
|  | ||||
| #include <debug_util.h> | ||||
| #include <AMD.h> | ||||
|  | ||||
| #if CONFIG(CPU_AMD_AGESA_OPENSOURCE) | ||||
|   | ||||
		Reference in New Issue
	
	Block a user