Commit 08bd168696 ("ASoC: Add driver for the Lyrtech SFFSDR board") added a reference to a Kconfig symbol SFFSDR_FPGA. Commit ac37373b64 ("ASoC: DaVinci: Fix SFFSDR compilation error.") added a macro CONFIG_SFFSDR_FPGA. But there's no Kconfig symbol SFFSDR_FPGA and no define of CONFIG_SFFSDR_FPGA. So the select of SFFSDR_FPGA is a nop. It can be dropped. The checks for CONFIG_SFFSDR_FPGA can also be dropped.
Signed-off-by: Paul Bolle <pebo...@tiscali.nl> --- 0) First sent over a year ago. Reapplied on top of v3.9-rc1. 1) Still untested, and this might need (build, etc.) testing if we want to be thorough. But I do not have the hardware and neither do I have the toolchain needed at hand. The patch should be correct anyhow. 2) Note that there's no file named *plat-sffsdr/sffsdr-fpga.h in the mainline tree. Neither is there a function or a macro named sffsdr_fpga_set_codec_fs(). So enabling SFFSDR_FPGA can currently never work. sound/soc/davinci/Kconfig | 1 - sound/soc/davinci/davinci-sffsdr.c | 9 --------- 2 files changed, 10 deletions(-) diff --git a/sound/soc/davinci/Kconfig b/sound/soc/davinci/Kconfig index 9e11a14..6a91206 100644 --- a/sound/soc/davinci/Kconfig +++ b/sound/soc/davinci/Kconfig @@ -59,7 +59,6 @@ config SND_DAVINCI_SOC_SFFSDR depends on SND_DAVINCI_SOC && MACH_SFFSDR select SND_DAVINCI_SOC_I2S select SND_SOC_PCM3008 - select SFFSDR_FPGA help Say Y if you want to add support for SoC audio on Lyrtech SFFSDR board. diff --git a/sound/soc/davinci/davinci-sffsdr.c b/sound/soc/davinci/davinci-sffsdr.c index 5be65aa..ec4cb3a 100644 --- a/sound/soc/davinci/davinci-sffsdr.c +++ b/sound/soc/davinci/davinci-sffsdr.c @@ -24,9 +24,6 @@ #include <asm/dma.h> #include <asm/mach-types.h> -#ifdef CONFIG_SFFSDR_FPGA -#include <asm/plat-sffsdr/sffsdr-fpga.h> -#endif #include <mach/edma.h> @@ -53,13 +50,11 @@ static int sffsdr_hw_params(struct snd_pcm_substream *substream, /* Fsref can be 32000, 44100 or 48000. */ fs = params_rate(params); -#ifndef CONFIG_SFFSDR_FPGA /* Without the FPGA module, the Fs is fixed at 44100 Hz */ if (fs != 44100) { pr_debug("warning: only 44.1 kHz is supported without SFFSDR FPGA module\n"); return -EINVAL; } -#endif /* set cpu DAI configuration */ ret = snd_soc_dai_set_fmt(cpu_dai, AUDIO_FORMAT); @@ -68,11 +63,7 @@ static int sffsdr_hw_params(struct snd_pcm_substream *substream, pr_debug("sffsdr_hw_params: rate = %d Hz\n", fs); -#ifndef CONFIG_SFFSDR_FPGA return 0; -#else - return sffsdr_fpga_set_codec_fs(fs); -#endif } static struct snd_soc_ops sffsdr_ops = { -- 1.7.11.7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/