gpiosim: use size_t in the public interface where applicable
authorBartosz Golaszewski <brgl@bgdev.pl>
Wed, 9 Mar 2022 09:08:03 +0000 (10:08 +0100)
committerBartosz Golaszewski <brgl@bgdev.pl>
Wed, 9 Mar 2022 09:09:24 +0000 (10:09 +0100)
Use the standard integer type for representing sizes in the API of
libgpiod wherever it makes sense.

Signed-off-by: Bartosz Golaszewski <brgl@bgdev.pl>
tests/gpiosim/gpiosim.c
tests/gpiosim/gpiosim.h

index 338fbc17148e56d6a89d1a9f359cb67d0d2febb5..ea5357a0005fb023daca1bb1a875a190db023ef5 100644 (file)
@@ -319,7 +319,7 @@ struct gpiosim_bank {
        char *dev_path;
        int cfs_dir_fd;
        int sysfs_dir_fd;
-       unsigned int num_lines;
+       size_t num_lines;
 };
 
 static int ctx_open_configfs_dir(struct gpiosim_ctx *ctx, const char *cfs_path)
@@ -830,7 +830,7 @@ GPIOSIM_API int gpiosim_bank_set_label(struct gpiosim_bank *bank,
 }
 
 GPIOSIM_API int gpiosim_bank_set_num_lines(struct gpiosim_bank *bank,
-                                          unsigned int num_lines)
+                                          size_t num_lines)
 {
        char buf[32];
        int ret;
@@ -838,7 +838,7 @@ GPIOSIM_API int gpiosim_bank_set_num_lines(struct gpiosim_bank *bank,
        if (!dev_check_pending(bank->dev))
                return -1;
 
-       snprintf(buf, sizeof(buf), "%u", num_lines);
+       snprintf(buf, sizeof(buf), "%zu", num_lines);
 
        ret = open_write_close(bank->cfs_dir_fd, "num_lines", buf);
        if (ret)
@@ -958,8 +958,7 @@ GPIOSIM_API int gpiosim_bank_clear_hog(struct gpiosim_bank *bank,
 }
 
 static int sysfs_read_bank_attr(struct gpiosim_bank *bank, unsigned int offset,
-                               const char *attr, char *buf,
-                               unsigned int bufsize)
+                               const char *attr, char *buf, size_t bufsize)
 {
        struct gpiosim_dev *dev = bank->dev;
        char where[32];
index de38f3f17df249f086ebe97b2cc39d0e21aefebb..25b245af28f62282e6a94caa9d64bc363045df9e 100644 (file)
@@ -5,6 +5,7 @@
 #define __GPIOD_GPIOSIM_H__
 
 #include <stdbool.h>
+#include <stddef.h>
 
 #ifdef __cplusplus
 extern "C" {
@@ -49,8 +50,7 @@ const char *gpiosim_bank_get_chip_name(struct gpiosim_bank *bank);
 const char *gpiosim_bank_get_dev_path(struct gpiosim_bank *bank);
 
 int gpiosim_bank_set_label(struct gpiosim_bank *bank, const char *label);
-int gpiosim_bank_set_num_lines(struct gpiosim_bank *bank,
-                              unsigned int num_lines);
+int gpiosim_bank_set_num_lines(struct gpiosim_bank *bank, size_t num_lines);
 int gpiosim_bank_set_line_name(struct gpiosim_bank *bank,
                               unsigned int offset, const char *name);
 int gpiosim_bank_hog_line(struct gpiosim_bank *bank, unsigned int offset,