diff options
author | Heikki Krogerus <heikki.krogerus@linux.intel.com> | 2016-03-17 17:22:38 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-03-17 18:09:34 -0400 |
commit | efc2cd7937fd3c32a8e0ffcdfa2603989f0a2faa (patch) | |
tree | ff614d9c04a4e366fd3934f4203f2ff1d467303f | |
parent | 9adb9254f4b0f591282f713de5356a9e75a44da3 (diff) |
usb: common: 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: Heikki Krogerus <heikki.krogerus@linux.intel.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/usb/common/common.c | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/drivers/usb/common/common.c b/drivers/usb/common/common.c index e6ec125e4485..677b3f01deaa 100644 --- a/drivers/usb/common/common.c +++ b/drivers/usb/common/common.c | |||
@@ -64,18 +64,15 @@ EXPORT_SYMBOL_GPL(usb_speed_string); | |||
64 | enum usb_device_speed usb_get_maximum_speed(struct device *dev) | 64 | enum usb_device_speed usb_get_maximum_speed(struct device *dev) |
65 | { | 65 | { |
66 | const char *maximum_speed; | 66 | const char *maximum_speed; |
67 | int err; | 67 | int ret; |
68 | int i; | ||
69 | 68 | ||
70 | err = device_property_read_string(dev, "maximum-speed", &maximum_speed); | 69 | ret = device_property_read_string(dev, "maximum-speed", &maximum_speed); |
71 | if (err < 0) | 70 | if (ret < 0) |
72 | return USB_SPEED_UNKNOWN; | 71 | return USB_SPEED_UNKNOWN; |
73 | 72 | ||
74 | for (i = 0; i < ARRAY_SIZE(speed_names); i++) | 73 | ret = match_string(speed_names, ARRAY_SIZE(speed_names), maximum_speed); |
75 | if (strcmp(maximum_speed, speed_names[i]) == 0) | ||
76 | return i; | ||
77 | 74 | ||
78 | return USB_SPEED_UNKNOWN; | 75 | return (ret < 0) ? USB_SPEED_UNKNOWN : ret; |
79 | } | 76 | } |
80 | EXPORT_SYMBOL_GPL(usb_get_maximum_speed); | 77 | EXPORT_SYMBOL_GPL(usb_get_maximum_speed); |
81 | 78 | ||
@@ -109,13 +106,10 @@ static const char *const usb_dr_modes[] = { | |||
109 | 106 | ||
110 | static enum usb_dr_mode usb_get_dr_mode_from_string(const char *str) | 107 | static enum usb_dr_mode usb_get_dr_mode_from_string(const char *str) |
111 | { | 108 | { |
112 | int i; | 109 | int ret; |
113 | |||
114 | for (i = 0; i < ARRAY_SIZE(usb_dr_modes); i++) | ||
115 | if (!strcmp(usb_dr_modes[i], str)) | ||
116 | return i; | ||
117 | 110 | ||
118 | return USB_DR_MODE_UNKNOWN; | 111 | ret = match_string(usb_dr_modes, ARRAY_SIZE(usb_dr_modes), str); |
112 | return (ret < 0) ? USB_DR_MODE_UNKNOWN : ret; | ||
119 | } | 113 | } |
120 | 114 | ||
121 | enum usb_dr_mode usb_get_dr_mode(struct device *dev) | 115 | enum usb_dr_mode usb_get_dr_mode(struct device *dev) |