PCI: Use kstrtobool() directly, sans strtobool() wrapper
authorKrzysztof Wilczyński <kw@linux.com>
Wed, 15 Sep 2021 23:01:27 +0000 (23:01 +0000)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 28 Sep 2021 23:06:29 +0000 (18:06 -0500)
strtobool() is a wrapper around kstrtobool() that has been added for
backward compatibility.

There is no reason to use the old API, so use kstrtobool() directly.

Related: ef951599074b ("lib: move strtobool() to kstrtobool()")

Link: https://lore.kernel.org/r/20210915230127.2495723-3-kw@linux.com
Signed-off-by: Krzysztof Wilczyński <kw@linux.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/p2pdma.c
drivers/pci/pcie/aspm.c

index 50cdde3e9a8b2b34bc9d62def0218da7c7ae368d..4fccdcf9186f0871e0d46cc42a6a4a8ec9795a7c 100644 (file)
@@ -943,7 +943,7 @@ EXPORT_SYMBOL_GPL(pci_p2pdma_unmap_sg_attrs);
  *
  * Parses an attribute value to decide whether to enable p2pdma.
  * The value can select a PCI device (using its full BDF device
- * name) or a boolean (in any format strtobool() accepts). A false
+ * name) or a boolean (in any format kstrtobool() accepts). A false
  * value disables p2pdma, a true value expects the caller
  * to automatically find a compatible device and specifying a PCI device
  * expects the caller to use the specific provider.
@@ -975,11 +975,11 @@ int pci_p2pdma_enable_store(const char *page, struct pci_dev **p2p_dev,
        } else if ((page[0] == '0' || page[0] == '1') && !iscntrl(page[1])) {
                /*
                 * If the user enters a PCI device that  doesn't exist
-                * like "0000:01:00.1", we don't want strtobool to think
+                * like "0000:01:00.1", we don't want kstrtobool to think
                 * it's a '0' when it's clearly not what the user wanted.
                 * So we require 0's and 1's to be exactly one character.
                 */
-       } else if (!strtobool(page, use_p2pdma)) {
+       } else if (!kstrtobool(page, use_p2pdma)) {
                return 0;
        }
 
index 013a47f587ceae2b9c87e623c6ac4223efa9f487..52c74682601a914e8fa6efd0ff0c0a1e958145ef 100644 (file)
@@ -1219,7 +1219,7 @@ static ssize_t aspm_attr_store_common(struct device *dev,
        struct pcie_link_state *link = pcie_aspm_get_link(pdev);
        bool state_enable;
 
-       if (strtobool(buf, &state_enable) < 0)
+       if (kstrtobool(buf, &state_enable) < 0)
                return -EINVAL;
 
        down_read(&pci_bus_sem);
@@ -1276,7 +1276,7 @@ static ssize_t clkpm_store(struct device *dev,
        struct pcie_link_state *link = pcie_aspm_get_link(pdev);
        bool state_enable;
 
-       if (strtobool(buf, &state_enable) < 0)
+       if (kstrtobool(buf, &state_enable) < 0)
                return -EINVAL;
 
        down_read(&pci_bus_sem);