diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2016-03-17 17:22:17 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-03-17 18:09:34 -0400 |
commit | a7c1d0a987ee3be0b87db5c95aa4fbadf7c3c1c2 (patch) | |
tree | f06cf8ee8b6a4e565c02f150187b69d171e604eb /drivers/base | |
parent | 56b060814e2d87d6646a85a2f4609c73587399ca (diff) |
device property: convert to use match_string() helper
The new helper returns index of the mathing string in an array. We
would use it here.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Mika Westerberg <mika.westerberg@linux.intel.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/base')
-rw-r--r-- | drivers/base/property.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/drivers/base/property.c b/drivers/base/property.c index 76628a7b45f1..9b1a65debd49 100644 --- a/drivers/base/property.c +++ b/drivers/base/property.c | |||
@@ -651,7 +651,7 @@ int fwnode_property_match_string(struct fwnode_handle *fwnode, | |||
651 | const char *propname, const char *string) | 651 | const char *propname, const char *string) |
652 | { | 652 | { |
653 | const char **values; | 653 | const char **values; |
654 | int nval, ret, i; | 654 | int nval, ret; |
655 | 655 | ||
656 | nval = fwnode_property_read_string_array(fwnode, propname, NULL, 0); | 656 | nval = fwnode_property_read_string_array(fwnode, propname, NULL, 0); |
657 | if (nval < 0) | 657 | if (nval < 0) |
@@ -668,13 +668,9 @@ int fwnode_property_match_string(struct fwnode_handle *fwnode, | |||
668 | if (ret < 0) | 668 | if (ret < 0) |
669 | goto out; | 669 | goto out; |
670 | 670 | ||
671 | ret = -ENODATA; | 671 | ret = match_string(values, nval, string); |
672 | for (i = 0; i < nval; i++) { | 672 | if (ret < 0) |
673 | if (!strcmp(values[i], string)) { | 673 | ret = -ENODATA; |
674 | ret = i; | ||
675 | break; | ||
676 | } | ||
677 | } | ||
678 | out: | 674 | out: |
679 | kfree(values); | 675 | kfree(values); |
680 | return ret; | 676 | return ret; |