Re: [alsa-devel] [PATCH RFC v3 0/8] Beaglebone-Black HDMI audio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/27/2014 08:06 PM, Jean-Francois Moine wrote:
On Mon, 27 Jan 2014 18:17:54 +0200
Jyri Sarha <jsarha@xxxxxx> wrote:

Support for S16_LE could maybe be added if the tda998x specific codec
would fiddle with CTS_N predivider-setting (K select) according to the
used sample width. But it appears Cobox plays all the sample formats
fine without this change, so the question is if playing around with
CTS_N predivider would break already working support there (something to
be tested).

Jyri,

Setting

	cts_n = CTS_N_M(3) | CTS_N_K(1);

instead of K(3) in the I2S sequence of tda998x_configure_audio() in the
tda998x driver works fine for me with S16_LE and S24_LE.

Does this solve your problem?


With that setting S16_LE works, but S32_LE does not. S24_LE does not really work at all on BBB, or it works but because the 8 most significant bits are zero the audio output is attenuated by 48 dB.

On the other hand S24_3LE does work, but then the problem is the clock rate support. On BBB the McASP IP is the bit clock master and provides the bit clock by simple divider from 24.576MHz oscillator. With 24 bit samples neither 44100 or 48000Hz sample rate can be supported (on the other hand 32kHz could be supported 24576000 % (24*2*32000) = 0).

The BBB HW price has squeezed so low that it can only implement the most basic use cases.

I would suggest to leave the CTS_N_K to the current setting (3), unless we can change the CTS_N_K on the fly according to the used sample format.

Best regards,
Jyri
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux