diff options
author | Wolfram Sang <w.sang@pengutronix.de> | 2011-01-12 20:00:23 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-01-13 11:03:14 -0500 |
commit | 0fdae42d361bbb431ca0ab0efed5126a94821177 (patch) | |
tree | 958a045aca3e40ca487dac67597496a7ed885115 /include/linux/gpio.h | |
parent | 49a367937fe4250144e24440e5a11ae4344202b1 (diff) |
gpiolib: annotate gpio-intialization with __must_check
Because GPIOs can have crucial functions especially in embedded systems,
we are better safe than sorry regarding their configuration. For
gpio_request, the documentation is simply enforced: <quote>"The return
value of gpio_request() must be checked."</quote> For gpio_direction_* and
gpio_request_*, we now act accordingly.
Signed-off-by: Wolfram Sang <w.sang@pengutronix.de>
Cc: David Brownell <dbrownell@users.sourceforge.net>
Cc: Greg KH <gregkh@suse.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'include/linux/gpio.h')
-rw-r--r-- | include/linux/gpio.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/linux/gpio.h b/include/linux/gpio.h index e41f7dd1ae67..1d5214a89110 100644 --- a/include/linux/gpio.h +++ b/include/linux/gpio.h | |||
@@ -29,7 +29,7 @@ static inline int gpio_is_valid(int number) | |||
29 | return 0; | 29 | return 0; |
30 | } | 30 | } |
31 | 31 | ||
32 | static inline int gpio_request(unsigned gpio, const char *label) | 32 | static inline int __must_check gpio_request(unsigned gpio, const char *label) |
33 | { | 33 | { |
34 | return -ENOSYS; | 34 | return -ENOSYS; |
35 | } | 35 | } |
@@ -42,12 +42,12 @@ static inline void gpio_free(unsigned gpio) | |||
42 | WARN_ON(1); | 42 | WARN_ON(1); |
43 | } | 43 | } |
44 | 44 | ||
45 | static inline int gpio_direction_input(unsigned gpio) | 45 | static inline int __must_check gpio_direction_input(unsigned gpio) |
46 | { | 46 | { |
47 | return -ENOSYS; | 47 | return -ENOSYS; |
48 | } | 48 | } |
49 | 49 | ||
50 | static inline int gpio_direction_output(unsigned gpio, int value) | 50 | static inline int __must_check gpio_direction_output(unsigned gpio, int value) |
51 | { | 51 | { |
52 | return -ENOSYS; | 52 | return -ENOSYS; |
53 | } | 53 | } |