aboutsummaryrefslogtreecommitdiff
path: root/src/arch/x86
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2023-02-08 11:39:16 +0100
committerFelix Held <felix-coreboot@felixheld.de>2023-02-09 16:54:11 +0000
commit1e78165cdcccd95306604928a145a515498e794a (patch)
tree60cf3e2f6c4a444017af9d47563b7e36b6826180 /src/arch/x86
parent24f3dc8a17240c6869e247c99c94a96af94be0ef (diff)
arch/x86/include/cpu: introduce CPU_TABLE_END CPU table terminator
Instead of having a magic entry in the CPU device ID table list to tell find_cpu_driver that it has reached the end of the list, introduce and use CPU_TABLE_END. Since the vendor entry in the CPU device ID struct is compared against X86_VENDOR_INVALID which is 0, use X86_VENDOR_INVALID instead of the 0 in the CPU_TABLE_END definition. TEST=Timeless build for Mandolin results in identical image. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Suggested-by: Angel Pons <th3fanbus@gmail.com> Change-Id: I0cae6d65b2265cf5ebf90fe1a9d885d0c489eb92 Reviewed-on: https://review.coreboot.org/c/coreboot/+/72888 Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Diffstat (limited to 'src/arch/x86')
-rw-r--r--src/arch/x86/include/arch/cpu.h2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/arch/x86/include/arch/cpu.h b/src/arch/x86/include/arch/cpu.h
index a0a5840f78..b8c990e631 100644
--- a/src/arch/x86/include/arch/cpu.h
+++ b/src/arch/x86/include/arch/cpu.h
@@ -130,6 +130,8 @@ static inline bool cpuid_match(uint32_t a, uint32_t b, uint32_t mask)
return (a & mask) == (b & mask);
}
+#define CPU_TABLE_END { X86_VENDOR_INVALID, 0, 0 }
+
struct cpu_device_id {
unsigned int vendor;
uint32_t device;