From: Enric Balletbo Serra <eballetbo@gmail.com>
To: Peter Ujfalusi <peter.ujfalusi@ti.com>
Cc: alsa-devel@alsa-project.org, Lars-Peter Clausen <lars@metafoo.de>,
Takashi Iwai <tiwai@suse.de>, Liam Girdwood <lgirdwood@gmail.com>,
Jyri Sarha <jsarha@ti.com>, Mark Brown <broonie@kernel.org>
Subject: Re: Audio play problem with tlv320aic3106 in mainline.
Date: Fri, 29 May 2015 11:24:50 +0200 [thread overview]
Message-ID: <CAFqH_511rLmDPEBLmXou6_iqNTj9t2uFNVvaW=+RmpumM9GmyA@mail.gmail.com> (raw)
In-Reply-To: <5568133A.6010006@ti.com>
Thanks for the answer Peter,
2015-05-29 9:20 GMT+02:00 Peter Ujfalusi <peter.ujfalusi@ti.com>:
> On 05/28/2015 02:36 PM, Enric Balletbo Serra wrote:
>> Dear all,
>>
>> Recently I sent some patches to add support for a new board based on
>> AM335x processor [1]. The board has the TLV320AIC3106 audio codec, the
>> MCLK signal is provided by an external oscillator of 12MHz. The
>> problem is that when I play a wav file for example I heard the song a
>> double speed. This is with v4.0 and current mainline.
>>
>> In dts file I defined "ti,codec-clock-rate = <12000000>;" which is the
>> correct value. Curiously when I changed the clock rate value to
>> 24000000 I heard the song at normal speed. AFAIK the correct value for
>> clock-rate is 12MHz so I suspect that something is wrong in the
>> driver, maybe it sets a wrong clock divider/multiplier ? Anyone can
>> point me in the right direction to fix the issue ? I see that
>> am335x-evm and am335x-pepper boards also uses this audio codec, did
>> anyone with those boards have the same problem?
>
> It might be really 24MHz.
> If the MCLK in your board is wired like am335x-evmsk or how BeagleBone's audio
> capes are wired (MCLK is coming from AM335x's EVENT_INTR0/1 pin as CLKOUT1/2)
> then 24MHz is correct. On the am335x-evm there is a dedicated crystal
> providing the 12MHz.
>
Well, in my board there is also a dedicated crystal providing the
12MHz, so I expect that the correct value for clock-frequency is 12MHz
not 24MHz.
clocks {
compatible = "simple-bus";
#address-cells = <1>;
#size-cells = <0>;
/* audio external oscillator */
tlv320aic3x_mclk: oscillator@0 {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <12000000>; /* 12MHz */
};
};
sound {
compatible = "ti,da830-evm-audio";
ti,model = "AM335x-SL50";
ti,audio-codec = <&audio_codec>;
ti,mcasp-controller = <&mcasp0>;
clocks = <&tlv320aic3x_mclk>;
clock-names = "mclk";
ti,audio-routing =
"Headphone Jack", "HPLOUT",
"Headphone Jack", "HPROUT",
"LINE1R", "Line In",
"LINE1L", "Line In";
};
For me it's really odd have to set the clock-frequency to 24MHz to make it work
clock-frequency = <24000000>;
For that reason I'm wondering if there is a bug in the driver, did
anyone test with an am335x-evm board ?
>>
>> [1] https://patchwork.kernel.org/patch/6480071/
>>
>> Thanks in advance,
>> Enric
>>
>
> --
> Péter
Thanks,
Enric
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2015-05-29 9:24 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-28 11:36 Audio play problem with tlv320aic3106 in mainline Enric Balletbo Serra
2015-05-29 7:20 ` Peter Ujfalusi
2015-05-29 9:24 ` Enric Balletbo Serra [this message]
2015-05-29 12:16 ` Peter Ujfalusi
2015-06-01 9:16 ` Enric Balletbo Serra
2015-06-01 12:10 ` Peter Ujfalusi
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='CAFqH_511rLmDPEBLmXou6_iqNTj9t2uFNVvaW=+RmpumM9GmyA@mail.gmail.com' \
--to=eballetbo@gmail.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=jsarha@ti.com \
--cc=lars@metafoo.de \
--cc=lgirdwood@gmail.com \
--cc=peter.ujfalusi@ti.com \
--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;
as well as URLs for NNTP newsgroup(s).