On 5/22/19 10:58 PM, Randy Dunlap wrote:
From: Randy Dunlap <rdun...@infradead.org>

Several drivers in sound/soc/intel/boards/ #include header files
that only exist in arch/x86/include/asm.  This causes build errors,
so make these drivers depend on X86.

Fixes these build errors (on ia64):

../sound/soc/intel/boards/bxt_da7219_max98357a.c:19:10: fatal error: 
asm/cpu_device_id.h: No such file or directory
  #include <asm/cpu_device_id.h>
../sound/soc/intel/boards/bytcr_rt5640.c:31:10: fatal error: 
asm/cpu_device_id.h: No such file or directory
  #include <asm/cpu_device_id.h>
../sound/soc/intel/boards/bytcr_rt5651.c:33:10: fatal error: 
asm/cpu_device_id.h: No such file or directory
  #include <asm/cpu_device_id.h>
../sound/soc/intel/boards/cht_bsw_rt5645.c:29:10: fatal error: 
asm/cpu_device_id.h: No such file or directory
  #include <asm/cpu_device_id.h>
../sound/soc/intel/boards/bytcht_es8316.c:33:10: fatal error: 
asm/cpu_device_id.h: No such file or directory
  #include <asm/cpu_device_id.h>
../sound/soc/intel/boards/bytcht_da7213.c:26:10: fatal error: 
asm/platform_sst_audio.h: No such file or directory
  #include <asm/platform_sst_audio.h>

And more drivers determined by:
grep "include.*asm.cpu_device_id.h" *.c
bxt_da7219_max98357a.c:#include <asm/cpu_device_id.h>
bytcht_es8316.c:#include <asm/cpu_device_id.h>
bytcr_rt5640.c:#include <asm/cpu_device_id.h>
bytcr_rt5651.c:#include <asm/cpu_device_id.h>
cht_bsw_rt5645.c:#include <asm/cpu_device_id.h>
sof_rt5682.c:#include <asm/cpu_device_id.h>
   and
grep "include.*asm.platform_sst_audio.h" *.c
bytcht_da7213.c:#include <asm/platform_sst_audio.h>
bytcht_es8316.c:#include <asm/platform_sst_audio.h>

Signed-off-by: Randy Dunlap <rdun...@infradead.org>
Cc: Mark Brown <broo...@kernel.org>
Cc: Pierre-Louis Bossart <pierre-louis.boss...@linux.intel.com>
Cc: Liam Girdwood <liam.r.girdw...@linux.intel.com>
Cc: Jie Yang <yang....@linux.intel.com>
Cc: alsa-de...@alsa-project.org
---
  sound/soc/intel/boards/Kconfig |    6 ++++++
  1 file changed, 6 insertions(+)

--- lnx-52-rc1.orig/sound/soc/intel/boards/Kconfig
+++ lnx-52-rc1/sound/soc/intel/boards/Kconfig
@@ -87,6 +87,7 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
        tristate "Baytrail and Baytrail-CR with RT5640 codec"
        depends on I2C && ACPI
        depends on X86_INTEL_LPSS || COMPILE_TEST
+       depends on X86

How does this improve the results?

config X86_INTEL_LPSS
        bool "Intel Low Power Subsystem Support"
        depends on X86 && ACPI && PCI

So the X86 dependency is already there. Does this happen with COMPILE_TEST set? If yes, maybe that's the part that needs to be changed? The addition of COMPILE_TEST here is quite recent and might need to be reverted.

Reply via email to