The if-else block in aesni_init does not follow required coding
conventions. If other conditionals are added to the block, it
becomes very difficult to parse. Use the correct coding style
instead.

Reviewed-by: Ira Weiny <ira.we...@intel.com>
Signed-off-by: Megha Dey <megha....@intel.com>
---
 arch/x86/crypto/aesni-intel_glue.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/arch/x86/crypto/aesni-intel_glue.c 
b/arch/x86/crypto/aesni-intel_glue.c
index 2144e54..e2aae07 100644
--- a/arch/x86/crypto/aesni-intel_glue.c
+++ b/arch/x86/crypto/aesni-intel_glue.c
@@ -1146,8 +1146,7 @@ static int __init aesni_init(void)
                pr_info("AVX2 version of gcm_enc/dec engaged.\n");
                static_branch_enable(&gcm_use_avx);
                static_branch_enable(&gcm_use_avx2);
-       } else
-       if (boot_cpu_has(X86_FEATURE_AVX)) {
+       } else if (boot_cpu_has(X86_FEATURE_AVX)) {
                pr_info("AVX version of gcm_enc/dec engaged.\n");
                static_branch_enable(&gcm_use_avx);
        } else {
-- 
2.7.4

Reply via email to