From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751899AbdIEHhx (ORCPT ); Tue, 5 Sep 2017 03:37:53 -0400 Received: from mail-out.m-online.net ([212.18.0.9]:34318 "EHLO mail-out.m-online.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751386AbdIEHhw (ORCPT ); Tue, 5 Sep 2017 03:37:52 -0400 X-Auth-Info: NIm/NrlAvkbULmwbFD9jbplqsHEVjYEYfNFbEyCSq+U= Subject: Re: [PATCH] sound: soc: fsl: Do not set DAI sysclk when it is equal to system freq To: Nicolin Chen Cc: Timur Tabi , Xiubo Li , Fabio Estevam , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , alsa-devel@alsa-project.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org References: <1504436701-20700-1-git-send-email-lukma@denx.de> <20170905050602.GA2774@Asurada-CZ80> From: =?UTF-8?Q?=c5=81ukasz_Majewski?= Organization: DENX Message-ID: Date: Tue, 5 Sep 2017 09:37:43 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <20170905050602.GA2774@Asurada-CZ80> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/05/2017 07:06 AM, Nicolin Chen wrote: > On Sun, Sep 03, 2017 at 01:05:01PM +0200, Lukasz Majewski wrote: >> The problem is visible in the following setup (on the imx6q): >> "simple-audio-card" -> ssi2 -> I2S + I2C -> codec >> >> The function call log (simple-card probe -> CONFIG_SND_SIMPLE_CARD): >> >> asoc_simple_card_init_dai() @ sound/soc/generic/simple-card-utils.c >> snd_soc_dai_set_sysclk() >> fsl_ssi_set_dai_sysclk() @ sound/soc/fsl/fsl_ssi.c >> >> The last call is changing the bit clock (BCLK) frequency to SSI's IP >> block clock (ipg = 66 MHz) [1]. > > I think a bigger question here is why the routine sets BCLK to 66MHz. Yes, exactly. In my case the bclk is set to ipg clock, which is the SSI IP block clock (ipg). > >> This is wrong, since IMX SSI block requires the I2S BCLK to be less >> than 1/5 of [1]. >> >> As a result the driver initialization passes without any errors, but the >> speaker-test test case breaks. >> >> This commit checks if the fsl_ssi_set_dai_sysclk() frequency passed is >> not equal to [1]. > > I don't feel it's quite comprehensive...what if it's being set to 67MHz. I think that this clock is not changing for the SoC. It should be 66 MHz fixed. > > Thanks > Nicolin > >> Signed-off-by: Lukasz Majewski >> --- >> sound/soc/fsl/fsl_ssi.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/sound/soc/fsl/fsl_ssi.c b/sound/soc/fsl/fsl_ssi.c >> index 173cb84..1186fa9 100644 >> --- a/sound/soc/fsl/fsl_ssi.c >> +++ b/sound/soc/fsl/fsl_ssi.c >> @@ -809,6 +809,8 @@ static int fsl_ssi_set_dai_sysclk(struct snd_soc_dai *cpu_dai, >> int clk_id, unsigned int freq, int dir) >> { >> struct fsl_ssi_private *ssi_private = snd_soc_dai_get_drvdata(cpu_dai); >> + if (clk_get_rate(ssi_private->clk) == freq) >> + return 0; >> >> ssi_private->bitclk_freq = freq; >> >> -- >> 2.1.4 >> > -- Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de