diff options
author | Michael Hennerich <michael.hennerich@analog.com> | 2008-10-13 11:35:22 +0800 |
---|---|---|
committer | Bryan Wu <cooloney@kernel.org> | 2008-10-13 11:35:22 +0800 |
commit | e9fae189caae7c1cf306e30f5b67c6d226ed69cf (patch) | |
tree | 059ece3216ac9c01b943e41b059f26c6ff577685 /arch/blackfin/kernel/bfin_gpio.c | |
parent | 46aa04f9b678d1d6f3558429109326775ca87715 (diff) | |
download | op-kernel-dev-e9fae189caae7c1cf306e30f5b67c6d226ed69cf.zip op-kernel-dev-e9fae189caae7c1cf306e30f5b67c6d226ed69cf.tar.gz |
Blackfin arch: Fix bogus str_ident check in gpio code
Signed-off-by: Michael Hennerich <michael.hennerich@analog.com>
Signed-off-by: Bryan Wu <cooloney@kernel.org>
Diffstat (limited to 'arch/blackfin/kernel/bfin_gpio.c')
-rw-r--r-- | arch/blackfin/kernel/bfin_gpio.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/arch/blackfin/kernel/bfin_gpio.c b/arch/blackfin/kernel/bfin_gpio.c index 12710ed..6e08f42 100644 --- a/arch/blackfin/kernel/bfin_gpio.c +++ b/arch/blackfin/kernel/bfin_gpio.c @@ -238,7 +238,7 @@ static void gpio_error(unsigned gpio) static void set_label(unsigned short ident, const char *label) { - if (label && str_ident) { + if (label) { strncpy(str_ident[ident].name, label, RESOURCE_LABEL_SIZE); str_ident[ident].name[RESOURCE_LABEL_SIZE - 1] = 0; @@ -247,9 +247,6 @@ static void set_label(unsigned short ident, const char *label) static char *get_label(unsigned short ident) { - if (!str_ident) - return "UNKNOWN"; - return (*str_ident[ident].name ? str_ident[ident].name : "UNKNOWN"); } @@ -260,7 +257,7 @@ static int cmp_label(unsigned short ident, const char *label) printk(KERN_ERR "Please provide none-null label\n"); } - if (label && str_ident) + if (label) return strncmp(str_ident[ident].name, label, strlen(label)); else |