diff options
| author |   Michael Hennerich <michael.hennerich@analog.com> | 2007-07-24 18:03:45 +0800 |
|---|---|---|
| committer |   Bryan Wu <bryan.wu@analog.com> | 2007-07-24 18:03:45 +0800 |
| commit | 314c98d589b9da40af42cfe7213eff04c74caea3 (patch) | |
| tree | 3c91afc0d6b0f0150aa7f545c0060f96d751a9d4 /arch/blackfin/kernel/bfin_gpio.c | |
| parent | Blackfin arch: Fix define - SPORT0_DTPRI is first function (diff) | |
| download | linux-dev-314c98d589b9da40af42cfe7213eff04c74caea3.tar.xz linux-dev-314c98d589b9da40af42cfe7213eff04c74caea3.zip | |
Blackfin arch: add missing gpio error handling to make sure we roll back requests in case one fails
Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Signed-off-by: Bryan Wu <bryan.wu@analog.com>
Diffstat (limited to '')
| -rw-r--r-- | arch/blackfin/kernel/bfin_gpio.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/arch/blackfin/kernel/bfin_gpio.c b/arch/blackfin/kernel/bfin_gpio.c index b818a8da1215..979cf79d3799 100644 --- a/arch/blackfin/kernel/bfin_gpio.c +++ b/arch/blackfin/kernel/bfin_gpio.c @@ -711,9 +711,15 @@ int peripheral_request_list(unsigned short per[], const char *label) int ret; for (cnt = 0; per[cnt] != 0; cnt++) { + ret = peripheral_request(per[cnt], label); - if (ret < 0) - return ret; + + if (ret < 0) { + for ( ; cnt > 0; cnt--) { + peripheral_free(per[cnt - 1]); + } + return ret; + } } return 0; |