From 12378e1c3ff8259d6269980dcfd0cbb46735ade7 Mon Sep 17 00:00:00 2001 From: Thorsten Blum Date: Sat, 1 Feb 2025 14:09:54 +0100 Subject: [PATCH] EDAC/amd64: Simplify return statement in dct_ecc_enabled() Simplify the return statement to improve the code's readability. No functional changes. Signed-off-by: Thorsten Blum Signed-off-by: Borislav Petkov (AMD) Reviewed-by: Qiuxu Zhuo Reviewed-by: Yazen Ghannam Link: https://lore.kernel.org/r/20250201130953.1377-2-thorsten.blum@linux.dev --- drivers/edac/amd64_edac.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c index d133a5be5890..90f0eb7cc5b9 100644 --- a/drivers/edac/amd64_edac.c +++ b/drivers/edac/amd64_edac.c @@ -3354,10 +3354,7 @@ static bool dct_ecc_enabled(struct amd64_pvt *pvt) edac_dbg(3, "Node %d: DRAM ECC %s.\n", nid, str_enabled_disabled(ecc_en)); - if (!ecc_en || !nb_mce_en) - return false; - else - return true; + return ecc_en && nb_mce_en; } static bool umc_ecc_enabled(struct amd64_pvt *pvt)