Re: [PATCH] ASoC: rt5670: Enable Braswell platform workaround for Dell Wyse 3040

[Vinod Koul <vinod.koul@intel.com>]

On Wed, Nov 30, 2016 at 05:30:25PM +0100, Takashi Iwai wrote:
> On Wed, 30 Nov 2016 17:36:15 +0100,
> Vinod Koul wrote:
> > 
> > On Wed, Nov 30, 2016 at 05:19:20PM +0100, Takashi Iwai wrote:
> > > Dell Wyse 3040 requires the platform workaround for enabling dmic and
> > > jack detection as defined for Braswell CRB.
> > > 
> > > Signed-off-by: Takashi Iwai <tiwai@suse.de>
> > > 
> > > ---
> > > 
> > > I guess the platform data should be applied generically to all CHT
> > > machines instead of DMI matching?  DMI matching doesn't scale,
> > > obviously.
> > 
> > we ahve been doing this on other RT codecs like RT286/298 as well for combo
> > jack to work. Yeah a better solution would be welcomed :)
> 
> Well, the question is how generic the given platform-specific setup
> is.  I guess we can keep the dmi thingy for a while, although we'll
> need more generic solution (or a pattern matching for a wider range of
> machines). 

Mostly if people stuck to Intel ref designs, then it would same, which has
been a problem on embedded side, but who knows.. :(

> So, if you think the addition of DMI entries is OK, give
> your ack :)

Absolutely atm :)

Acked-by: Vinod Koul <vinod.koul@intel.com>

-- 
~Vinod