summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSebastien Guiriec2012-10-16 17:23:20 +0200
committerWolfram Sang2012-11-14 17:44:37 +0100
commit2d4b4520a5eaed6701b0c9f7540c8fd99a26e449 (patch)
tree760d720fc68cafd7f44cbaac54c3abe233bbff61
parenti2c: omap: ensure writes to dev->buf_len are ordered (diff)
downloadkernel-qcow2-linux-2d4b4520a5eaed6701b0c9f7540c8fd99a26e449.tar.gz
kernel-qcow2-linux-2d4b4520a5eaed6701b0c9f7540c8fd99a26e449.tar.xz
kernel-qcow2-linux-2d4b4520a5eaed6701b0c9f7540c8fd99a26e449.zip
i2c: omap: adopt pinctrl support
Some GPIO expanders need some early pin control muxing. Due to legacy boards sometimes the driver uses subsys_initcall instead of module_init. This patch takes advantage of defer probe feature and pin control in order to wait until pin control probing before GPIO driver probing. It has been tested on OMAP5 board with TCA6424 driver. Signed-off-by: Sebastien Guiriec <s-guiriec@ti.com> Acked-by: Shubhrajyoti D <shubhrajyoti@ti.com> Reviewed-by: Felipe Balbi <balbi@ti.com> Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
-rw-r--r--drivers/i2c/busses/i2c-omap.c13
1 files changed, 13 insertions, 0 deletions
diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
index 3525c9e62cb0..16afb289cca7 100644
--- a/drivers/i2c/busses/i2c-omap.c
+++ b/drivers/i2c/busses/i2c-omap.c
@@ -43,6 +43,7 @@
#include <linux/slab.h>
#include <linux/i2c-omap.h>
#include <linux/pm_runtime.h>
+#include <linux/pinctrl/consumer.h>
/* I2C controller revisions */
#define OMAP_I2C_OMAP1_REV_2 0x20
@@ -213,6 +214,8 @@ struct omap_i2c_dev {
u16 syscstate;
u16 westate;
u16 errata;
+
+ struct pinctrl *pins;
};
static const u8 reg_map_ip_v1[] = {
@@ -1104,6 +1107,16 @@ omap_i2c_probe(struct platform_device *pdev)
dev->dtrev = pdata->rev;
}
+ dev->pins = devm_pinctrl_get_select_default(&pdev->dev);
+ if (IS_ERR(dev->pins)) {
+ if (PTR_ERR(dev->pins) == -EPROBE_DEFER)
+ return -EPROBE_DEFER;
+
+ dev_warn(&pdev->dev, "did not get pins for i2c error: %li\n",
+ PTR_ERR(dev->pins));
+ dev->pins = NULL;
+ }
+
dev->dev = &pdev->dev;
dev->irq = irq;