aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorElyes Haouas <ehaouas@noos.fr>2023-08-26 16:56:58 +0200
committerFelix Singer <service+coreboot-gerrit@felixsinger.de>2023-08-28 03:11:35 +0000
commit94ffda9b84df382605648c424f81f83be8881332 (patch)
treee468d71970af9f273b59268a09da590c2e1a5789
parent77d00b27af77ab3d680e5b94458635cb43b9fc01 (diff)
arch/x86/cpu: Remove unnecessary parentheses
Change-Id: I157a3a700ed998b1012c85857c5fad78f60d62dc Signed-off-by: Elyes Haouas <ehaouas@noos.fr> Reviewed-on: https://review.coreboot.org/c/coreboot/+/77525 Reviewed-by: Felix Singer <service+coreboot-gerrit@felixsinger.de> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
-rw-r--r--src/arch/x86/cpu.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/arch/x86/cpu.c b/src/arch/x86/cpu.c
index 60b236c3c1..cb616c96ca 100644
--- a/src/arch/x86/cpu.c
+++ b/src/arch/x86/cpu.c
@@ -124,8 +124,8 @@ static const char *cpu_vendor_name(int vendor)
{
const char *name;
name = "<invalid CPU vendor>";
- if ((vendor < ARRAY_SIZE(x86_vendor_name)) &&
- (x86_vendor_name[vendor] != 0))
+ if (vendor < ARRAY_SIZE(x86_vendor_name) &&
+ x86_vendor_name[vendor] != 0)
name = x86_vendor_name[vendor];
return name;
}
@@ -145,7 +145,7 @@ static void identify_cpu(struct device *cpu)
cpu->device = 0x00000400; /* 486 */
else
cpu->device = 0x00000300; /* 386 */
- if ((cpu->device == 0x00000400) && test_cyrix_52div())
+ if (cpu->device == 0x00000400 && test_cyrix_52div())
memcpy(vendor_name, "CyrixInstead", 13);
/* If we ever care we can enable cpuid here */
/* Detect NexGen with old hypercode */
@@ -195,7 +195,7 @@ struct cpu_driver *find_cpu_driver(struct device *cpu)
const struct cpu_device_id *id;
for (id = driver->id_table;
id->vendor != X86_VENDOR_INVALID; id++) {
- if ((cpu->vendor == id->vendor) &&
+ if (cpu->vendor == id->vendor &&
cpuid_match(cpu->device, id->device, id->device_match_mask))
return driver;
if (id->vendor == X86_VENDOR_ANY)