>>> On 04.03.16 at 07:15, <zhaoshengl...@huawei.com> wrote:
> From: Naresh Bhat <naresh.b...@linaro.org>
> 
> Add ACPI support on arm64 xen hypervisor. Enable EFI support on ARM.
> 
> Cc: Jan Beulich <jbeul...@suse.com>
> Signed-off-by: Naresh Bhat <naresh.b...@linaro.org>
> Signed-off-by: Shannon Zhao <shannon.z...@linaro.org>
> ---
> v5: make ACPI selectable option
> ---
>  xen/arch/arm/Kconfig         |  9 +++++++++
>  xen/common/efi/runtime.c     | 12 +++++++-----

For the change to this file:
Acked-by: Jan Beulich <jbeul...@suse.com>
However ...

> +++ b/xen/arch/arm/Kconfig
> @@ -33,6 +33,15 @@ menu "Architecture Features"
>  
>  source "arch/Kconfig"
>  
> +config ACPI
> +     bool "ACPI (Advanced Configuration and Power Interface) Support"
> +     depends on ARM_64
> +     default y

... I recall Stefano having asked for this, but - are both of you sure?
Have you seen the earlier discussion about the introduction of new
user selectable config options? I.e. shouldn't the prompt at least
become conditional on EXPERT = "y", just like done elsewhere?

Jan


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

Reply via email to