[PATCH] [v2] ASoC: Intel: sof_rt5682: Add support for max98390 speaker amp

Pierre-Louis Bossart pierre-louis.bossart at linux.intel.com
Tue Aug 24 18:22:09 CEST 2021


Looks mostly good, minor comments below

On 8/24/21 8:21 AM, Mark Hsieh wrote:
> Configure adl_max98390_rt5682 to support the rt5682 headset codec and max98390 speaker
> 
> BUG=b:191811888
> TEST=emerge-brya chromeos-kernel-5_10

I don't think we need this for upstream?

> +struct snd_soc_ops max_98390_ops = {
> +	.hw_params = max_98390_hw_params,
> +};

this could be const. We didn't do it for previous codecs but that was a
miss. I'll send a patch for the max98373

> +extern struct snd_soc_ops max_98390_ops;

const



More information about the Alsa-devel mailing list