From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtpout-04.galae.net (smtpout-04.galae.net [185.171.202.116]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 617933101A0 for ; Wed, 18 Feb 2026 12:19:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.171.202.116 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771417147; cv=none; b=n/fsydHzxbiK7ZUsdGcrQO30Xpyk656FPc/yM66UqjSk8jXsvkY7c2+ABNTkuExVJ9PDfyc+TqNbOtvkmVNEOWqRDz6dW1NJSY48OjFKOCSU2qfGMvUZ1Xb/Uw4H3zRpZhVrenLMLLREtKMEj3d2p52lqsgy0xczLyaBT+6LP6Q= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771417147; c=relaxed/simple; bh=VWnqBmocNDaroG7fE5OS7c7etZQiW5vivxnVhjDJN1g=; h=Mime-Version:Content-Type:Date:Message-Id:Cc:To:From:Subject: References:In-Reply-To; b=WKQ8hbBpuXOQU35flwHahhMb37t674fgzrK6Fexrk7bBxCK2HTkPQeTZvttwQf4ivuep0kJnKBAWdCLSxGEKNB5y7lHIzrlISQARSKa8IYlEUJ3P9GfUyjCNaj7QYNEvYWerFvRB7iEeqb373FfsKISMAkxYAgZcvPQL3Ec9mX0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com; spf=pass smtp.mailfrom=bootlin.com; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b=xhCNSzyB; arc=none smtp.client-ip=185.171.202.116 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=bootlin.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bootlin.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=bootlin.com header.i=@bootlin.com header.b="xhCNSzyB" Received: from smtpout-01.galae.net (smtpout-01.galae.net [212.83.139.233]) by smtpout-04.galae.net (Postfix) with ESMTPS id 90F3CC111A3; Wed, 18 Feb 2026 12:19:16 +0000 (UTC) Received: from mail.galae.net (mail.galae.net [212.83.136.155]) by smtpout-01.galae.net (Postfix) with ESMTPS id A23D960738; Wed, 18 Feb 2026 12:19:04 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) by localhost (Mailerdaemon) with ESMTPSA id 224DF10368B83; Wed, 18 Feb 2026 13:19:01 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=dkim; t=1771417143; h=from:subject:date:message-id:to:cc:mime-version:content-type: content-transfer-encoding:in-reply-to:references; bh=G981jZtYq9+m6LGqFjQopp1Q4KFWAzKgyvqnHiUisc4=; b=xhCNSzyB7mBPbkbEi/35CRJeSC7kjISQzTti7ikeGonDDxTn39Zzm3dF7jAwRzizWzSGxP GSAFP35plXgFaVfcJpJmhFuh8Q4rkekH7uGM7xdabuynvx/KC+ClSMbN7LVT6kLKVpHT9d +HBGIRtztWn9pplPYXpb+aImCBPfb45cUheK8qFRzg18QaMYdgSiUgJqcRvGhFXjccBNnb fZF5TjysHDl+C8BQLDDdmMlfNTW5z5NJSVp0AxEBash0XkXip/x5gZm6+aXjkGiIwof8ZH UK0KlAo/42u4JCh07OVqXBI9wg/XVOfNZqD+AZeEDnzUQpJBjMbqmVXcQmlOtA== Precedence: bulk X-Mailing-List: linux-sound@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Wed, 18 Feb 2026 13:19:00 +0100 Message-Id: Cc: "Nicolas Frattaroli" , "Liam Girdwood" , "Mark Brown" , "Jaroslav Kysela" , "Takashi Iwai" , "Heiko Stuebner" , , , , , , To: "Luca Ceresoli" , "Detlev Casanova" , From: "Luca Ceresoli" Subject: Re: [REGRESSION] Re: [PATCH v1] ASoC: rockchip: i2s_tdm: Re-add the set_sysclk callback X-Mailer: aerc 0.20.1 References: <20250117163102.65807-1-detlev.casanova@collabora.com> In-Reply-To: X-Last-TLS-Session-Version: TLSv1.3 Hello Detlev, On Tue Feb 10, 2026 at 10:54 AM CET, Luca Ceresoli wrote: > Hello Detlev, > > On Fri Jan 17, 2025 at 5:31 PM CET, Detlev Casanova wrote: >> In commit >> 9e2ab4b18ebd ("ASoC: rockchip: i2s-tdm: Fix inaccurate sampling rates"), >> the set_sysclk callback was removed as considered unused as the mclk rat= e >> can be set in the hw_params callback. >> The difference between hw_params and set_sysclk is that the former is >> called with the audio sampling rate set in the params (e.g.: 48000 Hz) >> while the latter is called with a clock rate already computed with >> sampling_rate * mclk-fs (e.g.: 48000 * 256) >> >> For HDMI audio using the Rockchip I2S TDM driver, the mclk-fs value must >> be set to 128 instead of the default 256, and that value is set in the >> device tree at the machine driver level (like a simple-audio-card >> compatible node). >> Therefore, the i2s_tdm driver has no idea that another mclk-fs value can >> be configured and simply computes the mclk rate in the hw_params callbac= k >> with DEFAULT_MCLK_FS * params_rate(params), which is wrong for HDMI >> audio. >> >> Re-add the set_sysclk callback so that the mclk rate is computed by the >> machine driver which has the correct mclk-fs value set in its device tre= e >> node. > > I'm afraid I just found this commit breaks audio capture on the RK3308. > > Using 'arecord -Vmono -d 2 -c 8 -f S16_LE -r 96000 /dev/null' I get: > > rockchip-i2s-tdm ff320000.i2s: ASoC error (-22): at snd_soc_dai_hw_para= ms() on ff320000.i2s > ff320000.i2s-rk3308-hifi: ASoC error (-22): at __soc_pcm_hw_params() o= n ff320000.i2s-rk3308-hifi > > Tested on: > > * Radxa Rock Pi S > * Upstream kernel > * arm64 defconfig > > Tested kernel versions: > > * v6.12 works > * 5323186e2e8d (this commit) fails > * 5323186e2e8d^ works > * 21cfbeae7d7c (same patch on stable/linux-6.12.y) fails > * 21cfbeae7d7c^ works > * v6.19 fails > * v6.19 + 'git revert 5323186e2e8d' works > > I don't have more information at the moment. Let me add some more info to my previous report. The function failing is clk_set_rate() in rockchip_i2s_tdm_hw_params(). I added some logging to rockchip_i2s_tdm_hw_params() and here's the output: On v6.19-11566-g254edc893f3a (current Linux master) At boot: [ 3.324622] rockchip-i2s-tdm ff320000.i2s: The target mclk_tx freq i= s: 50176000 [ 3.324720] rockchip-i2s-tdm ff320000.i2s: using zero-initialized fl= at cache, this may cause unexpected behavior When launching arecord: [ 18.429389] rockchip_i2s_tdm_hw_params:653 i2s_tdm->is_master_mode = =3D 1 [ 18.430154] rockchip_i2s_tdm_hw_params:654 i2s_tdm->mclk_tx clk_i2s2= _8ch_tx [ 18.430815] rockchip_i2s_tdm_hw_params:655 i2s_tdm->mclk_rx clk_i2s2= _8ch_rx [ 18.431465] rockchip_i2s_tdm_hw_params:656 i2s_tdm->mclk_tx_freq 501= 76000 [ 18.432097] rockchip_i2s_tdm_hw_params:657 i2s_tdm->mclk_rx_freq 0 [ 18.432677] rockchip_i2s_tdm_hw_params:658 i2s_tdm->clk_trcm 0 [ 18.433230] rockchip_i2s_tdm_hw_params:659 substream->stream 1 [ 18.433858] rockchip_i2s_tdm_hw_params:678 mclk clk_i2s2_8ch_rx [ 18.434423] rockchip_i2s_tdm_hw_params:679 mclk_rate 0 [ 18.434977] rockchip_i2s_tdm_hw_params:682 clk_set_rate(clk_i2s2_8ch= _rx, 0) returned -22 [ 18.435743] rockchip-i2s-tdm ff320000.i2s: ASoC error (-22): at snd_= soc_dai_hw_params() on ff320000.i2s [ 18.436699] ff320000.i2s-rk3308-hifi: ASoC error (-22): at __soc_pc= m_hw_params() on ff320000.i2s-rk3308-hifi On v6.19-11566-g254edc893f3a + 'git revert 5323186e2e8d' At boot: [ 3.283853] rockchip-i2s-tdm ff320000.i2s: using zero-initialized fl= at cache, this may cause unexpected behavior When launching arecord: [ 11.952290] rockchip_i2s_tdm_hw_params:631 i2s_tdm->is_master_mode = =3D 1 [ 11.952955] rockchip_i2s_tdm_hw_params:632 i2s_tdm->mclk_tx clk_i2s2= _8ch_tx [ 11.953615] rockchip_i2s_tdm_hw_params:633 i2s_tdm->mclk_rx clk_i2s2= _8ch_rx [ 11.954355] rockchip_i2s_tdm_hw_params:636 i2s_tdm->clk_trcm 0 [ 11.954917] rockchip_i2s_tdm_hw_params:637 substream->stream 1 [ 11.955477] rockchip_i2s_tdm_hw_params:652 mclk clk_i2s2_8ch_rx [ 11.956048] rockchip_i2s_tdm_hw_params:653 params_rate(params) 96000 [ 11.956769] rockchip_i2s_tdm_hw_params:656 clk_set_rate(clk_i2s2_8ch= _rx, 24576000) returned 0 I cannot point to the right fix but the value 0 in i2s_tdm->mclk_rx_freq appears as a good clue, perhaps it has never been set. I hope this helps. Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com