topology2: pcm_caps: Remove defaults for rate_min/rate_max

[ranj063]

Use 'rates' instead to specify that 48K is the default supported rate.

[kv2019i]

Tests failing on all platforms:
https://sof-ci.01.org/sofpr/PR9017/build3818/devicetest/index.html

[ujfalusi]

Tests failing on all platforms: https://sof-ci.01.org/sofpr/PR9017/build3818/devicetest/index.html

and no dmesg

[ujfalusi]

Something is wrong with CI, in the failing cases it tries to use 0Hz rate:
aplay -Dhw:0,0 -r 0 -c 2 -f S16_LE -d 10 /dev/zero -v -q
arecord -Dhw:0,0 -r 0 -c 2 -f S16_LE -d 3 /dev/null -v -q

resulting bad speed value 0

[ujfalusi]

Something is wrong with CI, in the failing cases it tries to use 0Hz rate: aplay -Dhw:0,0 -r 0 -c 2 -f S16_LE -d 10 /dev/zero -v -q arecord -Dhw:0,0 -r 0 -c 2 -f S16_LE -d 3 /dev/null -v -q

resulting bad speed value 0

The reason is that the sof-test is parsing the topology and uses rate_min, rate_max which is now 0 if it is not set explicitly. The same issue with #9004 (converted back to draft).
I have not looked at the sof-test that closely, but tools/sof-tplgreader.py loadFile() only reads rate_min, rate_max

[ujfalusi]

PIPELINE_LST[3]="pcm=HDA Analog;id=0;type=playback;cap_name=Analog Playback;eq=eqiir.2.1 eqfir.2.1;fmts=S16_LE S24_LE S32_LE;fmt=S16_LE;rate_min=48000;rate_max=48000;ch_min=2;ch_max=2;rate=48000;channel=2;dev=hw:0,0;snd=/dev/snd/pcmC0D0p;"

vs

PIPELINE_LST[3]="pcm=HDA Analog;id=0;type=playback;cap_name=Analog Playback;eq=eqiir.2.1 eqfir.2.1;fmts=S16_LE S24_LE S32_LE;fmt=S16_LE;rate_min=0;rate_max=0;ch_min=2;ch_max=2;rate=0;channel=2;dev=hw:0,0;snd=/dev/snd/pcmC0D0p;"

The PCM itself shows correct information:

./alsa_conformance_test/alsa_conformance_test -P hw:0,0 --dev_info_only
------DEVICE INFORMATION------
PCM handle name: hw:0,0
PCM type: HW
card: sofhdadsp [sof-hda-dsp]
device: HDA Analog (*) []
stream: PLAYBACK
available channels: 2
available formats: S16_LE S24_LE S32_LE
rate range: [48000, 48000]
available rates: 48000
period size range: [24, 4096]
buffer size range: [192, 16384]
------------------------------

[plbossart]

that's a spectacular dependency on sof-test parsing of topology. Oh well. Glad it was caught, our CI results just improved over the last week, probably not wise to run CI into the ground again...

[ranj063]

Something is wrong with CI, in the failing cases it tries to use 0Hz rate: aplay -Dhw:0,0 -r 0 -c 2 -f S16_LE -d 10 /dev/zero -v -q arecord -Dhw:0,0 -r 0 -c 2 -f S16_LE -d 3 /dev/null -v -q
resulting bad speed value 0

The reason is that the sof-test is parsing the topology and uses rate_min, rate_max which is now 0 if it is not set explicitly. The same issue with #9004 (converted back to draft). I have not looked at the sof-test that closely, but tools/sof-tplgreader.py loadFile() only reads rate_min, rate_max

@ujfalusi yes softest relies on rate min/max. I have a fix coming soon

[lgirdwood]

@ujfalusi any update ?

[fredoh9]

SOFCI TEST

[fredoh9]

After thesofproject/sof-test#1174 merged, I re-triggered the device test. It looks good so far.

[marc-hb]

Another problem with this:

• topology2: intel: bt-generic.conf: fix rate constraints #9068