From e6a4d6054d6c4c9c2d8796255659bb6b3fcca635 Mon Sep 17 00:00:00 2001 From: Johannes Zellner Date: Fri, 25 May 2012 16:42:55 -0700 Subject: evdev: Cleanup mouse and keyboard plugins Remove fallback event0 device usage. All available devices should be detected automatically from DeviceDiscovery either through udev or the static fallback discovery method. This also removes the never used 'key' argument, which was pulled over from Qt4. Change-Id: Iffe8bd80f0770664024019a220cbbc6f98843340 Reviewed-by: Girish Ramakrishnan --- .../generic/evdevkeyboard/qevdevkeyboardhandler.cpp | 15 +++++---------- src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.h | 5 +++-- .../generic/evdevkeyboard/qevdevkeyboardmanager.cpp | 9 +-------- 3 files changed, 9 insertions(+), 20 deletions(-) (limited to 'src/plugins/generic/evdevkeyboard') diff --git a/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.cpp b/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.cpp index 4634f81029..5574457fac 100644 --- a/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.cpp +++ b/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.cpp @@ -62,8 +62,8 @@ QT_BEGIN_NAMESPACE // simple builtin US keymap #include "qevdevkeyboard_defaultmap.h" -QEvdevKeyboardHandler::QEvdevKeyboardHandler(int deviceDescriptor, bool disableZap, bool enableCompose, const QString &keymapFile) - : m_fd(deviceDescriptor), +QEvdevKeyboardHandler::QEvdevKeyboardHandler(const QString &device, int fd, bool disableZap, bool enableCompose, const QString &keymapFile) + : m_device(device), m_fd(fd), m_modifiers(0), m_composing(0), m_dead_unicode(0xffff), m_no_zap(disableZap), m_do_compose(enableCompose), m_keymap(0), m_keymap_size(0), m_keycompose(0), m_keycompose_size(0) @@ -93,16 +93,13 @@ QEvdevKeyboardHandler::~QEvdevKeyboardHandler() qt_safe_close(m_fd); } -QEvdevKeyboardHandler *QEvdevKeyboardHandler::createLinuxInputKeyboardHandler(const QString &key, const QString &specification) +QEvdevKeyboardHandler *QEvdevKeyboardHandler::create(const QString &device, const QString &specification) { #ifdef QT_QPA_KEYMAP_DEBUG - qWarning() << "Try to create keyboard handler with" << key << specification; -#else - Q_UNUSED(key) + qWarning() << "Try to create keyboard handler for" << device << specification; #endif QString keymapFile; - QString device = QLatin1String("/dev/input/event0"); int repeatDelay = 400; int repeatRate = 80; bool disableZap = false; @@ -120,8 +117,6 @@ QEvdevKeyboardHandler *QEvdevKeyboardHandler::createLinuxInputKeyboardHandler(co repeatDelay = arg.mid(13).toInt(); else if (arg.startsWith(QLatin1String("repeat-rate="))) repeatRate = arg.mid(12).toInt(); - else if (arg.startsWith(QLatin1String("/dev/"))) - device = arg; } #ifdef QT_QPA_KEYMAP_DEBUG @@ -136,7 +131,7 @@ QEvdevKeyboardHandler *QEvdevKeyboardHandler::createLinuxInputKeyboardHandler(co ::ioctl(fd, EVIOCSREP, kbdrep); } - return new QEvdevKeyboardHandler(fd, disableZap, enableCompose, keymapFile); + return new QEvdevKeyboardHandler(device, fd, disableZap, enableCompose, keymapFile); } else { qWarning("Cannot open keyboard input device '%s': %s", qPrintable(device), strerror(errno)); return 0; diff --git a/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.h b/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.h index 5a1253857f..6b7901f42e 100644 --- a/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.h +++ b/src/plugins/generic/evdevkeyboard/qevdevkeyboardhandler.h @@ -125,7 +125,7 @@ class QEvdevKeyboardHandler : public QObject { Q_OBJECT public: - QEvdevKeyboardHandler(int deviceDescriptor, bool disableZap, bool enableCompose, const QString &keymapFile); + QEvdevKeyboardHandler(const QString &device, int fd, bool disableZap, bool enableCompose, const QString &keymapFile); ~QEvdevKeyboardHandler(); enum KeycodeAction { @@ -147,7 +147,7 @@ public: SwitchConsoleMask = 0x0000007f }; - static QEvdevKeyboardHandler *createLinuxInputKeyboardHandler(const QString &key, const QString &specification); + static QEvdevKeyboardHandler *create(const QString &device, const QString &specification); static Qt::KeyboardModifiers toQtModifiers(quint8 mod) { @@ -173,6 +173,7 @@ private: void processKeyEvent(int unicode, int keycode, Qt::KeyboardModifiers modifiers, bool isPress, bool autoRepeat); void switchLed(int, bool); + QString m_device; int m_fd; // keymap handling diff --git a/src/plugins/generic/evdevkeyboard/qevdevkeyboardmanager.cpp b/src/plugins/generic/evdevkeyboard/qevdevkeyboardmanager.cpp index 412695579d..8a1adfdbf9 100644 --- a/src/plugins/generic/evdevkeyboard/qevdevkeyboardmanager.cpp +++ b/src/plugins/generic/evdevkeyboard/qevdevkeyboardmanager.cpp @@ -105,15 +105,8 @@ void QEvdevKeyboardManager::addKeyboard(const QString &deviceNode) qWarning() << "Adding keyboard at" << deviceNode; #endif - QString specification = m_spec; - - if (!deviceNode.isEmpty()) { - specification.append(QLatin1Char(':')); - specification.append(deviceNode); - } - QEvdevKeyboardHandler *keyboard; - keyboard = QEvdevKeyboardHandler::createLinuxInputKeyboardHandler(QLatin1String("EvdevKeyboard"), specification); + keyboard = QEvdevKeyboardHandler::create(deviceNode, m_spec); if (keyboard) m_keyboards.insert(deviceNode, keyboard); else -- cgit v1.2.3