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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 16181C54EE9 for ; Thu, 1 Sep 2022 07:33:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232730AbiIAHdN (ORCPT ); Thu, 1 Sep 2022 03:33:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38752 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232572AbiIAHc4 (ORCPT ); Thu, 1 Sep 2022 03:32:56 -0400 Received: from mail-lf1-x12e.google.com (mail-lf1-x12e.google.com [IPv6:2a00:1450:4864:20::12e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5EE95B5A60 for ; Thu, 1 Sep 2022 00:30:28 -0700 (PDT) Received: by mail-lf1-x12e.google.com with SMTP id bt10so23172404lfb.1 for ; Thu, 01 Sep 2022 00:30:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc; bh=Yq1f5sXSl9dJBBsa4JJtzhgEzf3Vcbcgdkoadank0UY=; b=djMelikWbC/lJ29b+RcknkU1Y10YBy9OxcmWeKREbcDJZ7sIMbL7ZYDHEtfg7WWj/e C3PYlYa3kq3VB8KYoyf9BFYkRuA92YeiaVrBAE7jPxx90EZaorcQn4NE7uKUwV+x87ci 4+EE7VD/aU1mjDn5J4ZjZoDb/drWpjXr0y9Dgnbpz124meHVprM6tAfv3z3P4IxXIwQi H8mZ/M9p+vrlDwmZNgDeZGS9rYUMB98eJP+9QXE25bCbHVOziulP5lvdpsZhI/6WnGDa k6SrtklCLROaDxYaqRxIe/BfgB96gPTfuLve0ZIP1B4LRFM95B5IDAbwI+SXdvVZY+Rd uDxw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc; bh=Yq1f5sXSl9dJBBsa4JJtzhgEzf3Vcbcgdkoadank0UY=; b=q49EJvADjPWQ2EHxYp0Gmfmeiw1yXcGPJA3Ic/1O/qeXYRTVQ/EIzP6QIagawVMqeV FfG/x0TD42xcbzjHf+cdyMydtJIhsdwCtxAxy2onoANmwSkAsuHUkaNLQ6gOS/AdJfZ3 TNAzg+nzPRv6UWcK7brtOKWen5WdecC+junvQ9O+I78RFEufOWgjFH2Wd2BzEL/DrIgI dtIHSCWdqqEaowXhklVqYm6LS95aorW+lFIzJ5h41QjX+/2LPiMv8+FtCg60pLwN5iv3 eHYCFNZXDRVEmAcYrOKMnJUNP1bOflVTTRph+LwlNd5HOyzMmbY/XE7mZjjvbX9Fd9QX MxYg== X-Gm-Message-State: ACgBeo0NUjyPGXHRRythHO98+WFI4VszzEQxhOdV6ONrBYUOXki4K2Q2 eyk0638vYzQWf7jQKFQlelg+Jg== X-Google-Smtp-Source: AA6agR7yPEX4lBca6MaRVV7FLC8JsJi9f1Qnc5BJpKL2yg/lvtRzib/see+UUQqoafpytn52CBFq6A== X-Received: by 2002:a05:6512:1395:b0:48d:81c:5159 with SMTP id p21-20020a056512139500b0048d081c5159mr11687844lfa.375.1662017426392; Thu, 01 Sep 2022 00:30:26 -0700 (PDT) Received: from [192.168.28.124] (balticom-73-99-134.balticom.lv. [109.73.99.134]) by smtp.gmail.com with ESMTPSA id h6-20020a2ea486000000b00268bc2c1ed0sm151744lji.22.2022.09.01.00.30.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 01 Sep 2022 00:30:25 -0700 (PDT) Message-ID: Date: Thu, 1 Sep 2022 10:30:24 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.13.0 Subject: Re: [PATCH 6/6] ASoC: codecs: tx-macro: add support for sm8450 and sc8280xp Content-Language: en-US To: Srinivas Kandagatla , broonie@kernel.org Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, perex@perex.cz, tiwai@suse.com, pierre-louis.bossart@linux.intel.com, linux-arm-msm@vger.kernel.org, alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20220818134619.3432-1-srinivas.kandagatla@linaro.org> <20220818134619.3432-7-srinivas.kandagatla@linaro.org> From: Krzysztof Kozlowski In-Reply-To: <20220818134619.3432-7-srinivas.kandagatla@linaro.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 18/08/2022 16:46, Srinivas Kandagatla wrote: > LPASS VA Macro now has soundwire master to deal with access to > analog mic in low power island use cases. > > This is added after sc8280xp, add support for this. > Along with this also add compatibles for sm8450 and sc8280xp. > > Signed-off-by: Srinivas Kandagatla > --- > sound/soc/codecs/lpass-va-macro.c | 64 ++++++++++++++++++++++++++++++- > 1 file changed, 62 insertions(+), 2 deletions(-) > > diff --git a/sound/soc/codecs/lpass-va-macro.c b/sound/soc/codecs/lpass-va-macro.c > index a35f684053d2..f8b0c8caa1db 100644 > --- a/sound/soc/codecs/lpass-va-macro.c > +++ b/sound/soc/codecs/lpass-va-macro.c > @@ -25,6 +25,10 @@ > #define CDC_VA_FS_CONTROL_EN BIT(0) > #define CDC_VA_FS_COUNTER_CLR BIT(1) > #define CDC_VA_CLK_RST_CTRL_SWR_CONTROL (0x0008) > +#define CDC_VA_SWR_RESET_MASK BIT(1) > +#define CDC_VA_SWR_RESET_ENABLE BIT(1) > +#define CDC_VA_SWR_CLK_EN_MASK BIT(0) > +#define CDC_VA_SWR_CLK_ENABLE BIT(0) > #define CDC_VA_TOP_CSR_TOP_CFG0 (0x0080) > #define CDC_VA_FS_BROADCAST_EN BIT(1) > #define CDC_VA_TOP_CSR_DMIC0_CTL (0x0084) > @@ -66,6 +70,8 @@ > #define CDC_VA_TOP_CSR_SWR_MIC_CTL0 (0x00D0) > #define CDC_VA_TOP_CSR_SWR_MIC_CTL1 (0x00D4) > #define CDC_VA_TOP_CSR_SWR_MIC_CTL2 (0x00D8) > +#define CDC_VA_SWR_MIC_CLK_SEL_0_1_MASK (0xEE) > +#define CDC_VA_SWR_MIC_CLK_SEL_0_1_DIV1 (0xCC) > #define CDC_VA_TOP_CSR_SWR_CTRL (0x00DC) > #define CDC_VA_INP_MUX_ADC_MUX0_CFG0 (0x0100) > #define CDC_VA_INP_MUX_ADC_MUX0_CFG1 (0x0104) > @@ -194,6 +200,8 @@ struct va_macro { > unsigned long active_ch_mask[VA_MACRO_MAX_DAIS]; > unsigned long active_ch_cnt[VA_MACRO_MAX_DAIS]; > u16 dmic_clk_div; > + bool has_swr_master; > + bool reset_swr; > > int dec_mode[VA_MACRO_NUM_DECIMATORS]; > struct regmap *regmap; > @@ -325,6 +333,9 @@ static bool va_is_rw_register(struct device *dev, unsigned int reg) > case CDC_VA_TOP_CSR_DMIC2_CTL: > case CDC_VA_TOP_CSR_DMIC3_CTL: > case CDC_VA_TOP_CSR_DMIC_CFG: > + case CDC_VA_TOP_CSR_SWR_MIC_CTL0: > + case CDC_VA_TOP_CSR_SWR_MIC_CTL1: > + case CDC_VA_TOP_CSR_SWR_MIC_CTL2: > case CDC_VA_TOP_CSR_DEBUG_BUS: > case CDC_VA_TOP_CSR_DEBUG_EN: > case CDC_VA_TOP_CSR_TX_I2S_CTL: > @@ -1306,12 +1317,40 @@ static const struct snd_soc_component_driver va_macro_component_drv = { > > static int fsgen_gate_enable(struct clk_hw *hw) > { > - return va_macro_mclk_enable(to_va_macro(hw), true); > + struct va_macro *va = to_va_macro(hw); > + struct regmap *regmap = va->regmap; > + int ret; > + > + ret = va_macro_mclk_enable(va, true); > + if (!va->has_swr_master) > + return ret; > + > + if (va->reset_swr) > + regmap_update_bits(regmap, CDC_VA_CLK_RST_CTRL_SWR_CONTROL, > + CDC_VA_SWR_RESET_MASK, > + CDC_VA_SWR_RESET_ENABLE); > + > + regmap_update_bits(regmap, CDC_VA_CLK_RST_CTRL_SWR_CONTROL, > + CDC_VA_SWR_CLK_EN_MASK, > + CDC_VA_SWR_CLK_ENABLE); > + if (va->reset_swr) > + regmap_update_bits(regmap, CDC_VA_CLK_RST_CTRL_SWR_CONTROL, > + CDC_VA_SWR_RESET_MASK, 0x0); > + va->reset_swr = false; > + > + return ret; > } > > static void fsgen_gate_disable(struct clk_hw *hw) > { > - va_macro_mclk_enable(to_va_macro(hw), false); > + struct va_macro *va = to_va_macro(hw); > + struct regmap *regmap = va->regmap; > + > + if (va->has_swr_master) > + regmap_update_bits(regmap, CDC_VA_CLK_RST_CTRL_SWR_CONTROL, > + CDC_VA_SWR_CLK_EN_MASK, 0x0); > + > + va_macro_mclk_enable(va, false); > } > > static int fsgen_gate_is_enabled(struct clk_hw *hw) > @@ -1459,6 +1498,11 @@ static int va_macro_probe(struct platform_device *pdev) > > dev_set_drvdata(dev, va); > > + if (of_device_is_compatible(dev->of_node, "qcom,sm8450-lpass-va-macro") || > + of_device_is_compatible(dev->of_node, "qcom,sc8280xp-lpass-va-macro")) { > + va->has_swr_master = true; > + va->reset_swr = true; This should go to driver_data. Either via quirks/flags or device type (enum for each device). Usually the first (flags) is more flexible if you want to support many devices. This also explains Rob's concerns about unneeded entries in of_device_id table. Best regards, Krzysztof