[Xen-devel,03/13] xen/arm: traps: Switch from bool_t to bool

Message ID 1481114033-11024-4-git-send-email-julien.grall@arm.com
State New
Headers show

Commit Message

Julien Grall Dec. 7, 2016, 12:33 p.m.
Since commit 9202342 "xen/build: Use C99 booleans", bool_t is an alias
to bool. Going forward, there is a preference to use bool rather than
bool_t. Also replace 0 and 1 by true and false when relevant.

Signed-off-by: Julien Grall <julien.grall@arm.com>
---
 xen/arch/arm/traps.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

Comments

Stefano Stabellini Dec. 7, 2016, 9:51 p.m. | #1
On Wed, 7 Dec 2016, Julien Grall wrote:
> Since commit 9202342 "xen/build: Use C99 booleans", bool_t is an alias
> to bool. Going forward, there is a preference to use bool rather than
> bool_t. Also replace 0 and 1 by true and false when relevant.
> 
> Signed-off-by: Julien Grall <julien.grall@arm.com>

Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>


> ---
>  xen/arch/arm/traps.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c
> index ae921d7..fb07ae1 100644
> --- a/xen/arch/arm/traps.c
> +++ b/xen/arch/arm/traps.c
> @@ -154,13 +154,13 @@ static void print_xen_info(void)
>  }
>  
>  #ifdef CONFIG_ARM_32
> -static inline bool_t is_zero_register(int reg)
> +static inline bool is_zero_register(int reg)
>  {
>      /* There is no zero register for ARM32 */
> -    return 0;
> +    return false;
>  }
>  #else
> -static inline bool_t is_zero_register(int reg)
> +static inline bool is_zero_register(int reg)
>  {
>      /*
>       * For store/load and sysreg instruction, the encoding 31 always
> @@ -1500,7 +1500,7 @@ static void do_trap_hypercall(struct cpu_user_regs *regs, register_t *nr,
>  #endif
>  }
>  
> -static bool_t check_multicall_32bit_clean(struct multicall_entry *multi)
> +static bool check_multicall_32bit_clean(struct multicall_entry *multi)
>  {
>      int i;
>  
> @@ -1661,7 +1661,7 @@ static void advance_pc(struct cpu_user_regs *regs, const union hsr hsr)
>  /* Read as zero and write ignore */
>  static void handle_raz_wi(struct cpu_user_regs *regs,
>                            int regidx,
> -                          bool_t read,
> +                          bool read,
>                            const union hsr hsr,
>                            int min_el)
>  {
> @@ -1680,7 +1680,7 @@ static void handle_raz_wi(struct cpu_user_regs *regs,
>  /* Write only as write ignore */
>  static void handle_wo_wi(struct cpu_user_regs *regs,
>                           int regidx,
> -                         bool_t read,
> +                         bool read,
>                           const union hsr hsr,
>                           int min_el)
>  {
> @@ -1699,7 +1699,7 @@ static void handle_wo_wi(struct cpu_user_regs *regs,
>  /* Read only as read as zero */
>  static void handle_ro_raz(struct cpu_user_regs *regs,
>                            int regidx,
> -                          bool_t read,
> +                          bool read,
>                            const union hsr hsr,
>                            int min_el)
>  {
> -- 
> 1.9.1
>

Patch hide | download patch | download mbox

diff --git a/xen/arch/arm/traps.c b/xen/arch/arm/traps.c
index ae921d7..fb07ae1 100644
--- a/xen/arch/arm/traps.c
+++ b/xen/arch/arm/traps.c
@@ -154,13 +154,13 @@  static void print_xen_info(void)
 }
 
 #ifdef CONFIG_ARM_32
-static inline bool_t is_zero_register(int reg)
+static inline bool is_zero_register(int reg)
 {
     /* There is no zero register for ARM32 */
-    return 0;
+    return false;
 }
 #else
-static inline bool_t is_zero_register(int reg)
+static inline bool is_zero_register(int reg)
 {
     /*
      * For store/load and sysreg instruction, the encoding 31 always
@@ -1500,7 +1500,7 @@  static void do_trap_hypercall(struct cpu_user_regs *regs, register_t *nr,
 #endif
 }
 
-static bool_t check_multicall_32bit_clean(struct multicall_entry *multi)
+static bool check_multicall_32bit_clean(struct multicall_entry *multi)
 {
     int i;
 
@@ -1661,7 +1661,7 @@  static void advance_pc(struct cpu_user_regs *regs, const union hsr hsr)
 /* Read as zero and write ignore */
 static void handle_raz_wi(struct cpu_user_regs *regs,
                           int regidx,
-                          bool_t read,
+                          bool read,
                           const union hsr hsr,
                           int min_el)
 {
@@ -1680,7 +1680,7 @@  static void handle_raz_wi(struct cpu_user_regs *regs,
 /* Write only as write ignore */
 static void handle_wo_wi(struct cpu_user_regs *regs,
                          int regidx,
-                         bool_t read,
+                         bool read,
                          const union hsr hsr,
                          int min_el)
 {
@@ -1699,7 +1699,7 @@  static void handle_wo_wi(struct cpu_user_regs *regs,
 /* Read only as read as zero */
 static void handle_ro_raz(struct cpu_user_regs *regs,
                           int regidx,
-                          bool_t read,
+                          bool read,
                           const union hsr hsr,
                           int min_el)
 {