pinctrl: mxs: atomically switch mux and drive strength config
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Thu, 18 May 2017 09:23:55 +0000 (11:23 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 23 May 2017 09:27:53 +0000 (11:27 +0200)
commitda6c2addf66d7ff7d0b090d6267d4292f951e4e6
tree10f4412dc93b1774c49802bafa47988805e58baa
parent2a8209fa68236ad65363dba03db5dbced520268a
pinctrl: mxs: atomically switch mux and drive strength config

To set the mux mode of a pin two bits must be set. Up to now this is
implemented using the following idiom:

writel(mask, reg + CLR);
writel(value, reg + SET);

. This however results in the mux mode being 0 between the two writes.

On my machine there is an IC's reset pin connected to LCD_D20. The
bootloader configures this pin as GPIO output-high (i.e. not holding the
IC in reset). When Linux reconfigures the pin to GPIO the short time
LCD_D20 is muxed as LCD_D20 instead of GPIO_1_20 is enough to confuse
the connected IC.

The same problem is present for the pin's drive strength setting which is
reset to low drive strength before using the right value.

So instead of relying on the hardware to modify the register setting
using two writes implement the bit toggling using read-modify-write.

Fixes: 17723111e64f ("pinctrl: add pinctrl-mxs support")
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Acked-by: Shawn Guo <shawnguo@kernel.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/freescale/pinctrl-mxs.c