[Sound-open-firmware] [PATCH] topology: byt-rt5640: Fix DAI config

[Liam Girdwood]

On Thu, 2018-03-15 at 16:37 -0500, Pierre-Louis Bossart wrote:
> 
> On 03/15/2018 04:33 PM, Pierre-Louis Bossart wrote:
> > 
> > 
> > On 03/15/2018 04:20 PM, Liam Girdwood wrote:
> > > DAI config was coming from deprecated machine driver. Fix it to use
> > > bytcr-rt5640 machine driver.
> > > 
> > > Signed-off-by: Liam Girdwood <liam.r.girdwood at linux.intel.com>
> > > ---
> > >   topology/reef-byt-rt5640.m4 | 6 +++---
> > >   1 file changed, 3 insertions(+), 3 deletions(-)
> > > 
> > > diff --git a/topology/reef-byt-rt5640.m4 b/topology/reef-byt-rt5640.m4
> > > index 879e9a9..bf2b004 100644
> > > --- a/topology/reef-byt-rt5640.m4
> > > +++ b/topology/reef-byt-rt5640.m4
> > > @@ -1,5 +1,5 @@
> > >   #
> > > -# Topology for generic Baytrail board with no codec.
> > > +# Topology for generic Baytrail board with no RT5640.
> > >   #
> > >     # Include topology builder
> > > @@ -76,7 +76,7 @@ SectionGraph."pipe-byt-rt5640" {
> > >   # playback DAI is SSP2 using 2 periods
> > >   # Buffers use s24le format, with 48 frame per 1000us on core 0 with 
> > > priority 0
> > >   DAI_ADD(sof/pipe-dai-playback.m4,
> > > -    1, SSP, 2, SSP-Codec
> > > +    1, SSP, 2, SSP-Codec,
> > 
> > Could it also be that it's SSP2-Codec in all those files? That might 
> > explain the issue I have with DAI configs...
> 
> Actually should be SSP2-Codec here, all the other files have this.
> Still doesn't tell me what's wrong with those files.
> 
Your right, now fixed. I've been thinking maybe qemu is the best way to test all
these (without the HW) since qemu can virtualise all the codecs as dummy
devices....

Liam