McASP0 interface with tlv320aic3101

Hi,

I am changes in sound audio-codec & mcasp-controller .but i dont understand Codec-clock-rate  bec i dont use external crystal use internal clock it is correct 24MHz.

      sound {
                compatible = "ti,da830-evm-audio";
                ti,model = "AM335x-EDGE";
                ti,audio-codec = <&tlv320aic310x>;
                ti,mcasp-controller = <&mcasp0>;
                ti,codec-clock-rate = <24000000>;
                ti,audio-routing =
                        "Headphone Jack",       "HPLOUT",
                        "Headphone Jack",       "HPROUT";
        };

but my sound card not register i m test on evm bec our hardware is not ready.

Thanks

Tejas

Hi Tejas,

tejas patil said:

but i dont understand Codec-clock-rate  bec i dont use external crystal use internal clock it is correct 24MHz.

      sound {
                compatible = "ti,da830-evm-audio";
                ti,model = "AM335x-EDGE";
                ti,audio-codec = <&tlv320aic310x>;
                ti,mcasp-controller = <&mcasp0>;
                ti,codec-clock-rate = <24000000>;
                ti,audio-routing =
                        "Headphone Jack",       "HPLOUT",
                        "Headphone Jack",       "HPROUT";
        };

but my sound card not register i m test on evm bec our hardware is not ready.

You need to configure the <codec>_hw_params function correctly, you have to use: 

ret = snd_soc_dai_set_sysclk(cpu_dai, 0, sysclk, SND_SOC_CLOCK_OUT);

Which tells the ALSA that internal 24MHz clock will be used. See: http://processors.wiki.ti.com/index.php/Sitara_Linux_Audio_DAC_Example#Creating_pcm5102a_hw_params_Function 

Best Regards,

Yordan

Hi VISWANATH,

We are design custom board  AM335x.
McASP0 interface with codec tlv320aic3106 chip. We are using linux kernel 3.14 & codec driver use sound/soc/codecs/tlv320aic31xx.c

my configuration in davinci-evm.c file :-

static struct snd_soc_dai_link evm_dai_tlv320aic3x = {
        .name           = "TLV320AIC3X",
        .stream_name    = "AIC3X",
        .codec_dai_name = "tlv320aic31xx-hifi",
        //.codec_dai_name       = "tlv320aic3x-hifi",
        .ops            = &evm_ops,
        .init           = evm_aic3x_init,
        .dai_fmt = SND_SOC_DAIFMT_I2S | SND_SOC_DAIFMT_CBS_CFS |
                   SND_SOC_DAIFMT_NB_NF,
};

& dts file :-

  sound {
                compatible = "ti,da830-evm-audio";
                ti,model = "AM335x-EDGE";
                ti,audio-codec = <&tlv320aic310x>;
                ti,mcasp-controller = <&mcasp0>;
                ti,codec-clock-rate = <24000000>;
                ti,audio-routing =
                        "Headphone Jack",       "HPLOUT",
                        "Headphone Jack",       "HPROUT";
        };

mcasp0_pins: mcasp0_pins {
                pinctrl-single,pins = <
                        0x120 (PIN_OUTPUT | MUX_MODE6)  /* K15 MII1_TXD2 mcasp0_ahclkx-MCLK */
                        0x12C (PIN_OUTPUT | MUX_MODE6)  /* K18 MII1_TX_CLK mcasp0_aclkx-BCLK */
                        0x130 (PIN_OUTPUT | MUX_MODE6) /* L18 MII1_RX_CLK mcasp0_fsx-WCLK */
                        0x118 (PIN_INPUT | MUX_MODE6)  /* J17 MII1_RX_DV mcasp0_aclkr-BCLK */
                        0x11C (PIN_INPUT | MUX_MODE6)   /* J18 MII1_TXD3 mcasp0_fsr-WCLK */
                        0x134 (PIN_OUTPUT| MUX_MODE6)  /* L17 MII1_RXD3 mcasp0_axr0-TXD */
                        0x138 (PIN_INPUT | MUX_MODE6)  /* L16 MII1_RXD2 mcasp0_axr1-RXD */
                >;
        };

        mcasp0_pins_sleep: mcasp0_pins_sleep {
                pinctrl-single,pins = <
                        0x120 (PIN_OUTPUT | MUX_MODE7)  /* K15 MII1_TXD2 mcasp0_ahclkx-MCLK */
                        0x12C (PIN_OUTPUT | MUX_MODE7)  /* K18 MII1_TX_CLK mcasp0_aclkx-BCLK */
                        0x130 (PIN_OUTPUT | MUX_MODE7) /* L18 MII1_RX_CLK mcasp0_fsx-WCLK */
                        0x118 (PIN_INPUT | MUX_MODE7)  /* J17 MII1_RX_DV mcasp0_aclkr-BCLK */
                        0x11C (PIN_INPUT | MUX_MODE7)   /* J18 MII1_TXD3 mcasp0_fsr-WCLK */
                        0x134 (PIN_OUTPUT | MUX_MODE7)  /* L17 MII1_RXD3 mcasp0_axr0-TXD */
                        0x138 (PIN_INPUT | MUX_MODE7)  /* L16 MII1_RXD2 mcasp0_axr1-RXD */
                >;
        };
&mcasp0 {
        pinctrl-names = "default", "sleep";
        pinctrl-0 = <&mcasp0_pins>;
        pinctrl-1 = <&mcasp0_pins_sleep>;

        status = "okay";

        op-mode = <0>;  /* MCASP_IIS_MODE */
        tdm-slots = <2>;
        /* 4 serializers */
        serial-dir = <  /* 0: INACTIVE, 1: TX, 2: RX  AXR0-TXD AXR1-RXD */
                1 2 0 0
        >;
        tx-num-evt = <32>;
        rx-num-evt = <32>;
};

&i2c0 {
        pinctrl-names = "default";
        pinctrl-0 = <&i2c0_pins>;

        status = "okay";
        clock-frequency = <400000>;

        tlv320aic310x: tlv320aic310x@18 {
                compatible = "ti,tlv320aic310x";
                reg = <0x18>;
                status = "okay";
                gpio-reset = <&gpio3 13 GPIO_ACTIVE_LOW>; /*GPIO3_13*/
                /* Regulators */
                AVDD-supply = <&vaux2_reg>;
                IOVDD-supply = <&vaux2_reg>;
                DRVDD-supply = <&vaux2_reg>;

                DVDD-supply = <&vaux2_reg1>;
        };
};

aplay test.wav
Playing WAVE 'test.wav' : Signed 16 bit Little Endian, Rate 48000 Hz, Stereo
[   47.821634] sysclk=24000000
[   47.839602] davinci-mcasp 48038000.mcasp: Inaccurate BCLK: 24000000 Hz / 16 != 1536000 Hz

any changes is need in davinci-evm.c file ?

MCLK is comes proper but BCLK & WCLK is not proper .

Thanks

Tejas

tejas patil said:

[ 113.142438] davinci-mcasp 48038000.mcasp: Inaccurate BCLK: 12000000 Hz / 8 != 1536000 Hz

You are trying to play a 48kHz 16-bit stereo file. This needs a 48 000 * 16 *2 = 1 536 000Hz bit clock. This clock cannot be derived from the internal master clock of 24MHz. You need an external master clock source of 1 536 000 * 16 = 24 576 000Hz.

Also, you should not have to write any code to make this work since the tlv320aic31xx codec driver already exists.  An entry like this will make everything work assuming you have your other codec node defined correct:

 sound {
    compatible = "simple-audio-card";
    simple-audio-card,format = "i2s";

    simple-audio-card,cpu {
      sound-dai = <&mcasp0>;
      system-clock-frequency = <24000000>;
      system-clock-direction = "out";
    };

    simple-audio-card,codec {
      sound-dai = <&tlv320aic310x>;      
      bitclock-master;
      frame-master;
    };
    
  };

Take a look at simple-card.c and 

http://lxr.free-electrons.com/source/Documentation/devicetree/bindings/sound/simple-card.txt?v=3.14

That is wrong. Also it shows that you are trying to write kernel/driver code without understanding how a device tree works! Do yourself a favor and at least understand how the "compatible" field works. ;-)

lxr.free-electrons.com/.../usage-model.txt

So do not change the compatible fields arbitrarily. It should look more like this:

sound {
compatible = "simple-audio-card";
simple-audio-card,format = "dsp_b";
simple-audio-card,routing =
"Headphone Jack", "HPL",
"Headphone Jack", "HPR";

simple-audio-card,cpu {
sound-dai = <&mcasp0>;
system-clock-frequency = <24000000>;
system-clock-direction = "out";
};

simple-audio-card,codec {
sound-dai = <&tlv320aic310x>;
bitclock-master;
frame-master;
};

};

You might need to also add

#sound-dai-cells = <0>;

to both your mcasp0 node and tlv320aic310x node.

Hi,

I am setting i2c is correct in dts file .

&i2c0 {
        pinctrl-names = "default";
        pinctrl-0 = <&i2c0_pins>;

        status = "okay";
        clock-frequency = <400000>;

        tlv320aic310x: tlv320aic310x@18 {
                #sound-dai-cells = <0>;
                compatible = "ti,tlv320aic310x";
                reg = <0x18>;
                status = "okay";
                gpio-reset = <&gpio3 13 GPIO_ACTIVE_LOW>; /*GPIO3_13*/
                /* Regulators */
                AVDD-supply = <&vaux2_reg>;
                IOVDD-supply = <&vaux2_reg>;
                DRVDD-supply = <&vaux2_reg>;

                DVDD-supply = <&vaux2_reg1>;
        };
};

how to solve above error ? i have no idea.

Thanks

Tejas

Have you taken the codec out of reset using gpio3.13?

Hi,

Yes,

gpio-reset = <&gpio3 13 GPIO_ACTIVE_LOW>; /*GPIO3_13*/

I am using gpio3.13 for reset tlv.

thanks

Tejas

That line does not do anything. It is just a mapping of the gpio pin to a string: "gpio-reset". In your codec's probe() function, you need to call devm_gpiod_get(&pdev->dev,"reset") to get access to that gpio pin and then manipulate the gpio pin to bring your codec out of reset as according to the TLV320AIC3101 datasheet.

I'm going to stop helping you here. Please spend more time reading the documentation and code. Good luck.