TLV320AIC3104: Clock related issue

Kokila Karuppusamy

Part Number: TLV320AIC3104

Tool/software:

Hello Team,

We are integrating TLV320aic3104 audio codec to our processor. We are facing this error

01-01 12:14:56.427 4157 4157 E aic3x_hw_params(): unable to setup PLL
01-01 12:14:56.427 4157 4157 E tlv320aic3x 29-0018: ASoC: error at snd_soc_dai_hw_params on tlv320aic3x-hifi: -22

and I see that sysclk value is 0. and also I see that .set_sysclk = aic3x_set_dai_sysclk, This function is not called. At which stage this function will be called?

And we want the value of the register 102 as 0xA2. And i see this register is modified in the above function only. Could you pls confirm this?

Also If i want to use bclk and not mclk is there any specific changes we need to do in DTS?

Thanks & Regards,

Kokila K

23 days ago

0 Mir Jeffres 22 days ago

TI__Genius 12695 points

Hi,

I will get back to you on Tuesday about this, sorry about the delay.

0 Mir Jeffres 18 days ago in reply to Mir Jeffres

TI__Genius 12695 points

Hi Kokila,

Hi again. I believe it will be called at runtime, and if not, it would be when you set the clock in your DTS file. Can you provide your DTS and let's see where we need to define the MCLK/BCLK rate. I would think it is in the <clocks> section.

https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/Documentation/devicetree/bindings/sound/ti,tlv320aic3x.yaml

Best,
Mir

0 Kokila Karuppusamy 18 days ago in reply to Mir Jeffres

Prodigy 10 points

Hi Mir,

This is my DTS entry for the TLV. and for the clocks param what is the expected value we need to give?

tlv320aic320: tlv320aic320@18 {
                compatible = "ti,tlv320aic3104";
                reg = <0x18>;
                #sound-dai-cells = <0>;

                ai3x-micbias-vg = <3>;
                //reset-gpios = <&io_expander 11 GPIO_ACTIVE_LOW>;
                AVDD-supply = <&codec_3v3_reg>;
                IOVDD-supply = <&codec_3v3_reg>;
                DRVDD-supply = <&codec_1v8_reg>;
                DVDD-supply = <&codec_1v8_reg>;
        };

0 Mir Jeffres 17 days ago in reply to Kokila Karuppusamy

TI__Genius 12695 points

Hi,

I found this old e2e talking about the same problem: https://e2e.ti.com/support/audio-group/audio/f/audio-forum/702805/rtos-tlv320aic3105-tlv320aic3105-codec-showing-system-clock-sysclk-as-0

They suggest adding in the line aic3x->sysclk = <value> in the driver itself... but according to other forum posts, it may be expecting your SoC to change the MCLK rate. You can check out this forum for some suggestions for the DTS or changing the driver to a hard coded MCLK:

https://community.nxp.com/t5/i-MX-Processors/i-MX8QXP-SAI-MCLK-frequency/m-p/1097084#M160628

https://www.programmersought.com/article/40346906740/

Sorry I am not more help, I am not a Linux DTS expert.

Best,
Mir

0 Kokila Karuppusamy 5 days ago in reply to Mir Jeffres

Prodigy 10 points

Hello Mir,

Now my PLL error is solved, and Iam able to capture, but after capturing when i try to check the wav file, there is no valid data.

exynosauto9_easdk_android_vm:/ # i2cdump -y -f 29 0x18
     0  1  2  3  4  5  6  7  8  9  a  b  c  d  e  f    0123456789abcdef
00: 00 00 00 91 10 0c dc 0a 00 00 00 01 00 00 00 20    ...?????...?...
10: 20 ff ff 04 78 78 04 78 78 06 00 fe 00 00 fe 00     ???xx?xx?.?..?.
20: 00 00 00 00 cc 00 00 00 40 00 00 af af 00 2f af    ....?...@..??./?
30: 00 00 00 0c 00 2f af 00 00 00 0c 00 00 00 00 2f    ...?./?...?..../
40: af 0c 00 00 00 00 2f af 0c 00 00 00 00 00 00 00    ??..../??.......
50: 00 2f af 00 00 00 08 00 00 00 00 2f af 08 00 00    ./?...?..../??..
60: 00 00 00 00 00 00 a2 00 00 00 00 00 00 00 00 00    ......?.........
70: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00    ................
80: 00 00 00 91 10 0c dc 0a 00 00 00 01 00 00 00 20    ...?????...?...
90: 20 ff ff 04 78 78 04 78 78 06 00 fe 00 00 fe 00     ???xx?xx?.?..?.
a0: 00 00 00 00 cc 00 00 00 40 00 00 af af 00 2f af    ....?...@..??./?
b0: 00 00 00 0c 00 2f af 00 00 00 0c 00 00 00 00 2f    ...?./?...?..../
c0: af 0c 00 00 00 00 2f af 0c 00 00 00 00 00 00 00    ??..../??.......
d0: 00 2f af 00 00 00 08 00 00 00 00 2f af 08 00 00    ./?...?..../??..
e0: 00 00 00 00 00 00 a2 00 00 00 00 00 00 00 00 00    ......?.........
f0: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00    ................
exynosauto9_easdk_android_vm:/ #

Also, Let me know for routing of the audio, should we set any value for mixer?

Please find the logs while try to capture the audio.

exynosauto9_easdk_android_vm:/ # tinycap /mnt/test.wav -d 16 -c 2 -r 48000 -b 16
Capturing sample: 2 ch, 48000 hz, 16 bit
Captured 417792 frames
exynosauto9_easdk_android_vm:/ #

0 Mir Jeffres 4 days ago in reply to Kokila Karuppusamy

TI__Genius 12695 points

Hi,

Can you share what you changed to fix your PLL/clocks? Just for future reference.

And yes, you need to add dapm_routes to the DTS file. You can find the options by searching "snd_soc_dapm_route" in the driver aic3x.c file: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/sound/soc/codecs/tlv320aic3x.c And the codec pin names are in the yaml help file for the driver: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/Documentation/devicetree/bindings/sound/ti,tlv320aic3x.yaml

There are common values in the "intercon" section but the AIC3104 has its own few options as well. It depends which pins you are trying to use.

Best,
Mir

0 Kokila Karuppusamy 4 days ago in reply to Mir Jeffres

Prodigy 10 points

Hi Mir,

For PLL issue, I tried to set the sysclk from the master, This automatically calls the set_sysclk function, which resolves PLL error.

 ret = snd_soc_dai_set_sysclk(codec_dai, 2, 24080000, SND_SOC_CLOCK_OUT);
        if (ret) {
            dev_err(card->dev, "failed to set tlv320aic31xx sysclk!\n");
            return ret;
        }

Now, Pls help me with the Mixer settings, what are the value we need to set for the Mic capture.?

0 Kokila Karuppusamy 4 days ago in reply to Kokila Karuppusamy

Prodigy 10 points

I tried to capture the audio, The files is captured there is no valid audio data, I think iam missing some of the mixer routing. Please help me with the Mixer configuration

0 Mir Jeffres 3 days ago in reply to Kokila Karuppusamy

TI__Genius 12695 points

Hi Kokila,

You will want to include some routing in your dts file, like this:

         simple-audio-card,widgets =
                "Microphone", "Microphone Jack",
                "Microphone", "Internal Microphone",
                "Headphone", "Headphone Jack",
                "Line", "Line Out",
                "Speaker", "Internal Speaker";
            simple-audio-card,routing =
                "Line Out", "HPLCOM",
                "Line Out", "HPRCOM",
                "Headphone Jack", "HPLOUT",
                "Headphone Jack", "HPROUT",
                "Microphone Jack", "MIC2L",
                "Internal Microphone", "MIC1L",
                "Internal Speaker", "LLOUT",
                "Internal Speaker", "RLOUT";

-Mir

Audio

Audio forum

TLV320AIC3104: Clock related issue