diff options
author | Takashi Iwai <tiwai@suse.de> | 2008-01-08 12:05:26 -0500 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2008-01-31 11:29:47 -0500 |
commit | d948035a928400ae127c873fbf771389bee18949 (patch) | |
tree | 1f8c98f21627e0ae1747c471dfbc3148923d80dd /sound/core/oss | |
parent | fa5717f2099aadb2083d5df4d19af8f9685fa03e (diff) |
[ALSA] Remove PCM xfer_align sw params
The xfer_align sw_params parameter has never been used in a sane manner,
and no one understands what this does exactly. The current
implementation looks also buggy because it allows write of shorter size
than xfer_align. So, if you do partial writes, the write isn't actually
aligned at all.
Removing this parameter will make some pcm_lib_* code more readable
(and less buggy).
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
Diffstat (limited to 'sound/core/oss')
-rw-r--r-- | sound/core/oss/pcm_oss.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/sound/core/oss/pcm_oss.c b/sound/core/oss/pcm_oss.c index d0c4ceb9f0b4..be089ccd736d 100644 --- a/sound/core/oss/pcm_oss.c +++ b/sound/core/oss/pcm_oss.c | |||
@@ -988,7 +988,6 @@ static int snd_pcm_oss_change_params(struct snd_pcm_substream *substream) | |||
988 | sw_params->sleep_min = 0; | 988 | sw_params->sleep_min = 0; |
989 | sw_params->avail_min = substream->stream == SNDRV_PCM_STREAM_PLAYBACK ? | 989 | sw_params->avail_min = substream->stream == SNDRV_PCM_STREAM_PLAYBACK ? |
990 | 1 : runtime->period_size; | 990 | 1 : runtime->period_size; |
991 | sw_params->xfer_align = 1; | ||
992 | if (atomic_read(&substream->mmap_count) || | 991 | if (atomic_read(&substream->mmap_count) || |
993 | substream->oss.setup.nosilence) { | 992 | substream->oss.setup.nosilence) { |
994 | sw_params->silence_threshold = 0; | 993 | sw_params->silence_threshold = 0; |