From cacc11de4f8ca1f5c65b9cfa53fe042e05c6e88b Mon Sep 17 00:00:00 2001 From: Maximilian Brune Date: Wed, 7 Jan 2026 22:12:09 +0100 Subject: [PATCH] include/cpu/x86/msr.h: Update return types from int -> bool For all functions which check flags, return a bool type instead of an int type. Signed-off-by: Maximilian Brune Change-Id: I39f0e2f392ec999f7622ed28c5755dd4d0eecf42 Reviewed-on: https://review.coreboot.org/c/coreboot/+/90776 Tested-by: build bot (Jenkins) Reviewed-by: Felix Held --- src/include/cpu/x86/msr.h | 30 +++++++++++++++--------------- 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/src/include/cpu/x86/msr.h b/src/include/cpu/x86/msr.h index 2fdb064608..f4f33b40f6 100644 --- a/src/include/cpu/x86/msr.h +++ b/src/include/cpu/x86/msr.h @@ -142,77 +142,77 @@ static inline void mca_clear_status(void) /* Helpers for interpreting MC[i]_STATUS */ -static inline int mca_valid(msr_t msr) +static inline bool mca_valid(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_VAL); } -static inline int mca_over(msr_t msr) +static inline bool mca_over(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_OVERFLOW); } -static inline int mca_uc(msr_t msr) +static inline bool mca_uc(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_UC); } -static inline int mca_en(msr_t msr) +static inline bool mca_en(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_EN); } -static inline int mca_miscv(msr_t msr) +static inline bool mca_miscv(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_MISCV); } -static inline int mca_addrv(msr_t msr) +static inline bool mca_addrv(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_ADDRV); } -static inline int mca_pcc(msr_t msr) +static inline bool mca_pcc(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_PCC); } -static inline int mca_tcc(msr_t msr) +static inline bool mca_tcc(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_TCC); } -static inline int mca_syndv(msr_t msr) +static inline bool mca_syndv(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_SYNDV); } -static inline int mca_idv(msr_t msr) +static inline bool mca_idv(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_COREID_VAL); } -static inline int mca_cecc(msr_t msr) +static inline bool mca_cecc(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_CECC); } -static inline int mca_uecc(msr_t msr) +static inline bool mca_uecc(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_UECC); } -static inline int mca_defd(msr_t msr) +static inline bool mca_defd(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_DEFERRED); } -static inline int mca_poison(msr_t msr) +static inline bool mca_poison(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_POISON); } -static inline int mca_sublink(msr_t msr) +static inline bool mca_sublink(msr_t msr) { return !!(msr.hi & MCA_STATUS_HI_SUBLINK); }