From mboxrd@z Thu Jan 1 00:00:00 1970 From: "chaithrika" Subject: Re: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed Date: Wed, 22 Jul 2009 18:30:09 +0530 Message-ID: <01ff01ca0acc$580a8110$081f8330$@com> References: <1248263104-12142-1-git-send-email-chaithrika@ti.com> <20090722124214.GA28188@rakim.wolfsonmicro.main> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from bear.ext.ti.com (bear.ext.ti.com [192.94.94.41]) by alsa0.perex.cz (Postfix) with ESMTP id 7B1F71038F6 for ; Wed, 22 Jul 2009 15:01:08 +0200 (CEST) In-Reply-To: <20090722124214.GA28188@rakim.wolfsonmicro.main> Content-Language: en-us 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: 'Mark Brown' Cc: khilman@deeprootsystems.com, alsa-devel@alsa-project.org, davinci-linux-open-source@linux.davincidsp.com List-Id: alsa-devel@alsa-project.org On Wed, Jul 22, 2009 at 18:12:14, Mark Brown wrote: > On Wed, Jul 22, 2009 at 07:45:04AM -0400, Chaithrika U S wrote: > > PLL was not being enabled when it was not bypassed. This patch > > enables the PLL when it is used. Additionally, it disables the PLL > > when it is bypassed. > > > > Without this patch, the audio on TI DM646x EVM and DM355 EVM > > does not work properly. The bit clocks and the frame sync signals > > from the codec are not correct and hence the playback/record are faster > > than usual for most sample rates. The reason for this was that the PLL > > was not enabled when it was not bypassed. > > Looks reasonable though I've no knowledge of the CODEC. Sounds like > this is needed in 2.6.31? > Yes, it is needed in 2.6.31. Regards, Chaithrika