Re: [PATCH 5/9] ASoC: mediatek: mt2701: add HDMI audio memif, FE and BE DAIs

Mark Brown
2026-04-16T14:23:19Z

[-- Attachment #1: Type: text/plain, Size: 1066 bytes --]

On Wed, Apr 15, 2026 at 04:23:59PM +0100, Daniel Golle wrote:
> Extend the MT2701/MT7623N AFE driver with the HDMI playback path:

> +static int mt2701_afe_hdmi_hw_params(struct snd_pcm_substream *substream,
> + struct snd_pcm_hw_params *params,
> + struct snd_soc_dai *dai)
> +{

> + /* Channel count into the HDMI output memif (bits [7:4]). */
> + regmap_update_bits(afe->regmap, AFE_HDMI_OUT_CON0,
> + 0x000000f0, channels << 4);

Are the other fields guaranteed to be already set up properly?

> + /*
> + * 8-channel I2S framing: standard I2S, 32-bit slots,
> + * LRCK/BCK inverted. The wire protocol is fixed.
> + */
> + regmap_update_bits(afe->regmap, AFE_8CH_I2S_OUT_CON,
> + AFE_8CH_I2S_OUT_CON_WLEN_MASK |
> + AFE_8CH_I2S_OUT_CON_I2S_DELAY |
> + AFE_8CH_I2S_OUT_CON_LRCK_INV |
> + AFE_8CH_I2S_OUT_CON_BCK_INV,
> + AFE_8CH_I2S_OUT_CON_WLEN_32BIT |
> + AFE_8CH_I2S_OUT_CON_I2S_DELAY |
> + AFE_8CH_I2S_OUT_CON_LRCK_INV |
> + AFE_8CH_I2S_OUT_CON_BCK_INV);

Does this need suspend/resume handling?

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]