diff options
author | Arthur Heymans <arthur@aheymans.xyz> | 2022-03-30 13:34:38 +0200 |
---|---|---|
committer | Paul Fagerburg <pfagerburg@chromium.org> | 2022-04-01 13:44:46 +0000 |
commit | 56ed0bee8697a56b08ecfac0b8cb8a9cc312592a (patch) | |
tree | 19303ae40782b2cb6b949d82a514a93e4a41a3e0 | |
parent | 10e47d80cbe5bb2464931a0c6c6d5a7f7077b28e (diff) |
drivers/intel/fsp1_1: Reduce scope of functions
Reduce scope of get_next_hob and drop unused functions.
Change-Id: I81007295ed2d1592c4d829cbb277c0726d89ea4b
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63203
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Elyes Haouas <ehaouas@noos.fr>
-rw-r--r-- | src/drivers/intel/fsp1_1/hob.c | 25 | ||||
-rw-r--r-- | src/drivers/intel/fsp1_1/include/fsp/util.h | 6 |
2 files changed, 1 insertions, 30 deletions
diff --git a/src/drivers/intel/fsp1_1/hob.c b/src/drivers/intel/fsp1_1/hob.c index 076dd5f768..bba32d125c 100644 --- a/src/drivers/intel/fsp1_1/hob.c +++ b/src/drivers/intel/fsp1_1/hob.c @@ -24,7 +24,7 @@ void *get_hob_list(void) } /* Returns the next instance of a HOB type from the starting HOB. */ -void *get_next_hob(uint16_t type, const void *hob_start) +static void *get_next_hob(uint16_t type, const void *hob_start) { EFI_PEI_HOB_POINTERS hob; @@ -44,12 +44,6 @@ void *get_next_hob(uint16_t type, const void *hob_start) return NULL; } -/* Returns the first instance of a HOB type among the whole HOB list. */ -void *get_first_hob(uint16_t type) -{ - return get_next_hob(type, get_hob_list()); -} - /* Returns the next instance of the matched GUID HOB from the starting HOB. */ void *get_next_guid_hob(const EFI_GUID *guid, const void *hob_start) { @@ -66,14 +60,6 @@ void *get_next_guid_hob(const EFI_GUID *guid, const void *hob_start) } /* - * Returns the first instance of the matched GUID HOB among the whole HOB list. - */ -void *get_first_guid_hob(const EFI_GUID *guid) -{ - return get_next_guid_hob(guid, get_hob_list()); -} - -/* * Returns the next instance of the matching resource HOB from the starting HOB. */ void *get_next_resource_hob(const EFI_GUID *guid, const void *hob_start) @@ -90,15 +76,6 @@ void *get_next_resource_hob(const EFI_GUID *guid, const void *hob_start) return hob.Raw; } -/* - * Returns the first instance of the matching resource HOB among the whole HOB - * list. - */ -void *get_first_resource_hob(const EFI_GUID *guid) -{ - return get_next_resource_hob(guid, get_hob_list()); -} - static void print_hob_mem_attributes(void *hob_ptr) { EFI_MEMORY_TYPE hob_mem_type; diff --git a/src/drivers/intel/fsp1_1/include/fsp/util.h b/src/drivers/intel/fsp1_1/include/fsp/util.h index e1fb59605a..0c9a1fad2f 100644 --- a/src/drivers/intel/fsp1_1/include/fsp/util.h +++ b/src/drivers/intel/fsp1_1/include/fsp/util.h @@ -24,10 +24,7 @@ void fsp_early_init(FSP_INFO_HEADER *fsp_info); void fsp_notify(u32 phase); void print_hob_type_structure(u16 hob_type, void *hob_list_ptr); void print_fsp_info(FSP_INFO_HEADER *fsp_header); -void *get_next_type_guid_hob(UINT16 type, const EFI_GUID *guid, - const void *hob_start); void *get_next_resource_hob(const EFI_GUID *guid, const void *hob_start); -void *get_first_resource_hob(const EFI_GUID *guid); void fsp_display_upd_value(const char *name, uint32_t size, uint64_t old, uint64_t new); void report_fsp_output(void); @@ -80,10 +77,7 @@ extern void *FspHobListPtr; #endif void *get_hob_list(void); -void *get_next_hob(uint16_t type, const void *hob_start); -void *get_first_hob(uint16_t type); void *get_next_guid_hob(const EFI_GUID *guid, const void *hob_start); -void *get_first_guid_hob(const EFI_GUID *guid); asmlinkage void chipset_teardown_car_main(void); |