Re: [Xen-devel] [PATCH 07/12] arm/cpu{errata, feature}.[ch]: switch to plain bool

[Julien Grall <julien.grall@xxxxxxx>]

Hi Wei,

On 09/15/2017 10:42 AM, Wei Liu wrote:
> Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> ---
>   xen/arch/arm/cpuerrata.c         | 2 +-
>   xen/include/asm-arm/cpuerrata.h  | 8 ++++----
>   xen/include/asm-arm/cpufeature.h | 6 +++---
>   3 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/xen/arch/arm/cpuerrata.c b/xen/arch/arm/cpuerrata.c
> index f89c0abe7e..fe9e9facbe 100644
> --- a/xen/arch/arm/cpuerrata.c
> +++ b/xen/arch/arm/cpuerrata.c
> @@ -7,7 +7,7 @@
>       .midr_range_min = min,              \
>       .midr_range_max = max
>   
> -static bool_t __maybe_unused
> +static bool __maybe_unused
>   is_affected_midr_range(const struct arm_cpu_capabilities *entry)
>   {
>       return MIDR_IS_CPU_MODEL_RANGE(boot_cpu_data.midr.bits, entry->midr_model,
> diff --git a/xen/include/asm-arm/cpuerrata.h b/xen/include/asm-arm/cpuerrata.h
> index 7b504418da..18a7d495d2 100644
> --- a/xen/include/asm-arm/cpuerrata.h
> +++ b/xen/include/asm-arm/cpuerrata.h
> @@ -9,13 +9,13 @@ void check_local_cpu_errata(void);
>   #ifdef CONFIG_HAS_ALTERNATIVE
>   
>   #define CHECK_WORKAROUND_HELPER(erratum, feature, arch)         \
> -static inline bool_t check_workaround_##erratum(void)           \
> +static inline bool check_workaround_##erratum(void)             \
>   {                                                               \
>       if ( !IS_ENABLED(arch) )                                    \
>           return 0;                                               \
return false;

With that:

Acked-by: Julien Grall <julien.grall@xxxxxxx>

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel