diff mbox series

[05/30] nvdimm: Use kstrtobool() instead of strtobool()

Message ID	5990e1eadd81a25026e6f04fdda96bd43ff23a07.1667336095.git.christophe.jaillet@wanadoo.fr
State	New
Headers	Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; From: Christophe JAILLET <christophe.jaillet@wanadoo.fr> To: Dan Williams <dan.j.williams@intel.com>, Vishal Verma <vishal.l.verma@intel.com>, Dave Jiang <dave.jiang@intel.com>, Ira Weiny <ira.weiny@intel.com> Cc: linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Christophe JAILLET <christophe.jaillet@wanadoo.fr>, nvdimm@lists.linux.dev Subject: [PATCH 05/30] nvdimm: Use kstrtobool() instead of strtobool() Date: Tue, 1 Nov 2022 22:13:53 +0100 Message-Id: <5990e1eadd81a25026e6f04fdda96bd43ff23a07.1667336095.git.christophe.jaillet@wanadoo.fr> In-Reply-To: <cover.1667336095.git.christophe.jaillet@wanadoo.fr> References: <cover.1667336095.git.christophe.jaillet@wanadoo.fr> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	Remove strtobool() \| [00/30] Remove strtobool() [01/30] net: usb: Use kstrtobool() instead of strtobool() [02/30] wifi: Use kstrtobool() instead of strtobool() [03/30] irqchip: Use kstrtobool() instead of strtobool() [04/30] scsi: target: Use kstrtobool() instead of strtobool() [05/30] nvdimm: Use kstrtobool() instead of strtobool() [06/30] nvme: Use kstrtobool() instead of strtobool() [07/30] usb: core: Use kstrtobool() instead of strtobool() [08/30] usb: gadget: Use kstrtobool() instead of strtobool() [09/30] ACPI: sysfs: Use kstrtobool() instead of strtobool() [10/30] clocksource/drivers/arm_arch_timer: Use kstrtobool() instead of strtobool() [11/30] greybus: svc: Use kstrtobool() instead of strtobool() [12/30] input: Use kstrtobool() instead of strtobool() [13/30] platform/chrome: Use kstrtobool() instead of strtobool() [14/30] powercap: Use kstrtobool() instead of strtobool() [15/30] video: fbdev: omapfb: Use kstrtobool() instead of strtobool() [16/30] cifs: Use kstrtobool() instead of strtobool() [17/30] initramfs: Use kstrtobool() instead of strtobool() [18/30] module: Use kstrtobool() instead of strtobool() [19/30] kernel/params.c: Use kstrtobool() instead of strtobool() [20/30] mm/damon: Use kstrtobool() instead of strtobool() [21/30] mm: Use kstrtobool() instead of strtobool() [22/30] Bluetooth: hci_debugfs: Use kstrtobool() instead of strtobool() [23/30] test_firmware: Use kstrtobool() instead of strtobool() [24/30] arm64: cpufeature: Use kstrtobool() instead of strtobool() [25/30] KVM: arm64: vgic-v3: Use kstrtobool() instead of strtobool() [26/30] s390/ipl: Use kstrtobool() instead of strtobool() [27/30] KVM: x86/mmu: Use kstrtobool() instead of strtobool() [28/30] x86/xen: Use kstrtobool() instead of strtobool() [29/30] driver core: Use kstrtobool() instead of strtobool() [30/30] kstrtox: Remove strtobool()

Commit Message

Christophe JAILLET Nov. 1, 2022, 9:13 p.m. UTC

  strtobool() is the same as kstrtobool().
However, the latter is more used within the kernel.

In order to remove strtobool() and slightly simplify kstrtox.h, switch to
the other function name.

While at it, include the corresponding header file (<linux/kstrtox.h>)

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
This patch is part of a serie that axes all usages of strtobool().
Each patch can be applied independently from the other ones.

The last patch of the serie removes the definition of strtobool().

You may not be in copy of the cover letter. So, if needed, it is available
at [1].

[1]: https://lore.kernel.org/all/cover.1667336095.git.christophe.jaillet@wanadoo.fr/
---
 drivers/nvdimm/namespace_devs.c | 3 ++-
 drivers/nvdimm/pmem.c           | 3 ++-
 drivers/nvdimm/region_devs.c    | 5 +++--
 3 files changed, 7 insertions(+), 4 deletions(-)

diff mbox series

Patch

diff --git a/drivers/nvdimm/namespace_devs.c b/drivers/nvdimm/namespace_devs.c
index c60ec0b373c5..07177eadc56e 100644
--- a/drivers/nvdimm/namespace_devs.c
+++ b/drivers/nvdimm/namespace_devs.c
@@ -2,6 +2,7 @@ 
 /*
  * Copyright(c) 2013-2015 Intel Corporation. All rights reserved.
  */
+#include <linux/kstrtox.h>
 #include <linux/module.h>
 #include <linux/device.h>
 #include <linux/sort.h>
@@ -1338,7 +1339,7 @@  static ssize_t force_raw_store(struct device *dev,
 		struct device_attribute *attr, const char *buf, size_t len)
 {
 	bool force_raw;
-	int rc = strtobool(buf, &force_raw);
+	int rc = kstrtobool(buf, &force_raw);
 
 	if (rc)
 		return rc;
diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c
index 3c63dc2cdc81..46475d146f64 100644
--- a/drivers/nvdimm/pmem.c
+++ b/drivers/nvdimm/pmem.c
@@ -17,6 +17,7 @@ 
 #include <linux/moduleparam.h>
 #include <linux/badblocks.h>
 #include <linux/memremap.h>
+#include <linux/kstrtox.h>
 #include <linux/vmalloc.h>
 #include <linux/blk-mq.h>
 #include <linux/pfn_t.h>
@@ -408,7 +409,7 @@  static ssize_t write_cache_store(struct device *dev,
 	bool write_cache;
 	int rc;
 
-	rc = strtobool(buf, &write_cache);
+	rc = kstrtobool(buf, &write_cache);
 	if (rc)
 		return rc;
 	dax_write_cache(pmem->dax_dev, write_cache);
diff --git a/drivers/nvdimm/region_devs.c b/drivers/nvdimm/region_devs.c
index e0875d369762..76e1ae6f830a 100644
--- a/drivers/nvdimm/region_devs.c
+++ b/drivers/nvdimm/region_devs.c
@@ -5,6 +5,7 @@ 
 #include <linux/scatterlist.h>
 #include <linux/memregion.h>
 #include <linux/highmem.h>
+#include <linux/kstrtox.h>
 #include <linux/sched.h>
 #include <linux/slab.h>
 #include <linux/hash.h>
@@ -229,7 +230,7 @@  static ssize_t deep_flush_store(struct device *dev, struct device_attribute *att
 		const char *buf, size_t len)
 {
 	bool flush;
-	int rc = strtobool(buf, &flush);
+	int rc = kstrtobool(buf, &flush);
 	struct nd_region *nd_region = to_nd_region(dev);
 
 	if (rc)
@@ -484,7 +485,7 @@  static ssize_t read_only_store(struct device *dev,
 		struct device_attribute *attr, const char *buf, size_t len)
 {
 	bool ro;
-	int rc = strtobool(buf, &ro);
+	int rc = kstrtobool(buf, &ro);
 	struct nd_region *nd_region = to_nd_region(dev);
 
 	if (rc)