From 8e2e33a044262aa5b74cd2cb330fb4943d14b59f Mon Sep 17 00:00:00 2001 From: Subrata Banik Date: Sat, 23 Sep 2023 18:42:47 +0000 Subject: [PATCH] commonlib: Make CBMEM_ID_CSE_BP_INFO little endian, fix id for string This patch fixes the mistake introduced with 'commit 17cea380d985 ("commonlib: Add CBMEM ID to store CSE Boot Partition Info")' where single CBMEM ID name `CBMEM_ID_CSE_INFO` is associated with two different name description. Additionally, use little endian format for `CBMEM_ID_CSE_INFO` cbmem id. TEST=Build and boot google/rex. Able to fix the issue introduced in commit 17cea380d985 while running cbmem --list and verify that the associated name string is proper. Change-Id: I4235f1f6881ab86ccb252065e922d5d526f7f1f7 Signed-off-by: Subrata Banik Reviewed-on: https://review.coreboot.org/c/coreboot/+/78110 Reviewed-by: Patrick Georgi Reviewed-by: Dinesh Gehlot Reviewed-by: sridhar siricilla Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) Reviewed-by: Krishna P Bhat D --- src/commonlib/bsd/include/commonlib/bsd/cbmem_id.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/commonlib/bsd/include/commonlib/bsd/cbmem_id.h b/src/commonlib/bsd/include/commonlib/bsd/cbmem_id.h index a848702f26..3536c46f7b 100644 --- a/src/commonlib/bsd/include/commonlib/bsd/cbmem_id.h +++ b/src/commonlib/bsd/include/commonlib/bsd/cbmem_id.h @@ -88,7 +88,7 @@ #define CBMEM_ID_AMD_STB 0x5f425453 #define CBMEM_ID_AMD_MP2 0x5f32504d #define CBMEM_ID_CSE_INFO 0x4553435F -#define CBMEM_ID_CSE_BP_INFO 0x43534542 +#define CBMEM_ID_CSE_BP_INFO 0x42455343 #define CBMEM_ID_TO_NAME_TABLE \ { CBMEM_ID_ACPI, "ACPI " }, \ @@ -170,5 +170,5 @@ { CBMEM_ID_AMD_STB, "AMD STB"},\ { CBMEM_ID_AMD_MP2, "AMD MP2 BUFFER"},\ { CBMEM_ID_CSE_INFO, "CSE SPECIFIC INFO"},\ - { CBMEM_ID_CSE_INFO, "CSE BP INFO"} + { CBMEM_ID_CSE_BP_INFO, "CSE BP INFO"} #endif /* _CBMEM_ID_H_ */