> > 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