[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <YjiV9DfLC7T1uweb@google.com>
Date: Mon, 21 Mar 2022 23:12:52 +0800
From: Tzung-Bi Shih <tzungbi@...nel.org>
To: Trevor Wu <trevor.wu@...iatek.com>
Cc: broonie@...nel.org, tiwai@...e.com, robh+dt@...nel.org,
matthias.bgg@...il.com, alsa-devel@...a-project.org,
linux-mediatek@...ts.infradead.org,
linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org, aaronyu@...gle.com,
angelogioacchino.delregno@...labora.com, yc.hung@...iatek.com
Subject: Re: [PATCH v3 5/6] ASoC: mediatek: mt8195: add machine support for
max98390 and rt5682
On Mon, Mar 21, 2022 at 03:23:11PM +0800, Trevor Wu wrote:
> diff --git a/sound/soc/mediatek/mt8195/mt8195-mt6359.c b/sound/soc/mediatek/mt8195/mt8195-mt6359.c
[...]
> #define RT1011_SPEAKER_AMP_PRESENT BIT(0)
> #define RT1019_SPEAKER_AMP_PRESENT BIT(1)
> +#define MAX98390_SPEAKER_AMP_PRESENT BIT(2)
> +
> +#define MAX98390_CODEC_DAI "max98390-aif1"
> +#define MAX98390_DEV0_NAME "max98390.2-0038" /* right */
> +#define MAX98390_DEV1_NAME "max98390.2-0039" /* left */
>
> #define RT1011_CODEC_DAI "rt1011-aif"
> #define RT1011_DEV0_NAME "rt1011.2-0038"
The quirk bit order: RT1011, RT1019, and MAX98390. To be neat, move
MAX98390_CODEC_DAI, MAX98390_DEV0_NAME, and MAX98390_DEV1_NAME after
RT1019_DEV0_NAME to maintain the order.
> +static int mt8195_max98390_init(struct snd_soc_pcm_runtime *rtd)
[...]
> +
> + ret = snd_soc_dapm_add_routes(&card->dapm, mt8195_max98390_routes,
> + ARRAY_SIZE(mt8195_max98390_routes));
> +
> + if (ret)
Remove the extra blank line.
Powered by blists - more mailing lists