diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2018-08-06 14:30:34 -0500 |
---|---|---|
committer | Bjorn Helgaas <bhelgaas@google.com> | 2018-08-06 14:30:34 -0500 |
commit | 36131ce9a0b23ed6feb04f8a3ffaa44b804abd5e (patch) | |
tree | 8ccec8940b8359e955f05a2284acafdb11e6725c /drivers/pci | |
parent | ce397d215ccd07b8ae3f71db689aedb85d56ab40 (diff) | |
download | talos-obmc-linux-36131ce9a0b23ed6feb04f8a3ffaa44b804abd5e.tar.gz talos-obmc-linux-36131ce9a0b23ed6feb04f8a3ffaa44b804abd5e.zip |
PCI/ASPM: Convert to use sysfs_match_string() helper
The sysfs_match_string() helper returns index of the matching string in an
array. Use it in pcie_aspm_set_policy() to simplify the code.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
[bhelgaas: squash sysfs_match_string() fix into original patch for issue
Reported-by: Heiner Kallweit <hkallweit1@gmail.com>]
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Diffstat (limited to 'drivers/pci')
-rw-r--r-- | drivers/pci/pcie/aspm.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/pci/pcie/aspm.c b/drivers/pci/pcie/aspm.c index c687c817b47d..5326916715d2 100644 --- a/drivers/pci/pcie/aspm.c +++ b/drivers/pci/pcie/aspm.c @@ -1127,11 +1127,9 @@ static int pcie_aspm_set_policy(const char *val, if (aspm_disabled) return -EPERM; - for (i = 0; i < ARRAY_SIZE(policy_str); i++) - if (!strncmp(val, policy_str[i], strlen(policy_str[i]))) - break; - if (i >= ARRAY_SIZE(policy_str)) - return -EINVAL; + i = sysfs_match_string(policy_str, val); + if (i < 0) + return i; if (i == aspm_policy) return 0; |