From 0805c7010a5fcfe732a38dfac065b615d577aa4d Mon Sep 17 00:00:00 2001 From: Fabio Aiuto Date: Fri, 19 Aug 2022 17:25:25 +0200 Subject: src/arch/x86/smbios.c: remove unneeded braces fix the following checkpatch errors: WARNING:BRACES: braces {} are not necessary for any arm of this statement 354: FILE: src/arch/x86/smbios.c:354: + if (CONFIG_ROM_SIZE >= 1 * GiB) { [...] + } else { [...] WARNING:BRACES: braces {} are not necessary for single statement blocks 561: FILE: src/arch/x86/smbios.c:561: + if (leaf_b_threads == 0) { + leaf_b_threads = 1; + } Signed-off-by: Fabio Aiuto Change-Id: I14c29e4358cad4cd5ef169ebab7079db2129d8fa Reviewed-on: https://review.coreboot.org/c/coreboot/+/66925 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/arch/x86/smbios.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/arch/x86/smbios.c b/src/arch/x86/smbios.c index fd41ff23f1..4f8de420c3 100644 --- a/src/arch/x86/smbios.c +++ b/src/arch/x86/smbios.c @@ -351,11 +351,10 @@ static int smbios_write_type0(unsigned long *current, int handle) rom_size = MIN(CONFIG_ROM_SIZE, 16 * MiB); t->bios_rom_size = (rom_size / 65535) - 1; - if (CONFIG_ROM_SIZE >= 1 * GiB) { + if (CONFIG_ROM_SIZE >= 1 * GiB) t->extended_bios_rom_size = DIV_ROUND_UP(CONFIG_ROM_SIZE, GiB) | (1 << 14); - } else { + else t->extended_bios_rom_size = DIV_ROUND_UP(CONFIG_ROM_SIZE, MiB); - } t->system_bios_major_release = coreboot_major_revision; t->system_bios_minor_release = coreboot_minor_revision; @@ -558,9 +557,9 @@ static int smbios_write_type4(unsigned long *current, int handle) res = cpuid_ext(0xb, 0); leaf_b_threads = res.ebx; /* if hyperthreading is not available, pretend this is 1 */ - if (leaf_b_threads == 0) { + if (leaf_b_threads == 0) leaf_b_threads = 1; - } + t->core_count2 = leaf_b_cores / leaf_b_threads; t->core_count = t->core_count2 > 0xff ? 0xff : t->core_count2; t->thread_count2 = leaf_b_cores; -- cgit v1.2.3