ASoC: Intel: Fix block is enabled multiple times issue
During FW parsing and loading, block_list_prepare() may be called for each raw data block copying and this may made the hsw_block_enable() called mutiple times, which increase block->users many times. The result of this is hsw_block_disable() can't power gated the related block when trying to free the blocks during suspend, and the power gating status also confused. Here check the block user status, only calling enable() for those blocks who has no user yet. Remember that this works correctlly on current case, where there are enough SRAM memory so different module won't share a memory block. For further usage, we may need restructure the struct sst_mem_block to save the module list who is using it. Signed-off-by: Jie Yang <yang.jie@intel.com> Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
2e4f75919e
commit
35c0a8c017
|
@ -149,7 +149,7 @@ static int block_list_prepare(struct sst_dsp *dsp,
|
|||
/* enable each block so that's it'e ready for data */
|
||||
list_for_each_entry(block, block_list, module_list) {
|
||||
|
||||
if (block->ops && block->ops->enable) {
|
||||
if (block->ops && block->ops->enable && !block->users) {
|
||||
ret = block->ops->enable(block);
|
||||
if (ret < 0) {
|
||||
dev_err(dsp->dev,
|
||||
|
|
Loading…
Reference in New Issue