diff mbox series

[XEN] x86/cpu: Address a violation of MISRA C:2012 Rule 8.3

Message ID 163937bc59c7a8a5b2f292466090f97b7ec8f623.1691421539.git.federico.serafini@bugseng.com (mailing list archive)
State Superseded
Headers show
Series [XEN] x86/cpu: Address a violation of MISRA C:2012 Rule 8.3 | expand

Commit Message

Federico Serafini Aug. 7, 2023, 3:24 p.m. UTC
Keep consistency between object declarations to address a violation
of MISRA C:2012 Rule 8.3 ("All declarations of an object or function
shall use the same names and type qualifiers").

No functional change.

Signed-off-by: Federico Serafini <federico.serafini@bugseng.com>
---
 xen/arch/x86/cpu/cpu.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jan Beulich Aug. 7, 2023, 3:26 p.m. UTC | #1
On 07.08.2023 17:24, Federico Serafini wrote:
> Keep consistency between object declarations to address a violation
> of MISRA C:2012 Rule 8.3 ("All declarations of an object or function
> shall use the same names and type qualifiers").
> 
> No functional change.
> 
> Signed-off-by: Federico Serafini <federico.serafini@bugseng.com>

I think this wants both a Reported-by: and a Fixes: tag. The
description would likely also better not follow the boilerplate you
use elsewhere, but instead describe what has happened and hence why
the further change is wanted.

Jan
diff mbox series

Patch

diff --git a/xen/arch/x86/cpu/cpu.h b/xen/arch/x86/cpu/cpu.h
index 85a67771f7..e3d06278b3 100644
--- a/xen/arch/x86/cpu/cpu.h
+++ b/xen/arch/x86/cpu/cpu.h
@@ -7,7 +7,7 @@  struct cpu_dev {
 extern const struct cpu_dev intel_cpu_dev, amd_cpu_dev, centaur_cpu_dev,
     shanghai_cpu_dev, hygon_cpu_dev;
 
-extern bool_t opt_arat;
+extern bool opt_arat;
 extern unsigned int opt_cpuid_mask_ecx, opt_cpuid_mask_edx;
 extern unsigned int opt_cpuid_mask_xsave_eax;
 extern unsigned int opt_cpuid_mask_ext_ecx, opt_cpuid_mask_ext_edx;