ecore_drm2: Don't try to disable all devices when session deactivates

We get this callback after we've lost the drm device to logind, so
deactivating stuff here will just generate a lot of ERR messages
and break our internal book-keeping.

Instead, we just turn on DPMS on session activation instead of trying
to go through the output enable path (that will bail if it's already
enabled)

This could potentially result in a display that's enabled and DPMS
off being switched back on during session activation - if that's a real
problem we can restore the previous dpms state instead...

@fix T5483

diff --git a/src/lib/ecore_drm2/ecore_drm2_device.c b/src/lib/ecore_drm2/ecore_drm2_device.c
index 41d2128..2a1680c 100644 (file)
--- a/src/lib/ecore_drm2/ecore_drm2_device.c
+++ b/src/lib/ecore_drm2/ecore_drm2_device.c
@@ -22,8 +22,9 @@ _cb_session_active(void *data, int type EINA_UNUSED, void *event)
    ev = event;
    device = data;
 
-   EINA_LIST_FOREACH(device->outputs, l, output)
-     ecore_drm2_output_enabled_set(output, ev->active);
+   if (ev->active)
+     EINA_LIST_FOREACH(device->outputs, l, output)
+       ecore_drm2_output_dpms_set(output, DRM_MODE_DPMS_ON);
 
    ea = calloc(1, sizeof(Ecore_Drm2_Event_Activate));
    if (!ea) return ECORE_CALLBACK_RENEW;