simple: remove redundant line releases
authorBartosz Golaszewski <bartekgola@gmail.com>
Sun, 24 Sep 2017 19:59:45 +0000 (21:59 +0200)
committerBartosz Golaszewski <bartekgola@gmail.com>
Sun, 24 Sep 2017 19:59:45 +0000 (21:59 +0200)
We're closing the chip right after calling the release routines in all
three cases. The chip close function releases all line resources
anyway so no need to do it twice.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>
src/lib/simple.c

index 2b23e4b58b6a7275d7dcdff0fe0f2287735b9446..12766e6769f1dd43e908244e6f2286cf86d26d50 100644 (file)
@@ -70,7 +70,6 @@ int gpiod_simple_get_value_multiple(const char *consumer, const char *device,
        memset(values, 0, sizeof(*values) * num_lines);
        status = gpiod_line_get_value_bulk(&bulk, values);
 
-       gpiod_line_release_bulk(&bulk);
        gpiod_chip_close(chip);
 
        return status;
@@ -130,7 +129,6 @@ int gpiod_simple_set_value_multiple(const char *consumer, const char *device,
        if (cb)
                cb(data);
 
-       gpiod_line_release_bulk(&bulk);
        gpiod_chip_close(chip);
 
        return 0;
@@ -192,7 +190,6 @@ int gpiod_simple_event_loop(const char *consumer, const char *device,
        }
 
 out:
-       gpiod_line_release(line);
        gpiod_chip_close(chip);
 
        return status;