All of lore.kernel.org
 help / color / mirror / Atom feed
From: Giuliano Pochini <pochini@shiny.it>
To: Olaf Hering <olh@suse.de>
Cc: alsa-devel@alsa-project.org
Subject: RE: snd-powermac produces only low quality dirty so
Date: Wed, 15 Dec 2004 11:10:02 +0100 (CET)	[thread overview]
Message-ID: <XFMail.20041215111002.pochini@shiny.it> (raw)
In-Reply-To: <20041214213549.GA22308@suse.de>

[-- Attachment #1: Type: text/plain, Size: 488 bytes --]


On 14-Dec-2004 Olaf Hering wrote:

> All 2.6 versions I tried so far produce only "dirty" low quality sound
> when playing mp3 with xmms with either the alsa or oss plugin.
> But it works as expected when I use dmasound_pmac and the liboss plugin.
>
> Any ideas how to debug it?

It seems that the chip is not happy with some period/buffer sizes.
Someone was trying to find the right constraints to set for those
parameters. Maybe the attached patch fixes it (untested).



--
Giuliano.

[-- Attachment #2: pmac.patch --]
[-- Type: text/plain, Size: 551 bytes --]

--- alsa-kernel/ppc/pmac.c__	Wed Dec 15 10:15:37 2004
+++ alsa-kernel/ppc/pmac.c	Wed Dec 15 10:37:15 2004
@@ -538,6 +538,11 @@
 	runtime->private_data = rec;
 	rec->substream = subs;
 
+	if ((i = snd_pcm_hw_constraint_pow2(runtime, 0, SNDRV_PCM_HW_PARAM_PERIOD_SIZE
)) < 0)
+		return i;
+	if ((i = snd_pcm_hw_constraint_pow2(runtime, 0, SNDRV_PCM_HW_PARAM_BUFFER_SIZE
)) < 0)
+		return i;
+
 #if 0 /* FIXME: still under development.. */
 	snd_pcm_hw_rule_add(runtime, 0, SNDRV_PCM_HW_PARAM_RATE,
 			    snd_pmac_hw_rule_rate, chip, rec->stream, -1);

  reply	other threads:[~2004-12-15 10:10 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-12-14 21:35 snd-powermac produces only low quality dirty sound Olaf Hering
2004-12-15 10:10 ` Giuliano Pochini [this message]
2004-12-15 10:32   ` Giuliano Pochini
2004-12-15 18:05   ` snd-powermac produces only low quality dirty so Olaf Hering
2004-12-15 18:32   ` danny
2004-12-15 18:42     ` Olaf Hering
2004-12-15 18:56     ` Olaf Hering

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=XFMail.20041215111002.pochini@shiny.it \
    --to=pochini@shiny.it \
    --cc=alsa-devel@alsa-project.org \
    --cc=olh@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.