pinctrl: pinctrl-imx: free allocated pinctrl_map structure only once and use kernel...
authorDevendra Naga <devendra.aaru@gmail.com>
Fri, 8 Jun 2012 19:22:11 +0000 (00:52 +0530)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 12 Jun 2012 11:35:32 +0000 (13:35 +0200)
commit3a86a5f8abb33956446ae31b1e9c149d7b2d1d21
treee5a6c3a69c7b8e3e67b9d45fc2ee4aab9f47e790
parente85bbc19d5a9ad2b48bf8f2fd3277d6fe58f23ce
pinctrl: pinctrl-imx: free allocated pinctrl_map structure only once and use kernel facilities for IMX_PMX_DUMP

a) as we allocate the pinctrl_map structure at imx_dt_node_to_map at  line 167, anyway
   if its an element, or a num_elements * (sizeof(type)) elements allocated to one single
   pointer must be freed only once.

CASE. A)

as new_map is not moved and allocated like,

for (i = 0; i < MAX_ELEMS; i++) {
    new_map[i] = kmalloc(numelems * size, GFP_KERNEL);
}

its freed as

for (i = 0; i < MAX_ELEMS; i++) {
    kfree(new_map[i]);
}

CASE. B)
and its allocated like
        new_map = kmalloc(numelems * size, GFP_KERNEL);

it just needs kfree not as case A's.

b) use KERN_DEBUG facility for the IMX_PMX_DUMP macro.

Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Acked-by: Dong Aisheng <dong.aisheng@linaro.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-imx.c