diff options
author | Michael Hennerich <michael.hennerich@analog.com> | 2007-07-24 06:03:45 -0400 |
---|---|---|
committer | Bryan Wu <bryan.wu@analog.com> | 2007-07-24 06:03:45 -0400 |
commit | 314c98d589b9da40af42cfe7213eff04c74caea3 (patch) | |
tree | 3c91afc0d6b0f0150aa7f545c0060f96d751a9d4 /arch/blackfin/mach-bf548 | |
parent | 06039e90b90af4029184d577b3c66e59f6039a9e (diff) |
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 'arch/blackfin/mach-bf548')
-rw-r--r-- | arch/blackfin/mach-bf548/gpio.c | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/arch/blackfin/mach-bf548/gpio.c b/arch/blackfin/mach-bf548/gpio.c index c073ab36066d..f3b9deacd332 100644 --- a/arch/blackfin/mach-bf548/gpio.c +++ b/arch/blackfin/mach-bf548/gpio.c | |||
@@ -212,11 +212,18 @@ int peripheral_request_list(unsigned short per[], const char *label) | |||
212 | int ret; | 212 | int ret; |
213 | 213 | ||
214 | for (cnt = 0; per[cnt] != 0; cnt++) { | 214 | for (cnt = 0; per[cnt] != 0; cnt++) { |
215 | |||
215 | ret = peripheral_request(per[cnt], label); | 216 | ret = peripheral_request(per[cnt], label); |
216 | if (ret < 0) | 217 | |
217 | return ret; | 218 | if (ret < 0) { |
219 | for ( ; cnt > 0; cnt--) { | ||
220 | peripheral_free(per[cnt - 1]); | ||
221 | } | ||
222 | return ret; | ||
223 | } | ||
218 | } | 224 | } |
219 | 225 | ||
226 | |||
220 | return 0; | 227 | return 0; |
221 | } | 228 | } |
222 | EXPORT_SYMBOL(peripheral_request_list); | 229 | EXPORT_SYMBOL(peripheral_request_list); |