pinctrl: don't print unavailable function groups

There is no reason to try to print groups associated to a function if
get_function_groups returns an error. Moreover, it can lead to a NULL
pointer dereference error.

Signed-off-by: Ludovic Desroches <ludovic.desroches@atmel.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
This commit is contained in:
Ludovic Desroches 2015-06-08 17:16:37 +02:00 committed by Linus Walleij
parent 1bb6fad359
commit 9d7ebbbf22

View File

@ -557,9 +557,12 @@ static int pinmux_functions_show(struct seq_file *s, void *what)
ret = pmxops->get_function_groups(pctldev, func_selector, ret = pmxops->get_function_groups(pctldev, func_selector,
&groups, &num_groups); &groups, &num_groups);
if (ret) if (ret) {
seq_printf(s, "function %s: COULD NOT GET GROUPS\n", seq_printf(s, "function %s: COULD NOT GET GROUPS\n",
func); func);
func_selector++;
continue;
}
seq_printf(s, "function: %s, groups = [ ", func); seq_printf(s, "function: %s, groups = [ ", func);
for (i = 0; i < num_groups; i++) for (i = 0; i < num_groups; i++)