From: Thorsten Blum <thorsten.blum@linux.dev>
Date: Sat, 1 Feb 2025 13:09:54 +0000 (+0100)
Subject: EDAC/amd64: Simplify return statement in dct_ecc_enabled()
X-Git-Tag: v6.15-rc1~198^2^3~3
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=12378e1c3ff8259d6269980dcfd0cbb46735ade7;p=thirdparty%2Flinux.git

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 <thorsten.blum@linux.dev>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Reviewed-by: Qiuxu Zhuo <qiuxu.zhuo@intel.com>
Reviewed-by: Yazen Ghannam <yazen.ghannam@amd.com>
Link: https://lore.kernel.org/r/20250201130953.1377-2-thorsten.blum@linux.dev
---

diff --git a/drivers/edac/amd64_edac.c b/drivers/edac/amd64_edac.c
index d133a5be58905..90f0eb7cc5b9b 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)