From: Lee Jones <lee.jones@linaro.org>
To: linux-kernel@vger.kernel.org
Cc: linus.walleij@linaro.org,
Alexandre Torgue <alexandre.torgue@stericsson.com>,
Lee Jones <lee.jones@linaro.org>,
Yang QU <yang.qu@stericsson.com>
Subject: [PATCH 16/21] gpio: ab8500: Add internal pull up on GPIO of ab8540
Date: Fri, 14 Dec 2012 16:19:34 +0000 [thread overview]
Message-ID: <1355501979-1157-17-git-send-email-lee.jones@linaro.org> (raw)
In-Reply-To: <1355501979-1157-1-git-send-email-lee.jones@linaro.org>
From: Alexandre Torgue <alexandre.torgue@stericsson.com>
Add 4 GPIOs named GPIOx_VBAT which have both internal pull up
and pull down capability on ab8540. We can also select different
voltage domains for GPIOx_VBAT.
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Yang QU <yang.qu@stericsson.com>
Reviewed-by: Alexandre TORGUE <alexandre.torgue@stericsson.com>
Reviewed-by: Philippe LANGLAIS <philippe.langlais@stericsson.com>
Tested-by: Alexandre TORGUE <alexandre.torgue@stericsson.com>
Tested-by: Philippe LANGLAIS <philippe.langlais@stericsson.com>
---
drivers/gpio/gpio-ab8500.c | 64 ++++++++++++++++++++++++++++
include/linux/mfd/abx500/ab8500-gpio.h | 71 +++++++++++---------------------
2 files changed, 87 insertions(+), 48 deletions(-)
diff --git a/drivers/gpio/gpio-ab8500.c b/drivers/gpio/gpio-ab8500.c
index 757ae34..ff8df1f 100644
--- a/drivers/gpio/gpio-ab8500.c
+++ b/drivers/gpio/gpio-ab8500.c
@@ -72,6 +72,8 @@
#define AB8500_GPIO_IN5_REG 0x44
#define AB8500_GPIO_IN6_REG 0x45
#define AB9540_GPIO_IN7_REG 0x46
+#define AB8540_GPIO_VINSEL_REG 0x47
+#define AB8540_GPIO_PULL_UPDOWN_REG 0x48
#define AB8500_GPIO_ALTFUN_REG 0x50
#define AB8500_ALTFUN_REG_INDEX 6
#define AB9540_ALTFUN_REG_INDEX 7
@@ -80,6 +82,8 @@
#define AB8505_NUM_GPIO 53
#define AB8540_NUM_GPIO 56
#define AB8500_NUM_VIR_GPIO_IRQ 16
+#define AB8540_GPIO_PULL_UPDOWN_MASK 0x03
+#define AB8540_GPIO_VINSEL_MASK 0x03
enum ab8500_gpio_action {
NONE,
@@ -552,6 +556,20 @@ static int __devinit ab8500_gpio_probe(struct platform_device *pdev)
if (ret < 0)
goto out_free;
}
+
+ if (is_ab8540(parent)) {
+ ret = abx500_set_register_interruptible(ab8500_gpio->dev,
+ AB8500_MISC, AB8540_GPIO_VINSEL_REG,
+ pdata->config_vinsel);
+ if (ret < 0)
+ goto out_free;
+ ret = abx500_set_register_interruptible(ab8500_gpio->dev,
+ AB8500_MISC, AB8540_GPIO_PULL_UPDOWN_REG,
+ pdata->config_pullupdown);
+ if (ret < 0)
+ goto out_free;
+ }
+
ret = abx500_set_register_interruptible(ab8500_gpio->dev, AB8500_MISC,
AB8500_GPIO_ALTFUN_REG,
pdata->config_reg[altfun_reg_index]);
@@ -619,6 +637,52 @@ int ab8500_config_pulldown(struct device *dev,
EXPORT_SYMBOL(ab8500_config_pulldown);
/*
+ * ab8540_config_pull_updown() only available for following GPIOs:
+ * GPIO51(GPIO1_VBAT)
+ * GPIO52(GPIO2_VBAT)
+ * GPIO53(GPIO3_VBAT)
+ * GPIO54(GPIO4_VBAT)
+ */
+int ab8540_config_pull_updown(struct device *dev,
+ int gpio, enum ab8540_gpio_pull_updown reg)
+{
+ u8 offset = gpio - AB8500_PIN_GPIO(51);
+ u8 pos = 2 * offset;
+ int ret;
+
+ ret = abx500_mask_and_set_register_interruptible(dev,
+ AB8500_MISC, AB8540_GPIO_PULL_UPDOWN_REG,
+ AB8540_GPIO_PULL_UPDOWN_MASK << pos, reg << pos);
+ if (ret < 0)
+ dev_err(dev, "%s failed (%d)\n", __func__, ret);
+ return ret;
+}
+EXPORT_SYMBOL(ab8540_config_pull_updown);
+
+/*
+ * ab8540_gpio_config_vinsel() only available for following GPIOs:
+ * GPIO51(GPIO1_VBAT)
+ * GPIO52(GPIO2_VBAT)
+ * GPIO53(GPIO3_VBAT)
+ * GPIO54(GPIO4_VBAT)
+ */
+int ab8540_gpio_config_vinsel(struct device *dev,
+ int gpio, enum ab8540_gpio_vinsel reg)
+{
+ u8 offset = gpio - AB8500_PIN_GPIO(51);
+ u8 pos = 2 * offset;
+ int ret;
+
+ ret = abx500_mask_and_set_register_interruptible(dev,
+ AB8500_MISC, AB8540_GPIO_VINSEL_REG,
+ AB8540_GPIO_VINSEL_MASK << pos, reg << pos);
+ if (ret < 0)
+ dev_err(dev, "%s failed (%d)\n", __func__, ret);
+ return ret;
+}
+EXPORT_SYMBOL(ab8540_gpio_config_vinsel);
+
+/*
* ab8500_gpio_config_select()
*
* Configure functionality of pin, either specific use or GPIO.
diff --git a/include/linux/mfd/abx500/ab8500-gpio.h b/include/linux/mfd/abx500/ab8500-gpio.h
index 48325db..20df07d 100644
--- a/include/linux/mfd/abx500/ab8500-gpio.h
+++ b/include/linux/mfd/abx500/ab8500-gpio.h
@@ -18,62 +18,37 @@ struct ab8500_gpio_platform_data {
int gpio_base;
u32 irq_base;
u8 config_reg[8];
- u8 config_direction[6];
- u8 config_pullups[6];
+ u8 config_direction[7];
+ u8 config_pullups[7];
+ u8 config_vinsel;
+ u8 config_pullupdown;
};
-enum ab8500_pin {
- AB8500_PIN_GPIO1 = 0,
- AB8500_PIN_GPIO2,
- AB8500_PIN_GPIO3,
- AB8500_PIN_GPIO4,
- AB8500_PIN_GPIO5,
- AB8500_PIN_GPIO6,
- AB8500_PIN_GPIO7,
- AB8500_PIN_GPIO8,
- AB8500_PIN_GPIO9,
- AB8500_PIN_GPIO10,
- AB8500_PIN_GPIO11,
- AB8500_PIN_GPIO12,
- AB8500_PIN_GPIO13,
- AB8500_PIN_GPIO14,
- AB8500_PIN_GPIO15,
- AB8500_PIN_GPIO16,
- AB8500_PIN_GPIO17,
- AB8500_PIN_GPIO18,
- AB8500_PIN_GPIO19,
- AB8500_PIN_GPIO20,
- AB8500_PIN_GPIO21,
- AB8500_PIN_GPIO22,
- AB8500_PIN_GPIO23,
- AB8500_PIN_GPIO24,
- AB8500_PIN_GPIO25,
- AB8500_PIN_GPIO26,
- AB8500_PIN_GPIO27,
- AB8500_PIN_GPIO28,
- AB8500_PIN_GPIO29,
- AB8500_PIN_GPIO30,
- AB8500_PIN_GPIO31,
- AB8500_PIN_GPIO32,
- AB8500_PIN_GPIO33,
- AB8500_PIN_GPIO34,
- AB8500_PIN_GPIO35,
- AB8500_PIN_GPIO36,
- AB8500_PIN_GPIO37,
- AB8500_PIN_GPIO38,
- AB8500_PIN_GPIO39,
- AB8500_PIN_GPIO40,
- AB8500_PIN_GPIO41,
- AB8500_PIN_GPIO42,
+enum ab8540_gpio_pull_updown {
+ AB8540_GPIO_PULL_DOWN = 0x0,
+ AB8540_GPIO_PULL_NONE = 0x1,
+ AB8540_GPIO_PULL_UP = 0x3,
+};
+
+enum ab8540_gpio_vinsel {
+ AB8540_GPIO_VINSEL_VBAT = 0x0,
+ AB8540_GPIO_VINSEL_VIN_1V8 = 0x1,
+ AB8540_GPIO_VINSEL_VDD_BIF = 0x2,
};
int ab8500_config_pulldown(struct device *dev,
- enum ab8500_pin gpio, bool enable);
+ int gpio, bool enable);
+
+int ab8540_config_pull_updown(struct device *dev,
+ int gpio, enum ab8540_gpio_pull_updown reg);
+
+int ab8540_gpio_config_vinsel(struct device *dev,
+ int gpio, enum ab8540_gpio_vinsel reg);
int ab8500_gpio_config_select(struct device *dev,
- enum ab8500_pin gpio, bool gpio_select);
+ int gpio, bool gpio_select);
int ab8500_gpio_config_get_select(struct device *dev,
- enum ab8500_pin gpio, bool *gpio_select);
+ int gpio, bool *gpio_select);
#endif /* _AB8500_GPIO_H */
--
1.7.9.5
next prev parent reply other threads:[~2012-12-14 16:20 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-14 16:19 [PATCH 00/21] gpio: ab8500: Bring AB8500 back to life Lee Jones
2012-12-14 16:19 ` [PATCH 01/21] gpio: ab8500: Activate and port AB8500 GPIO driver to new framework Lee Jones
2012-12-14 16:19 ` [PATCH 02/21] gpio: ab8500: Make pins configurable Lee Jones
2012-12-14 16:19 ` [PATCH 03/21] gpio: ab8500: Fix alternate function register address Lee Jones
2012-12-14 16:19 ` [PATCH 04/21] gpio: ab8500: Read register corrected in get_value api Lee Jones
2012-12-14 16:19 ` [PATCH 05/21] gpio: ab8500: Allow direction and pullups configuration Lee Jones
2012-12-14 16:19 ` [PATCH 06/21] gpio: ab8500: Add api to enable pulldown Lee Jones
2012-12-14 16:19 ` [PATCH 07/21] gpio: ab8500: Write argument value instead of hardwired 1 Lee Jones
2012-12-14 16:19 ` [PATCH 08/21] gpio: ab8500: Fix gpio offset bounds for irq mapping Lee Jones
2012-12-14 16:19 ` [PATCH 09/21] gpio: ab8500: Fix bad include name after renaming Lee Jones
2012-12-14 16:19 ` [PATCH 10/21] gpio: ab8500: Add support for the AB9540 Lee Jones
2012-12-14 16:19 ` [PATCH 11/21] gpio: ab8500: Add support for AB8505 Chip Lee Jones
2012-12-14 16:19 ` [PATCH 12/21] gpio: ab8500: Allow gpios to wake the system from suspend Lee Jones
2012-12-14 16:19 ` [PATCH 13/21] gpio: ab8500: Use most recent run-time platform checker Lee Jones
2012-12-14 16:19 ` [PATCH 14/21] gpio: ab8500: Remove ENUMs from linux/mfd/abx500/ab8500-gpio.h Lee Jones
2012-12-14 16:19 ` [PATCH 15/21] gpio: ab8500: Add support for the ab8540 Lee Jones
2012-12-14 16:19 ` Lee Jones [this message]
2012-12-14 16:19 ` [PATCH 17/21] gpio: ab8500: Add the action range for the internal pull up function on GPIO Lee Jones
2012-12-14 16:19 ` [PATCH 18/21] gpio: ab8500: Fix parameter uninitialized warning for ab8540 Lee Jones
2012-12-14 16:19 ` [PATCH 19/21] gpio: ab8500: Update gpio ab8540 interrupt mapping Lee Jones
2012-12-14 16:19 ` [PATCH 20/21] gpio: ab8500: Fix ab8540 setting direction output error Lee Jones
2012-12-14 16:19 ` [PATCH 21/21] gpio: ab8500: Add explicit dependencies Lee Jones
2012-12-19 14:18 ` [PATCH 00/21] gpio: ab8500: Bring AB8500 back to life Linus Walleij
2012-12-19 22:49 ` Grant Likely
2012-12-20 7:51 ` Lee Jones
2013-01-04 14:17 ` Linus Walleij
2013-01-04 14:46 ` Lee Jones
2012-12-19 22:26 ` Grant Likely
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1355501979-1157-17-git-send-email-lee.jones@linaro.org \
--to=lee.jones@linaro.org \
--cc=alexandre.torgue@stericsson.com \
--cc=linus.walleij@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=yang.qu@stericsson.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).