From 645680d62b82455d24fcb178f1cc5208f9942e38 Mon Sep 17 00:00:00 2001 From: Amaury Pouly Date: Thu, 10 May 2012 00:08:03 +0200 Subject: imx233: add pinctrl debug code to track pin uses Past development has proved that one can mistakely use the same pin for two uses without noticing. Since this causes extremely hard to find bugs, the infrastructure will allow to register pin uses and panic when a conflict is detected. The pinctrl debug now shows the pin uses when its support is compiled in. Change-Id: Idb2d5235ce09207d77aa474d6f158e72b933761a --- firmware/target/arm/imx233/pinctrl-imx233.c | 39 +++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+) (limited to 'firmware/target/arm/imx233/pinctrl-imx233.c') diff --git a/firmware/target/arm/imx233/pinctrl-imx233.c b/firmware/target/arm/imx233/pinctrl-imx233.c index 7997292fcb..be2d8b2262 100644 --- a/firmware/target/arm/imx233/pinctrl-imx233.c +++ b/firmware/target/arm/imx233/pinctrl-imx233.c @@ -21,8 +21,47 @@ #include "system.h" #include "system-target.h" #include "cpu.h" +#include "string.h" #include "pinctrl-imx233.h" +#ifdef IMX233_PINCTRL_DEBUG +// 4 banks of 32 pins +static const char *pin_use[4][32]; + +void imx233_pinctrl_acquire_pin(unsigned bank, unsigned pin, const char *name) +{ + if(pin_use[bank][pin] != NULL && pin_use[bank][pin] != name) + panicf("acquire B%dP%02d for %s, was %s!", bank, pin, name, pin_use[bank][pin]); + pin_use[bank][pin] = name; +} + +void imx233_pinctrl_acquire_pin_mask(unsigned bank, uint32_t mask, const char *name) +{ + for(unsigned pin = 0; pin < 32; pin++) + if(mask & (1 << pin)) + imx233_pinctrl_acquire_pin(bank, pin, name); +} + +void imx233_pinctrl_release_pin(unsigned bank, unsigned pin, const char *name) +{ + if(pin_use[bank][pin] != NULL && pin_use[bank][pin] != name) + panicf("release B%dP%02d for %s: was %s!", bank, pin, name, pin_use[bank][pin]); + pin_use[bank][pin] = NULL; +} + +void imx233_pinctrl_release_pin_mask(unsigned bank, uint32_t mask, const char *name) +{ + for(unsigned pin = 0; pin < 32; pin++) + if(mask & (1 << pin)) + imx233_pinctrl_release_pin(bank, pin, name); +} + +const char *imx233_pinctrl_get_pin_use(unsigned bank, unsigned pin) +{ + return pin_use[bank][pin]; +} +#endif + static pin_irq_cb_t pin_cb[3][32]; /* 3 banks, 32 pins/bank */ static void INT_GPIO(int bank) -- cgit v1.2.3