From 6953f3a501dfee5064a12558a2e2515b69b35189 Mon Sep 17 00:00:00 2001 From: Laszlo Agocs Date: Mon, 27 Jan 2014 13:40:42 +0100 Subject: [PATCH] eglfs: Mouse hotplugging support On Embedded Linux with libudev support hotplugging was already working, except that the mouse cursor was not shown and hidden. This is now corrected so that the cursor disappears when all mice become disconnected and reappears if a mouse gets plugged in later on. [ChangeLog][QtGui] Mouse hotplugging is now fully supported in eglfs when running on Embedded Linux systems with libudev support enabled. Task-number: QTBUG-36374 Change-Id: Iec7c1557ba6085e3958dd357460cc032896fb174 Reviewed-by: Andy Nichols --- .../eglconvenience/eglconvenience.pri | 3 ++ .../eglconvenience/qeglplatformcursor.cpp | 52 ++++++++++++++++--- .../eglconvenience/qeglplatformcursor_p.h | 23 ++++++++ .../qeglplatformintegration.cpp | 22 ++++++++ .../qeglplatformintegration_p.h | 2 + .../input/evdevmouse/qevdevmousemanager_p.h | 2 + .../platforms/eglfs/qeglfsintegration.cpp | 18 +------ .../platforms/eglfs/qeglfsintegration.h | 2 - 8 files changed, 98 insertions(+), 26 deletions(-) diff --git a/src/platformsupport/eglconvenience/eglconvenience.pri b/src/platformsupport/eglconvenience/eglconvenience.pri index 4af0df58c7..6da0ea4fd1 100644 --- a/src/platformsupport/eglconvenience/eglconvenience.pri +++ b/src/platformsupport/eglconvenience/eglconvenience.pri @@ -27,6 +27,9 @@ contains(QT_CONFIG,egl) { $$PWD/qeglplatformintegration.cpp } + # Avoid X11 header collision + DEFINES += MESA_EGL_NO_X11_HEADERS + contains(QT_CONFIG,xlib) { HEADERS += \ $$PWD/qxlibeglintegration_p.h diff --git a/src/platformsupport/eglconvenience/qeglplatformcursor.cpp b/src/platformsupport/eglconvenience/qeglplatformcursor.cpp index 55590dbc9e..1c87e1d27d 100644 --- a/src/platformsupport/eglconvenience/qeglplatformcursor.cpp +++ b/src/platformsupport/eglconvenience/qeglplatformcursor.cpp @@ -39,7 +39,6 @@ ** ****************************************************************************/ -#include "qeglplatformcursor_p.h" #include #include #include @@ -50,6 +49,9 @@ #include +#include "qeglplatformcursor_p.h" +#include "qeglplatformintegration_p.h" + QT_BEGIN_NAMESPACE /*! @@ -66,15 +68,12 @@ QEGLPlatformCursor::QEGLPlatformCursor(QPlatformScreen *screen) m_program(0), m_vertexCoordEntry(0), m_textureCoordEntry(0), - m_textureEntry(0) + m_textureEntry(0), + m_deviceListener(0) { QByteArray hideCursorVal = qgetenv("QT_QPA_EGLFS_HIDECURSOR"); - if (hideCursorVal.isEmpty()) { - QScopedPointer dis(QDeviceDiscovery::create(QDeviceDiscovery::Device_Mouse)); - m_visible = !dis->scanConnectedDevices().isEmpty(); - } else { + if (!hideCursorVal.isEmpty()) m_visible = hideCursorVal.toInt() == 0; - } if (!m_visible) return; @@ -92,6 +91,45 @@ QEGLPlatformCursor::QEGLPlatformCursor(QPlatformScreen *screen) QEGLPlatformCursor::~QEGLPlatformCursor() { resetResources(); + delete m_deviceListener; +} + +void QEGLPlatformCursor::setMouseDeviceDiscovery(QDeviceDiscovery *dd) +{ + if (m_visible && dd) { + m_deviceListener = new QEGLPlatformCursorDeviceListener(dd, this); + updateMouseStatus(); + } +} + +void QEGLPlatformCursor::updateMouseStatus() +{ + m_visible = m_deviceListener->hasMouse(); +} + +QEGLPlatformCursorDeviceListener::QEGLPlatformCursorDeviceListener(QDeviceDiscovery *dd, QEGLPlatformCursor *cursor) + : m_cursor(cursor) +{ + m_mouseCount = dd->scanConnectedDevices().count(); + connect(dd, SIGNAL(deviceDetected(QString)), SLOT(onDeviceAdded())); + connect(dd, SIGNAL(deviceRemoved(QString)), SLOT(onDeviceRemoved())); +} + +bool QEGLPlatformCursorDeviceListener::hasMouse() const +{ + return m_mouseCount > 0; +} + +void QEGLPlatformCursorDeviceListener::onDeviceAdded() +{ + ++m_mouseCount; + m_cursor->updateMouseStatus(); +} + +void QEGLPlatformCursorDeviceListener::onDeviceRemoved() +{ + --m_mouseCount; + m_cursor->updateMouseStatus(); } void QEGLPlatformCursor::resetResources() diff --git a/src/platformsupport/eglconvenience/qeglplatformcursor_p.h b/src/platformsupport/eglconvenience/qeglplatformcursor_p.h index 52359b84cd..d1402d1457 100644 --- a/src/platformsupport/eglconvenience/qeglplatformcursor_p.h +++ b/src/platformsupport/eglconvenience/qeglplatformcursor_p.h @@ -48,6 +48,25 @@ QT_BEGIN_NAMESPACE class QOpenGLShaderProgram; +class QDeviceDiscovery; +class QEGLPlatformCursor; + +class QEGLPlatformCursorDeviceListener : public QObject +{ + Q_OBJECT + +public: + QEGLPlatformCursorDeviceListener(QDeviceDiscovery *dd, QEGLPlatformCursor *cursor); + bool hasMouse() const; + +private slots: + void onDeviceAdded(); + void onDeviceRemoved(); + +private: + QEGLPlatformCursor *m_cursor; + int m_mouseCount; +}; class QEGLPlatformCursor : public QPlatformCursor { @@ -66,6 +85,9 @@ public: void paintOnScreen(); void resetResources(); + void setMouseDeviceDiscovery(QDeviceDiscovery *dd); + void updateMouseStatus(); + private: #ifndef QT_NO_CURSOR bool setCurrentCursor(QCursor *cursor); @@ -107,6 +129,7 @@ private: int m_vertexCoordEntry; int m_textureCoordEntry; int m_textureEntry; + QEGLPlatformCursorDeviceListener *m_deviceListener; }; QT_END_NAMESPACE diff --git a/src/platformsupport/eglconvenience/qeglplatformintegration.cpp b/src/platformsupport/eglconvenience/qeglplatformintegration.cpp index 343375a14a..5fc1721f6c 100644 --- a/src/platformsupport/eglconvenience/qeglplatformintegration.cpp +++ b/src/platformsupport/eglconvenience/qeglplatformintegration.cpp @@ -41,6 +41,7 @@ #include #include +#include #include #include @@ -49,11 +50,18 @@ #include #include +#if !defined(QT_NO_EVDEV) && (!defined(Q_OS_ANDROID) || defined(Q_OS_ANDROID_NO_SDK)) +#include +#include +#include +#endif + #include "qeglplatformintegration_p.h" #include "qeglplatformcontext_p.h" #include "qeglplatformwindow_p.h" #include "qeglplatformbackingstore_p.h" #include "qeglplatformscreen_p.h" +#include "qeglplatformcursor_p.h" QT_BEGIN_NAMESPACE @@ -256,4 +264,18 @@ QPlatformNativeInterface::NativeResourceForContextFunction QEGLPlatformIntegrati return 0; } +void QEGLPlatformIntegration::createInputHandlers() +{ +#if !defined(QT_NO_EVDEV) && (!defined(Q_OS_ANDROID) || defined(Q_OS_ANDROID_NO_SDK)) + new QEvdevKeyboardManager(QLatin1String("EvdevKeyboard"), QString() /* spec */, this); + QEvdevMouseManager *mouseMgr = new QEvdevMouseManager(QLatin1String("EvdevMouse"), QString() /* spec */, this); + Q_FOREACH (QScreen *screen, QGuiApplication::screens()) { + QEGLPlatformCursor *cursor = static_cast(screen->handle()->cursor()); + if (cursor) + cursor->setMouseDeviceDiscovery(mouseMgr->deviceDiscovery()); + } + new QEvdevTouchScreenHandlerThread(QString() /* spec */, this); +#endif +} + QT_END_NAMESPACE diff --git a/src/platformsupport/eglconvenience/qeglplatformintegration_p.h b/src/platformsupport/eglconvenience/qeglplatformintegration_p.h index a5b131ced6..d3c3f07aff 100644 --- a/src/platformsupport/eglconvenience/qeglplatformintegration_p.h +++ b/src/platformsupport/eglconvenience/qeglplatformintegration_p.h @@ -85,6 +85,8 @@ protected: virtual QEGLPlatformWindow *createWindow(QWindow *window) const = 0; virtual EGLNativeDisplayType nativeDisplay() const { return EGL_DEFAULT_DISPLAY; } + void createInputHandlers(); + private: QEGLPlatformScreen *m_screen; EGLDisplay m_display; diff --git a/src/platformsupport/input/evdevmouse/qevdevmousemanager_p.h b/src/platformsupport/input/evdevmouse/qevdevmousemanager_p.h index 6abe933371..d52a16ea75 100644 --- a/src/platformsupport/input/evdevmouse/qevdevmousemanager_p.h +++ b/src/platformsupport/input/evdevmouse/qevdevmousemanager_p.h @@ -59,6 +59,8 @@ public: QEvdevMouseManager(const QString &key, const QString &specification, QObject *parent = 0); ~QEvdevMouseManager(); + QDeviceDiscovery *deviceDiscovery() { return m_deviceDiscovery; } + public slots: void handleMouseEvent(int x, int y, Qt::MouseButtons buttons); void handleWheelEvent(int delta, Qt::Orientation orientation); diff --git a/src/plugins/platforms/eglfs/qeglfsintegration.cpp b/src/plugins/platforms/eglfs/qeglfsintegration.cpp index dfe240a888..472e58cc72 100644 --- a/src/plugins/platforms/eglfs/qeglfsintegration.cpp +++ b/src/plugins/platforms/eglfs/qeglfsintegration.cpp @@ -43,6 +43,7 @@ #include "qeglfswindow.h" #include "qeglfshooks.h" +#include "qeglfscontext.h" #include @@ -50,12 +51,6 @@ #include #include -#if !defined(QT_NO_EVDEV) && (!defined(Q_OS_ANDROID) || defined(Q_OS_ANDROID_NO_SDK)) -#include -#include -#include -#endif - #include #include #include @@ -63,8 +58,6 @@ #include #include -#include "qeglfscontext.h" - #include static void initResources() @@ -167,13 +160,4 @@ EGLConfig QEglFSIntegration::chooseConfig(EGLDisplay display, const QSurfaceForm return chooser.chooseConfig(); } -void QEglFSIntegration::createInputHandlers() -{ -#if !defined(QT_NO_EVDEV) && (!defined(Q_OS_ANDROID) || defined(Q_OS_ANDROID_NO_SDK)) - new QEvdevKeyboardManager(QLatin1String("EvdevKeyboard"), QString() /* spec */, this); - new QEvdevMouseManager(QLatin1String("EvdevMouse"), QString() /* spec */, this); - new QEvdevTouchScreenHandlerThread(QString() /* spec */, this); -#endif -} - QT_END_NAMESPACE diff --git a/src/plugins/platforms/eglfs/qeglfsintegration.h b/src/plugins/platforms/eglfs/qeglfsintegration.h index 581b523c76..9ff5bd6f3b 100644 --- a/src/plugins/platforms/eglfs/qeglfsintegration.h +++ b/src/plugins/platforms/eglfs/qeglfsintegration.h @@ -70,8 +70,6 @@ protected: EGLNativeDisplayType nativeDisplay() const Q_DECL_OVERRIDE; private: - void createInputHandlers(); - bool mDisableInputHandlers; };