thesofproject / sof-test

BSD 3-Clause "New" or "Revised" License
13 stars 45 forks source link

Add test for no-i915 configurations #291

Open kv2019i opened 4 years ago

kv2019i commented 4 years ago

SOF driver now supports configurations where the HDMI/DP audio via integrated graphics (i915) is disabled. No separate driver or topology is needed to make this work. However, currently the SOF CI is not testing this configuration and we've had multiple regresssions and bugs that only occur in this configuration. Some examples:

cc:

mengdonglin commented 4 years ago

@plbossart @xiulipan and @RanderWang aligned on how to support this configuration in SOF CI. It seems we cannot support this in sof-test, but in CI test framework.

plbossart commented 4 years ago

@mengdonglin can you elaborate? I don't get at all the explanations.

xiulipan commented 4 years ago

@plbossart I think @mengdonglin would like us to add some fail case in CI. Platform with non-i915 support will be one of this kind of test

marc-hb commented 2 years ago

@kv2019i is this still needed?

kv2019i commented 2 years ago

@marc-hb This is still a valid and popular configuration that we are not explicitly testing in sof-test. Based on data from 2021-22, we've not had any regressions, so that would indicate priority can be lowered.

fredoh9 commented 1 year ago

I came here to add this PR, but it is already mentioned.