Commit b9ddd81b authored by Pierre-Louis Bossart's avatar Pierre-Louis Bossart Committed by Mark Brown

ASoC: SOF: Intel: hda: initial SoundWire machine driver autodetect

For now we have a limited number of machine driver configurations, and
we can detect them based on the link configuration returned after
checking hardware and firmware (BIOS) configurations.

The link configuration is checked with a link_mask as well as a list
of _ADR descriptors for each link.

There is a chance that in extreme cases where the BIOS contains too
much information we would need to detect which Slave devices actually
report as 'attached'. This would be more accurate than static
table-based solutions, but it also introduces timing dependencies
since we don't know when those devices might become attached, so will
only be only be looked at if we see limitations with static methods
and the usual quirks based e.g. on DMI information.
Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: default avatarBard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: default avatarRander Wang <rander.wang@intel.com>
Link: https://lore.kernel.org/r/20200325215027.28716-6-pierre-louis.bossart@linux.intel.comSigned-off-by: default avatarMark Brown <broonie@kernel.org>
parent d2c383aa
...@@ -20,6 +20,7 @@ ...@@ -20,6 +20,7 @@
#include <linux/acpi.h> #include <linux/acpi.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/soundwire/sdw.h>
#include <linux/soundwire/sdw_intel.h> #include <linux/soundwire/sdw_intel.h>
#include <sound/intel-nhlt.h> #include <sound/intel-nhlt.h>
#include <sound/sof.h> #include <sound/sof.h>
...@@ -136,11 +137,8 @@ static int hda_sdw_probe(struct snd_sof_dev *sdev) ...@@ -136,11 +137,8 @@ static int hda_sdw_probe(struct snd_sof_dev *sdev)
{ {
struct sof_intel_hda_dev *hdev; struct sof_intel_hda_dev *hdev;
struct sdw_intel_res res; struct sdw_intel_res res;
acpi_handle handle;
void *sdw; void *sdw;
handle = ACPI_HANDLE(sdev->dev);
hdev = sdev->pdata->hw_pdata; hdev = sdev->pdata->hw_pdata;
memset(&res, 0, sizeof(res)); memset(&res, 0, sizeof(res));
...@@ -180,6 +178,9 @@ int hda_sdw_startup(struct snd_sof_dev *sdev) ...@@ -180,6 +178,9 @@ int hda_sdw_startup(struct snd_sof_dev *sdev)
hdev = sdev->pdata->hw_pdata; hdev = sdev->pdata->hw_pdata;
if (!hdev->sdw)
return 0;
return sdw_intel_startup(hdev->sdw); return sdw_intel_startup(hdev->sdw);
} }
...@@ -536,24 +537,31 @@ static int hda_init_caps(struct snd_sof_dev *sdev) ...@@ -536,24 +537,31 @@ static int hda_init_caps(struct snd_sof_dev *sdev)
/* scan SoundWire capabilities exposed by DSDT */ /* scan SoundWire capabilities exposed by DSDT */
ret = hda_sdw_acpi_scan(sdev); ret = hda_sdw_acpi_scan(sdev);
if (ret < 0) { if (ret < 0) {
dev_err(sdev->dev, "error: SoundWire ACPI scan error\n"); dev_dbg(sdev->dev, "skipping SoundWire, ACPI scan error\n");
return ret; goto skip_soundwire;
} }
link_mask = hdev->info.link_mask; link_mask = hdev->info.link_mask;
if (!link_mask) { if (!link_mask) {
/* dev_dbg(sdev->dev, "skipping SoundWire, no links enabled\n");
* probe/allocated SoundWire resources. goto skip_soundwire;
* The hardware configuration takes place in hda_sdw_startup
* after power rails are enabled.
*/
ret = hda_sdw_probe(sdev);
if (ret < 0) {
dev_err(sdev->dev, "error: SoundWire probe error\n");
return ret;
}
} }
/*
* probe/allocate SoundWire resources.
* The hardware configuration takes place in hda_sdw_startup
* after power rails are enabled.
* It's entirely possible to have a mix of I2S/DMIC/SoundWire
* devices, so we allocate the resources in all cases.
*/
ret = hda_sdw_probe(sdev);
if (ret < 0) {
dev_err(sdev->dev, "error: SoundWire probe error\n");
return ret;
}
skip_soundwire:
#if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA) #if IS_ENABLED(CONFIG_SND_SOC_SOF_HDA)
if (bus->mlcap) if (bus->mlcap)
snd_hdac_ext_bus_get_ml_capabilities(bus); snd_hdac_ext_bus_get_ml_capabilities(bus);
...@@ -949,6 +957,123 @@ static int hda_generic_machine_select(struct snd_sof_dev *sdev) ...@@ -949,6 +957,123 @@ static int hda_generic_machine_select(struct snd_sof_dev *sdev)
} }
#endif #endif
#if IS_ENABLED(CONFIG_SND_SOC_SOF_INTEL_SOUNDWIRE)
/* Check if all Slaves defined on the link can be found */
static bool link_slaves_found(struct snd_sof_dev *sdev,
const struct snd_soc_acpi_link_adr *link,
struct sdw_intel_ctx *sdw)
{
struct hdac_bus *bus = sof_to_bus(sdev);
struct sdw_intel_slave_id *ids = sdw->ids;
int num_slaves = sdw->num_slaves;
unsigned int part_id, link_id, unique_id, mfg_id;
int i, j;
for (i = 0; i < link->num_adr; i++) {
u64 adr = link->adr_d[i].adr;
mfg_id = SDW_MFG_ID(adr);
part_id = SDW_PART_ID(adr);
link_id = SDW_DISCO_LINK_ID(adr);
for (j = 0; j < num_slaves; j++) {
if (ids[j].link_id != link_id ||
ids[j].id.part_id != part_id ||
ids[j].id.mfg_id != mfg_id)
continue;
/*
* we have to check unique id
* if there is more than one
* Slave on the link
*/
unique_id = SDW_UNIQUE_ID(adr);
if (link->num_adr == 1 ||
ids[j].id.unique_id == SDW_IGNORED_UNIQUE_ID ||
ids[j].id.unique_id == unique_id) {
dev_dbg(bus->dev,
"found %x at link %d\n",
part_id, link_id);
break;
}
}
if (j == num_slaves) {
dev_dbg(bus->dev,
"Slave %x not found\n",
part_id);
return false;
}
}
return true;
}
static int hda_sdw_machine_select(struct snd_sof_dev *sdev)
{
struct snd_sof_pdata *pdata = sdev->pdata;
const struct snd_soc_acpi_link_adr *link;
struct hdac_bus *bus = sof_to_bus(sdev);
struct snd_soc_acpi_mach *mach;
struct sof_intel_hda_dev *hdev;
u32 link_mask;
int i;
hdev = pdata->hw_pdata;
link_mask = hdev->info.link_mask;
/*
* Select SoundWire machine driver if needed using the
* alternate tables. This case deals with SoundWire-only
* machines, for mixed cases with I2C/I2S the detection relies
* on the HID list.
*/
if (link_mask && !pdata->machine) {
for (mach = pdata->desc->alt_machines;
mach && mach->link_mask; mach++) {
if (mach->link_mask != link_mask)
continue;
/* No need to match adr if there is no links defined */
if (!mach->links)
break;
link = mach->links;
for (i = 0; i < hdev->info.count && link->num_adr;
i++, link++) {
/*
* Try next machine if any expected Slaves
* are not found on this link.
*/
if (!link_slaves_found(sdev, link, hdev->sdw))
break;
}
/* Found if all Slaves are checked */
if (i == hdev->info.count || !link->num_adr)
break;
}
if (mach && mach->link_mask) {
dev_dbg(bus->dev,
"SoundWire machine driver %s topology %s\n",
mach->drv_name,
mach->sof_tplg_filename);
pdata->machine = mach;
mach->mach_params.links = mach->links;
mach->mach_params.link_mask = mach->link_mask;
mach->mach_params.platform = dev_name(sdev->dev);
pdata->fw_filename = mach->sof_fw_filename;
pdata->tplg_filename = mach->sof_tplg_filename;
} else {
dev_info(sdev->dev,
"No SoundWire machine driver found\n");
}
}
return 0;
}
#else
static int hda_sdw_machine_select(struct snd_sof_dev *sdev)
{
return 0;
}
#endif
void hda_set_mach_params(const struct snd_soc_acpi_mach *mach, void hda_set_mach_params(const struct snd_soc_acpi_mach *mach,
struct device *dev) struct device *dev)
{ {
...@@ -968,8 +1093,18 @@ void hda_machine_select(struct snd_sof_dev *sdev) ...@@ -968,8 +1093,18 @@ void hda_machine_select(struct snd_sof_dev *sdev)
if (mach) { if (mach) {
sof_pdata->tplg_filename = mach->sof_tplg_filename; sof_pdata->tplg_filename = mach->sof_tplg_filename;
sof_pdata->machine = mach; sof_pdata->machine = mach;
if (mach->link_mask) {
mach->mach_params.links = mach->links;
mach->mach_params.link_mask = mach->link_mask;
}
} }
/*
* If I2S fails, try SoundWire
*/
hda_sdw_machine_select(sdev);
/* /*
* Choose HDA generic machine driver if mach is NULL. * Choose HDA generic machine driver if mach is NULL.
* Otherwise, set certain mach params. * Otherwise, set certain mach params.
......
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