Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel <alsa-devel@alsa-project.org>
Subject: Re: [PATCH] Fix buffer position for ATI SB4x0
Date: Fri, 6 Jun 2008 11:52:32 -0300	[thread overview]
Message-ID: <20080606115232.411c13e3@gaivota> (raw)
In-Reply-To: <s5hod6pchui.wl%tiwai@suse.de>

On Thu, 29 May 2008 16:20:21 +0200
Takashi Iwai <tiwai@suse.de> wrote:

> At Thu, 29 May 2008 11:10:22 -0300,
> Mauro Carvalho Chehab wrote:
> > 
> > ATI SB4x0 doesn't need any fix at position.
> 
> It's not about the position fixing but whether to use the
> position-buffer.  The devices on the blacklist are the ones that have
> no position buffer.  So, it would fall into LPIB mode, and this list
> avoids it from the beginning.
> 
> > This patch solves the issue of receiving several clicks during capture on those
> > devices.
> > 
> > Tested with a Gateway Notebook MX-6453.
> 
> The click noise is often a different problem.  Did you already try
> the patch below?

The click seems associated to some residual samples inside the buffer. Here it
is a sample of the king of click noise I'm hearing here (captured from CD input entry):
	http://linuxtv.org/~mchehab/snd.ogg

I've started/stopped the CD some times (with cdplay). When volume goes to zero,
after an "echoed click", it disappears.

I've did some trials with position_fix changes, but nothing solved. So, my
previous patch were wrong. The click is somewhere else (still, I suspect that
it is linked to the buffer handling).

Cheers,
Mauro

  parent reply	other threads:[~2008-06-06 14:52 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-29 14:10 [PATCH] Fix buffer position for ATI SB4x0 Mauro Carvalho Chehab
2008-05-29 14:20 ` Takashi Iwai
2008-06-02 14:14   ` Mauro Carvalho Chehab
2008-06-06 14:52   ` Mauro Carvalho Chehab [this message]
2008-06-06 16:49     ` Takashi Iwai
2008-06-06 18:31       ` Mauro Carvalho Chehab
2008-06-07  8:27         ` Takashi Iwai
2008-06-09 13:17           ` Takashi Iwai
2008-06-09 20:29             ` Mauro Carvalho Chehab

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=20080606115232.411c13e3@gaivota \
    --to=mchehab@infradead.org \
    --cc=alsa-devel@alsa-project.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox