diff options
| author | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-10 08:45:30 -0800 |
|---|---|---|
| committer | Lee Leahy <leroy.p.leahy@intel.com> | 2017-03-10 20:08:02 +0100 |
| commit | e686ee8bf7b40d3aed16a4d6d9a42f99ab31ad8e (patch) | |
| tree | bcbb1f8f71e89e8fd700a3ec07d753bb7648a7af /src/drivers/intel/fsp2_0/memory_init.c | |
| parent | b2b97a5db27befc10b015c8971c35e7ff6993d09 (diff) | |
drivers/intel/fsp2_0: Remove braces for single statements
Fix the following warning detected by checkpatch.pl:
WARNING: braces {} are not necessary for single statement blocks
TEST=Build and run on Galileo Gen2
Change-Id: Ibd351703e60acebbacd6ae5b1a2fa1cb34fd3ff9
Signed-off-by: Lee Leahy <leroy.p.leahy@intel.com>
Reviewed-on: https://review.coreboot.org/18745
Tested-by: build bot (Jenkins)
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'src/drivers/intel/fsp2_0/memory_init.c')
| -rw-r--r-- | src/drivers/intel/fsp2_0/memory_init.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/drivers/intel/fsp2_0/memory_init.c b/src/drivers/intel/fsp2_0/memory_init.c index 46dd2ef8d5..559d83e9df 100644 --- a/src/drivers/intel/fsp2_0/memory_init.c +++ b/src/drivers/intel/fsp2_0/memory_init.c @@ -293,9 +293,8 @@ static void do_fsp_memory_init(struct fsp_header *hdr, bool s3wake, upd = (FSPM_UPD *)(hdr->cfg_region_offset + hdr->image_base); - if (upd->FspUpdHeader.Signature != FSPM_UPD_SIGNATURE) { + if (upd->FspUpdHeader.Signature != FSPM_UPD_SIGNATURE) die("Invalid FSPM signature!\n"); - } /* Copy the default values from the UPD area */ memcpy(&fspm_upd, upd, sizeof(fspm_upd)); @@ -417,9 +416,8 @@ void fsp_memory_init(bool s3wake) else status = load_fspm_xip(&hdr, &file_data); - if (status != CB_SUCCESS) { + if (status != CB_SUCCESS) die("Loading FSPM failed!\n"); - } /* Signal that FSP component has been loaded. */ prog_segment_loaded(hdr.image_base, hdr.image_size, SEG_FINAL); |