DP AUX busses can both be accessed by DRM, and through any of the
userspace dev nodes in /dev/drm_dp_auxN. We need to make sure the GPU
stays on in all of these codepaths.

Signed-off-by: Lyude Paul <ly...@redhat.com>
Cc: Karol Herbst <karolher...@gmail.com>
Cc: sta...@vger.kernel.org
---
 drivers/gpu/drm/nouveau/nvkm/subdev/i2c/aux.c | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/i2c/aux.c 
b/drivers/gpu/drm/nouveau/nvkm/subdev/i2c/aux.c
index 4c1f547da463..6276b113065c 100644
--- a/drivers/gpu/drm/nouveau/nvkm/subdev/i2c/aux.c
+++ b/drivers/gpu/drm/nouveau/nvkm/subdev/i2c/aux.c
@@ -98,18 +98,28 @@ nvkm_i2c_aux_release(struct nvkm_i2c_aux *aux)
        AUX_TRACE(aux, "release");
        nvkm_i2c_pad_release(pad);
        mutex_unlock(&aux->mutex);
+       pm_runtime_put_autosuspend(pad->i2c->subdev.device->dev);
 }
 
 int
 nvkm_i2c_aux_acquire(struct nvkm_i2c_aux *aux)
 {
        struct nvkm_i2c_pad *pad = aux->pad;
+       struct device *dev = pad->i2c->subdev.device->dev;
        int ret;
+
        AUX_TRACE(aux, "acquire");
+
+       ret = pm_runtime_get_sync(dev);
+       if (ret < 0 && ret != -EACCES)
+               return ret;
+
        mutex_lock(&aux->mutex);
        ret = nvkm_i2c_pad_acquire(pad, NVKM_I2C_PAD_AUX);
-       if (ret)
+       if (ret) {
                mutex_unlock(&aux->mutex);
+               pm_runtime_put_autosuspend(dev);
+       }
        return ret;
 }
 
-- 
2.17.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to