pinctrl: single: fix infinite loop caused by bad mask
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Thu, 9 Jan 2014 12:50:29 +0000 (14:50 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 15 Jan 2014 07:31:53 +0000 (08:31 +0100)
commit 4e7e8017a80e1 (pinctrl: pinctrl-single:
enhance to configure multiple pins of different modules) improved
support for pinctrl-single,bits option, but also caused a regression
in parsing badly configured mask data.

If the masks in DT data are not quite right,
pcs_parse_bits_in_pinctrl_entry() can end up in an infinite loop,
trashing memory at the same time.

Add a check to verify that each loop actually removes bits from the
'mask', so that the loop can eventually end.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Acked-by: Tony Lindgren <tony@atomide.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-single.c

Simple merge