Commit f5172a7e authored by Takashi Iwai's avatar Takashi Iwai

ALSA: hda - Check the existing path in snd_hda_add_new_path()

If the requested path has been already added, return the existing path
instance instead of adding a duplicated instance.
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 1e0b5286
...@@ -116,11 +116,9 @@ EXPORT_SYMBOL_HDA(snd_hda_gen_spec_free); ...@@ -116,11 +116,9 @@ EXPORT_SYMBOL_HDA(snd_hda_gen_spec_free);
* parsing paths * parsing paths
*/ */
/* get the path between the given NIDs; static struct nid_path *get_nid_path(struct hda_codec *codec,
* passing 0 to either @pin or @dac behaves as a wildcard hda_nid_t from_nid, hda_nid_t to_nid,
*/ int with_aa_mix)
struct nid_path *snd_hda_get_nid_path(struct hda_codec *codec,
hda_nid_t from_nid, hda_nid_t to_nid)
{ {
struct hda_gen_spec *spec = codec->spec; struct hda_gen_spec *spec = codec->spec;
int i; int i;
...@@ -130,11 +128,23 @@ struct nid_path *snd_hda_get_nid_path(struct hda_codec *codec, ...@@ -130,11 +128,23 @@ struct nid_path *snd_hda_get_nid_path(struct hda_codec *codec,
if (path->depth <= 0) if (path->depth <= 0)
continue; continue;
if ((!from_nid || path->path[0] == from_nid) && if ((!from_nid || path->path[0] == from_nid) &&
(!to_nid || path->path[path->depth - 1] == to_nid)) (!to_nid || path->path[path->depth - 1] == to_nid)) {
return path; if (with_aa_mix == HDA_PARSE_ALL ||
path->with_aa_mix == with_aa_mix)
return path;
}
} }
return NULL; return NULL;
} }
/* get the path between the given NIDs;
* passing 0 to either @pin or @dac behaves as a wildcard
*/
struct nid_path *snd_hda_get_nid_path(struct hda_codec *codec,
hda_nid_t from_nid, hda_nid_t to_nid)
{
return get_nid_path(codec, from_nid, to_nid, HDA_PARSE_ALL);
}
EXPORT_SYMBOL_HDA(snd_hda_get_nid_path); EXPORT_SYMBOL_HDA(snd_hda_get_nid_path);
/* check whether the given DAC is already found in any existing paths */ /* check whether the given DAC is already found in any existing paths */
...@@ -248,6 +258,8 @@ static bool __parse_nid_path(struct hda_codec *codec, ...@@ -248,6 +258,8 @@ static bool __parse_nid_path(struct hda_codec *codec,
found: found:
path->path[path->depth] = conn[i]; path->path[path->depth] = conn[i];
if (conn[i] == spec->mixer_nid)
path->with_aa_mix = true;
path->idx[path->depth + 1] = i; path->idx[path->depth + 1] = i;
if (nums > 1 && get_wcaps_type(get_wcaps(codec, to_nid)) != AC_WID_AUD_MIX) if (nums > 1 && get_wcaps_type(get_wcaps(codec, to_nid)) != AC_WID_AUD_MIX)
path->multi[path->depth + 1] = 1; path->multi[path->depth + 1] = 1;
...@@ -290,6 +302,11 @@ snd_hda_add_new_path(struct hda_codec *codec, hda_nid_t from_nid, ...@@ -290,6 +302,11 @@ snd_hda_add_new_path(struct hda_codec *codec, hda_nid_t from_nid,
if (from_nid && to_nid && !is_reachable_path(codec, from_nid, to_nid)) if (from_nid && to_nid && !is_reachable_path(codec, from_nid, to_nid))
return NULL; return NULL;
/* check whether the path has been already added */
path = get_nid_path(codec, from_nid, to_nid, with_aa_mix);
if (path)
return path;
path = snd_array_new(&spec->paths); path = snd_array_new(&spec->paths);
if (!path) if (!path)
return NULL; return NULL;
......
...@@ -53,6 +53,7 @@ struct nid_path { ...@@ -53,6 +53,7 @@ struct nid_path {
unsigned char multi[MAX_NID_PATH_DEPTH]; unsigned char multi[MAX_NID_PATH_DEPTH];
unsigned int ctls[NID_PATH_NUM_CTLS]; /* NID_PATH_XXX_CTL */ unsigned int ctls[NID_PATH_NUM_CTLS]; /* NID_PATH_XXX_CTL */
bool active; bool active;
bool with_aa_mix;
}; };
/* mic/line-in auto switching entry */ /* mic/line-in auto switching entry */
......
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