Re: [PATCH] ASoC: sgtl5000: add support for a fixed sysclk frequency

[Jakob Unterwurzacher]

> On 16. Oct 2017, at 12:51, Fabio Estevam <festevam@xxxxxxxxx> wrote:
> 
> Hi Jakob,
> 
> On Mon, Oct 16, 2017 at 7:46 AM, Jakob Unterwurzacher
> <jakob.unterwurzacher@xxxxxxxxxxxxxxxxxxxxx> wrote:
>> Hi Fabio, thatâs interesting.
>> 
>> Looks like the fsl,imx-audio-sgtl5000 sound card driver that is used on the imx51-babbage
>> does not try to adjust MCLK in response to the playback sample rate.
>> 
>> The simple-audio-card driver does that (it calls snd_soc_dai_set_sysclk whenever playback
>> starts).
> 
> Could you please test this with linux-next tree?
> 
> Looks like that the recent b0a7043d5c2c("ASoC: fsl_ssi: Caculate bit
> clock rate using slot number and width") would avoid the issue.
> 

Hi Fabio,

if this ( https://patchwork.kernel.org/patch/9952385/ ) is the patch, then it seems to be
specific to the Freescale I2S controller. We use the SGTL5000 together with a Rockchip CPU.

I think I see a way to fix this without adding a DTS property, Iâll send out a patch later this afternoon.

Best regards,
Jakob