Commit a386267a authored by Linus Walleij's avatar Linus Walleij

pinctrl: pinconf: take the right mutex

The pinconf_dgb_config_print() takes the per-pincontroller
mutex, when what it wants to take is actually the pin maps
mutex.
Reported-by: default avatarJames Hogan <james.hogan@imgtec.com>
Cc: Patrice Chotard <patrice.chotard@st.com>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent d72f88a4
...@@ -610,7 +610,7 @@ static int pinconf_dbg_config_print(struct seq_file *s, void *d) ...@@ -610,7 +610,7 @@ static int pinconf_dbg_config_print(struct seq_file *s, void *d)
bool found = false; bool found = false;
unsigned long config; unsigned long config;
mutex_lock(&pctldev->mutex); mutex_lock(&pinctrl_maps_mutex);
/* Parse the pinctrl map and look for the elected pin/state */ /* Parse the pinctrl map and look for the elected pin/state */
for_each_maps(maps_node, i, map) { for_each_maps(maps_node, i, map) {
...@@ -659,7 +659,7 @@ static int pinconf_dbg_config_print(struct seq_file *s, void *d) ...@@ -659,7 +659,7 @@ static int pinconf_dbg_config_print(struct seq_file *s, void *d)
confops->pin_config_config_dbg_show(pctldev, s, config); confops->pin_config_config_dbg_show(pctldev, s, config);
exit: exit:
mutex_unlock(&pctldev->mutex); mutex_unlock(&pinctrl_maps_mutex);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment