gpiolib: Use list_first_entry()/list_last_entry()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Mon, 7 Mar 2022 10:54:14 +0000 (12:54 +0200)
committerBartosz Golaszewski <brgl@bgdev.pl>
Tue, 8 Mar 2022 08:52:57 +0000 (09:52 +0100)
Use list_first_entry()/list_last_entry() instead of open coded variants.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
drivers/gpio/gpiolib.c

index b4d8bf3..9b2b3a2 100644 (file)
@@ -262,14 +262,14 @@ static int gpiodev_add_to_list(struct gpio_device *gdev)
                return 0;
        }
 
-       next = list_entry(gpio_devices.next, struct gpio_device, list);
+       next = list_first_entry(&gpio_devices, struct gpio_device, list);
        if (gdev->base + gdev->ngpio <= next->base) {
                /* add before first entry */
                list_add(&gdev->list, &gpio_devices);
                return 0;
        }
 
-       prev = list_entry(gpio_devices.prev, struct gpio_device, list);
+       prev = list_last_entry(&gpio_devices, struct gpio_device, list);
        if (prev->base + prev->ngpio <= gdev->base) {
                /* add behind last entry */
                list_add_tail(&gdev->list, &gpio_devices);
@@ -4423,7 +4423,7 @@ static void *gpiolib_seq_next(struct seq_file *s, void *v, loff_t *pos)
        if (list_is_last(&gdev->list, &gpio_devices))
                ret = NULL;
        else
-               ret = list_entry(gdev->list.next, struct gpio_device, list);
+               ret = list_first_entry(&gdev->list, struct gpio_device, list);
        spin_unlock_irqrestore(&gpio_lock, flags);
 
        s->private = "\n";