Re: [PATCH v4 3/8] ASoC: qcom: lpass-rx-macro: Add shikra compatible

[Ajay Kumar Nandam <ajay.nandam@oss.qualcomm.com>]

On 9/22/2026 3:12 PM, Mohammad Rafi Shaik wrote:
> Add shikra RX macro compatible data and hook it into the LPASS codec
> v4.1 flow.
> 
> The RX macro register layout of LPASS codec v4.1 matches the v2.5+
> layout, so reuse the existing v2.5 register access helpers, controls
> and widgets for it.
> 
> v4.1 additionally requires the FS counter to be bypassed before it is
> enabled during MCLK enable. This is a property of the codec version and
> not of the board, so key it off the version detected by the VA macro
> rather than off match data.
> 
> Signed-off-by: Mohammad Rafi Shaik <mohammad.rafi.shaik@oss.qualcomm.com>

LGTM

Reviewed-by: Ajay Kumar Nandam <ajay.nandam@oss.qualcomm.com>


> ---
>   sound/soc/codecs/lpass-macro-common.h |  3 +++
>   sound/soc/codecs/lpass-rx-macro.c     | 14 ++++++++++++++
>   2 files changed, 17 insertions(+)
> 
> diff --git a/sound/soc/codecs/lpass-macro-common.h b/sound/soc/codecs/lpass-macro-common.h
> index 573b2145e1c4..b52665b11e35 100644
> --- a/sound/soc/codecs/lpass-macro-common.h
> +++ b/sound/soc/codecs/lpass-macro-common.h
> @@ -30,6 +30,7 @@ enum lpass_codec_version {
>   	LPASS_CODEC_VERSION_2_7,
>   	LPASS_CODEC_VERSION_2_8,
>   	LPASS_CODEC_VERSION_2_9,
> +	LPASS_CODEC_VERSION_4_1,
>   };
>   
>   struct lpass_macro {
> @@ -70,6 +71,8 @@ static inline const char *lpass_macro_get_codec_version_string(int version)
>   		return "v2.8";
>   	case LPASS_CODEC_VERSION_2_9:
>   		return "v2.9";
> +	case LPASS_CODEC_VERSION_4_1:
> +		return "v4.1";
>   	default:
>   		break;
>   	}
> diff --git a/sound/soc/codecs/lpass-rx-macro.c b/sound/soc/codecs/lpass-rx-macro.c
> index c41eb8de6160..93ceee878b78 100644
> --- a/sound/soc/codecs/lpass-rx-macro.c
> +++ b/sound/soc/codecs/lpass-rx-macro.c
> @@ -60,6 +60,7 @@
>   #define CDC_RX_FS_MCLK_CNT_ENABLE	BIT(0)
>   #define CDC_RX_FS_MCLK_CNT_CLR_MASK	BIT(1)
>   #define CDC_RX_FS_MCLK_CNT_CLR		BIT(1)
> +#define CDC_RX_FS_CNT_BYPASS		BIT(7)
>   #define CDC_RX_CLK_RST_CTRL_SWR_CONTROL	(0x0108)
>   #define CDC_RX_SWR_CLK_EN_MASK		BIT(0)
>   #define CDC_RX_SWR_RESET_MASK		BIT(1)
> @@ -652,6 +653,7 @@ struct rx_macro {
>   	bool is_ear_mode_on;
>   	bool hph_pwr_mode;
>   	bool hph_hd2_mode;
> +	bool fs_cnt_bypass;
>   	struct snd_soc_component *component;
>   	unsigned long active_ch_mask[RX_MACRO_MAX_DAIS];
>   	unsigned long active_ch_cnt[RX_MACRO_MAX_DAIS];
> @@ -1613,6 +1615,7 @@ static bool rx_is_rw_register(struct device *dev, unsigned int reg)
>   	case LPASS_CODEC_VERSION_2_6:
>   	case LPASS_CODEC_VERSION_2_7:
>   	case LPASS_CODEC_VERSION_2_8:
> +	case LPASS_CODEC_VERSION_4_1:
>   		return rx_2_5_is_rw_register(dev, reg);
>   	default:
>   		break;
> @@ -2045,6 +2048,11 @@ static int rx_macro_mclk_enable(struct rx_macro *rx, bool mclk_enable)
>   					   CDC_RX_CLK_MCLK2_ENABLE);
>   			regmap_update_bits(regmap, CDC_RX_CLK_RST_CTRL_FS_CNT_CONTROL,
>   					   CDC_RX_FS_MCLK_CNT_CLR_MASK, 0x00);
> +
> +			if (rx->fs_cnt_bypass)
> +				regmap_set_bits(regmap, CDC_RX_CLK_RST_CTRL_FS_CNT_CONTROL,
> +						CDC_RX_FS_CNT_BYPASS);
> +
>   			regmap_update_bits(regmap, CDC_RX_CLK_RST_CTRL_FS_CNT_CONTROL,
>   					   CDC_RX_FS_MCLK_CNT_EN_MASK,
>   					   CDC_RX_FS_MCLK_CNT_ENABLE);
> @@ -3652,6 +3660,7 @@ static int rx_macro_component_probe(struct snd_soc_component *component)
>   	case LPASS_CODEC_VERSION_2_6:
>   	case LPASS_CODEC_VERSION_2_7:
>   	case LPASS_CODEC_VERSION_2_8:
> +	case LPASS_CODEC_VERSION_4_1:
>   		controls = rx_macro_2_5_snd_controls;
>   		num_controls = ARRAY_SIZE(rx_macro_2_5_snd_controls);
>   		widgets = rx_macro_2_5_dapm_widgets;
> @@ -3837,8 +3846,10 @@ static int rx_macro_probe(struct platform_device *pdev)
>   	case LPASS_CODEC_VERSION_2_6:
>   	case LPASS_CODEC_VERSION_2_7:
>   	case LPASS_CODEC_VERSION_2_8:
> +	case LPASS_CODEC_VERSION_4_1:
>   		rx->rxn_reg_stride = 0xc0;
>   		rx->rxn_reg_stride2 = 0x0;
> +		rx->fs_cnt_bypass = (rx->codec_version == LPASS_CODEC_VERSION_4_1);
>   		def_count = ARRAY_SIZE(rx_defaults) + ARRAY_SIZE(rx_2_5_defaults);
>   		reg_defaults = kmalloc_objs(struct reg_default, def_count);
>   		if (!reg_defaults)
> @@ -3951,6 +3962,9 @@ static const struct of_device_id rx_macro_dt_match[] = {
>   	}, {
>   		.compatible = "qcom,sc8280xp-lpass-rx-macro",
>   		.data = (void *)LPASS_MACRO_FLAG_HAS_NPL_CLOCK,
> +	}, {
> +		.compatible = "qcom,shikra-lpass-rx-macro",
> +		.data = (void *)LPASS_MACRO_FLAG_HAS_NPL_CLOCK,
>   	},
>   	{ }
>   };
>