fdo-mirrors/libinput
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/libinput/libinput.git synced 2026-05-09 08:18:02 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

evdev: reuse code for device removal

Browse source 
Just like device_added, now the routines to close the compositor and vt switch
leave are using the same code to remove a device.

This patch also closes properly a mtdev device, bug spotted by Christopher
Michael.

Signed-off-by: Tiago Vignatti <tiago.vignatti@intel.com>
This commit is contained in:
Tiago Vignatti 2012-03-27 21:26:01 +03:00 • committed by Jonas Ådahl
parent 8fb6507f54
commit 9dae619646
1 changed files with 21 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
src/evdev.c
View file

@ -544,13 +544,8 @@ device_added(struct udev_device *udev_device, struct evdev_input *master)
} }
static void static void
device_removed(struct udev_device *udev_device, struct evdev_input *master) device_removed(struct evdev_input_device *device)
{ {
const char *devnode = udev_device_get_devnode(udev_device);
struct evdev_input_device *device, *next;
wl_list_for_each_safe(device, next, &master->devices_list, link) {
if (!strcmp(device->devnode, devnode)) {
wl_event_source_remove(device->source); wl_event_source_remove(device->source);
wl_list_remove(&device->link); wl_list_remove(&device->link);
if (device->mtdev) if (device->mtdev)
@ -558,9 +553,6 @@ device_removed(struct udev_device *udev_device, struct evdev_input *master)
close(device->fd); close(device->fd);
free(device->devnode); free(device->devnode);
free(device); free(device);
break;
}
}
} }
void void
@ -604,7 +596,9 @@ evdev_udev_handler(int fd, uint32_t mask, void *data)
{ {
struct evdev_input *master = data; struct evdev_input *master = data;
struct udev_device *udev_device; struct udev_device *udev_device;
struct evdev_input_device *device, *next;
const char *action; const char *action;
const char *devnode;
udev_device = udev_monitor_receive_device(master->udev_monitor); udev_device = udev_monitor_receive_device(master->udev_monitor);
if (!udev_device) if (!udev_device)
@ -618,8 +612,15 @@ evdev_udev_handler(int fd, uint32_t mask, void *data)
if (!strcmp(action, "add")) { if (!strcmp(action, "add")) {
device_added(udev_device, master); device_added(udev_device, master);
} }
else if (!strcmp(action, "remove")) else if (!strcmp(action, "remove")) {
device_removed(udev_device, master); devnode = udev_device_get_devnode(udev_device);
wl_list_for_each_safe(device, next,
&master->devices_list, link)
if (!strcmp(device->devnode, devnode)) {
device_removed(device);
break;
}
}
} }
udev_device_unref(udev_device); udev_device_unref(udev_device);
@ -685,13 +686,8 @@ evdev_remove_devices(struct weston_input_device *input_base)
struct evdev_input *input = (struct evdev_input *) input_base; struct evdev_input *input = (struct evdev_input *) input_base;
struct evdev_input_device *device, *next; struct evdev_input_device *device, *next;
wl_list_for_each_safe(device, next, &input->devices_list, link) { wl_list_for_each_safe(device, next, &input->devices_list, link)
wl_event_source_remove(device->source); device_removed(device);
wl_list_remove(&device->link);
close(device->fd);
free(device->devnode);
free(device);
}
} }
void void