From: "Gerhard Pircher" <gerhard_pircher@gmx.net>
To: Takashi Iwai <tiwai@suse.de>
Cc: linuxppc-dev@ozlabs.org, alsa-devel@alsa-project.org, cjg@cruxppc.org
Subject: Re: ALSA fixes for non-coherent archs (Re: [PATCH] Sam440ep support)
Date: Wed, 14 May 2008 23:01:10 +0200 [thread overview]
Message-ID: <20080514210110.155050@gmx.net> (raw)
In-Reply-To: <s5hr6c5ax6a.wl%tiwai@suse.de>
-------- Original-Nachricht --------
> Datum: Wed, 14 May 2008 14:26:53 +0200
> Von: Takashi Iwai <tiwai@suse.de>
> An: "Gerhard Pircher" <gerhard_pircher@gmx.net>
> CC: benh@kernel.crashing.org, galak@kernel.crashing.org, cjg@cruxppc.org, linuxppc-dev@ozlabs.org, alsa-devel@alsa-project.org
> Betreff: ALSA fixes for non-coherent archs (Re: [PATCH] Sam440ep support)
> At Tue, 06 May 2008 11:16:22 +0200,
> Gerhard Pircher wrote:
> >
> > Naturally I can do some tests, if you or Takashi come up with a new
> > patch.
>
> OK, here is another patch for testing. Since I lost my old patch
> somewhere (and it's not worth to dig the archive), I wrote it up
> quickly from scratch. This version should cover both SG and non-SG
> buffers. It's against the latest git tree.
>
> The patch adds a hackish verison of dma_mmap_coherent() for some
> architectures in sound/core/pcm_native.c. I'm not sure whether this
> works. I just tested it on X86. It'd be appreciated if someone can
> test this.
>
> Also, this disables HDSPM driver for non-X86/IA64 since the driver has
> own copy and silence methods that are incompatible with the new
> SG-buffer data.
>
>
> And, yes, I know we need to clean up huge messes in ALSA memory
> handling routines. But, let's fix obvious bugs before starting a big
> rewrite...
Okay, it seems to work fine so far. No lock-ups or kernel oopses. Tested
with kernel v2.6.26-rc2 and a Soundblaster 5.1/EMU10k1 device (which was
not working with the old patch) on my non cache coherent G4 system.
Thanks again!
Gerhard
--
Psssst! Schon vom neuen GMX MultiMessenger gehört?
Der kann`s mit allen: http://www.gmx.net/de/go/multimessenger
next prev parent reply other threads:[~2008-05-14 21:01 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-05 18:23 [PATCH] Sam440ep support Giuseppe Coviello
2008-05-05 18:27 ` Kumar Gala
2008-05-05 19:50 ` Gerhard Pircher
2008-05-05 23:44 ` Benjamin Herrenschmidt
2008-05-06 7:51 ` Gerhard Pircher
2008-05-06 8:48 ` Benjamin Herrenschmidt
2008-05-06 9:16 ` Gerhard Pircher
2008-05-06 10:12 ` Benjamin Herrenschmidt
2008-05-06 11:14 ` Takashi Iwai
2008-05-06 11:25 ` Benjamin Herrenschmidt
2008-05-06 11:27 ` Benjamin Herrenschmidt
2008-05-06 11:31 ` Takashi Iwai
2008-05-06 11:34 ` Benjamin Herrenschmidt
2008-05-14 12:26 ` ALSA fixes for non-coherent archs (Re: [PATCH] Sam440ep support) Takashi Iwai
2008-05-14 12:26 ` Takashi Iwai
2008-05-14 12:50 ` Gerhard Pircher
2008-05-14 21:01 ` Gerhard Pircher [this message]
2008-05-15 5:42 ` Takashi Iwai
2008-05-15 5:42 ` Takashi Iwai
2008-05-19 17:23 ` Giuseppe Coviello
2008-05-20 12:48 ` Takashi Iwai
2008-05-20 12:48 ` Takashi Iwai
2008-05-05 23:38 ` [PATCH] Sam440ep support Benjamin Herrenschmidt
2008-05-05 18:54 ` Josh Boyer
2008-05-05 23:40 ` Benjamin Herrenschmidt
2008-05-06 0:09 ` Josh Boyer
2008-05-05 23:36 ` Benjamin Herrenschmidt
2008-05-06 4:18 ` Sean MacLennan
2008-05-06 16:37 ` Giuseppe Coviello
2008-05-09 15:53 ` Giuseppe Coviello
2008-05-09 20:26 ` Josh Boyer
2008-05-19 12:47 ` Josh Boyer
2008-05-19 15:20 ` Giuseppe Coviello
2008-05-20 12:50 ` Josh Boyer
2008-05-20 13:34 ` Josh Boyer
2008-05-22 17:51 ` Giuseppe Coviello
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080514210110.155050@gmx.net \
--to=gerhard_pircher@gmx.net \
--cc=alsa-devel@alsa-project.org \
--cc=cjg@cruxppc.org \
--cc=linuxppc-dev@ozlabs.org \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.