[alsa-devel] [PATCH] ASoC: core: Move pcm_mutex up to card level from snd_soc_pcm_runtime

Peter Ujfalusi peter.ujfalusi at ti.com
Fri Aug 16 12:49:24 CEST 2019



On 15/08/2019 16.56, Pierre-Louis Bossart wrote:
>>> Actually you *can* test by submitting a PR for SOF, it'll trigger some
>>> tests on Intel platforms using DPCM. It's not going to test anything
>>> related to the compressed API but it's better than nothing.
>>
>> Good to know and thanks.
>> I would not thought of abusing the SOF project to run tests, not that I
>> know how to trigger the right tests ;)
> 
> well we will run those tests anyways when we do the weekly merges, and
> when in doubt it might be more productive for an ALSA contributor to run
> the tests ahead of time than report that things are broken after the
> merge (as seen 2 weeks in a row with the module_put and compilation issues)

I'll keep in mind, do you have some documentation on the tests?

>>> I took this patch and created one PR as an example
>>> https://github.com/thesofproject/linux/pull/1132
>>
>> Should I be worried because of your comment there saying 'I have no idea
>> why the BYT_NOCODEC mode fails, there's no information provided.' ?
> 
> Redoing the tests removes that error, so this looks fine. Must have been
> a CI glitch

Hrm, did it had the same failure w/o my patch? Reordering tests might
just hide a bug somewhere...

>>> Will share results when I have them.
> 
> So far no issues. so FWIW since it's just 1st order automatic testing
> 
> Tested-by: Pierre-Louis Bossart <pierre-louis.bossart at linux.intel.com>

Thank you!
- Péter

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki


More information about the Alsa-devel mailing list