Hi Wolfgang,

On Mon, Feb 3, 2020 at 9:06 PM Wolfgang Wallner
<wolfgang.wall...@br-automation.com> wrote:
>
> Currently the ITSS driver is built unconditionally. Add a Kconfig option
> to support enabling/disabling the inclusion of the ITSS driver depending
> on the platform.
>
> Signed-off-by: Wolfgang Wallner <wolfgang.wall...@br-automation.com>
> ---
>
>  arch/x86/Kconfig                   | 6 ++++++
>  arch/x86/cpu/intel_common/Makefile | 2 +-
>  2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> index 89b93e5de2..b733d2264e 100644
> --- a/arch/x86/Kconfig
> +++ b/arch/x86/Kconfig
> @@ -709,6 +709,12 @@ config ROM_TABLE_SIZE
>         hex
>         default 0x10000
>
> +config HAVE_ITSS
> +       bool "Enable ITSS"
> +       help
> +         Select this to include the driver for the Interrupt Timer
> +         Subsystem (ITSS) which is found on several Intel devices.
> +
>  menu "System tables"
>         depends on !EFI && !SYS_COREBOOT
>
> diff --git a/arch/x86/cpu/intel_common/Makefile 
> b/arch/x86/cpu/intel_common/Makefile
> index 266e6e26fa..e22c70781d 100644
> --- a/arch/x86/cpu/intel_common/Makefile
> +++ b/arch/x86/cpu/intel_common/Makefile
> @@ -27,7 +27,7 @@ obj-y += microcode.o
>  endif
>  endif
>  obj-y += pch.o
> -obj-y += itss.o
> +obj-$(CONFIG_HAVE_ITSS) += itss.o
>

I will have to squash this patch into the previous one, because the
unconditional build causes issues for other x86 targets, see:
https://dev.azure.com/bmeng/GitHub/_build/results?buildId=153&view=logs&j=8a1d3be7-a4c9-55b6-774d-e7f1a8f80847&t=b2f224a7-1103-5b52-edbc-3784ae727e03

Regards,
Bin

Reply via email to