Commit f0cea797 authored by Takashi Iwai's avatar Takashi Iwai

ALSA: hda - Fix dynamic ADC change working again

The commit eb541337
    ALSA: hda - Make converter setups sticky
changes the semantics of snd_hda_codec_cleanup_stream() not to clean up
the stream at that moment but delay the action.  This broke the codes
expecting that the clean-up is done immediately, such as dynamic ADC
changes in some codec drivers.

This patch fixes the issue by introducing a lower helper,
__snd_hda_codec_cleanup_stream(), to allow the immediate clean up.
The original snd_hda_codec_cleanup_stream() is kept as is now.
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent bbbe3390
...@@ -1261,12 +1261,17 @@ void snd_hda_codec_setup_stream(struct hda_codec *codec, hda_nid_t nid, ...@@ -1261,12 +1261,17 @@ void snd_hda_codec_setup_stream(struct hda_codec *codec, hda_nid_t nid,
} }
EXPORT_SYMBOL_HDA(snd_hda_codec_setup_stream); EXPORT_SYMBOL_HDA(snd_hda_codec_setup_stream);
static void really_cleanup_stream(struct hda_codec *codec,
struct hda_cvt_setup *q);
/** /**
* snd_hda_codec_cleanup_stream - clean up the codec for closing * __snd_hda_codec_cleanup_stream - clean up the codec for closing
* @codec: the CODEC to clean up * @codec: the CODEC to clean up
* @nid: the NID to clean up * @nid: the NID to clean up
* @do_now: really clean up the stream instead of clearing the active flag
*/ */
void snd_hda_codec_cleanup_stream(struct hda_codec *codec, hda_nid_t nid) void __snd_hda_codec_cleanup_stream(struct hda_codec *codec, hda_nid_t nid,
int do_now)
{ {
struct hda_cvt_setup *p; struct hda_cvt_setup *p;
...@@ -1274,14 +1279,19 @@ void snd_hda_codec_cleanup_stream(struct hda_codec *codec, hda_nid_t nid) ...@@ -1274,14 +1279,19 @@ void snd_hda_codec_cleanup_stream(struct hda_codec *codec, hda_nid_t nid)
return; return;
snd_printdd("hda_codec_cleanup_stream: NID=0x%x\n", nid); snd_printdd("hda_codec_cleanup_stream: NID=0x%x\n", nid);
/* here we just clear the active flag; actual clean-ups will be done
* in purify_inactive_streams()
*/
p = get_hda_cvt_setup(codec, nid); p = get_hda_cvt_setup(codec, nid);
if (p) if (p) {
/* here we just clear the active flag when do_now isn't set;
* actual clean-ups will be done later in
* purify_inactive_streams() called from snd_hda_codec_prpapre()
*/
if (do_now)
really_cleanup_stream(codec, p);
else
p->active = 0; p->active = 0;
}
} }
EXPORT_SYMBOL_HDA(snd_hda_codec_cleanup_stream); EXPORT_SYMBOL_HDA(__snd_hda_codec_cleanup_stream);
static void really_cleanup_stream(struct hda_codec *codec, static void really_cleanup_stream(struct hda_codec *codec,
struct hda_cvt_setup *q) struct hda_cvt_setup *q)
......
...@@ -963,7 +963,10 @@ void snd_hda_codec_cleanup(struct hda_codec *codec, ...@@ -963,7 +963,10 @@ void snd_hda_codec_cleanup(struct hda_codec *codec,
void snd_hda_codec_setup_stream(struct hda_codec *codec, hda_nid_t nid, void snd_hda_codec_setup_stream(struct hda_codec *codec, hda_nid_t nid,
u32 stream_tag, u32 stream_tag,
int channel_id, int format); int channel_id, int format);
void snd_hda_codec_cleanup_stream(struct hda_codec *codec, hda_nid_t nid); void __snd_hda_codec_cleanup_stream(struct hda_codec *codec, hda_nid_t nid,
int do_now);
#define snd_hda_codec_cleanup_stream(codec, nid) \
__snd_hda_codec_cleanup_stream(codec, nid, 0)
unsigned int snd_hda_calc_stream_format(unsigned int rate, unsigned int snd_hda_calc_stream_format(unsigned int rate,
unsigned int channels, unsigned int channels,
unsigned int format, unsigned int format,
......
...@@ -656,7 +656,7 @@ static int change_cur_input(struct hda_codec *codec, unsigned int idx, ...@@ -656,7 +656,7 @@ static int change_cur_input(struct hda_codec *codec, unsigned int idx,
return 0; return 0;
if (spec->cur_adc && spec->cur_adc != spec->adc_nid[idx]) { if (spec->cur_adc && spec->cur_adc != spec->adc_nid[idx]) {
/* stream is running, let's swap the current ADC */ /* stream is running, let's swap the current ADC */
snd_hda_codec_cleanup_stream(codec, spec->cur_adc); __snd_hda_codec_cleanup_stream(codec, spec->cur_adc, 1);
spec->cur_adc = spec->adc_nid[idx]; spec->cur_adc = spec->adc_nid[idx];
snd_hda_codec_setup_stream(codec, spec->cur_adc, snd_hda_codec_setup_stream(codec, spec->cur_adc,
spec->cur_adc_stream_tag, 0, spec->cur_adc_stream_tag, 0,
......
...@@ -1733,7 +1733,7 @@ static void cxt5051_portc_automic(struct hda_codec *codec) ...@@ -1733,7 +1733,7 @@ static void cxt5051_portc_automic(struct hda_codec *codec)
new_adc = spec->adc_nids[spec->cur_adc_idx]; new_adc = spec->adc_nids[spec->cur_adc_idx];
if (spec->cur_adc && spec->cur_adc != new_adc) { if (spec->cur_adc && spec->cur_adc != new_adc) {
/* stream is running, let's swap the current ADC */ /* stream is running, let's swap the current ADC */
snd_hda_codec_cleanup_stream(codec, spec->cur_adc); __snd_hda_codec_cleanup_stream(codec, spec->cur_adc, 1);
spec->cur_adc = new_adc; spec->cur_adc = new_adc;
snd_hda_codec_setup_stream(codec, new_adc, snd_hda_codec_setup_stream(codec, new_adc,
spec->cur_adc_stream_tag, 0, spec->cur_adc_stream_tag, 0,
......
...@@ -1037,7 +1037,7 @@ static void alc_dual_mic_adc_auto_switch(struct hda_codec *codec) ...@@ -1037,7 +1037,7 @@ static void alc_dual_mic_adc_auto_switch(struct hda_codec *codec)
new_adc = spec->adc_nids[spec->cur_adc_idx]; new_adc = spec->adc_nids[spec->cur_adc_idx];
if (spec->cur_adc && spec->cur_adc != new_adc) { if (spec->cur_adc && spec->cur_adc != new_adc) {
/* stream is running, let's swap the current ADC */ /* stream is running, let's swap the current ADC */
snd_hda_codec_cleanup_stream(codec, spec->cur_adc); __snd_hda_codec_cleanup_stream(codec, spec->cur_adc, 1);
spec->cur_adc = new_adc; spec->cur_adc = new_adc;
snd_hda_codec_setup_stream(codec, new_adc, snd_hda_codec_setup_stream(codec, new_adc,
spec->cur_adc_stream_tag, 0, spec->cur_adc_stream_tag, 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