From 216712ae01993e83265470de1e29744a0970e4fa Mon Sep 17 00:00:00 2001 From: Lee Leahy Date: Fri, 17 Mar 2017 11:23:32 -0700 Subject: drivers/intel/fsp1_1: Fix issues detected by checkpatch MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fix the following error and warnings detected by checkpatch.pl: ERROR: "foo * bar" should be "foo *bar" WARNING: line over 80 characters WARNING: else is not generally useful after a break or return WARNING: braces {} are not necessary for single statement blocks WARNING: suspect code indent for conditional statements (16, 32) WARNING: Comparisons should place the constant on the right side of the test TEST=Build and run on Galileo Gen2 Change-Id: I9f56c0b0e3baf84989411e4a4b98f935725c013f Signed-off-by: Lee Leahy Reviewed-on: https://review.coreboot.org/18886 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin Reviewed-by: Philippe Mathieu-Daudé --- src/drivers/intel/fsp1_1/include/fsp/util.h | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'src/drivers/intel/fsp1_1/include/fsp') diff --git a/src/drivers/intel/fsp1_1/include/fsp/util.h b/src/drivers/intel/fsp1_1/include/fsp/util.h index eea0c33a76..32ac99ee28 100644 --- a/src/drivers/intel/fsp1_1/include/fsp/util.h +++ b/src/drivers/intel/fsp1_1/include/fsp/util.h @@ -92,12 +92,11 @@ int fsp_relocate(struct prog *fsp_relocd, const struct region_device *fsp_src); extern void *FspHobListPtr; #endif -/* TODO: Remove the EFI types and decorations from coreboot implementations. */ -VOID * EFIAPI get_hob_list(VOID); -VOID * EFIAPI get_next_hob(UINT16 type, CONST VOID *hob_start); -VOID * EFIAPI get_first_hob(UINT16 type); -VOID * EFIAPI get_next_guid_hob(CONST EFI_GUID * guid, CONST VOID *hob_start); -VOID * EFIAPI get_first_guid_hob(CONST EFI_GUID * guid); +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); /* * Writes number_of_bytes data bytes from buffer to the console. @@ -106,6 +105,7 @@ VOID * EFIAPI get_first_guid_hob(CONST EFI_GUID * guid); * If number_of_bytes is zero, don't output any data but instead wait until * the console has output all data, then return 0. */ -size_t EFIAPI fsp_write_line(uint8_t *buffer, size_t number_of_bytes); +__attribute__((cdecl)) size_t fsp_write_line(uint8_t *buffer, + size_t number_of_bytes); #endif /* FSP1_1_UTIL_H */ -- cgit v1.2.3