From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5061134C826 for ; Wed, 15 Apr 2026 11:06:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776251218; cv=none; b=ixwSLquswVLKQ32RKBRSBIlaHmIjBPD/IcLGlEUrVJz2kYQLnvqc6teFBDjH68zMeBpLoXLIRet4svduQwZbmq2nnktzF5aC38Z8ADVRuQZC+XiGgag/tCycpZhUUT5TIS/Uay7XoboAKGwnneLY3RcWwaX2HL4A/3aJ3DOp1vA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1776251218; c=relaxed/simple; bh=LDI3mtUhr2Rd5w5ikgiCoRnemB42IOlavCohdIScs2s=; h=Mime-Version:Content-Type:Date:Message-Id:To:Cc:Subject:From: References:In-Reply-To; b=VvjysYNwNP55is0rwBHSSxqmTatoMnbsuUb0xembDxaRlHxKo4JLFcaY1RiZXJvzWWWEf4VQaTYLV4YQbejO7+MMSQGuJj3IrcTxEFddiew7v6X7+lg/SKnD4bgtNU6pugVQn6rwZuC/XdYommn3x7jcF8gCWe6Jqr80KCZy8c0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=FC1dNizW; arc=none smtp.client-ip=209.85.221.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="FC1dNizW" Received: by mail-wr1-f46.google.com with SMTP id ffacd0b85a97d-43d0deb7ad5so5235759f8f.2 for ; Wed, 15 Apr 2026 04:06:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1776251215; x=1776856015; darn=vger.kernel.org; h=in-reply-to:references:from:subject:cc:to:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=Rw116FfYB2wR32SWs/TCFerpwP/9uG3dqe/oInwd9Xs=; b=FC1dNizWonp4QpzI2N3Rq2W2xJ4iMmgVfWaMZPlRlepTj+O46bHcZeMyOvzm+f4p5B 0uL6aFM3ptRN6stlO+7R8SL3cN4C1leb1oZKGUZqvWTFFrL+9ofe30xrw7dS1bEepTjQ J0ADmXh+JsoVhe2SX/nQRIjjwzpx/zqWnrwl+GJAsYiUS0ZzXbnZwadHVOurA4W8rzRO 9qm1UgBqRSE9VOZzXNUgmlPQxITuvHMnnuGTsAUeT7wZ700vKzw0QCgtOVH2VY4mS9Kr klEEn4HbmUjn44Ac0GBJAeenFRV01M79S3CFRgUHuQKuQYuVeYJmzUFydARFzyqUq7GD Uihw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1776251215; x=1776856015; h=in-reply-to:references:from:subject:cc:to:message-id:date :content-transfer-encoding:mime-version:x-gm-gg:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=Rw116FfYB2wR32SWs/TCFerpwP/9uG3dqe/oInwd9Xs=; b=SjcStqA4w41R969iriXAlZT2ByY7VgdYXVlpI0CyeeUamkSiI2n/+RQBY4V+SI8ufP sGUG/cdSvQjfoNwRKyQ1PMJvsfwJRNCxyBUbSRXGTJoCT7vWBCI3IZJ1PxvuyoCOhnUn sjz4/7Ks0aBe/Tit/JECexbxQ984433d2PVdHDbkzLLoqwqCSvbuR50wWeGaxxxc0plc 8yq2pOXE4FQ7r3zy7Iu/TQICjtjWiRkjOePadgzL8JmUoBb9qjveHFV8MTW1YKwdZHDx IngpDd2b79m1iVtwcs9019ux/GZfvBdfDoo4gOH5kANyBr3+MW/KTgfJXFPBeyJP1kP2 tBWg== X-Gm-Message-State: AOJu0YyD/GBcBtU9+SveWKpumHD3tHYn7zPprcyvlrvtLQa1BG/ln02S W4OVpEKNzliJF0fJSor0U51CY55uv3TgonaAMQEfBRgbJbermZjZPrAQv868PH+xefc= X-Gm-Gg: AeBDietoya96NQZ6qmx0je9Vc5N/cL+MHwgX6zzIw3+4fCvOkzub2wdI4FD7lybFGZN n6SkXsAxL/0oxMpmhQeoasBh7Ay0rN314CW78aGXN1HAzLBP0tPBY6IJmMnG1E+/TLtHrGz8aho zR90cHyYbl/m75aI9nrZXnYUIFsIuaPBF4T/BNjX6bAWFR+x3pqvGejXjek/bBPSR82uXis2Ahi 9Vt6eWjx12dSd+aUsrkJ61N7Ho7edQFMJXr+ibDcNrD3l+6WChJ44OXENgWp0KP+h3k8Mw1NYPk t/1HmYRqF+ert28VzxFgPIArZKU01ydKqBs153BuPexOlcBgezXApnDx62REetYLmUfGB3JohZN wLViu26+qT2qiWeKje738FDwlbzhwdhpc8yL6Cl2LuZpd3fpZqRYNnyhrbSb+0eH72EQ/tPrZDa 2FG+v6GzJgaDHiOpdyN7cWVlQPELtJM/5DN4VroZMSkLBZaB3x4LqbBUDRz/gubPilJTtwdk3c1 vrliS+rt+LC4JLmag== X-Received: by 2002:a05:6000:184e:b0:43e:aa88:f1a6 with SMTP id ffacd0b85a97d-43eaa88f2cemr4326833f8f.45.1776251214558; Wed, 15 Apr 2026 04:06:54 -0700 (PDT) Received: from localhost ([212.187.182.164]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-43ead3e0091sm3867007f8f.25.2026.04.15.04.06.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 15 Apr 2026 04:06:53 -0700 (PDT) Precedence: bulk X-Mailing-List: linux-arm-msm@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, 15 Apr 2026 12:06:53 +0100 Message-Id: To: "Kumar Anurag" , "Bjorn Andersson" , "Konrad Dybcio" , "Rob Herring" , "Krzysztof Kozlowski" , "Conor Dooley" , "Srinivas Kandagatla" , "Liam Girdwood" , "Mark Brown" , "Jaroslav Kysela" , "Takashi Iwai" Cc: , , , Subject: Re: [PATCH v1 4/4] ASoC: qcom: sc8280xp: don't force S16_LE in hw_params fixup From: "Alexey Klimov" X-Mailer: aerc 0.21.0 References: <20260413091937.134469-1-kumar.singh@oss.qualcomm.com> <20260413091937.134469-5-kumar.singh@oss.qualcomm.com> In-Reply-To: <20260413091937.134469-5-kumar.singh@oss.qualcomm.com> On Mon Apr 13, 2026 at 10:19 AM BST, Kumar Anurag wrote: > The machine driver was unconditionally forcing S16_LE in > sc8280xp_be_hw_params_fixup(), which prevents links (e.g. HDMI bridges) > that require 32-bit formats from working. Drop the format override and > keep only the fixed rate/channels constraints. Why the format override is no longer needed? Please add it in the description. > Signed-off-by: Kumar Anurag > --- > sound/soc/qcom/sc8280xp.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/sound/soc/qcom/sc8280xp.c b/sound/soc/qcom/sc8280xp.c > index 7925aa3f63ba..c00eabf200b7 100644 > --- a/sound/soc/qcom/sc8280xp.c > +++ b/sound/soc/qcom/sc8280xp.c > @@ -75,10 +75,8 @@ static int sc8280xp_be_hw_params_fixup(struct snd_soc_= pcm_runtime *rtd, > SNDRV_PCM_HW_PARAM_RATE); > struct snd_interval *channels =3D hw_param_interval(params, > SNDRV_PCM_HW_PARAM_CHANNELS); > - struct snd_mask *fmt =3D hw_param_mask(params, SNDRV_PCM_HW_PARAM_FORMA= T); > =20 > rate->min =3D rate->max =3D 48000; > - snd_mask_set_format(fmt, SNDRV_PCM_FORMAT_S16_LE); Does compressed playback work after this change? How did you test it? Will it be possible with this series to have compressed playback via HDMI (with 32-bit format, right?)? You might need to add some functionality for this in topology. Thanks, Alexey