diff mbox series

[PULL,27/31] target/arm/cpu: Update coding style to make checkpatch.pl happy

Message ID 20200430115142.13430-28-peter.maydell@linaro.org
State Accepted
Commit 51c510aa5876a681cd0059ed3bacaa17590dc2d5
Headers show
Series target-arm queue | expand

Commit Message

Peter Maydell April 30, 2020, 11:51 a.m. UTC
From: Philippe Mathieu-Daudé <philmd@redhat.com>


We will move this code in the next commit. Clean it up
first to avoid checkpatch.pl errors.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Message-id: 20200423073358.27155-5-philmd@redhat.com
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---
 target/arm/cpu.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

-- 
2.20.1

Comments

Philippe Mathieu-Daudé April 30, 2020, 2:52 p.m. UTC | #1
On 4/30/20 1:51 PM, Peter Maydell wrote:
> From: Philippe Mathieu-Daudé <philmd@redhat.com>

> 

> We will move this code in the next commit. Clean it up

> first to avoid checkpatch.pl errors.


Oops this isn't the next commit anymore :S

> 

> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>

> Message-id: 20200423073358.27155-5-philmd@redhat.com

> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

> ---

>   target/arm/cpu.c | 9 ++++++---

>   1 file changed, 6 insertions(+), 3 deletions(-)

> 

> diff --git a/target/arm/cpu.c b/target/arm/cpu.c

> index 30e961f7754..a1e38b38ba1 100644

> --- a/target/arm/cpu.c

> +++ b/target/arm/cpu.c

> @@ -582,7 +582,8 @@ static bool arm_v7m_cpu_exec_interrupt(CPUState *cs, int interrupt_request)

>       CPUARMState *env = &cpu->env;

>       bool ret = false;

>   

> -    /* ARMv7-M interrupt masking works differently than -A or -R.

> +    /*

> +     * ARMv7-M interrupt masking works differently than -A or -R.

>        * There is no FIQ/IRQ distinction. Instead of I and F bits

>        * masking FIQ and IRQ interrupts, an exception is taken only

>        * if it is higher priority than the current execution priority

> @@ -1912,7 +1913,8 @@ static void arm1026_initfn(Object *obj)

>   static void arm1136_r2_initfn(Object *obj)

>   {

>       ARMCPU *cpu = ARM_CPU(obj);

> -    /* What qemu calls "arm1136_r2" is actually the 1136 r0p2, ie an

> +    /*

> +     * What qemu calls "arm1136_r2" is actually the 1136 r0p2, ie an

>        * older core than plain "arm1136". In particular this does not

>        * have the v6K features.

>        * These ID register values are correct for 1136 but may be wrong

> @@ -2698,7 +2700,8 @@ static const ARMCPUInfo arm_cpus[] = {

>       { .name = "arm926",      .initfn = arm926_initfn },

>       { .name = "arm946",      .initfn = arm946_initfn },

>       { .name = "arm1026",     .initfn = arm1026_initfn },

> -    /* What QEMU calls "arm1136-r2" is actually the 1136 r0p2, i.e. an

> +    /*

> +     * What QEMU calls "arm1136-r2" is actually the 1136 r0p2, i.e. an

>        * older core than plain "arm1136". In particular this does not

>        * have the v6K features.

>        */

>
diff mbox series

Patch

diff --git a/target/arm/cpu.c b/target/arm/cpu.c
index 30e961f7754..a1e38b38ba1 100644
--- a/target/arm/cpu.c
+++ b/target/arm/cpu.c
@@ -582,7 +582,8 @@  static bool arm_v7m_cpu_exec_interrupt(CPUState *cs, int interrupt_request)
     CPUARMState *env = &cpu->env;
     bool ret = false;
 
-    /* ARMv7-M interrupt masking works differently than -A or -R.
+    /*
+     * ARMv7-M interrupt masking works differently than -A or -R.
      * There is no FIQ/IRQ distinction. Instead of I and F bits
      * masking FIQ and IRQ interrupts, an exception is taken only
      * if it is higher priority than the current execution priority
@@ -1912,7 +1913,8 @@  static void arm1026_initfn(Object *obj)
 static void arm1136_r2_initfn(Object *obj)
 {
     ARMCPU *cpu = ARM_CPU(obj);
-    /* What qemu calls "arm1136_r2" is actually the 1136 r0p2, ie an
+    /*
+     * What qemu calls "arm1136_r2" is actually the 1136 r0p2, ie an
      * older core than plain "arm1136". In particular this does not
      * have the v6K features.
      * These ID register values are correct for 1136 but may be wrong
@@ -2698,7 +2700,8 @@  static const ARMCPUInfo arm_cpus[] = {
     { .name = "arm926",      .initfn = arm926_initfn },
     { .name = "arm946",      .initfn = arm946_initfn },
     { .name = "arm1026",     .initfn = arm1026_initfn },
-    /* What QEMU calls "arm1136-r2" is actually the 1136 r0p2, i.e. an
+    /*
+     * What QEMU calls "arm1136-r2" is actually the 1136 r0p2, i.e. an
      * older core than plain "arm1136". In particular this does not
      * have the v6K features.
      */