From b2b97a5db27befc10b015c8971c35e7ff6993d09 Mon Sep 17 00:00:00 2001 From: Lee Leahy Date: Fri, 10 Mar 2017 08:40:18 -0800 Subject: drivers/intel/fsp2_0: Fix spacing issues Fix the following errors detected by checkpatch.pl: ERROR: space prohibited before that close parenthesis ')' ERROR: space required before the open parenthesis '(' ERROR: space prohibited before open square bracket '[' ERROR: spaces required around that ':' (ctx:VxE) TEST=Build and run on Galileo Gen2 Change-Id: I085aaaa9e276c60eded6edf3be0325ed2402702a Signed-off-by: Lee Leahy Reviewed-on: https://review.coreboot.org/18744 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- src/drivers/intel/fsp2_0/header_display.c | 6 +++--- src/drivers/intel/fsp2_0/hob_display.c | 4 ++-- src/drivers/intel/fsp2_0/memory_init.c | 2 +- src/drivers/intel/fsp2_0/util.c | 2 +- 4 files changed, 7 insertions(+), 7 deletions(-) (limited to 'src/drivers/intel/fsp2_0') diff --git a/src/drivers/intel/fsp2_0/header_display.c b/src/drivers/intel/fsp2_0/header_display.c index 543c075846..f10e2fa136 100644 --- a/src/drivers/intel/fsp2_0/header_display.c +++ b/src/drivers/intel/fsp2_0/header_display.c @@ -28,7 +28,7 @@ void fsp_print_header_info(const struct fsp_header *hdr) revision.val = hdr->fsp_revision; - printk(BIOS_SPEW, "Spec version: v%u.%u\n", (hdr->spec_version >> 4 ), + printk(BIOS_SPEW, "Spec version: v%u.%u\n", (hdr->spec_version >> 4), hdr->spec_version & 0xf); printk(BIOS_SPEW, "Revision: %u.%u.%u, Build Number %u\n", revision.rev.major, @@ -36,8 +36,8 @@ void fsp_print_header_info(const struct fsp_header *hdr) revision.rev.revision, revision.rev.bld_num); printk(BIOS_SPEW, "Type: %s/%s\n", - (hdr->component_attribute & 1 ) ? "release" : "debug", - (hdr->component_attribute & 2 ) ? "test" : "official"); + (hdr->component_attribute & 1) ? "release" : "debug", + (hdr->component_attribute & 2) ? "test" : "official"); printk(BIOS_SPEW, "image ID: %s, base 0x%lx + 0x%zx\n", hdr->image_id, hdr->image_base, hdr->image_size); printk(BIOS_SPEW, "\tConfig region 0x%zx + 0x%zx\n", diff --git a/src/drivers/intel/fsp2_0/hob_display.c b/src/drivers/intel/fsp2_0/hob_display.c index c56075b873..7257aec7c5 100644 --- a/src/drivers/intel/fsp2_0/hob_display.c +++ b/src/drivers/intel/fsp2_0/hob_display.c @@ -17,7 +17,7 @@ struct hob_type_name { const char *name; } __attribute__((packed)); -static const struct hob_type_name hob_type_names [] = { +static const struct hob_type_name hob_type_names[] = { { HOB_TYPE_HANDOFF, "HOB_TYPE_HANDOFF" }, { HOB_TYPE_MEMORY_ALLOCATION, "HOB_TYPE_MEMORY_ALLOCATION" }, { HOB_TYPE_RESOURCE_DESCRIPTOR, "HOB_TYPE_RESOURCE_DESCRIPTOR" }, @@ -190,7 +190,7 @@ void fsp_display_hobs(void) /* Display the HOB header */ printk(BIOS_SPEW, "0x%p, 0x%08x bytes: %s\n", hob, hob->length, fsp_get_hob_type_name(hob)); - switch(hob->type) { + switch (hob->type) { default: soc_display_hob(hob); break; diff --git a/src/drivers/intel/fsp2_0/memory_init.c b/src/drivers/intel/fsp2_0/memory_init.c index e9386d6f8d..46dd2ef8d5 100644 --- a/src/drivers/intel/fsp2_0/memory_init.c +++ b/src/drivers/intel/fsp2_0/memory_init.c @@ -228,7 +228,7 @@ static void fsp_fill_mrc_cache(FSPM_ARCH_UPD *arch_upd, bool s3wake, /* MRC cache found */ arch_upd->NvsBufferPtr = data; arch_upd->BootMode = s3wake ? - FSP_BOOT_ON_S3_RESUME: + FSP_BOOT_ON_S3_RESUME : FSP_BOOT_ASSUMING_NO_CONFIGURATION_CHANGES; printk(BIOS_SPEW, "MRC cache found, size %zx bootmode:%d\n", region_device_sz(&rdev), arch_upd->BootMode); diff --git a/src/drivers/intel/fsp2_0/util.c b/src/drivers/intel/fsp2_0/util.c index 7453b599ae..f84d69cec5 100644 --- a/src/drivers/intel/fsp2_0/util.c +++ b/src/drivers/intel/fsp2_0/util.c @@ -107,7 +107,7 @@ void fsp_handle_reset(uint32_t status) printk(BIOS_SPEW, "FSP: handling reset type %x\n", status); - switch(status) { + switch (status) { case FSP_STATUS_RESET_REQUIRED_COLD: hard_reset(); break; -- cgit v1.2.3