On 01/27/2014 05:51 PM, Lars-Peter Clausen wrote:

Hi,

I think you should try to get this in sync with the work Jean-Francois is
currently doing[1]. Having the HDMI transmitter register a CODEC device is
definitely the right approach, compared to the rather 'interesting'
constraints stuff you do in patch 4.

Oh, Jean-Francois's patches are now out. I'll take those into use, but I 
am afraid it still does not save me from the constraint stuff.

The most complex piece of code comes from limited sample-rate 
availability, which is coming Beaglebone-Black desing (available i2s 
master clock), not from the HDMI encoder itself. It does not help with 
the stereo only limitation either, because it comes from the one wire 
only audio data connection on BBB.

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).


Best regards,
Jyri