linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
To: Randy Dunlap <rdunlap@infradead.org>,
	LKML <linux-kernel@vger.kernel.org>,
	moderated for non-subscribers <alsa-devel@alsa-project.org>
Cc: Mark Brown <broonie@kernel.org>,
	Liam Girdwood <liam.r.girdwood@linux.intel.com>,
	Jie Yang <yang.jie@linux.intel.com>
Subject: Re: [PATCH v2] ASoC: drop COMPILE_TEST for sound/soc/intel/boards/ that use X86_INTEL_LPSS
Date: Fri, 24 May 2019 08:07:28 -0500	[thread overview]
Message-ID: <c05d5438-d369-e69d-e314-a0b104a3c71e@linux.intel.com> (raw)
In-Reply-To: <03640e4f-1f8e-9090-c03b-364918e633d3@infradead.org>



On 5/23/19 10:45 PM, Randy Dunlap wrote:
> From: Randy Dunlap <rdunlap@infradead.org>
> 
> This is a partial revert of 164a263bf8d0, which causes build errors
> on non-X86 platforms (specifically seen on IA64) when COMPILE_TEST
> is set/enabled.
> 
> Fixes these build errors (on ia64):
> i../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>

Randy, your v2 removes COMPILE_TEST for all boards even the ones that 
compiled fine?

Also maybe the 'right' way to fix this is to have a helper that compiles 
out on non X86 platforms (or use an existing one?). IIRC the dependency 
on asm/cpu_device_id is just to find out if the device is baytrail or 
cherrytrail.

> 
> Fixes: 164a263bf8d0 ("ASoC: Intel: Make boards more available for compile test")
> 
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Mark Brown <broonie@kernel.org>
> Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> Cc: Liam Girdwood <liam.r.girdwood@linux.intel.com>
> Cc: Jie Yang <yang.jie@linux.intel.com>
> Cc: alsa-devel@alsa-project.org
> ---
>   sound/soc/intel/boards/Kconfig |   30 +++++++++++++++---------------
>   1 file changed, 15 insertions(+), 15 deletions(-)
> 
> --- lnx-52-rc1.orig/sound/soc/intel/boards/Kconfig
> +++ lnx-52-rc1/sound/soc/intel/boards/Kconfig
> @@ -18,7 +18,7 @@ config SND_SOC_INTEL_HASWELL_MACH
>   	tristate "Haswell Lynxpoint"
>   	depends on I2C
>   	depends on I2C_DESIGNWARE_PLATFORM || COMPILE_TEST
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_RT5640
>   	help
>   	  This adds support for the Lynxpoint Audio DSP on Intel(R) Haswell
> @@ -35,7 +35,7 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
>   	depends on I2C
>   	depends on I2C_DESIGNWARE_PLATFORM || COMPILE_TEST
>   	depends on GPIOLIB || COMPILE_TEST
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_RT5677
>   	help
>   	  This adds support for Intel Broadwell platform based boards with
> @@ -47,7 +47,7 @@ config SND_SOC_INTEL_BROADWELL_MACH
>   	tristate "Broadwell Wildcatpoint"
>   	depends on I2C
>   	depends on I2C_DESIGNWARE_PLATFORM || COMPILE_TEST
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_RT286
>   	help
>   	  This adds support for the Wilcatpoint Audio DSP on Intel(R) Broadwell
> @@ -61,7 +61,7 @@ if SND_SOC_INTEL_BAYTRAIL
>   config SND_SOC_INTEL_BYT_MAX98090_MACH
>   	tristate "Baytrail with MAX98090 codec"
>   	depends on I2C
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_MAX98090
>   	help
>   	  This adds audio driver for Intel Baytrail platform based boards
> @@ -72,7 +72,7 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
>   config SND_SOC_INTEL_BYT_RT5640_MACH
>   	tristate "Baytrail with RT5640 codec"
>   	depends on I2C
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_RT5640
>   	help
>   	  This adds audio driver for Intel Baytrail platform based boards
> @@ -86,7 +86,7 @@ if SND_SST_ATOM_HIFI2_PLATFORM || SND_SO
>   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_INTEL_LPSS
>   	select SND_SOC_ACPI
>   	select SND_SOC_RT5640
>   	help
> @@ -98,7 +98,7 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
>   config SND_SOC_INTEL_BYTCR_RT5651_MACH
>   	tristate "Baytrail and Baytrail-CR with RT5651 codec"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_ACPI
>   	select SND_SOC_RT5651
>   	help
> @@ -110,7 +110,7 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
>   config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
>   	tristate "Cherrytrail & Braswell with RT5672 codec"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_ACPI
>   	select SND_SOC_RT5670
>           help
> @@ -122,7 +122,7 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
>   config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
>   	tristate "Cherrytrail & Braswell with RT5645/5650 codec"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_ACPI
>   	select SND_SOC_RT5645
>   	help
> @@ -134,7 +134,7 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
>   config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
>   	tristate "Cherrytrail & Braswell with MAX98090 & TI codec"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_MAX98090
>   	select SND_SOC_TS3A227E
>   	help
> @@ -146,7 +146,7 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI
>   config SND_SOC_INTEL_CHT_BSW_NAU8824_MACH
>   	tristate "Cherrytrail & Braswell with NAU88L24 codec"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_ACPI
>   	select SND_SOC_NAU8824
>   	help
> @@ -158,7 +158,7 @@ config SND_SOC_INTEL_CHT_BSW_NAU8824_MAC
>   config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
>   	tristate "Baytrail & Cherrytrail with DA7212/7213 codec"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_ACPI
>   	select SND_SOC_DA7213
>   	help
> @@ -170,7 +170,7 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
>   config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
>   	tristate "Baytrail & Cherrytrail with ES8316 codec"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	select SND_SOC_ACPI
>   	select SND_SOC_ES8316
>   	help
> @@ -186,7 +186,7 @@ if SND_SST_ATOM_HIFI2_PLATFORM
>   config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
>   	tristate "Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
>   	depends on I2C && ACPI
> -	depends on X86_INTEL_LPSS || COMPILE_TEST
> +	depends on X86_INTEL_LPSS
>   	help
>   	  This adds support for ASoC machine driver for the MinnowBoard Max or
>   	  Up boards and provides access to I2S signals on the Low-Speed
> @@ -248,7 +248,7 @@ if SND_SOC_INTEL_APL
>   config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
>   	tristate "Broxton with DA7219 and MAX98357A in I2S Mode"
>   	depends on I2C && ACPI
> -	depends on MFD_INTEL_LPSS || COMPILE_TEST
> +	depends on MFD_INTEL_LPSS
>   	select SND_SOC_DA7219
>   	select SND_SOC_MAX98357A
>   	select SND_SOC_DMIC
> 

      reply	other threads:[~2019-05-24 13:07 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-24  3:45 [PATCH v2] ASoC: drop COMPILE_TEST for sound/soc/intel/boards/ that use X86_INTEL_LPSS Randy Dunlap
2019-05-24 13:07 ` Pierre-Louis Bossart [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=c05d5438-d369-e69d-e314-a0b104a3c71e@linux.intel.com \
    --to=pierre-louis.bossart@linux.intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=liam.r.girdwood@linux.intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@infradead.org \
    --cc=yang.jie@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).