[Qemu-devel] [PATCH v3] target/mips: remove a space before open parenthesis to fix checkpatch errors

Jules Irenge posted 1 patch 5 years ago
Test docker-clang@ubuntu passed
Test checkpatch passed
Test asan passed
Test docker-mingw@fedora passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190327145308.17838-1-jbi.octave@gmail.com
Maintainers: Aleksandar Markovic <amarkovic@wavecomp.com>, Aurelien Jarno <aurelien@aurel32.net>, Aleksandar Rikalo <arikalo@wavecomp.com>
target/mips/cp0_timer.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
[Qemu-devel] [PATCH v3] target/mips: remove a space before open parenthesis to fix checkpatch errors
Posted by Jules Irenge 5 years ago
Remove a space before open parenthesis to fix errors reported by checkpatch.pl tool
"ERROR: space prohibited between function name and open parenthesis"
within the file "target/mips/cp0_timer.c".

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
---
v3: corrects log grammar errors and clarifies the scope of the patch
 target/mips/cp0_timer.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/target/mips/cp0_timer.c b/target/mips/cp0_timer.c
index f4716395df..742f4b2890 100644
--- a/target/mips/cp0_timer.c
+++ b/target/mips/cp0_timer.c
@@ -29,7 +29,7 @@
 #define TIMER_PERIOD 10 /* 10 ns period for 100 Mhz frequency */
 
 /* XXX: do not use a global */
-uint32_t cpu_mips_get_random (CPUMIPSState *env)
+uint32_t cpu_mips_get_random(CPUMIPSState *env)
 {
     static uint32_t seed = 1;
     static uint32_t prev_idx = 0;
@@ -73,7 +73,7 @@ static void cpu_mips_timer_expire(CPUMIPSState *env)
     qemu_irq_raise(env->irq[(env->CP0_IntCtl >> CP0IntCtl_IPTI) & 0x7]);
 }
 
-uint32_t cpu_mips_get_count (CPUMIPSState *env)
+uint32_t cpu_mips_get_count(CPUMIPSState *env)
 {
     if (env->CP0_Cause & (1 << CP0Ca_DC)) {
         return env->CP0_Count;
@@ -91,7 +91,7 @@ uint32_t cpu_mips_get_count (CPUMIPSState *env)
     }
 }
 
-void cpu_mips_store_count (CPUMIPSState *env, uint32_t count)
+void cpu_mips_store_count(CPUMIPSState *env, uint32_t count)
 {
     /*
      * This gets called from cpu_state_reset(), potentially before timer init.
@@ -109,7 +109,7 @@ void cpu_mips_store_count (CPUMIPSState *env, uint32_t count)
     }
 }
 
-void cpu_mips_store_compare (CPUMIPSState *env, uint32_t value)
+void cpu_mips_store_compare(CPUMIPSState *env, uint32_t value)
 {
     env->CP0_Compare = value;
     if (!(env->CP0_Cause & (1 << CP0Ca_DC)))
@@ -131,7 +131,7 @@ void cpu_mips_stop_count(CPUMIPSState *env)
                                  TIMER_PERIOD);
 }
 
-static void mips_timer_cb (void *opaque)
+static void mips_timer_cb(void *opaque)
 {
     CPUMIPSState *env;
 
@@ -151,7 +151,7 @@ static void mips_timer_cb (void *opaque)
     env->CP0_Count--;
 }
 
-void cpu_mips_clock_init (MIPSCPU *cpu)
+void cpu_mips_clock_init(MIPSCPU *cpu)
 {
     CPUMIPSState *env = &cpu->env;
 
-- 
2.20.1


Re: [Qemu-devel] [PATCH v3] target/mips: remove a space before open parenthesis to fix checkpatch errors
Posted by Aleksandar Markovic 5 years ago
> From: Jules Irenge <jbi.octave@gmail.com>
> Subject: [PATCH v3] target/mips: remove a space before open parenthesis to fix checkpatch errors
> 
> Remove a space before open parenthesis to fix errors reported by checkpatch.pl tool
> "ERROR: space prohibited between function name and open parenthesis"
> within the file "target/mips/cp0_timer.c".
> 
> Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
> ---
> v3: corrects log grammar errors and clarifies the scope of the patch
>  target/mips/cp0_timer.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 

Thanks!

Reviewed-by: Aleksandar Markovic <amarkovic@wavecomp.com>

Jules, you don't need to do anything else regarding this patch,
but, in your future work, please insert complete history in commit
messages of all v3, v4, v5... versions. This means, v3 (as this
patch is) should contain history of what is done in both v3 and v2,
etc.

Yours,
Aleksandar


> diff --git a/target/mips/cp0_timer.c b/target/mips/cp0_timer.c
> index f4716395df..742f4b2890 100644
> --- a/target/mips/cp0_timer.c
> +++ b/target/mips/cp0_timer.c
> @@ -29,7 +29,7 @@
>  #define TIMER_PERIOD 10 /* 10 ns period for 100 Mhz frequency */
> 
>  /* XXX: do not use a global */
> -uint32_t cpu_mips_get_random (CPUMIPSState *env)
> +uint32_t cpu_mips_get_random(CPUMIPSState *env)
>  {
>      static uint32_t seed = 1;
>      static uint32_t prev_idx = 0;
> @@ -73,7 +73,7 @@ static void cpu_mips_timer_expire(CPUMIPSState *env)
>      qemu_irq_raise(env->irq[(env->CP0_IntCtl >> CP0IntCtl_IPTI) & 0x7]);
>  }
> 
> -uint32_t cpu_mips_get_count (CPUMIPSState *env)
> +uint32_t cpu_mips_get_count(CPUMIPSState *env)
>  {
>      if (env->CP0_Cause & (1 << CP0Ca_DC)) {
>          return env->CP0_Count;
> @@ -91,7 +91,7 @@ uint32_t cpu_mips_get_count (CPUMIPSState *env)
>      }
>  }
> 
> -void cpu_mips_store_count (CPUMIPSState *env, uint32_t count)
> +void cpu_mips_store_count(CPUMIPSState *env, uint32_t count)
>  {
>      /*
>       * This gets called from cpu_state_reset(), potentially before timer init.
> @@ -109,7 +109,7 @@ void cpu_mips_store_count (CPUMIPSState *env, uint32_t count)
>      }
>  }
> 
> -void cpu_mips_store_compare (CPUMIPSState *env, uint32_t value)
> +void cpu_mips_store_compare(CPUMIPSState *env, uint32_t value)
>  {
>      env->CP0_Compare = value;
>      if (!(env->CP0_Cause & (1 << CP0Ca_DC)))
> @@ -131,7 +131,7 @@ void cpu_mips_stop_count(CPUMIPSState *env)
>                                   TIMER_PERIOD);
>  }
> 
> -static void mips_timer_cb (void *opaque)
> +static void mips_timer_cb(void *opaque)
>  {
>      CPUMIPSState *env;
> 
> @@ -151,7 +151,7 @@ static void mips_timer_cb (void *opaque)
>      env->CP0_Count--;
>  }
> 
> -void cpu_mips_clock_init (MIPSCPU *cpu)
> +void cpu_mips_clock_init(MIPSCPU *cpu)
>  {
>      CPUMIPSState *env = &cpu->env;
> 
> --
> 2.20.1
> 
>