> 
> this change doesn't seem quite right, if you look the code below there 
> is no PM imbalance, is there?
> 
> int sst_pm_runtime_put(struct intel_sst_drv *sst_drv)
> {
>       int ret;
> 
>       pm_runtime_mark_last_busy(sst_drv->dev);
>       ret = pm_runtime_put_autosuspend(sst_drv->dev);
>       if (ret < 0)
>               return ret;
>       return 0;
> }

You are right. Thank you for your correction!

Regards,
Dinghao

Reply via email to