Re: [PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-22 Thread Sven Van Asbroeck
On Fri, Feb 22, 2019 at 4:36 PM Russell King - ARM Linux admin wrote: > There's actually two threads of conversation going, and I recently had > a reply from the maintainer of hdmi-codec suggesting a way forward - so > I've coded that up as the three RFC patches you should have just > received.

Re: [PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-22 Thread Russell King - ARM Linux admin
On Fri, Feb 22, 2019 at 04:18:33PM -0500, Sven Van Asbroeck wrote: > Russell, thank you so much for your patience, help and explanation, I really > appreciate it ! > > Yes, that would keep the current users in business without them having to > change anything. > > Of course, poor souls like

Re: [PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-22 Thread Sven Van Asbroeck
Russell, thank you so much for your patience, help and explanation, I really appreciate it ! On Fri, Feb 22, 2019 at 3:16 PM Russell King - ARM Linux admin wrote: > > A possible solution to that would be for hdmi-codec to default that to > zero unless it's been definitively provided by the ASoC

Re: [PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-22 Thread Russell King - ARM Linux admin
On Fri, Feb 22, 2019 at 04:27:35PM +, Russell King - ARM Linux admin wrote: > On Fri, Feb 22, 2019 at 10:47:35AM -0500, Sven Van Asbroeck wrote: > > The config structure which you need to fill in to init the audio has a > > "i2s qualifier" field, where you have the choice between 16 and 32

Re: [PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-22 Thread Russell King - ARM Linux admin
(Adding Mark, ASoC maintainer.) On Fri, Feb 22, 2019 at 10:47:35AM -0500, Sven Van Asbroeck wrote: > On Fri, Feb 22, 2019 at 8:21 AM Russell King - ARM Linux admin > wrote: > > > > On Thu, Feb 21, 2019 at 01:18:13PM -0500, Sven Van Asbroeck wrote: > > > > > [SNDRV_PCM_FORMAT_S24_LE] = { >

Re: [PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-22 Thread Sven Van Asbroeck
On Fri, Feb 22, 2019 at 8:21 AM Russell King - ARM Linux admin wrote: > > On Thu, Feb 21, 2019 at 01:18:13PM -0500, Sven Van Asbroeck wrote: > > > [SNDRV_PCM_FORMAT_S24_LE] = { > > .width = 24, .phys = 32, .le = 1, .signd = 1, > > .silence = {}, > > }, > >

Re: [PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-22 Thread Russell King - ARM Linux admin
On Thu, Feb 21, 2019 at 01:18:13PM -0500, Sven Van Asbroeck wrote: > My cpu dai driving the tda998x in master mode outputs > SNDRV_PCM_FORMAT_S24_LE, i2s left justified, two channels: > > [SNDRV_PCM_FORMAT_S24_LE] = { > .width = 24, .phys = 32, .le = 1, .signd = 1, >

[PATCH 1/2] drm/i2c: tda998x: adjust CTS_N audio pre-divider calculation

2019-02-21 Thread Sven Van Asbroeck
My cpu dai driving the tda998x in master mode outputs SNDRV_PCM_FORMAT_S24_LE, i2s left justified, two channels: [SNDRV_PCM_FORMAT_S24_LE] = { .width = 24, .phys = 32, .le = 1, .signd = 1, .silence = {}, }, This format has a sample width of 24