From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jarkko Nikula Subject: Re: [PATCH -next] ASoC: pandora: Add DAC regulator support Date: Sun, 7 Feb 2010 11:07:32 +0200 Message-ID: <20100207110732.9279b77a.jhnikula@gmail.com> References: <1265375612-9079-1-git-send-email-notasas@gmail.com> <1265378923.3177.11.camel@odin> <6ed0b2681002050629p1d38c19m19d1a124ddce307a@mail.gmail.com> <20100205172031.GA31523@rakim.wolfsonmicro.main> <6ed0b2681002060933x38009b77r237db131ab172fee@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ew0-f223.google.com (mail-ew0-f223.google.com [209.85.219.223]) by alsa0.perex.cz (Postfix) with ESMTP id 27699243EF for ; Sun, 7 Feb 2010 10:03:37 +0100 (CET) Received: by ewy23 with SMTP id 23so19254ewy.4 for ; Sun, 07 Feb 2010 01:03:35 -0800 (PST) In-Reply-To: <6ed0b2681002060933x38009b77r237db131ab172fee@mail.gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Grazvydas Ignotas Cc: alsa-devel@alsa-project.org, Mark Brown , Liam Girdwood List-Id: alsa-devel@alsa-project.org On Sat, 6 Feb 2010 19:33:08 +0200 Grazvydas Ignotas wrote: > It's on powerdown, there is a faint pop. It appears its the amp (final > part in the path), I tried to toggle it while playing /dev/zero > manually and it always pops on powerdown. On powerup there is a small > sound cutoff too, probably because of amp capacitors charging, but we > decided to leave it instead of halting every sound playing program > until the amp gets ready. > My side comment for the /dev/zero test. You may hear pops also if /dev/zero is played as unsigned 8-bit samples on signed DAC since then samples gets converted to maximum negative value. So there is a difference how the /dev/zero is played. First command below produces a pop because it's playing a pop and second produces a pop only if HW is producing it. aplay /dev/zero aplay -f S16_LE /dev/zero -- Jarkko