diff options
| author | Bill Richardson <wfrichar@chromium.org> | 2016-09-01 17:57:05 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-09-09 17:15:25 -0700 |
| commit | eb10d4518b0e37c483fbc0ec4915df47dee10f45 (patch) | |
| tree | bfc7afb6e9bacca7ca8268c88766fb3acefc87a6 /include | |
| parent | d9048f0896763f4ea30e3fca9f7db0b955ab2fd3 (diff) | |
| download | chrome-ec-eb10d4518b0e37c483fbc0ec4915df47dee10f45.tar.gz | |
Add check to prevent duplicate PIN assignments
All PIN() assignments in board/$BOARD/gpio.inc must be unique,
since otherwise you're just creating duplicate names and table
entries for the same core interrupt and may not be initializing
things the way you think.
BUG=none
BRANCH=none
TEST=make buildall; test on Cr50 hardware
Also verified that the image size is exactly the same before an
after this CL.
Change-Id: Ifb1805a010905f67fc5c0d246b6252af73715409
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/383773
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'include')
| -rw-r--r-- | include/gpio_list.h | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/include/gpio_list.h b/include/gpio_list.h index 5fa9839e13..802a943b0b 100644 --- a/include/gpio_list.h +++ b/include/gpio_list.h @@ -37,3 +37,18 @@ const int gpio_ih_count = ARRAY_SIZE(gpio_irq_handlers); #define GPIO_INT(name, pin, flags, signal) \ BUILD_ASSERT(GPIO_##name < ARRAY_SIZE(gpio_irq_handlers)); #include "gpio.wrap" + +/* + * All PIN(...) assignments must be unique, since otherwise you're just + * creating duplicate names for the same thing, and may not always be + * initializing them the way you think. + */ +#define GPIO(name, pin, flags) pin +#define GPIO_INT(name, pin, flags, signal) pin +/* + * The compiler will complain if we use the same name twice. The linker ignores + * anything that gets by. + */ +#define PIN(a, b...) static const int _pin_ ## a ## _ ## b \ + __attribute__((unused, section(".unused"))) = __LINE__; +#include "gpio.wrap" |