diff options
| author | Ksenija Stanojevic | 2016-01-03 20:44:44 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2016-01-29 07:23:38 +0100 |
| commit | 8aa7307b7802cb765fcc11305db3fdeee0920658 (patch) | |
| tree | 98e38bbacbd9c347c8edf601f08c353d29b52b65 /drivers/staging/panel | |
| parent | Staging: panel: Reduce value range for *name (diff) | |
| download | kernel-qcow2-linux-8aa7307b7802cb765fcc11305db3fdeee0920658.tar.gz kernel-qcow2-linux-8aa7307b7802cb765fcc11305db3fdeee0920658.tar.xz kernel-qcow2-linux-8aa7307b7802cb765fcc11305db3fdeee0920658.zip | |
Staging: panel: Make code more readable
Repace 'for' statement by strchr() function to make code more readable.
Signed-off-by: Ksenija Stanojevic <ksenija.stanojevic@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/panel')
| -rw-r--r-- | drivers/staging/panel/panel.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/staging/panel/panel.c b/drivers/staging/panel/panel.c index fcfcda7ff1f4..1219ff7884f8 100644 --- a/drivers/staging/panel/panel.c +++ b/drivers/staging/panel/panel.c @@ -2043,7 +2043,7 @@ static void init_scan_timer(void) static u8 input_name2mask(const char *name, __u64 *mask, __u64 *value, u8 *imask, u8 *omask) { - static char sigtab[10] = "EeSsPpAaBb"; + const char sigtab[] = "EeSsPpAaBb"; u8 im, om; __u64 m, v; @@ -2053,13 +2053,13 @@ static u8 input_name2mask(const char *name, __u64 *mask, __u64 *value, v = 0ULL; while (*name) { int in, out, bit, neg; + const char *idx; - for (in = 0; (in < sizeof(sigtab)) && (sigtab[in] != *name); - in++) - ; - - if (in >= sizeof(sigtab)) + idx = strchr(sigtab, *name); + if (!idx) return 0; /* input name not found */ + + in = idx - sigtab; neg = (in & 1); /* odd (lower) names are negated */ in >>= 1; im |= BIT(in); |