18 Nov
2021
18 Nov
'21
2:56 p.m.
On Thu, Nov 18, 2021 at 10:00:34AM +0100, Olivier Moysan wrote:
ret = snd_soc_add_component(component, NULL, 0);
- if (ret < 0)
- if (ret < 0) { dev_err(&pdev->dev, "%s: Failed to register PCM platform\n", __func__);
return ret;
- }
- return ret;
- pm_runtime_enable(&pdev->dev);
Enabling runtime PM after registering the component may potentially lead to a race where something manages to go in and starts using the device including what should be runtime PM stuff. That'd lead to a reference not being taken that should be. It's unlikely to actually happen but it's better to be safe.