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: 33+ 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:50 ` Gerhard Pircher
2008-05-14 21:01 ` Gerhard Pircher [this message]
2008-05-15 5:42 ` Takashi Iwai
2008-05-19 17:23 ` Giuseppe Coviello
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$(echo .)net \
--cc=alsa-devel@alsa-project$(echo .)org \
--cc=cjg@cruxppc$(echo .)org \
--cc=linuxppc-dev@ozlabs$(echo .)org \
--cc=tiwai@suse$(echo .)de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox