]> git.baikalelectronics.ru Git - kernel.git/commit
Subject: pinctrl: imx1-core: Fix debug output in .pin_config_set callback
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 17 Jul 2015 07:38:43 +0000 (09:38 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Mon, 20 Jul 2015 09:01:53 +0000 (11:01 +0200)
commitc950a729e5e2f17df5bb73a524af7d9f936b76f5
treedc3e265e2514ec7953b2cd4385fa08ec9a2d6d74
parent4cf1c48ab38867de94960d0ff8f0d1ec634c1eb8
Subject: pinctrl: imx1-core: Fix debug output in .pin_config_set callback

imx1_pinconf_set assumes that the array of pins in struct
imx1_pinctrl_soc_info can be indexed by pin id to get the
pinctrl_pin_desc for a pin. This used to be correct up to commit
cdef1d92fb6e which removed some entries from the array and so made it
wrong to access the array by pin id.

The result of this bug is a wrong pin name in the output for small pin
ids and an oops for the bigger ones.

This patch is the result of a discussion that includes patches by Markus
Pargmann and Chris Ruehl.

Fixes: cdef1d92fb6e ("pinctrl: i.MX27: Remove nonexistent pad definitions")
Cc: stable@vger.kernel.org
Reported-by: Chris Ruehl <chris.ruehl@gtsys.com.hk>
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Reviewed-by: Markus Pargmann <mpa@pengutronix.de>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/freescale/pinctrl-imx1-core.c