Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
Date:   Thu, 13 Jul 2023 19:36:49 +0800
From:   chenqingyun001@208suo.com
To:     catalin.marinas@arm.com, will@kernel.org
Cc:     linux-kernel@vger.kernel.org
Subject: [PATCH] arm64: errata:#include <linux/cpufeature.h> instead of
 <asm/cpufeature.h>            space required after that ','
In-Reply-To: <tencent_90950F0CFE28194211015CA269F07BE08C06@qq.com>
References: <tencent_90950F0CFE28194211015CA269F07BE08C06@qq.com>
User-Agent: Roundcube Webmail
Message-ID: <ac915cdb7b41c38a4af64fec89f9ddee@208suo.com>
Content-Type: text/plain; charset=US-ASCII;
 format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

The linux/cpufeature.h header file is a more general and
standardized header file, which includes the cpufeature.h
header files under different architectures, while the
asm/cpufeature.h header file is a header file specific
to a certain architecture. There may be some
incompatibilities or outdated issues.
If commas are used to separate multiple parameters
or expressions, there should be a space after the
commas to maintain code readability and style consistency.


Signed-off-by: Qingyun Chen <chenqingyun001@208suo.com>
---
  arch/arm64/kernel/cpu_errata.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/kernel/cpu_errata.c 
b/arch/arm64/kernel/cpu_errata.c
index 307faa2b4395..9b19380629c1 100644
--- a/arch/arm64/kernel/cpu_errata.c
+++ b/arch/arm64/kernel/cpu_errata.c
@@ -10,7 +10,7 @@
  #include <linux/cpu.h>
  #include <asm/cpu.h>
  #include <asm/cputype.h>
-#include <asm/cpufeature.h>
+#include <linux/cpufeature.h>
  #include <asm/kvm_asm.h>
  #include <asm/smp_plat.h>

@@ -726,7 +726,7 @@ const struct arm64_cpu_capabilities arm64_errata[] = 
{
          .capability = ARM64_WORKAROUND_2658417,
          /* Cortex-A510 r0p0 - r1p1 */
          ERRATA_MIDR_RANGE(MIDR_CORTEX_A510, 0, 0, 1, 1),
-        MIDR_FIXED(MIDR_CPU_VAR_REV(1,1), BIT(25)),
+        MIDR_FIXED(MIDR_CPU_VAR_REV(1, 1), BIT(25)),
          .cpu_enable = cpu_clear_bf16_from_user_emulation,
      },
  #endif