[Xen-devel,RFC,08/11] Add ACPI_IORT config

Message ID 20180102092809.1841-9-manish.jaggi@linaro.org
State New
Headers show
Series
  • acpi: arm: IORT Support for Xen
Related show

Commit Message

Manish Jaggi Jan. 2, 2018, 9:28 a.m.
From: Manish Jaggi <manish.jaggi@linaro.org>

Add ACPI_IORT config

Singed-off-by: Manish Jaggi <manish.jaggi@linaro.org>
---
 xen/arch/arm/Kconfig     | 5 +++++
 xen/drivers/acpi/Kconfig | 3 +++
 2 files changed, 8 insertions(+)

Comments

Julien Grall Jan. 18, 2018, 7:01 p.m. | #1
Hi  Manish,

The usual scripts/get_maintainers.pl.

On 02/01/18 09:28, manish.jaggi@linaro.org wrote:
> From: Manish Jaggi <manish.jaggi@linaro.org>
> 
> Add ACPI_IORT config
> 
> Singed-off-by: Manish Jaggi <manish.jaggi@linaro.org>
> ---
>   xen/arch/arm/Kconfig     | 5 +++++
>   xen/drivers/acpi/Kconfig | 3 +++
>   2 files changed, 8 insertions(+)
> 
> diff --git a/xen/arch/arm/Kconfig b/xen/arch/arm/Kconfig
> index f58019d6ed..d4767d6ea3 100644
> --- a/xen/arch/arm/Kconfig
> +++ b/xen/arch/arm/Kconfig
> @@ -42,6 +42,11 @@ config ACPI
>   	  Advanced Configuration and Power Interface (ACPI) support for Xen is
>   	  an alternative to device tree on ARM64.
>   
> +config ACPI_IORT
> +	bool
> +	prompt "ACPI IORT Support" if EXPERT = "y"

No need for EXPERT here. It will get picked up by ACPI.

Also, I don't think it is useful to let the user disabling IORT. You 
either need all ACPI or not. It will get into trouble if IORT is not 
handled in Xen.

Note that I am happy to see the config ACPI_IORT here. But not exposed 
to user's choice.

> +	depends on ACPI
> +
>   config HAS_GICV3
>   	bool
>   
> diff --git a/xen/drivers/acpi/Kconfig b/xen/drivers/acpi/Kconfig
> index b64d3731fb..15ae98140c 100644
> --- a/xen/drivers/acpi/Kconfig
> +++ b/xen/drivers/acpi/Kconfig
> @@ -5,5 +5,8 @@ config ACPI
>   config ACPI_LEGACY_TABLES_LOOKUP
>   	bool
>   
> +config ACPI_IORT
> +	bool

I am not sure to understand why you define ACPI_IORT again here. It is 
already done above?

However, I don't think it is necessary to have a separate patch just for 
adding the Kconfig. You can fold into the patch that is first using it. 
BTW, I would have expected this to be patch #1 and used to gate 
compilation for any of those file.

> +
>   config NUMA
>   	bool
> 

Cheers,

Patch

diff --git a/xen/arch/arm/Kconfig b/xen/arch/arm/Kconfig
index f58019d6ed..d4767d6ea3 100644
--- a/xen/arch/arm/Kconfig
+++ b/xen/arch/arm/Kconfig
@@ -42,6 +42,11 @@  config ACPI
 	  Advanced Configuration and Power Interface (ACPI) support for Xen is
 	  an alternative to device tree on ARM64.
 
+config ACPI_IORT
+	bool
+	prompt "ACPI IORT Support" if EXPERT = "y"
+	depends on ACPI
+
 config HAS_GICV3
 	bool
 
diff --git a/xen/drivers/acpi/Kconfig b/xen/drivers/acpi/Kconfig
index b64d3731fb..15ae98140c 100644
--- a/xen/drivers/acpi/Kconfig
+++ b/xen/drivers/acpi/Kconfig
@@ -5,5 +5,8 @@  config ACPI
 config ACPI_LEGACY_TABLES_LOOKUP
 	bool
 
+config ACPI_IORT
+	bool
+
 config NUMA
 	bool