Re: [PATCH] vt8500: pinctrl: Change devicetree data parsing

From: Tony Prisk
Date: Thu Jan 23 2014 - 03:59:48 EST


On 23/01/14 21:43, Linus Walleij wrote:
On Thu, Jan 23, 2014 at 7:31 AM, Tony Prisk <linux@xxxxxxxxxxxxxxx> wrote:

Due to an assumption in the VT8500 pinctrl driver, the value passed
from devicetree for 'wm,pull' was not explicitly translated before
being passed to pinconf.

With changes to 'enum pin_config_param', PIN_CONFIG_BIAS_PULL_(UP/DOWN)
no longer map 1-to-1 with the expected values in devicetree.

This patch adds a small translation between the devicetree values (0..2)
and the enum pin_config_param equivalent values.

Signed-off-by: Tony Prisk <linux@xxxxxxxxxxxxxxx>
Isn't this a regression fix for stable?

I wasn't sure how to handle this since it's been a problem since 3.10 but no one has mentioned it (or patched it).

I assume this is because:
a) pinctrl is only used for I2C in mainline - and there is no mainline I2C consumer device drivers for the WonderMedia devices
b) most users are using the non-mainline kernel which has more support (as nasty as some of it is).

Regards
Tony Prisk
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/