Swap conditions for ARRAY_FOR_EACH()
[platform/upstream/libinput.git] / src / udev-seat.c
index 1522c30..8d19894 100644 (file)
@@ -35,20 +35,20 @@ static const char default_seat[] = "seat0";
 static const char default_seat_name[] = "default";
 
 static struct udev_seat *
-udev_seat_create(struct udev_input *input, const char *seat_name);
+udev_seat_create(struct udev_input *input,
+                const char *device_seat,
+                const char *seat_name);
 static struct udev_seat *
 udev_seat_get_named(struct udev_input *input, const char *seat_name);
 
 static int
 device_added(struct udev_device *udev_device, struct udev_input *input)
 {
-       struct libinput *libinput = &input->base;
        struct evdev_device *device;
        const char *devnode;
        const char *sysname;
        const char *device_seat, *seat_name, *output_name;
        const char *calibration_values;
-       int fd;
        struct udev_seat *seat;
 
        device_seat = udev_device_get_property_value(udev_device, "ID_SEAT");
@@ -71,29 +71,20 @@ device_added(struct udev_device *udev_device, struct udev_input *input)
        if (seat)
                libinput_seat_ref(&seat->base);
        else {
-               seat = udev_seat_create(input, seat_name);
+               seat = udev_seat_create(input, device_seat, seat_name);
                if (!seat)
                        return -1;
        }
 
-       /* Use non-blocking mode so that we can loop on read on
-        * evdev_device_data() until all events on the fd are
-        * read.  mtdev_get() also expects this. */
-       fd = open_restricted(libinput, devnode, O_RDWR | O_NONBLOCK);
-       if (fd < 0) {
-               log_info("opening input device '%s' failed (%s).\n", devnode, strerror(-fd));
-               goto error;
-       }
+       device = evdev_device_create(&seat->base, devnode, sysname);
+       libinput_seat_unref(&seat->base);
 
-       device = evdev_device_create(&seat->base, devnode, sysname, fd);
        if (device == EVDEV_UNHANDLED_DEVICE) {
-               close_restricted(libinput, fd);
-               log_info("not using input device '%s'.\n", devnode);
-               goto error;
+               log_info(&input->base, "not using input device '%s'.\n", devnode);
+               return 0;
        } else if (device == NULL) {
-               close_restricted(libinput, fd);
-               log_info("failed to create input device '%s'.\n", devnode);
-               goto error;
+               log_info(&input->base, "failed to create input device '%s'.\n", devnode);
+               return 0;
        }
 
        calibration_values =
@@ -109,7 +100,8 @@ device_added(struct udev_device *udev_device, struct udev_input *input)
                                         &device->abs.calibration[4],
                                         &device->abs.calibration[5]) == 6) {
                device->abs.apply_calibration = 1;
-               log_info("Applying calibration: %f %f %f %f %f %f\n",
+               log_info(&input->base,
+                        "Applying calibration: %f %f %f %f %f %f\n",
                         device->abs.calibration[0],
                         device->abs.calibration[1],
                         device->abs.calibration[2],
@@ -123,10 +115,28 @@ device_added(struct udev_device *udev_device, struct udev_input *input)
                device->output_name = strdup(output_name);
 
        return 0;
-error:
-       if (seat)
-               libinput_seat_unref(&seat->base);
-       return 0;
+}
+
+static void
+device_removed(struct udev_device *udev_device, struct udev_input *input)
+{
+       const char *devnode;
+       struct evdev_device *device, *next;
+       struct udev_seat *seat;
+
+       devnode = udev_device_get_devnode(udev_device);
+       list_for_each(seat, &input->base.seat_list, base.link) {
+               list_for_each_safe(device, next,
+                                  &seat->base.devices_list, base.link) {
+                       if (!strcmp(device->devnode, devnode)) {
+                               log_info(&input->base,
+                                        "input device %s, %s removed\n",
+                                        device->devname, device->devnode);
+                               evdev_device_remove(device);
+                               break;
+                       }
+               }
+       }
 }
 
 static int
@@ -167,12 +177,8 @@ static void
 evdev_udev_handler(void *data)
 {
        struct udev_input *input = data;
-       struct libinput *libinput = &input->base;
        struct udev_device *udev_device;
-       struct evdev_device *device, *next;
        const char *action;
-       const char *devnode;
-       struct udev_seat *seat;
 
        udev_device = udev_monitor_receive_device(input->udev_monitor);
        if (!udev_device)
@@ -185,24 +191,10 @@ evdev_udev_handler(void *data)
        if (strncmp("event", udev_device_get_sysname(udev_device), 5) != 0)
                goto out;
 
-       if (!strcmp(action, "add")) {
+       if (!strcmp(action, "add"))
                device_added(udev_device, input);
-       }
-       else if (!strcmp(action, "remove")) {
-               devnode = udev_device_get_devnode(udev_device);
-               list_for_each(seat, &input->base.seat_list, base.link) {
-                       list_for_each_safe(device, next,
-                                          &seat->base.devices_list, base.link)
-                               if (!strcmp(device->devnode, devnode)) {
-                                       log_info("input device %s, %s removed\n",
-                                                device->devname, device->devnode);
-                                       close_restricted(libinput, device->fd);
-                                       evdev_device_remove(device);
-                                       libinput_seat_unref(&seat->base);
-                                       break;
-                               }
-               }
-       }
+       else if (!strcmp(action, "remove"))
+               device_removed(udev_device, input);
 
 out:
        udev_device_unref(udev_device);
@@ -218,23 +210,12 @@ udev_input_remove_devices(struct udev_input *input)
                libinput_seat_ref(&seat->base);
                list_for_each_safe(device, next,
                                   &seat->base.devices_list, base.link) {
-                       close_restricted(&input->base, device->fd);
                        evdev_device_remove(device);
-                       if (list_empty(&seat->base.devices_list)) {
-                               /* if the seat may be referenced by the
-                                  client, so make sure it's dropped from
-                                  the seat list now, to be freed whenever
-                                  the device is removed */
-                               list_remove(&seat->base.link);
-                               list_init(&seat->base.link);
-                       }
-                       libinput_seat_unref(&seat->base);
                }
                libinput_seat_unref(&seat->base);
        }
 }
 
-
 static void
 udev_input_disable(struct libinput *libinput)
 {
@@ -263,7 +244,8 @@ udev_input_enable(struct libinput *libinput)
 
        input->udev_monitor = udev_monitor_new_from_netlink(udev, "udev");
        if (!input->udev_monitor) {
-               log_info("udev: failed to create the udev monitor\n");
+               log_info(libinput,
+                        "udev: failed to create the udev monitor\n");
                return -1;
        }
 
@@ -271,7 +253,7 @@ udev_input_enable(struct libinput *libinput)
                        "input", NULL);
 
        if (udev_monitor_enable_receiving(input->udev_monitor)) {
-               log_info("udev: failed to bind the udev monitor\n");
+               log_info(libinput, "udev: failed to bind the udev monitor\n");
                udev_monitor_unref(input->udev_monitor);
                input->udev_monitor = NULL;
                return -1;
@@ -316,7 +298,9 @@ udev_seat_destroy(struct libinput_seat *seat)
 }
 
 static struct udev_seat *
-udev_seat_create(struct udev_input *input, const char *seat_name)
+udev_seat_create(struct udev_input *input,
+                const char *device_seat,
+                const char *seat_name)
 {
        struct udev_seat *seat;
 
@@ -324,8 +308,9 @@ udev_seat_create(struct udev_input *input, const char *seat_name)
        if (!seat)
                return NULL;
 
-       libinput_seat_init(&seat->base, &input->base, seat_name, udev_seat_destroy);
-       list_insert(&input->base.seat_list, &seat->base.link);
+       libinput_seat_init(&seat->base, &input->base,
+                          device_seat, seat_name,
+                          udev_seat_destroy);
 
        return seat;
 }
@@ -336,7 +321,7 @@ udev_seat_get_named(struct udev_input *input, const char *seat_name)
        struct udev_seat *seat;
 
        list_for_each(seat, &input->base.seat_list, base.link) {
-               if (strcmp(seat->base.name, seat_name) == 0)
+               if (strcmp(seat->base.logical_name, seat_name) == 0)
                        return seat;
        }
 
@@ -350,14 +335,13 @@ static const struct libinput_interface_backend interface_backend = {
 };
 
 LIBINPUT_EXPORT struct libinput *
-libinput_create_from_udev(const struct libinput_interface *interface,
-                         void *user_data,
-                         struct udev *udev,
-                         const char *seat_id)
+libinput_udev_create_context(const struct libinput_interface *interface,
+                            void *user_data,
+                            struct udev *udev)
 {
        struct udev_input *input;
 
-       if (!interface || !udev || !seat_id)
+       if (!interface || !udev)
                return NULL;
 
        input = zalloc(sizeof *input);
@@ -366,19 +350,36 @@ libinput_create_from_udev(const struct libinput_interface *interface,
 
        if (libinput_init(&input->base, interface,
                          &interface_backend, user_data) != 0) {
+               libinput_unref(&input->base);
                free(input);
                return NULL;
        }
 
        input->udev = udev_ref(udev);
-       input->seat_id = strdup(seat_id);
 
-       if (udev_input_enable(&input->base) < 0) {
-               udev_unref(udev);
-               libinput_destroy(&input->base);
-               free(input);
-               return NULL;
+       return &input->base;
+}
+
+LIBINPUT_EXPORT int
+libinput_udev_assign_seat(struct libinput *libinput,
+                         const char *seat_id)
+{
+       struct udev_input *input = (struct udev_input*)libinput;
+
+       if (!seat_id)
+               return -1;
+       if (input->seat_id != NULL)
+               return -1;
+
+       if (libinput->interface_backend != &interface_backend) {
+               log_bug_client(libinput, "Mismatching backends.\n");
+               return -1;
        }
 
-       return &input->base;
+       input->seat_id = strdup(seat_id);
+
+       if (udev_input_enable(&input->base) < 0)
+               return -1;
+
+       return 0;
 }