From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Francois Moine Subject: Re: [alsa-devel] [PATCH RFC v3 0/8] Beaglebone-Black HDMI audio Date: Mon, 27 Jan 2014 19:06:41 +0100 Message-ID: <20140127190641.273890c6@armhf> References: <52E6806A.2090109@metafoo.de> <52E686B2.9060806@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <52E686B2.9060806@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org To: Jyri Sarha , Russell King - ARM Linux Cc: alsa-devel@alsa-project.org, dri-devel@lists.freedesktop.org, peter.ujfalusi@ti.com, broonie@kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org List-Id: alsa-devel@alsa-project.org T24gTW9uLCAyNyBKYW4gMjAxNCAxODoxNzo1NCArMDIwMApKeXJpIFNhcmhhIDxqc2FyaGFAdGku Y29tPiB3cm90ZToKCj4gU3VwcG9ydCBmb3IgUzE2X0xFIGNvdWxkIG1heWJlIGJlIGFkZGVkIGlm IHRoZSB0ZGE5OTh4IHNwZWNpZmljIGNvZGVjIAo+IHdvdWxkIGZpZGRsZSB3aXRoIENUU19OIHBy ZWRpdmlkZXItc2V0dGluZyAoSyBzZWxlY3QpIGFjY29yZGluZyB0byB0aGUgCj4gdXNlZCBzYW1w bGUgd2lkdGguIEJ1dCBpdCBhcHBlYXJzIENvYm94IHBsYXlzIGFsbCB0aGUgc2FtcGxlIGZvcm1h dHMgCj4gZmluZSB3aXRob3V0IHRoaXMgY2hhbmdlLCBzbyB0aGUgcXVlc3Rpb24gaXMgaWYgcGxh eWluZyBhcm91bmQgd2l0aCAKPiBDVFNfTiBwcmVkaXZpZGVyIHdvdWxkIGJyZWFrIGFscmVhZHkg d29ya2luZyBzdXBwb3J0IHRoZXJlIChzb21ldGhpbmcgdG8gCj4gYmUgdGVzdGVkKS4KCkp5cmks CgpTZXR0aW5nCgoJY3RzX24gPSBDVFNfTl9NKDMpIHwgQ1RTX05fSygxKTsKCmluc3RlYWQgb2Yg SygzKSBpbiB0aGUgSTJTIHNlcXVlbmNlIG9mIHRkYTk5OHhfY29uZmlndXJlX2F1ZGlvKCkgaW4g dGhlCnRkYTk5OHggZHJpdmVyIHdvcmtzIGZpbmUgZm9yIG1lIHdpdGggUzE2X0xFIGFuZCBTMjRf TEUuCgpEb2VzIHRoaXMgc29sdmUgeW91ciBwcm9ibGVtPwoKUnVzc2VsbCwgZG8geW91IHNlZSBh bnkgcHJvYmxlbSBhYm91dCB0aGlzIGNoYW5nZT8KCi0tIApLZW4gYXIgYydoZW50YcOxCXwJICAg ICAgKiogQnJlaXpoIGhhIExpbnV4IGF0YXYhICoqCkplZgkJfAkJaHR0cDovL21vaW5lamYuZnJl ZS5mci8KX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJp LWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHA6 Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 From: moinejf@free.fr (Jean-Francois Moine) Date: Mon, 27 Jan 2014 19:06:41 +0100 Subject: [alsa-devel] [PATCH RFC v3 0/8] Beaglebone-Black HDMI audio In-Reply-To: <52E686B2.9060806@ti.com> References: <52E6806A.2090109@metafoo.de> <52E686B2.9060806@ti.com> Message-ID: <20140127190641.273890c6@armhf> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, 27 Jan 2014 18:17:54 +0200 Jyri Sarha 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? Russell, do you see any problem about this change? -- Ken ar c'henta? | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/