* [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed
@ 2009-07-22 11:45 Chaithrika U S
[not found] ` <1248263104-12142-1-git-send-email-chaithrika-l0cyMroinI0@public.gmane.org>
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Chaithrika U S @ 2009-07-22 11:45 UTC (permalink / raw)
To: alsa-devel; +Cc: khilman, Chaithrika U S, davinci-linux-open-source, broonie
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.
Tested on DM6467 EVM, playback tested on DM355 EVM.
Signed-off-by: Chaithrika U S <chaithrika@ti.com>
---
Applies to master branch of ALSA GIT tree at
http://git.kernel.org/?p=linux/kernel/git/tiwai/sound-2.6.git;a=summary
sound/soc/codecs/tlv320aic3x.c | 11 ++++++++++-
1 files changed, 10 insertions(+), 1 deletions(-)
diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c
index ab099f4..cb0d1bf 100644
--- a/sound/soc/codecs/tlv320aic3x.c
+++ b/sound/soc/codecs/tlv320aic3x.c
@@ -767,6 +767,7 @@ static int aic3x_hw_params(struct snd_pcm_substream *substream,
int codec_clk = 0, bypass_pll = 0, fsref, last_clk = 0;
u8 data, r, p, pll_q, pll_p = 1, pll_r = 1, pll_j = 1;
u16 pll_d = 1;
+ u8 reg;
/* select data word length */
data =
@@ -801,8 +802,16 @@ static int aic3x_hw_params(struct snd_pcm_substream *substream,
pll_q &= 0xf;
aic3x_write(codec, AIC3X_PLL_PROGA_REG, pll_q << PLLQ_SHIFT);
aic3x_write(codec, AIC3X_GPIOB_REG, CODEC_CLKIN_CLKDIV);
- } else
+ /* disable PLL if it is bypassed */
+ reg = aic3x_read_reg_cache(codec, AIC3X_PLL_PROGA_REG);
+ aic3x_write(codec, AIC3X_PLL_PROGA_REG, reg & ~PLL_ENABLE);
+
+ } else {
aic3x_write(codec, AIC3X_GPIOB_REG, CODEC_CLKIN_PLLDIV);
+ /* enable PLL when it is used */
+ reg = aic3x_read_reg_cache(codec, AIC3X_PLL_PROGA_REG);
+ aic3x_write(codec, AIC3X_PLL_PROGA_REG, reg | PLL_ENABLE);
+ }
/* Route Left DAC to left channel input and
* right DAC to right channel input */
--
1.5.6
^ permalink raw reply related [flat|nested] 6+ messages in thread[parent not found: <1248263104-12142-1-git-send-email-chaithrika-l0cyMroinI0@public.gmane.org>]
* RE: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed [not found] ` <1248263104-12142-1-git-send-email-chaithrika-l0cyMroinI0@public.gmane.org> @ 2009-07-22 12:39 ` Mani, Arun 2009-07-22 13:10 ` chaithrika 0 siblings, 1 reply; 6+ messages in thread From: Mani, Arun @ 2009-07-22 12:39 UTC (permalink / raw) To: Subrahmanya, Chaithrika, alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org, broonie-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org Hi Chaithrika, Do you know why it worked in dm644x. I don't see any specific path taken for the DM644x. Arun -----Original Message----- From: davinci-linux-open-source-bounces-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org [mailto:davinci-linux-open-source-bounces-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org] On Behalf Of Subrahmanya, Chaithrika Sent: Wednesday, July 22, 2009 7:45 AM To: alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org; broonie-yzvPICuk2ABfQMn51R22fg@public.gmane.orgsonmicro.com Subject: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed 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. Tested on DM6467 EVM, playback tested on DM355 EVM. Signed-off-by: Chaithrika U S <chaithrika-l0cyMroinI0@public.gmane.org> --- Applies to master branch of ALSA GIT tree at http://git.kernel.org/?p=linux/kernel/git/tiwai/sound-2.6.git;a=summary sound/soc/codecs/tlv320aic3x.c | 11 ++++++++++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c index ab099f4..cb0d1bf 100644 --- a/sound/soc/codecs/tlv320aic3x.c +++ b/sound/soc/codecs/tlv320aic3x.c @@ -767,6 +767,7 @@ static int aic3x_hw_params(struct snd_pcm_substream *substream, int codec_clk = 0, bypass_pll = 0, fsref, last_clk = 0; u8 data, r, p, pll_q, pll_p = 1, pll_r = 1, pll_j = 1; u16 pll_d = 1; + u8 reg; /* select data word length */ data = @@ -801,8 +802,16 @@ static int aic3x_hw_params(struct snd_pcm_substream *substream, pll_q &= 0xf; aic3x_write(codec, AIC3X_PLL_PROGA_REG, pll_q << PLLQ_SHIFT); aic3x_write(codec, AIC3X_GPIOB_REG, CODEC_CLKIN_CLKDIV); - } else + /* disable PLL if it is bypassed */ + reg = aic3x_read_reg_cache(codec, AIC3X_PLL_PROGA_REG); + aic3x_write(codec, AIC3X_PLL_PROGA_REG, reg & ~PLL_ENABLE); + + } else { aic3x_write(codec, AIC3X_GPIOB_REG, CODEC_CLKIN_PLLDIV); + /* enable PLL when it is used */ + reg = aic3x_read_reg_cache(codec, AIC3X_PLL_PROGA_REG); + aic3x_write(codec, AIC3X_PLL_PROGA_REG, reg | PLL_ENABLE); + } /* Route Left DAC to left channel input and * right DAC to right channel input */ -- 1.5.6 _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/@public.gmane.org http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed 2009-07-22 12:39 ` Mani, Arun @ 2009-07-22 13:10 ` chaithrika 0 siblings, 0 replies; 6+ messages in thread From: chaithrika @ 2009-07-22 13:10 UTC (permalink / raw) To: 'Mani, Arun', alsa-devel; +Cc: davinci-linux-open-source, broonie On Wed, Jul 22, 2009 at 18:09:58, Mani, Arun wrote: > Hi Chaithrika, > Do you know why it worked in dm644x. I don't see any specific path taken for the DM644x. > On DM644x EVM the MCLK frequency is different when compared to DM646x and DM355 EVM. Also, some sample frequencies bypass the PLL and will work fine. However for example in the case of 44100Hz sampling rate, the PLL should be used but was not enabled and the playback is slightly faster: a 60s audio file plays in about 56s. With this patch applied the playback seems to be correct. Regards, Chaithrika > Arun > > -----Original Message----- > From: davinci-linux-open-source-bounces@linux.davincidsp.com [mailto:davinci-linux-open-source-bounces@linux.davincidsp.com] On Behalf Of Subrahmanya, Chaithrika > Sent: Wednesday, July 22, 2009 7:45 AM > To: alsa-devel@alsa-project.org > Cc: davinci-linux-open-source@linux.davincidsp.com; broonie@opensource.wolfsonmicro.com > Subject: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed > > 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. > > Tested on DM6467 EVM, playback tested on DM355 EVM. > > Signed-off-by: Chaithrika U S <chaithrika@ti.com> > --- > Applies to master branch of ALSA GIT tree at > http://git.kernel.org/?p=linux/kernel/git/tiwai/sound-2.6.git;a=summary > > sound/soc/codecs/tlv320aic3x.c | 11 ++++++++++- > 1 files changed, 10 insertions(+), 1 deletions(-) > > diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c > index ab099f4..cb0d1bf 100644 > --- a/sound/soc/codecs/tlv320aic3x.c > +++ b/sound/soc/codecs/tlv320aic3x.c > @@ -767,6 +767,7 @@ static int aic3x_hw_params(struct snd_pcm_substream *substream, > int codec_clk = 0, bypass_pll = 0, fsref, last_clk = 0; > u8 data, r, p, pll_q, pll_p = 1, pll_r = 1, pll_j = 1; > u16 pll_d = 1; > + u8 reg; > > /* select data word length */ > data = > @@ -801,8 +802,16 @@ static int aic3x_hw_params(struct snd_pcm_substream *substream, > pll_q &= 0xf; > aic3x_write(codec, AIC3X_PLL_PROGA_REG, pll_q << PLLQ_SHIFT); > aic3x_write(codec, AIC3X_GPIOB_REG, CODEC_CLKIN_CLKDIV); > - } else > + /* disable PLL if it is bypassed */ > + reg = aic3x_read_reg_cache(codec, AIC3X_PLL_PROGA_REG); > + aic3x_write(codec, AIC3X_PLL_PROGA_REG, reg & ~PLL_ENABLE); > + > + } else { > aic3x_write(codec, AIC3X_GPIOB_REG, CODEC_CLKIN_PLLDIV); > + /* enable PLL when it is used */ > + reg = aic3x_read_reg_cache(codec, AIC3X_PLL_PROGA_REG); > + aic3x_write(codec, AIC3X_PLL_PROGA_REG, reg | PLL_ENABLE); > + } > > /* Route Left DAC to left channel input and > * right DAC to right channel input */ > -- > 1.5.6 > > _______________________________________________ > Davinci-linux-open-source mailing list > Davinci-linux-open-source@linux.davincidsp.com > http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed 2009-07-22 11:45 [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed Chaithrika U S [not found] ` <1248263104-12142-1-git-send-email-chaithrika-l0cyMroinI0@public.gmane.org> @ 2009-07-22 12:42 ` Mark Brown 2009-07-22 13:00 ` chaithrika 2009-07-23 7:22 ` Mark Brown 2 siblings, 1 reply; 6+ messages in thread From: Mark Brown @ 2009-07-22 12:42 UTC (permalink / raw) To: Chaithrika U S; +Cc: khilman, alsa-devel, davinci-linux-open-source 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? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed 2009-07-22 12:42 ` Mark Brown @ 2009-07-22 13:00 ` chaithrika 0 siblings, 0 replies; 6+ messages in thread From: chaithrika @ 2009-07-22 13:00 UTC (permalink / raw) To: 'Mark Brown'; +Cc: khilman, alsa-devel, davinci-linux-open-source 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed 2009-07-22 11:45 [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed Chaithrika U S [not found] ` <1248263104-12142-1-git-send-email-chaithrika-l0cyMroinI0@public.gmane.org> 2009-07-22 12:42 ` Mark Brown @ 2009-07-23 7:22 ` Mark Brown 2 siblings, 0 replies; 6+ messages in thread From: Mark Brown @ 2009-07-23 7:22 UTC (permalink / raw) To: Chaithrika U S; +Cc: khilman, alsa-devel, davinci-linux-open-source 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. Applied, thanks. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-07-23 7:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-07-22 11:45 [PATCH] ASoC: tlv320aic3x: Enable PLL when not bypassed Chaithrika U S
[not found] ` <1248263104-12142-1-git-send-email-chaithrika-l0cyMroinI0@public.gmane.org>
2009-07-22 12:39 ` Mani, Arun
2009-07-22 13:10 ` chaithrika
2009-07-22 12:42 ` Mark Brown
2009-07-22 13:00 ` chaithrika
2009-07-23 7:22 ` Mark Brown
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.