diff options
author | Randall Spangler <rspangler@chromium.org> | 2013-08-02 14:28:43 -0700 |
---|---|---|
committer | ChromeBot <chrome-bot@google.com> | 2013-08-07 12:43:35 -0700 |
commit | f2b56fcb9fe078d5a29f1c3744e47e77240cd4e7 (patch) | |
tree | 4ff810332c543871da7febb2cc21597eee800b6d /chip/lm4/gpio.c | |
parent | ce704b40005c1acdffabe58f47df42c1d7da33c2 (diff) | |
download | chrome-ec-f2b56fcb9fe078d5a29f1c3744e47e77240cd4e7.tar.gz |
Clean up configuring GPIO alternate functions
GPIO alternate functions used to be configured throughout the code,
which made it hard to tell which ones you needed to configure yourself
in board.c. It also sometimes (chip/lm4/i2c.c) led to GPIOs being
configured as alternate functions even if they weren't used on a given
board.
With this change, every board has a table in board.c which lists ALL
GPIOs which have alternate functions. This is now the only place
where alternate functions are configured. Each module then calls
gpio_init_module() to set up its GPIOs.
This also fixes a bug where gpio_set_flags() ignored most of the flags
passed to it (only direction and level were actually used).
On stm32f, gpio_set_alternate() does not exist, and pins are
configured via direct register writes from board.c. Rather than
attempt to change that in the same CL, I've stubbed out
gpio_set_alternate() for stm32f, and will fix the register writes in a
follow-up CL.
BUG=chrome-os-partner:21618
BRANCH=peppy (fixes I2C1 being initialized even though those pins are used
for other things)
TEST=boot link, falco, pit, spring
Change-Id: I40f47025d8f767e0723c6b40c80413af9ba8deba
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/64400
Diffstat (limited to 'chip/lm4/gpio.c')
-rw-r--r-- | chip/lm4/gpio.c | 80 |
1 files changed, 40 insertions, 40 deletions
diff --git a/chip/lm4/gpio.c b/chip/lm4/gpio.c index ab6423d570..d3a060904d 100644 --- a/chip/lm4/gpio.c +++ b/chip/lm4/gpio.c @@ -74,7 +74,6 @@ void gpio_set_alternate_function(int port, int mask, int func) } else { LM4_GPIO_AFSEL(port) &= ~mask; } - LM4_GPIO_DEN(port) |= mask; } test_mockable int gpio_get_level(enum gpio_signal signal) @@ -93,58 +92,59 @@ void gpio_set_level(enum gpio_signal signal, int value) gpio_list[signal].mask) = (value ? 0xff : 0); } -void gpio_set_flags(enum gpio_signal signal, int flags) +void gpio_set_flags_by_mask(uint32_t port, uint32_t mask, uint32_t flags) { - const struct gpio_info *g = gpio_list + signal; + /* + * Select open drain first, so that we don't glitch the signal + * when changing the line to an output. + */ + if (flags & GPIO_OPEN_DRAIN) + LM4_GPIO_ODR(port) |= mask; + else + LM4_GPIO_ODR(port) &= ~mask; - if (flags & GPIO_OUTPUT) { - /* - * Select open drain first, so that we don't glitch the signal - * when changing the line to an output. - */ - if (g->flags & GPIO_OPEN_DRAIN) - LM4_GPIO_ODR(g->port) |= g->mask; - else - LM4_GPIO_ODR(g->port) &= ~g->mask; - - LM4_GPIO_DIR(g->port) |= g->mask; - - /* Set level if necessary */ - if (flags & GPIO_HIGH) - gpio_set_level(signal, 1); - else if (flags & GPIO_LOW) - gpio_set_level(signal, 0); - } else { - /* Input */ - LM4_GPIO_DIR(g->port) &= ~g->mask; - } + if (flags & GPIO_OUTPUT) + LM4_GPIO_DIR(port) |= mask; + else + LM4_GPIO_DIR(port) &= ~mask; /* Handle pullup / pulldown */ - if (g->flags & GPIO_PULL_UP) { - LM4_GPIO_PUR(g->port) |= g->mask; - } else if (g->flags & GPIO_PULL_DOWN) { - LM4_GPIO_PDR(g->port) |= g->mask; + if (flags & GPIO_PULL_UP) { + LM4_GPIO_PUR(port) |= mask; + } else if (flags & GPIO_PULL_DOWN) { + LM4_GPIO_PDR(port) |= mask; } else { /* No pull up/down */ - LM4_GPIO_PUR(g->port) &= ~g->mask; - LM4_GPIO_PDR(g->port) &= ~g->mask; + LM4_GPIO_PUR(port) &= ~mask; + LM4_GPIO_PDR(port) &= ~mask; } /* Set up interrupt type */ - if (g->flags & GPIO_INT_LEVEL) - LM4_GPIO_IS(g->port) |= g->mask; + if (flags & GPIO_INT_LEVEL) + LM4_GPIO_IS(port) |= mask; else - LM4_GPIO_IS(g->port) &= ~g->mask; + LM4_GPIO_IS(port) &= ~mask; - if (g->flags & (GPIO_INT_RISING | GPIO_INT_HIGH)) - LM4_GPIO_IEV(g->port) |= g->mask; + if (flags & (GPIO_INT_RISING | GPIO_INT_HIGH)) + LM4_GPIO_IEV(port) |= mask; else - LM4_GPIO_IEV(g->port) &= ~g->mask; + LM4_GPIO_IEV(port) &= ~mask; - if (g->flags & GPIO_INT_BOTH) - LM4_GPIO_IBE(g->port) |= g->mask; + if (flags & GPIO_INT_BOTH) + LM4_GPIO_IBE(port) |= mask; else - LM4_GPIO_IBE(g->port) &= ~g->mask; + LM4_GPIO_IBE(port) &= ~mask; + + if (flags & GPIO_ANALOG) + LM4_GPIO_DEN(port) &= ~mask; + else + LM4_GPIO_DEN(port) |= mask; + + /* Set level */ + if (flags & GPIO_HIGH) + LM4_GPIO_DATA(port, mask) = 0xff; + else if (flags & GPIO_LOW) + LM4_GPIO_DATA(port, mask) = 0; } int gpio_enable_interrupt(enum gpio_signal signal) @@ -210,7 +210,7 @@ void gpio_pre_init(void) flags &= ~(GPIO_LOW | GPIO_HIGH); /* Set up GPIO based on flags */ - gpio_set_flags(i, flags); + gpio_set_flags_by_mask(g->port, g->mask, flags); /* Use as GPIO, not alternate function */ gpio_set_alternate_function(g->port, g->mask, -1); |