Commit f4ab5950 authored by Oswald Buddenhagen's avatar Oswald Buddenhagen Committed by Takashi Iwai

ALSA: emu10k1: make channel count of multi-channel playback flexible

There is no reason to nail it to 16 channels.
Signed-off-by: default avatarOswald Buddenhagen <oswald.buddenhagen@gmx.de>
Link: https://lore.kernel.org/r/20230523200709.236023-4-oswald.buddenhagen@gmx.deSigned-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 11ee59bd
...@@ -430,7 +430,7 @@ static int snd_emu10k1_efx_playback_prepare(struct snd_pcm_substream *substream) ...@@ -430,7 +430,7 @@ static int snd_emu10k1_efx_playback_prepare(struct snd_pcm_substream *substream)
struct snd_emu10k1_pcm *epcm = runtime->private_data; struct snd_emu10k1_pcm *epcm = runtime->private_data;
unsigned int start_addr; unsigned int start_addr;
unsigned int extra_size, channel_size; unsigned int extra_size, channel_size;
int i; unsigned int i;
start_addr = epcm->start_addr >> 1; // 16-bit voices start_addr = epcm->start_addr >> 1; // 16-bit voices
...@@ -441,7 +441,7 @@ static int snd_emu10k1_efx_playback_prepare(struct snd_pcm_substream *substream) ...@@ -441,7 +441,7 @@ static int snd_emu10k1_efx_playback_prepare(struct snd_pcm_substream *substream)
start_addr, start_addr + extra_size); start_addr, start_addr + extra_size);
epcm->ccca_start_addr = start_addr; epcm->ccca_start_addr = start_addr;
for (i = 0; i < NUM_EFX_PLAYBACK; i++) { for (i = 0; i < runtime->channels; i++) {
snd_emu10k1_pcm_init_voices(emu, epcm->voices[i], true, false, snd_emu10k1_pcm_init_voices(emu, epcm->voices[i], true, false,
start_addr, start_addr + channel_size, start_addr, start_addr + channel_size,
&emu->efx_pcm_mixer[i]); &emu->efx_pcm_mixer[i]);
...@@ -461,7 +461,7 @@ static const struct snd_pcm_hardware snd_emu10k1_efx_playback = ...@@ -461,7 +461,7 @@ static const struct snd_pcm_hardware snd_emu10k1_efx_playback =
.rates = SNDRV_PCM_RATE_48000, .rates = SNDRV_PCM_RATE_48000,
.rate_min = 48000, .rate_min = 48000,
.rate_max = 48000, .rate_max = 48000,
.channels_min = NUM_EFX_PLAYBACK, .channels_min = 1,
.channels_max = NUM_EFX_PLAYBACK, .channels_max = NUM_EFX_PLAYBACK,
.buffer_bytes_max = (128*1024), .buffer_bytes_max = (128*1024),
.period_bytes_max = (128*1024), .period_bytes_max = (128*1024),
...@@ -903,21 +903,21 @@ static int snd_emu10k1_efx_playback_trigger(struct snd_pcm_substream *substream, ...@@ -903,21 +903,21 @@ static int snd_emu10k1_efx_playback_trigger(struct snd_pcm_substream *substream,
case SNDRV_PCM_TRIGGER_PAUSE_RELEASE: case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
case SNDRV_PCM_TRIGGER_RESUME: case SNDRV_PCM_TRIGGER_RESUME:
mask = snd_emu10k1_efx_playback_voice_mask( mask = snd_emu10k1_efx_playback_voice_mask(
epcm, NUM_EFX_PLAYBACK); epcm, runtime->channels);
for (int i = 0; i < 10; i++) { for (int i = 0; i < 10; i++) {
// Note that the freeze is not interruptible, so we make no // Note that the freeze is not interruptible, so we make no
// effort to reset the bits outside the error handling here. // effort to reset the bits outside the error handling here.
snd_emu10k1_voice_set_loop_stop_multiple(emu, mask); snd_emu10k1_voice_set_loop_stop_multiple(emu, mask);
snd_emu10k1_efx_playback_freeze_voices( snd_emu10k1_efx_playback_freeze_voices(
emu, epcm, NUM_EFX_PLAYBACK); emu, epcm, runtime->channels);
snd_emu10k1_playback_prepare_voices( snd_emu10k1_playback_prepare_voices(
emu, epcm, true, false, NUM_EFX_PLAYBACK); emu, epcm, true, false, runtime->channels);
// It might seem to make more sense to unmute the voices only after // It might seem to make more sense to unmute the voices only after
// they have been started, to potentially avoid torturing the speakers // they have been started, to potentially avoid torturing the speakers
// if something goes wrong. However, we cannot unmute atomically, // if something goes wrong. However, we cannot unmute atomically,
// which means that we'd get some mild artifacts in the regular case. // which means that we'd get some mild artifacts in the regular case.
snd_emu10k1_efx_playback_unmute_voices(emu, epcm, NUM_EFX_PLAYBACK); snd_emu10k1_efx_playback_unmute_voices(emu, epcm, runtime->channels);
snd_emu10k1_playback_set_running(emu, epcm); snd_emu10k1_playback_set_running(emu, epcm);
result = snd_emu10k1_voice_clear_loop_stop_multiple_atomic(emu, mask); result = snd_emu10k1_voice_clear_loop_stop_multiple_atomic(emu, mask);
...@@ -928,7 +928,7 @@ static int snd_emu10k1_efx_playback_trigger(struct snd_pcm_substream *substream, ...@@ -928,7 +928,7 @@ static int snd_emu10k1_efx_playback_trigger(struct snd_pcm_substream *substream,
} }
snd_emu10k1_efx_playback_stop_voices( snd_emu10k1_efx_playback_stop_voices(
emu, epcm, NUM_EFX_PLAYBACK); emu, epcm, runtime->channels);
if (result != -EAGAIN) if (result != -EAGAIN)
break; break;
...@@ -941,7 +941,7 @@ static int snd_emu10k1_efx_playback_trigger(struct snd_pcm_substream *substream, ...@@ -941,7 +941,7 @@ static int snd_emu10k1_efx_playback_trigger(struct snd_pcm_substream *substream,
case SNDRV_PCM_TRIGGER_PAUSE_PUSH: case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
snd_emu10k1_playback_stop_voice(emu, epcm->extra); snd_emu10k1_playback_stop_voice(emu, epcm->extra);
snd_emu10k1_efx_playback_stop_voices( snd_emu10k1_efx_playback_stop_voices(
emu, epcm, NUM_EFX_PLAYBACK); emu, epcm, runtime->channels);
epcm->resume_pos = snd_emu10k1_playback_pointer(substream); epcm->resume_pos = snd_emu10k1_playback_pointer(substream);
break; break;
......
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