diff options
author | Takashi Iwai <tiwai@suse.de> | 2009-11-26 06:50:01 -0500 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2009-11-27 04:12:40 -0500 |
commit | 66b6cfacfc5aa2fda37b0d40cd54931ca5ef8cd7 (patch) | |
tree | 44dc65e1a1f38f7265cc060ccc4fa29d29b751cd /sound/core | |
parent | 9eb4a06788a598573c751af1a7e46639afc89513 (diff) |
ALSA: pcm - fix page conversion on non-coherent MIPS arch
The non-coherent MIPS arch doesn't give the correct address by a simple
virt_to_page() for pages allocated via dma_alloc_coherent().
Original patch by Wu Zhangjin <wuzj@lemote.com>.
[Ralf mentioned: "The origins of this patch go back far further.
The oldest patch I could find which is a superset of this was written
by Atsushi Nemoto and various incarnations of it have been sumitted
to and reject by me a number of times through the years."]
A proper check of the buffer allocation type was added to avoid the
wrong conversion.
Note that this doesn't fix perfectly: the pages should be marked with
proper pgprot value. This will be done in a future implementation like
the conversion to dma_mmap_coherent().
Acked-by: Ralf Baechle <ralf@linux-mips.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/core')
-rw-r--r-- | sound/core/pcm_native.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c index c906be26c312..e48c5f618578 100644 --- a/sound/core/pcm_native.c +++ b/sound/core/pcm_native.c | |||
@@ -3066,6 +3066,10 @@ static inline struct page * | |||
3066 | snd_pcm_default_page_ops(struct snd_pcm_substream *substream, unsigned long ofs) | 3066 | snd_pcm_default_page_ops(struct snd_pcm_substream *substream, unsigned long ofs) |
3067 | { | 3067 | { |
3068 | void *vaddr = substream->runtime->dma_area + ofs; | 3068 | void *vaddr = substream->runtime->dma_area + ofs; |
3069 | #if defined(CONFIG_MIPS) && defined(CONFIG_DMA_NONCOHERENT) | ||
3070 | if (substream->dma_buffer.dev.type == SNDRV_DMA_TYPE_DEV) | ||
3071 | return virt_to_page(CAC_ADDR(vaddr)); | ||
3072 | #endif | ||
3069 | return virt_to_page(vaddr); | 3073 | return virt_to_page(vaddr); |
3070 | } | 3074 | } |
3071 | 3075 | ||