[AMD Official Use Only - Internal Distribution Only] > -----Original Message----- > From: Mark Brown <broonie@xxxxxxxxxx> > Sent: Wednesday, May 6, 2020 10:55 PM > To: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> > Cc: Mukunda, Vijendar <Vijendar.Mukunda@xxxxxxx>; Alex Deucher > <alexdeucher@xxxxxxxxx>; tiwai@xxxxxxx; alsa-devel@xxxxxxxxxxxxxxxx; > Deucher, Alexander <Alexander.Deucher@xxxxxxx> > Subject: Re: [PATCH 14/14] ASoC: amd: enable build for RN machine driver > > On Wed, May 06, 2020 at 12:17:03PM -0500, Pierre-Louis Bossart wrote: > > On 5/6/20 11:43 AM, Mark Brown wrote: > > > > What you've got should be fine then, you should test with a !GPIOLIB > > > build just to be sure though and ideally submit a patch fixing the > > > SND_SOC_DMIC Kconfig. > > > if you don't care about gpios, then the only reason to use the dmic codec > > would be the modeswitch and wakeup delays properties. If you don't care > then > > a dummy codec would be just fine. > > Yes, it just decays to a dummy eventually if you don't use enough of the > features - the constraints aren't particularly useful here. Still it > doesn't hurt to explicitly say what the device is. Our requirement is more are like instead of going with a dummy codec driver, We want to stick to existing generic dmic driver .