On Wed, Dec 11, 2019 at 3:00 AM Jon Hunter jonathanh@nvidia.com wrote:
On 10/12/2019 19:52, Arnd Bergmann wrote:
When CONFIG_SPI is disabled, the newly added code for the DSP firmware loading fails to link:
ERROR: "rt5677_spi_hotword_detected" [sound/soc/codecs/snd-soc-rt5677.ko] undefined! ERROR: "rt5677_spi_write" [sound/soc/codecs/snd-soc-rt5677.ko] undefined!
Would it be better if the above functions or the functions that call these are conditional on CONFIG_SND_SOC_RT5677_SPI?
Add a dependency to prevent this configuration.
Note: the does not work with the DT probing, as there is no binding
Are you missing 'SPI' or something here?
for the SPI half of the driver, but nothing seems to be using that with the mainline kernel anyway.
From a Tegra perspective, given that we don't use SPI in conjunction with the rt5677 codec, only I2C so far, I am not sure we should make the tegra_rt5677 driver dependent upon it. We should be able to operate without the SPI bits enabled.
There should be no changes needed for tegra, this should be isolated to the bdw machine driver. The only things added to the machine driver were some dai links.
Cheers Jon
-- nvpublic