On Wed, Apr 26, 2017 at 10:10:28PM +0200, Uwe Kleine-König wrote:
> + int ret;
> +
> + ret = pm_runtime_get_sync(ssp->dev);
> + if (ret < 0) {
> + dev_err(ssp->dev, "Failed to runtime resume for transfer\n");
> + return ret;
> + }
There's a flag auto_runtime_pm the driver can set which should do what
you're doing here in the core (with the very slight advantage that it'll
hold the device on while the queue is busy so we don't bounce the power
in the event that there's transactions queued up).
Otherwise this looks good to me.
Attachment:
signature.asc
Description: PGP signature
