Received: by 2002:a05:7412:8d09:b0:fa:4c10:6cad with SMTP id bj9csp435122rdb; Tue, 16 Jan 2024 05:10:35 -0800 (PST) X-Google-Smtp-Source: AGHT+IEtH7rMs1ikFmeh+hEmHi8AI8cnaKhcVDKrWlnKPrkrltI2UnCN4td9BQSvb+ScBo1c9WAH X-Received: by 2002:a05:6a20:7b10:b0:19a:4d54:67e4 with SMTP id s16-20020a056a207b1000b0019a4d5467e4mr9002047pzh.2.1705410634909; Tue, 16 Jan 2024 05:10:34 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1705410634; cv=none; d=google.com; s=arc-20160816; b=z9NsaDGqsuGf0V3Jd38f/U4JtFmoDQ7dHWqcmqR/kNTLfvieeG132c9Ow2uoXODFmi 69DjX97n1mVWt5FVn5n8wo7aAp5nacrhUSCysboH2rrIIR+RAtVipXixm5uY/lATJARy 2peHPTYZS0KXIB8MxSwyaGWg+TIj39UTvaLgGBr4yral7mFRTBBjIuW3dNpGHjBxys64 MEiYEvBNVVSY9bXevsYh0BsN2Xf5ffzGQlFN7sh76y7rH+D5zEJe4Moo/hQHJsax06tI ZSqJ6GNB4Y44SI3ebGA1Uc2DYUSDeR7PQHnagc5Undg4x8BAYDzUGy5dldkVqkVmkiUx Uj1A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=pTSW1bngLMALPV1DsobJBqfa6J2FUJKrV8a+m5hT69k=; fh=qo50WMfUTlhwb5XidtyFAHckF52aP9qgjBAoI6YlSUw=; b=hqHdUdHsX/Xb1z75o9p47tEAW1Em5h1zcKeS9UzJj0RZr0pXdnxAAq0rB7BoWWOCGn f/c+OJcyjL7eqfY4DXgLFD4REJbzOZQLFVw/IcEt3Ha1pXlZDBU0TpkRoaMTEP1yXZB8 fJT8Y7v8siSBrmZuOtujVRrS8Zd3xNvnELzzGCSEOOt4O15ew8S85Os3gu+6qToEEXKi nRxBVDb/7Noo12yVWv9o7RQ8iZXfMPkQkBJjjikQDlgaRSKFLugD8VldCFri6LsMD8u/ rwvI4DDb0/9lxMH7Z3JzOMu/ZjBPPYkrs9XNFEC2r1QkEsvBTU1SK9UlzrgsNNTfdv/F DX1A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=oDhj0ufm; spf=pass (google.com: domain of linux-kernel+bounces-27406-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-27406-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [147.75.48.161]) by mx.google.com with ESMTPS id bj3-20020a056a02018300b005cf8b79e32bsi630605pgb.119.2024.01.16.05.10.34 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 16 Jan 2024 05:10:34 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-27406-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=oDhj0ufm; spf=pass (google.com: domain of linux-kernel+bounces-27406-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-27406-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 87216B22907 for ; Tue, 16 Jan 2024 13:10:27 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 270681BC4F; Tue, 16 Jan 2024 13:10:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="oDhj0ufm" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 4A7D66FBF; Tue, 16 Jan 2024 13:10:16 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B9C9BC433C7; Tue, 16 Jan 2024 13:10:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1705410616; bh=i4dry6M6WDVUsf16531MxNHkF/ieE3KAF4x/oxtezNo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=oDhj0ufmBZ5rhO3OPZzmZDuzn9J90cGHPEF3adOa7IMzTRLXaZV8Khdd0rmAMF7uV xHL3+MkX22jLdE1SRP3SEZdWUmdK/vaubuGSljwLzIZwx851xPFEnxuYQxDMZHmeN4 xQFtajGQnZNKQJ0qHzO1I+PRphGmAcAkMhVha4UnfFp2w9QL9TuWOJGylInnhAnxmH RTiGLN1xKE/9HhRETXXIV7h8u0Rw3e3e7IdFE6Aex92sBHh1GepwBotSwxhVGrNvCh Um+7ZN8Ltee3ql6l11b3vrvEY8inpgpGSS/NksYL6I5isd9hQ+MqSqW0149552QpOP djgnIp4WMSTJA== Received: from johan by xi.lan with local (Exim 4.96.2) (envelope-from ) id 1rPjCt-00065Y-1B; Tue, 16 Jan 2024 14:10:20 +0100 Date: Tue, 16 Jan 2024 14:10:19 +0100 From: Johan Hovold To: Srinivas Kandagatla Cc: Johan Hovold , Mark Brown , Banajit Goswami , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , alsa-devel@alsa-project.org, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCH 2/7] ASoC: codecs: lpass-wsa-macro: fix compander volume hack Message-ID: References: <20240116093903.19403-1-johan+linaro@kernel.org> <20240116093903.19403-3-johan+linaro@kernel.org> <8bb1cad6-6a85-444a-b881-c03ab0051009@linaro.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8bb1cad6-6a85-444a-b881-c03ab0051009@linaro.org> On Tue, Jan 16, 2024 at 11:10:21AM +0000, Srinivas Kandagatla wrote: > Thanks Johan for this patch, > > On 16/01/2024 09:38, Johan Hovold wrote: > > The LPASS WSA macro codec driver is updating the digital gain settings > > behind the back of user space on DAPM events if companding has been > > enabled. > > > > As compander control is exported to user space, this can result in the > > digital gain setting being incremented (or decremented) every time the > > sound server is started and the codec suspended depending on what the > > UCM configuration looks like. > > > > Soon enough playback will become distorted (or too quiet). > > > > This is specifically a problem on the Lenovo ThinkPad X13s as this > > bypasses the limit for the digital gain setting that has been set by the > > machine driver. > > > > Fix this by simply dropping the compander gain hack. If someone cares > > about modelling the impact of the compander setting this can possibly be > > done by exporting it as a volume control later. > > > This is not a hack, wsa codec requires gain to be programmed after the > clk is enabled for that particular interpolator. Ok, but then it's also broken as, as I mentioned off-list, these registers are cached so unless companding is enabled, the write on enable will have no effect. > However I agree with you on programming the gain that is different to > what user set it. > > This is because of unimplemented or half baked implementation of half-db > feature of gain control in this codec. > > We can clean that half baked implementation for now and still continue > to program the gain values after the clks are enabled. > > lets remove spkr_gain_offset and associated code paths in this codec, > which should fix the issue that you have reported and still do the right > thing of programming gain after clks are enabled. Removing the offset which can alter the gain, will cause both of these writes to become no-ops as the registers are cached (e.g. just as for the follow-on codec cleanups). So then we might as well just remove this too. How is the half-dB feature supposed to work? And are you sure that you need to reprogram the gain value after enabling the clock? Everything seems to work without doing so. Johan