From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 200A2ECAAD5 for ; Mon, 12 Sep 2022 12:02:48 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 4C3D216E7; Mon, 12 Sep 2022 14:01:56 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 4C3D216E7 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1662984166; bh=cOz4LC+vVivuX8Ts3PVkxuMj1HkgIMBETXMOxkFJ1pg=; h=Date:Subject:To:References:From:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=atZC2S+dT8K5WYMMeqCOVV3Gdlva9O3tz3lDjo2ixfbHXhf5YjTW5mhYXIQ59vljV 9EGDYo0ocxd1RzYRZtwY6BzmrwLOH0FOJDr7tcPdoHf+6SmOUQwzImr7+MF2vk8tFu 2f9BbLNpBLopgnAEzbTUCbROdBE77t3p5LPnxx9U= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 0538AF80567; Mon, 12 Sep 2022 14:00:42 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 64AD4F8056F; Mon, 12 Sep 2022 14:00:40 +0200 (CEST) Received: from mga06.intel.com (mga06b.intel.com [134.134.136.31]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 5DB85F8030F for ; Mon, 12 Sep 2022 14:00:32 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 5DB85F8030F Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="kyg2n/PE" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1662984034; x=1694520034; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=cOz4LC+vVivuX8Ts3PVkxuMj1HkgIMBETXMOxkFJ1pg=; b=kyg2n/PEDv1rAJqWlrWJXG3a0m9JbPEmYN+yk4F/fWYmhuGXTnXxmvKR VUGOiJmpilCNxfJKDeJP+VGhHWh+Sf/jyMWtE4A0KjVQf4sTWSheXSbIz iKpmpVsJ5kX6kQzS/GRTDhp0sGg+aE+Rra1GSEqaezSGZEbU7T0p1TLrs cb9joMwD9FZXRl/Juxlb8B3wQv+nByyyBhA6Axnxazw4CV8lx+3YEiBLU lc1ZuAiFfUqBDxk36xgN6lSAOBuko7CeoDlm0kWrwru68Wv5g41y5NmK+ vaGrKuFh0i8rkiEXdn4+zOH2HslUGB8vOhWNuUNtk+WNIT1ut1Ef6PE/D Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10467"; a="359570576" X-IronPort-AV: E=Sophos;i="5.93,310,1654585200"; d="scan'208";a="359570576" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Sep 2022 05:00:31 -0700 X-IronPort-AV: E=Sophos;i="5.93,310,1654585200"; d="scan'208";a="705126666" Received: from vtsymbal-mobl.ger.corp.intel.com (HELO [10.252.32.67]) ([10.252.32.67]) by fmsmga003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Sep 2022 05:00:26 -0700 Message-ID: <00ffdd42-2cb6-112e-3bb5-b9ef1105fc4f@linux.intel.com> Date: Mon, 12 Sep 2022 13:49:30 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Firefox/91.0 Thunderbird/91.11.0 Subject: Re: [PATCH v2] ASoC: Intel: sof_rt5682: Add support for jsl_rt5682_rt1019 Content-Language: en-US To: "Lu, Brent" , Sean Hong , "perex@perex.cz" , "tiwai@suse.com" References: <20220816075424.1245593-1-sean.hong@quanta.corp-partner.google.com> From: Pierre-Louis Bossart In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: "Rojewski, Cezary" , "kai.vehmanen@linux.intel.com" , "peter.ujfalusi@linux.intel.com" , "ranjani.sridharan@linux.intel.com" , "linux-kernel@vger.kernel.org" , "liam.r.girdwood@linux.intel.com" , "broonie@kernel.org" , "alsa-devel@alsa-project.org" , "yung-chuan.liao@linux.intel.com" X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" On 9/5/22 10:07, Lu, Brent wrote: > >> On 8/16/22 09:54, Sean Hong wrote: >>> This patch adds the driver data for rt5682 support jsl_rt5682_rt1019. >>> >>> Signed-off-by: Sean Hong >>> --- >>> sound/soc/intel/boards/sof_rt5682.c | 9 +++++++++ >>> sound/soc/intel/common/soc-acpi-intel-jsl-match.c | 12 ++++++++++++ >>> 2 files changed, 21 insertions(+) >>> >>> diff --git a/sound/soc/intel/boards/sof_rt5682.c >>> b/sound/soc/intel/boards/sof_rt5682.c >>> index 045965312..3a840f3a9 100644 >>> --- a/sound/soc/intel/boards/sof_rt5682.c >>> +++ b/sound/soc/intel/boards/sof_rt5682.c >>> @@ -1100,6 +1100,15 @@ static const struct platform_device_id >> board_ids[] = { >>> SOF_RT5682_SSP_AMP(1) | >>> SOF_RT5682_NUM_HDMIDEV(4)), >>> }, >>> + { >>> + .name = "jsl_rt5682_rt1019", >>> + .driver_data = (kernel_ulong_t)(SOF_RT5682_MCLK_EN | >>> + SOF_RT5682_MCLK_24MHZ | >> >> I see it's the same setting for all JSL devices but I am having doubts on this >> MCLK. Is this 24MHz value correct for JSL? It's derived from ICL so in theory >> the MCLK should be a multiple of 19.2MHz if the root frequency was the >> oscillator. >> >> Is this intentional? >> > > I've checked some internal wiki page and sof git log. It seems the first amp enabled > on JSL is max98373 running in TDM 4 slot 100fs so 24MHz MCLK seems to be a > reasonable choice. The commis is 5340225a7 ("topology: Add JSL da7219+max98373 support") > > +# SSP 1 (ID: 0) > +DAI_CONFIG(SSP, SPK_INDEX, 0, SPK_NAME, > + SSP_CONFIG(DSP_B, SSP_CLOCK(mclk, 24000000, codec_mclk_in), > + SSP_CLOCK(bclk, 4800000, codec_slave), > + SSP_CLOCK(fsync, 48000, codec_slave), > + SSP_TDM(4, 25, 3, 240), > + SSP_CONFIG_DATA(SSP, SPK_INDEX, 16))) > > I've also tested 19.2MHz on JSL boards and it also works. 1K sinetone playback sounds ok. > Since JSL boards are using 2.304 MHz bclk for max98360a and 3.072 MHz bclk for alc1015 now, > changing to 19.2 MHz mclk seems no benefit. Functionally there should be no difference, but the 24 MHz MCLK can only be created on JSL from the 96 MHz HDaudio PLL while the 19.2 MHz can be divided from the oscillator. I guess it's too late to optimize now that the topology files are in use so here is my Ack: Acked-by: Pierre-Louis Bossart but that's a clear miss in the topology reviews.