Commit 291bfb92 authored by Mark Brown's avatar Mark Brown

ASoC: topology: Revert recent changes while boot errors are investigated

Krzysztof Kozlowski reported a NULL dereference in _instantiate_card()
on Odroid XU3 and XU boards which he bisected to 45f8cb57 (ASoC:
core: Allow topology to override machine driver FE DAI link config).
Revert that commit for now, along with f11a5c27 (ASoC: core: Add
name prefix for machines with topology rewrites) due to dependency
issues, in order to keep things booting cleanly in -next.
Reported-by: default avatarKrzysztof Kozlowski <krzk@kernel.org>
Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 24ada035
...@@ -1012,14 +1012,6 @@ struct snd_soc_platform_driver { ...@@ -1012,14 +1012,6 @@ struct snd_soc_platform_driver {
/* platform stream compress ops */ /* platform stream compress ops */
const struct snd_compr_ops *compr_ops; const struct snd_compr_ops *compr_ops;
/* this platform uses topology and ignore machine driver FEs */
const char *ignore_machine;
const char *topology_name_prefix;
int (*be_hw_params_fixup)(struct snd_soc_pcm_runtime *rtd,
struct snd_pcm_hw_params *params);
bool use_dai_pcm_id; /* use the DAI link PCM ID as PCM device number */
int be_pcm_base; /* base device ID for all BE PCMs */
}; };
struct snd_soc_dai_link_component { struct snd_soc_dai_link_component {
...@@ -1126,9 +1118,6 @@ struct snd_soc_dai_link { ...@@ -1126,9 +1118,6 @@ struct snd_soc_dai_link {
/* pmdown_time is ignored at stop */ /* pmdown_time is ignored at stop */
unsigned int ignore_pmdown_time:1; unsigned int ignore_pmdown_time:1;
/* Do not create a PCM for this DAI link (Backend link) */
unsigned int ignore:1;
struct list_head list; /* DAI link list of the soc card */ struct list_head list; /* DAI link list of the soc card */
struct snd_soc_dobj dobj; /* For topology */ struct snd_soc_dobj dobj; /* For topology */
}; };
...@@ -1168,7 +1157,6 @@ struct snd_soc_card { ...@@ -1168,7 +1157,6 @@ struct snd_soc_card {
const char *long_name; const char *long_name;
const char *driver_name; const char *driver_name;
char dmi_longname[80]; char dmi_longname[80];
char topology_shortname[32];
struct device *dev; struct device *dev;
struct snd_card *snd_card; struct snd_card *snd_card;
......
...@@ -1050,9 +1050,6 @@ static int soc_bind_dai_link(struct snd_soc_card *card, ...@@ -1050,9 +1050,6 @@ static int soc_bind_dai_link(struct snd_soc_card *card,
const char *platform_name; const char *platform_name;
int i; int i;
if (dai_link->ignore)
return 0;
dev_dbg(card->dev, "ASoC: binding %s\n", dai_link->name); dev_dbg(card->dev, "ASoC: binding %s\n", dai_link->name);
if (soc_is_dai_link_bound(card, dai_link)) { if (soc_is_dai_link_bound(card, dai_link)) {
...@@ -1675,7 +1672,7 @@ static int soc_probe_link_dais(struct snd_soc_card *card, ...@@ -1675,7 +1672,7 @@ static int soc_probe_link_dais(struct snd_soc_card *card,
{ {
struct snd_soc_dai_link *dai_link = rtd->dai_link; struct snd_soc_dai_link *dai_link = rtd->dai_link;
struct snd_soc_dai *cpu_dai = rtd->cpu_dai; struct snd_soc_dai *cpu_dai = rtd->cpu_dai;
int i, ret, num; int i, ret;
dev_dbg(card->dev, "ASoC: probe %s dai link %d late %d\n", dev_dbg(card->dev, "ASoC: probe %s dai link %d late %d\n",
card->name, rtd->num, order); card->name, rtd->num, order);
...@@ -1721,23 +1718,9 @@ static int soc_probe_link_dais(struct snd_soc_card *card, ...@@ -1721,23 +1718,9 @@ static int soc_probe_link_dais(struct snd_soc_card *card,
soc_dpcm_debugfs_add(rtd); soc_dpcm_debugfs_add(rtd);
#endif #endif
/*
* most drivers will register their PCMs using DAI link ordering but
* topology based drivers can use the DAI link id field to set PCM
* device number and then use rtd + a base offset of the BEs.
*/
if (rtd->platform->driver->use_dai_pcm_id) {
if (rtd->dai_link->no_pcm)
num = rtd->platform->driver->be_pcm_base + rtd->num;
else
num = rtd->dai_link->id;
} else {
num = rtd->num;
}
if (cpu_dai->driver->compress_new) { if (cpu_dai->driver->compress_new) {
/*create compress_device"*/ /*create compress_device"*/
ret = cpu_dai->driver->compress_new(rtd, num); ret = cpu_dai->driver->compress_new(rtd, rtd->num);
if (ret < 0) { if (ret < 0) {
dev_err(card->dev, "ASoC: can't create compress %s\n", dev_err(card->dev, "ASoC: can't create compress %s\n",
dai_link->stream_name); dai_link->stream_name);
...@@ -1747,7 +1730,7 @@ static int soc_probe_link_dais(struct snd_soc_card *card, ...@@ -1747,7 +1730,7 @@ static int soc_probe_link_dais(struct snd_soc_card *card,
if (!dai_link->params) { if (!dai_link->params) {
/* create the pcm */ /* create the pcm */
ret = soc_new_pcm(rtd, num); ret = soc_new_pcm(rtd, rtd->num);
if (ret < 0) { if (ret < 0) {
dev_err(card->dev, "ASoC: can't create pcm %s :%d\n", dev_err(card->dev, "ASoC: can't create pcm %s :%d\n",
dai_link->stream_name, ret); dai_link->stream_name, ret);
...@@ -2093,67 +2076,6 @@ int snd_soc_set_dmi_name(struct snd_soc_card *card, const char *flavour) ...@@ -2093,67 +2076,6 @@ int snd_soc_set_dmi_name(struct snd_soc_card *card, const char *flavour)
EXPORT_SYMBOL_GPL(snd_soc_set_dmi_name); EXPORT_SYMBOL_GPL(snd_soc_set_dmi_name);
#endif /* CONFIG_DMI */ #endif /* CONFIG_DMI */
static void soc_check_tplg_fes(struct snd_soc_card *card)
{
struct snd_soc_platform *platform;
struct snd_soc_dai_link *dai_link;
int i;
list_for_each_entry(platform, &platform_list, list) {
/* does this platform override FEs ? */
if (!platform->driver->ignore_machine)
continue;
/* for this machine ? */
if (strcmp(platform->driver->ignore_machine,
card->dev->driver->name))
continue;
/* machine matches, so override the rtd data */
for (i = 0; i < card->num_links; i++) {
dai_link = &card->dai_link[i];
/* ignore this FE */
if (dai_link->dynamic) {
dai_link->ignore = true;
continue;
}
dev_info(card->dev, "info: override FE DAI link %s\n",
card->dai_link[i].name);
/* override platform */
dai_link->platform_name = platform->component.name;
dai_link->cpu_dai_name = platform->component.name;
/* convert non BE into BE */
dai_link->no_pcm = 1;
dai_link->dpcm_playback = 1;
dai_link->dpcm_capture = 1;
/* override any BE fixups */
dai_link->be_hw_params_fixup =
platform->driver->be_hw_params_fixup;
/* most BE links dont set stream name, so set it to
* dai link name if it's NULL to help bind widgets.
*/
if (!dai_link->stream_name)
dai_link->stream_name = dai_link->name;
}
/* Inform userspace we are using alternate topology */
if (platform->driver->topology_name_prefix) {
snprintf(card->topology_shortname, 32, "%s-%s",
platform->driver->topology_name_prefix,
card->name);
card->name = card->topology_shortname;
}
}
}
static int snd_soc_instantiate_card(struct snd_soc_card *card) static int snd_soc_instantiate_card(struct snd_soc_card *card)
{ {
struct snd_soc_codec *codec; struct snd_soc_codec *codec;
...@@ -2164,9 +2086,6 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card) ...@@ -2164,9 +2086,6 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
mutex_lock(&client_mutex); mutex_lock(&client_mutex);
mutex_lock_nested(&card->mutex, SND_SOC_CARD_CLASS_INIT); mutex_lock_nested(&card->mutex, SND_SOC_CARD_CLASS_INIT);
/* check whether any platform is ignore machine FE and using topology */
soc_check_tplg_fes(card);
/* bind DAIs */ /* bind DAIs */
for (i = 0; i < card->num_links; i++) { for (i = 0; i < card->num_links; i++) {
ret = soc_bind_dai_link(card, &card->dai_link[i]); ret = soc_bind_dai_link(card, &card->dai_link[i]);
......
...@@ -909,20 +909,8 @@ int soc_dai_hw_params(struct snd_pcm_substream *substream, ...@@ -909,20 +909,8 @@ int soc_dai_hw_params(struct snd_pcm_substream *substream,
struct snd_pcm_hw_params *params, struct snd_pcm_hw_params *params,
struct snd_soc_dai *dai) struct snd_soc_dai *dai)
{ {
struct snd_soc_pcm_runtime *rtd = substream->private_data;
int ret; int ret;
/* perform any topology hw_params fixups before DAI */
if (rtd->dai_link->be_hw_params_fixup) {
ret = rtd->dai_link->be_hw_params_fixup(rtd, params);
if (ret < 0) {
dev_err(rtd->dev,
"ASoC: hw_params topology fixup failed %d\n",
ret);
return ret;
}
}
if (dai->driver->ops->hw_params) { if (dai->driver->ops->hw_params) {
ret = dai->driver->ops->hw_params(substream, params, dai); ret = dai->driver->ops->hw_params(substream, params, dai);
if (ret < 0) { if (ret < 0) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment