}
 
 static void sx150x_gpio_set(struct gpio_chip *chip, unsigned int offset,
-                              int value)
+                           int value)
 {
        struct sx150x_pinctrl *pctl = gpiochip_get_data(chip);
 
 }
 
 static int sx150x_gpio_direction_input(struct gpio_chip *chip,
-                                     unsigned int offset)
+                                      unsigned int offset)
 {
        struct sx150x_pinctrl *pctl = gpiochip_get_data(chip);
 
 }
 
 static int sx150x_gpio_direction_output(struct gpio_chip *chip,
-                                      unsigned int offset, int value)
+                                       unsigned int offset, int value)
 {
        struct sx150x_pinctrl *pctl = gpiochip_get_data(chip);
        int ret;
         *      reg + 3 [ 3 3 2 2 1 1 0 0 ]
         *
         * SX1503 and SX1506 deviate from that data layout, instead storing
-        * thier contents as follows:
+        * their contents as follows:
         *
         *      reg     [ f f e e d d c c ]
         *      reg + 1 [ 7 7 6 6 5 5 4 4 ]
  *
  * This way the rest of the driver code, interfacing with the chip via
  * regmap API, can work assuming that each GPIO pin is represented by
- * a group of bits at an offset proportioan to GPIO number within a
+ * a group of bits at an offset proportional to GPIO number within a
  * given register.
- *
  */
 static int sx150x_regmap_reg_read(void *context, unsigned int reg,
                                  unsigned int *result)
        unsigned int idx, val;
 
        /*
-        * There are four potential cases coverd by this function:
+        * There are four potential cases covered by this function:
         *
         * 1) 8-pin chip, single configuration bit register
         *