summaryrefslogtreecommitdiffstats
path: root/drivers/pinctrl
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2011-11-30 13:28:14 +0100
committerLinus Walleij <linus.walleij@linaro.org>2012-01-03 09:10:02 +0100
commit59b099b04981917ee7fbd88b6f50eeaffc9f33cd (patch)
tree454627d0c9617bf2cb878fddc282fa0a25f4736d /drivers/pinctrl
parent97607d157c133ab18dfcd77fa836e37fa950a44a (diff)
downloadop-kernel-dev-59b099b04981917ee7fbd88b6f50eeaffc9f33cd.zip
op-kernel-dev-59b099b04981917ee7fbd88b6f50eeaffc9f33cd.tar.gz
pinctrl: make it possible to add multiple maps
Since we now anyway make a copy of the platform-supplied pinmux map, we can just as well make it possible to call the function adding maps several times, so as to simplify cases (as PXA) where several sets of disparate mappings need to be added depending on target platform. Acked-by: Haojian Zhuang <haojian.zhuang@gmail.com> Acked-by: Arnd Bergmann <arnd@linaro.org> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl')
-rw-r--r--drivers/pinctrl/pinmux.c60
1 files changed, 29 insertions, 31 deletions
diff --git a/drivers/pinctrl/pinmux.c b/drivers/pinctrl/pinmux.c
index f6e7d58..a11681b 100644
--- a/drivers/pinctrl/pinmux.c
+++ b/drivers/pinctrl/pinmux.c
@@ -33,7 +33,7 @@
static DEFINE_MUTEX(pinmux_list_mutex);
static LIST_HEAD(pinmux_list);
-/* Global pinmux maps, we allow one set only */
+/* Global pinmux maps */
static struct pinmux_map *pinmux_maps;
static unsigned pinmux_maps_num;
@@ -347,48 +347,30 @@ EXPORT_SYMBOL_GPL(pinmux_gpio_direction_output);
int __init pinmux_register_mappings(struct pinmux_map const *maps,
unsigned num_maps)
{
- int ret = 0;
+ void *tmp_maps;
int i;
- if (pinmux_maps_num != 0) {
- pr_err("pinmux mappings already registered, you can only "
- "register one set of maps\n");
- return -EINVAL;
- }
-
pr_debug("add %d pinmux maps\n", num_maps);
- /*
- * Make a copy of the map array - string pointers will end up in the
- * kernel const section anyway so these do not need to be deep copied.
- */
- pinmux_maps = kmemdup(maps, sizeof(struct pinmux_map) * num_maps,
- GFP_KERNEL);
- if (!pinmux_maps)
- return -ENOMEM;
-
+ /* First sanity check the new mapping */
for (i = 0; i < num_maps; i++) {
- /* Sanity check the mapping while copying it */
if (!maps[i].name) {
pr_err("failed to register map %d: "
"no map name given\n", i);
- ret = -EINVAL;
- goto err_out_free;
+ return -EINVAL;
}
if (!maps[i].ctrl_dev && !maps[i].ctrl_dev_name) {
pr_err("failed to register map %s (%d): "
"no pin control device given\n",
maps[i].name, i);
- ret = -EINVAL;
- goto err_out_free;
+ return -EINVAL;
}
if (!maps[i].function) {
pr_err("failed to register map %s (%d): "
"no function ID given\n", maps[i].name, i);
- ret = -EINVAL;
- goto err_out_free;
+ return -EINVAL;
}
if (!maps[i].dev && !maps[i].dev_name)
@@ -399,17 +381,33 @@ int __init pinmux_register_mappings(struct pinmux_map const *maps,
pr_debug("register map %s, function %s\n",
maps[i].name,
maps[i].function);
+ }
- pinmux_maps_num++;
+ /*
+ * Make a copy of the map array - string pointers will end up in the
+ * kernel const section anyway so these do not need to be deep copied.
+ */
+ if (!pinmux_maps_num) {
+ /* On first call, just copy them */
+ tmp_maps = kmemdup(maps,
+ sizeof(struct pinmux_map) * num_maps,
+ GFP_KERNEL);
+ if (!tmp_maps)
+ return -ENOMEM;
+ } else {
+ /* Subsequent calls, reallocate array to new size */
+ size_t oldsize = sizeof(struct pinmux_map) * pinmux_maps_num;
+ size_t newsize = sizeof(struct pinmux_map) * num_maps;
+
+ tmp_maps = krealloc(pinmux_maps, oldsize + newsize, GFP_KERNEL);
+ if (!tmp_maps)
+ return -ENOMEM;
+ memcpy((tmp_maps + oldsize), maps, newsize);
}
+ pinmux_maps = tmp_maps;
+ pinmux_maps_num += num_maps;
return 0;
-
-err_out_free:
- kfree(pinmux_maps);
- pinmux_maps = NULL;
- pinmux_maps_num = 0;
- return ret;
}
/**
OpenPOWER on IntegriCloud