#include <linux/gpio.h>
 #include "gpio-utils.h"
 
-int hammer_device(const char *device_name, unsigned int *lines, int nlines,
+int hammer_device(const char *device_name, unsigned int *lines, int num_lines,
                  unsigned int loops)
 {
        struct gpiohandle_data data;
        unsigned int iteration = 0;
 
        memset(&data.values, 0, sizeof(data.values));
-       ret = gpiotools_request_linehandle(device_name, lines, nlines,
+       ret = gpiotools_request_linehandle(device_name, lines, num_lines,
                                           GPIOHANDLE_REQUEST_OUTPUT, &data,
                                           "gpio-hammer");
        if (ret < 0)
                goto exit_close_error;
 
        fprintf(stdout, "Hammer lines [");
-       for (i = 0; i < nlines; i++) {
+       for (i = 0; i < num_lines; i++) {
                fprintf(stdout, "%d", lines[i]);
-               if (i != (nlines - 1))
+               if (i != (num_lines - 1))
                        fprintf(stdout, ", ");
        }
        fprintf(stdout, "] on %s, initial states: [", device_name);
-       for (i = 0; i < nlines; i++) {
+       for (i = 0; i < num_lines; i++) {
                fprintf(stdout, "%d", data.values[i]);
-               if (i != (nlines - 1))
+               if (i != (num_lines - 1))
                        fprintf(stdout, ", ");
        }
        fprintf(stdout, "]\n");
        j = 0;
        while (1) {
                /* Invert all lines so we blink */
-               for (i = 0; i < nlines; i++)
+               for (i = 0; i < num_lines; i++)
                        data.values[i] = !data.values[i];
 
                ret = gpiotools_set_values(fd, &data);
                        j = 0;
 
                fprintf(stdout, "[");
-               for (i = 0; i < nlines; i++) {
+               for (i = 0; i < num_lines; i++) {
                        fprintf(stdout, "%d: %d", lines[i], data.values[i]);
-                       if (i != (nlines - 1))
+                       if (i != (num_lines - 1))
                                fprintf(stdout, ", ");
                }
                fprintf(stdout, "]\r");
        const char *device_name = NULL;
        unsigned int lines[GPIOHANDLES_MAX];
        unsigned int loops = 0;
-       int nlines;
+       int num_lines;
        int c;
        int i;
 
                return -1;
        }
 
-       nlines = i;
+       num_lines = i;
 
-       if (!device_name || !nlines) {
+       if (!device_name || !num_lines) {
                print_usage();
                return -1;
        }
-       return hammer_device(device_name, lines, nlines, loops);
+       return hammer_device(device_name, lines, num_lines, loops);
 }
 
  *                     such as "gpiochip0"
  * @lines:             An array desired lines, specified by offset
  *                     index for the associated GPIO device.
- * @nline:             The number of lines to request.
+ * @num_lines:         The number of lines to request.
  * @flag:              The new flag for requsted gpio. Reference
  *                     "linux/gpio.h" for the meaning of flag.
  * @data:              Default value will be set to gpio when flag is
  *                     On failure return the errno.
  */
 int gpiotools_request_linehandle(const char *device_name, unsigned int *lines,
-                                unsigned int nlines, unsigned int flag,
+                                unsigned int num_lines, unsigned int flag,
                                 struct gpiohandle_data *data,
                                 const char *consumer_label)
 {
                goto exit_free_name;
        }
 
-       for (i = 0; i < nlines; i++)
+       for (i = 0; i < num_lines; i++)
                req.lineoffsets[i] = lines[i];
 
        req.flags = flag;
        strcpy(req.consumer_label, consumer_label);
-       req.lines = nlines;
+       req.lines = num_lines;
        if (flag & GPIOHANDLE_REQUEST_OUTPUT)
                memcpy(req.default_values, data, sizeof(req.default_values));
 
  *                     such as "gpiochip0".
  * @lines:             An array desired lines, specified by offset
  *                     index for the associated GPIO device.
- * @nline:             The number of lines to request.
+ * @num_lines:         The number of lines to request.
  * @data:              The array of values get from gpiochip.
  *
  * Return:             On success return 0;
  *                     On failure return the errno.
  */
 int gpiotools_gets(const char *device_name, unsigned int *lines,
-                  unsigned int nlines, struct gpiohandle_data *data)
+                  unsigned int num_lines, struct gpiohandle_data *data)
 {
        int fd;
        int ret;
        int ret_close;
 
-       ret = gpiotools_request_linehandle(device_name, lines, nlines,
+       ret = gpiotools_request_linehandle(device_name, lines, num_lines,
                                           GPIOHANDLE_REQUEST_INPUT, data,
                                           CONSUMER);
        if (ret < 0)
  *                     such as "gpiochip0".
  * @lines:             An array desired lines, specified by offset
  *                     index for the associated GPIO device.
- * @nline:             The number of lines to request.
+ * @num_lines:         The number of lines to request.
  * @data:              The array of values set to gpiochip, must be
  *                     0(low) or 1(high).
  *
  *                     On failure return the errno.
  */
 int gpiotools_sets(const char *device_name, unsigned int *lines,
-                  unsigned int nlines, struct gpiohandle_data *data)
+                  unsigned int num_lines, struct gpiohandle_data *data)
 {
        int ret;
 
-       ret = gpiotools_request_linehandle(device_name, lines, nlines,
+       ret = gpiotools_request_linehandle(device_name, lines, num_lines,
                                           GPIOHANDLE_REQUEST_OUTPUT, data,
                                           CONSUMER);
        if (ret < 0)
 
 }
 
 int gpiotools_request_linehandle(const char *device_name, unsigned int *lines,
-                                unsigned int nlines, unsigned int flag,
+                                unsigned int num_lines, unsigned int flag,
                                 struct gpiohandle_data *data,
                                 const char *consumer_label);
 int gpiotools_set_values(const int fd, struct gpiohandle_data *data);
 
 int gpiotools_get(const char *device_name, unsigned int line);
 int gpiotools_gets(const char *device_name, unsigned int *lines,
-                  unsigned int nlines, struct gpiohandle_data *data);
+                  unsigned int num_lines, struct gpiohandle_data *data);
 int gpiotools_set(const char *device_name, unsigned int line,
                  unsigned int value);
 int gpiotools_sets(const char *device_name, unsigned int *lines,
-                  unsigned int nlines, struct gpiohandle_data *data);
+                  unsigned int num_lines, struct gpiohandle_data *data);
 
 #endif /* _GPIO_UTILS_H_ */