Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/81299?usp=email )
Change subject: arch/x86: Directly return result of `IS_POWER_OF_2()` ......................................................................
arch/x86: Directly return result of `IS_POWER_OF_2()`
Change-Id: I314d726deaed30e69121126ba6834e4c7cafd090 Signed-off-by: Paul Menzel pmenzel@molgen.mpg.de Reviewed-on: https://review.coreboot.org/c/coreboot/+/81299 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Jérémy Compostella jeremy.compostella@intel.com --- M src/arch/x86/cpu_common.c 1 file changed, 1 insertion(+), 4 deletions(-)
Approvals: Jérémy Compostella: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/arch/x86/cpu_common.c b/src/arch/x86/cpu_common.c index ee07214..5f7cd5d 100644 --- a/src/arch/x86/cpu_common.c +++ b/src/arch/x86/cpu_common.c @@ -245,8 +245,5 @@
size_t cache_sets = cpu_get_cache_sets(&info);
- if (IS_POWER_OF_2(cache_sets)) - return true; - - return false; + return IS_POWER_OF_2(cache_sets); }