Re: [Xen-devel] [PATCH v3 37/62] arm/acpi: Define a enum for reserved tables

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Tue, 17 Nov 2015, shannon.zhao@xxxxxxxxxx wrote:
> From: Shannon Zhao <shannon.zhao@xxxxxxxxxx>
> 
> It needs to copy and change the contents of some ACPI and EFI tables for
> Dom0. Here define a enum for those tables.
> 
> Signed-off-by: Parth Dixit <parth.dixit@xxxxxxxxxx>
> Signed-off-by: Shannon Zhao <shannon.zhao@xxxxxxxxxx>
> ---
>  xen/include/asm-arm/acpi.h | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/xen/include/asm-arm/acpi.h b/xen/include/asm-arm/acpi.h
> index 214fc4e..f159d51 100644
> --- a/xen/include/asm-arm/acpi.h
> +++ b/xen/include/asm-arm/acpi.h
> @@ -30,6 +30,18 @@
>  
>  extern bool_t acpi_disabled;
>  
> +/* Tables marked as reserved in efi table */
> +typedef enum {
> +    TBL_FADT,
> +    TBL_MADT,
> +    TBL_STAO,
> +    TBL_XSDT,
> +    TBL_RSDP,
> +    TBL_EFIT,
> +    TBL_MMAP,
> +    TBL_MMAX,
> +} EFI_MEM_RES;

Given that in later patches you are using the enum as integer, you
should initialize each value explicitly.


>  #ifdef CONFIG_ACPI
>  bool_t __init acpi_psci_present(void);
>  bool_t __init acpi_psci_hvc_present(void);
> -- 
> 2.1.0
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel
> 

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