Re: [PATCH 2/6] ASoC: mediatek: mt8188: Add reference for dmic clocks

[AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>]

Il 18/02/25 21:52, Nícolas F. R. A. Prado ha scritto:
> Add the names for the dmic clocks, aud_afe_dmic* and aud_dmic_hires*, so
> they can be acquired and enabled by the platform driver.
> 
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>
> ---
>   sound/soc/mediatek/mt8188/mt8188-afe-clk.c | 8 ++++++++
>   sound/soc/mediatek/mt8188/mt8188-afe-clk.h | 8 ++++++++
>   2 files changed, 16 insertions(+)
> 
> diff --git a/sound/soc/mediatek/mt8188/mt8188-afe-clk.c b/sound/soc/mediatek/mt8188/mt8188-afe-clk.c
> index e69c1bb2cb239596dee50b166c20192d5408be10..44c25b6e3d873448163b22e70f5b94cb5070654d 100644
> --- a/sound/soc/mediatek/mt8188/mt8188-afe-clk.c
> +++ b/sound/soc/mediatek/mt8188/mt8188-afe-clk.c
> @@ -58,6 +58,14 @@ static const char *aud_clks[MT8188_CLK_NUM] = {
>   	[MT8188_CLK_AUD_ADC] = "aud_adc",
>   	[MT8188_CLK_AUD_DAC_HIRES] = "aud_dac_hires",
>   	[MT8188_CLK_AUD_A1SYS_HP] = "aud_a1sys_hp",
> +	[MT8188_CLK_AUD_AFE_DMIC1] = "aud_afe_dmic1",
> +	[MT8188_CLK_AUD_AFE_DMIC2] = "aud_afe_dmic2",
> +	[MT8188_CLK_AUD_AFE_DMIC3] = "aud_afe_dmic3",
> +	[MT8188_CLK_AUD_AFE_DMIC4] = "aud_afe_dmic4",
> +	[MT8188_CLK_AUD_DMIC_HIRES1] = "aud_dmic_hires1",
> +	[MT8188_CLK_AUD_DMIC_HIRES2] = "aud_dmic_hires2",
> +	[MT8188_CLK_AUD_DMIC_HIRES3] = "aud_dmic_hires3",
> +	[MT8188_CLK_AUD_DMIC_HIRES4] = "aud_dmic_hires4",
>   	[MT8188_CLK_AUD_ADC_HIRES] = "aud_adc_hires",
>   	[MT8188_CLK_AUD_I2SIN] = "aud_i2sin",
>   	[MT8188_CLK_AUD_TDM_IN] = "aud_tdm_in",
> diff --git a/sound/soc/mediatek/mt8188/mt8188-afe-clk.h b/sound/soc/mediatek/mt8188/mt8188-afe-clk.h
> index ec53c171c170a8b4b47900e63ef79d53641e9b12..68c46feb72271b950d4e538f63cedf524354147e 100644
> --- a/sound/soc/mediatek/mt8188/mt8188-afe-clk.h
> +++ b/sound/soc/mediatek/mt8188/mt8188-afe-clk.h
> @@ -54,6 +54,14 @@ enum {
>   	MT8188_CLK_AUD_ADC,
>   	MT8188_CLK_AUD_DAC_HIRES,
>   	MT8188_CLK_AUD_A1SYS_HP,
> +	MT8188_CLK_AUD_AFE_DMIC1,
> +	MT8188_CLK_AUD_AFE_DMIC2,
> +	MT8188_CLK_AUD_AFE_DMIC3,
> +	MT8188_CLK_AUD_AFE_DMIC4,
> +	MT8188_CLK_AUD_DMIC_HIRES1,
> +	MT8188_CLK_AUD_DMIC_HIRES2,
> +	MT8188_CLK_AUD_DMIC_HIRES3,
> +	MT8188_CLK_AUD_DMIC_HIRES4,
>   	MT8188_CLK_AUD_ADC_HIRES,

Following the comments from audsys, put those after CLK_AUD_ADC_HIRES.

Cheers,
Angelo

>   	MT8188_CLK_AUD_I2SIN,
>   	MT8188_CLK_AUD_TDM_IN,
>