On 10/5/21 11:35 PM, Adrian Hunter wrote:
On 05/10/2021 21:52, Bart Van Assche wrote:
The following code:
if (err == 0) {
pm_runtime_disable(dev);
err = pm_runtime_set_active(dev);
pm_runtime_enable(dev);
[ ... ]
}
has been introduced in scsi_dev_type_resume() by commit 3c31b52f96f7
("scsi: async sd resume"). I'm in favor of removing that code.
Presumably that code is necessary. Trying to remove it really seems a
separate issue.
I want to move that code into the sd driver before this patch goes upstream.
Thanks,
Bart.