pinctrl: copy per-pin driver private data to struct pin_desc
authorMasahiro Yamada <yamada.masahiro@socionext.com>
Wed, 25 May 2016 05:09:31 +0000 (14:09 +0900)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 31 May 2016 08:27:19 +0000 (10:27 +0200)
Currently, struct pinctrl_pin_desc can have per-pin driver private
data, but it is not copied to struct pin_desc.

For a driver with sparse pin space, for-loop search like below would
be necessary in order to get the driver-specific data for a desired
pin number.

  for (i = 0; i < pctldev->desc->npins; i++)
          if (pin_number == pctldev->desc->pins[i].number)
                  return pctldev->desc->pins[i].drv_data;

This is not efficient for a driver with a large number of pins.
So, copy the data to struct pin_desc when each pin is registered
for the faster radix tree lookup.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/core.c
drivers/pinctrl/core.h

index 03c08c156744d1f5cf457e20463b8be7700062d7..fb38e208f32dd2199f427207fe6a0e19d3987a5a 100644 (file)
@@ -225,13 +225,14 @@ static void pinctrl_free_pindescs(struct pinctrl_dev *pctldev,
 }
 
 static int pinctrl_register_one_pin(struct pinctrl_dev *pctldev,
-                                   unsigned number, const char *name)
+                                   const struct pinctrl_pin_desc *pin)
 {
        struct pin_desc *pindesc;
 
-       pindesc = pin_desc_get(pctldev, number);
+       pindesc = pin_desc_get(pctldev, pin->number);
        if (pindesc != NULL) {
-               dev_err(pctldev->dev, "pin %d already registered\n", number);
+               dev_err(pctldev->dev, "pin %d already registered\n",
+                       pin->number);
                return -EINVAL;
        }
 
@@ -245,10 +246,10 @@ static int pinctrl_register_one_pin(struct pinctrl_dev *pctldev,
        pindesc->pctldev = pctldev;
 
        /* Copy basic pin info */
-       if (name) {
-               pindesc->name = name;
+       if (pin->name) {
+               pindesc->name = pin->name;
        } else {
-               pindesc->name = kasprintf(GFP_KERNEL, "PIN%u", number);
+               pindesc->name = kasprintf(GFP_KERNEL, "PIN%u", pin->number);
                if (pindesc->name == NULL) {
                        kfree(pindesc);
                        return -ENOMEM;
@@ -256,9 +257,11 @@ static int pinctrl_register_one_pin(struct pinctrl_dev *pctldev,
                pindesc->dynamic_name = true;
        }
 
-       radix_tree_insert(&pctldev->pin_desc_tree, number, pindesc);
+       pindesc->drv_data = pin->drv_data;
+
+       radix_tree_insert(&pctldev->pin_desc_tree, pin->number, pindesc);
        pr_debug("registered pin %d (%s) on %s\n",
-                number, pindesc->name, pctldev->desc->name);
+                pin->number, pindesc->name, pctldev->desc->name);
        return 0;
 }
 
@@ -270,8 +273,7 @@ static int pinctrl_register_pins(struct pinctrl_dev *pctldev,
        int ret = 0;
 
        for (i = 0; i < num_descs; i++) {
-               ret = pinctrl_register_one_pin(pctldev,
-                                              pins[i].number, pins[i].name);
+               ret = pinctrl_register_one_pin(pctldev, &pins[i]);
                if (ret)
                        return ret;
        }
index ca08723b9ee1e90fdfc792666cc7491ad5fc492e..747c423c11f34ab65a491b3d76fddff7d15e0049 100644 (file)
@@ -134,6 +134,7 @@ struct pinctrl_setting {
  * @name: a name for the pin, e.g. the name of the pin/pad/finger on a
  *     datasheet or such
  * @dynamic_name: if the name of this pin was dynamically allocated
+ * @drv_data: driver-defined per-pin data. pinctrl core does not touch this
  * @mux_usecount: If zero, the pin is not claimed, and @owner should be NULL.
  *     If non-zero, this pin is claimed by @owner. This field is an integer
  *     rather than a boolean, since pinctrl_get() might process multiple
@@ -148,6 +149,7 @@ struct pin_desc {
        struct pinctrl_dev *pctldev;
        const char *name;
        bool dynamic_name;
+       void *drv_data;
        /* These fields only added when supporting pinmux drivers */
 #ifdef CONFIG_PINMUX
        unsigned mux_usecount;