05.09.2020 13:34, Mikko Perttunen пишет: ... > +static int vic_power_on(struct tegra_drm_client *client) > +{ > + struct vic *vic = to_vic(client); > + > + return pm_runtime_get_sync(vic->dev); Please keep in mind that RPM needs to be put in a case of error. Maybe it would be better if driver-core could take care of resuming/suspending client's RPM instead of putting that burden on each client individually?