sof-soundwire: split speaker configuration for 2 and 4ch cases

[plbossart]

Add tests and separate files for configurations

Fixes: thesofproject/linux#1998

Signed-off-by: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com

[plbossart]

@RanderWang I could use some help to figure out how to deal with the TGL 2-amp case. we may need a cfg-amp:<1/2/4> string to represent all possible topologies?

[RanderWang]

@plbossart yes, we can judge amp number by spk number now. I discussed with @bardliao, for static configured codec amp number can be derived from spk number. But for some codecs can be configured dynamically to support different number of spk, cfg-spk doesn't work. We can add cfg-amp in machine driver when we get this dynamic configuration or do it now ?

[plbossart]

@plbossart yes, we can judge amp number by spk number now. I discussed with @bardliao, for static configured codec amp number can be derived from spk number. But for some codecs can be configured dynamically to support different number of spk, cfg-spk doesn't work. We can add cfg-amp in machine driver when we get this dynamic configuration or do it now ?

I think it's better to add this now, and use it later. Do you want to send a PR for the machine driver?

[RanderWang]

@plbossart yes, we can judge amp number by spk number now. I discussed with @bardliao, for static configured codec amp number can be derived from spk number. But for some codecs can be configured dynamically to support different number of spk, cfg-spk doesn't work. We can add cfg-amp in machine driver when we get this dynamic configuration or do it now ?

I think it's better to add this now, and use it later. Do you want to send a PR for the machine driver?

yes , thesofproject/linux#2006

[libinyang]

@plbossart As we have supported RT1308-2 in this patch, we should use RT1308.conf instead of RT1308-1.conf in HiFi.conf

[plbossart]

@plbossart As we have supported RT1308-2 in this patch, we should use RT1308.conf instead of RT1308-1.conf in HiFi.conf

Sorry I don't understand this remark.

[libinyang]

Sorry I don't understand this remark.

@plbossart I mean in HiFi.conf,

<sof-soundwire/RT700.conf>
<sof-soundwire/RT711.conf>
<sof-soundwire/RT5682.conf>
- <sof-soundwire/RT1308-1.conf>
<sof-soundwire/RT715.conf>
<sof-soundwire/Hdmi.conf>

should be

<sof-soundwire/RT700.conf>
<sof-soundwire/RT711.conf>
<sof-soundwire/RT5682.conf>
+ <sof-soundwire/RT1308.conf>
<sof-soundwire/RT715.conf>
<sof-soundwire/Hdmi.conf>

Now. Maybe this should also be updated in this PR.

[plbossart]

fixed errors reported by @libinyang let's merge and submit PRs against this if it's broken.