[PATCH 00/30] Remove strtobool()

Christophe JAILLET posted 30 patches 3 years, 5 months ago
arch/arm64/kernel/cpufeature.c                |  5 ++--
arch/arm64/kvm/vgic/vgic-v3.c                 |  9 +++---
arch/s390/kernel/ipl.c                        |  7 +++--
arch/x86/kvm/mmu/mmu.c                        |  3 +-
arch/x86/xen/enlighten_pv.c                   |  3 +-
arch/x86/xen/setup.c                          |  3 +-
drivers/acpi/sysfs.c                          |  3 +-
drivers/base/core.c                           |  7 +++--
drivers/clocksource/arm_arch_timer.c          |  3 +-
drivers/greybus/svc.c                         |  3 +-
drivers/input/input.c                         |  3 +-
drivers/irqchip/irq-gic-v3.c                  |  3 +-
drivers/irqchip/irq-gic.c                     |  3 +-
drivers/net/usb/cdc_ncm.c                     |  3 +-
drivers/net/usb/qmi_wwan.c                    |  5 ++--
drivers/net/wireless/ath/ath10k/debug.c       |  5 ++--
drivers/net/wireless/ath/ath9k/ath9k.h        |  1 +
drivers/net/wireless/ath/ath9k/tx99.c         |  2 +-
.../net/wireless/marvell/mwifiex/debugfs.c    |  2 +-
drivers/net/wireless/marvell/mwifiex/main.h   |  1 +
drivers/nvdimm/namespace_devs.c               |  3 +-
drivers/nvdimm/pmem.c                         |  3 +-
drivers/nvdimm/region_devs.c                  |  5 ++--
drivers/nvme/host/pci.c                       |  3 +-
drivers/nvme/target/configfs.c                | 17 ++++++-----
drivers/platform/chrome/cros_ec_lightbar.c    |  3 +-
drivers/powercap/powercap_sys.c               |  3 +-
drivers/target/target_core_configfs.c         | 29 ++++++++++---------
drivers/target/target_core_fabric_configfs.c  |  3 +-
drivers/usb/core/port.c                       |  3 +-
drivers/usb/core/sysfs.c                      |  7 +++--
drivers/usb/gadget/configfs.c                 |  3 +-
drivers/usb/gadget/function/f_mass_storage.c  |  3 +-
drivers/usb/gadget/function/storage_common.c  |  9 +++---
drivers/usb/gadget/function/u_serial.c        |  3 +-
drivers/usb/gadget/legacy/serial.c            |  3 +-
.../fbdev/omap2/omapfb/dss/display-sysfs.c    |  7 +++--
.../fbdev/omap2/omapfb/dss/manager-sysfs.c    |  7 +++--
.../fbdev/omap2/omapfb/dss/overlay-sysfs.c    |  3 +-
.../video/fbdev/omap2/omapfb/omapfb-sysfs.c   |  3 +-
fs/cifs/cifs_debug.c                          |  5 ++--
include/linux/kernel.h                        |  2 +-
init/initramfs.c                              |  8 ++++-
kernel/module/main.c                          |  3 +-
kernel/params.c                               |  3 +-
lib/test_firmware.c                           |  3 +-
mm/damon/lru_sort.c                           |  3 +-
mm/damon/reclaim.c                            |  3 +-
mm/page_table_check.c                         |  3 +-
mm/usercopy.c                                 |  3 +-
net/bluetooth/hci_debugfs.c                   |  3 +-
51 files changed, 143 insertions(+), 90 deletions(-)
[PATCH 00/30] Remove strtobool()
Posted by Christophe JAILLET 3 years, 5 months ago
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>)


Each patch can be applied independently from the other ones.
The last patch of the serie removes the definition of strtobool().

All patches have been compile tested, except the ones against
arm64, s390 and irq-gic.


Finally, this gives the opportunity to remove some <linux/kernel.h> from
a few drivers. This will be addressed later when/if the serie is applied.


Christophe JAILLET (30):
  net: usb: Use kstrtobool() instead of strtobool()
  wifi: Use kstrtobool() instead of strtobool()
  irqchip: Use kstrtobool() instead of strtobool()
  scsi: target: Use kstrtobool() instead of strtobool()
  nvdimm: Use kstrtobool() instead of strtobool()
  nvme: Use kstrtobool() instead of strtobool()
  usb: core: Use kstrtobool() instead of strtobool()
  usb: gadget: Use kstrtobool() instead of strtobool()
  ACPI: sysfs: Use kstrtobool() instead of strtobool()
  clocksource/drivers/arm_arch_timer: Use kstrtobool() instead of
    strtobool()
  greybus: svc: Use kstrtobool() instead of strtobool()
  input: Use kstrtobool() instead of strtobool()
  platform/chrome: Use kstrtobool() instead of strtobool()
  powercap: Use kstrtobool() instead of strtobool()
  video: fbdev: omapfb: Use kstrtobool() instead of strtobool()
  cifs: Use kstrtobool() instead of strtobool()
  initramfs: Use kstrtobool() instead of strtobool()
  module: Use kstrtobool() instead of strtobool()
  kernel/params.c: Use kstrtobool() instead of strtobool()
  mm/damon: Use kstrtobool() instead of strtobool()
  mm: Use kstrtobool() instead of strtobool()
  Bluetooth: hci_debugfs:: Use kstrtobool() instead of strtobool()
  test_firmware: Use kstrtobool() instead of strtobool()
  arm64: cpufeature: Use kstrtobool() instead of strtobool()
  KVM: arm64: vgic-v3: Use kstrtobool() instead of strtobool()
  s390/ipl: Use kstrtobool() instead of strtobool()
  KVM: x86/mmu: Use kstrtobool() instead of strtobool()
  x86/xen: Use kstrtobool() instead of strtobool()
  driver core: Use kstrtobool() instead of strtobool()
  kstrtox: Remove strtobool()

 arch/arm64/kernel/cpufeature.c                |  5 ++--
 arch/arm64/kvm/vgic/vgic-v3.c                 |  9 +++---
 arch/s390/kernel/ipl.c                        |  7 +++--
 arch/x86/kvm/mmu/mmu.c                        |  3 +-
 arch/x86/xen/enlighten_pv.c                   |  3 +-
 arch/x86/xen/setup.c                          |  3 +-
 drivers/acpi/sysfs.c                          |  3 +-
 drivers/base/core.c                           |  7 +++--
 drivers/clocksource/arm_arch_timer.c          |  3 +-
 drivers/greybus/svc.c                         |  3 +-
 drivers/input/input.c                         |  3 +-
 drivers/irqchip/irq-gic-v3.c                  |  3 +-
 drivers/irqchip/irq-gic.c                     |  3 +-
 drivers/net/usb/cdc_ncm.c                     |  3 +-
 drivers/net/usb/qmi_wwan.c                    |  5 ++--
 drivers/net/wireless/ath/ath10k/debug.c       |  5 ++--
 drivers/net/wireless/ath/ath9k/ath9k.h        |  1 +
 drivers/net/wireless/ath/ath9k/tx99.c         |  2 +-
 .../net/wireless/marvell/mwifiex/debugfs.c    |  2 +-
 drivers/net/wireless/marvell/mwifiex/main.h   |  1 +
 drivers/nvdimm/namespace_devs.c               |  3 +-
 drivers/nvdimm/pmem.c                         |  3 +-
 drivers/nvdimm/region_devs.c                  |  5 ++--
 drivers/nvme/host/pci.c                       |  3 +-
 drivers/nvme/target/configfs.c                | 17 ++++++-----
 drivers/platform/chrome/cros_ec_lightbar.c    |  3 +-
 drivers/powercap/powercap_sys.c               |  3 +-
 drivers/target/target_core_configfs.c         | 29 ++++++++++---------
 drivers/target/target_core_fabric_configfs.c  |  3 +-
 drivers/usb/core/port.c                       |  3 +-
 drivers/usb/core/sysfs.c                      |  7 +++--
 drivers/usb/gadget/configfs.c                 |  3 +-
 drivers/usb/gadget/function/f_mass_storage.c  |  3 +-
 drivers/usb/gadget/function/storage_common.c  |  9 +++---
 drivers/usb/gadget/function/u_serial.c        |  3 +-
 drivers/usb/gadget/legacy/serial.c            |  3 +-
 .../fbdev/omap2/omapfb/dss/display-sysfs.c    |  7 +++--
 .../fbdev/omap2/omapfb/dss/manager-sysfs.c    |  7 +++--
 .../fbdev/omap2/omapfb/dss/overlay-sysfs.c    |  3 +-
 .../video/fbdev/omap2/omapfb/omapfb-sysfs.c   |  3 +-
 fs/cifs/cifs_debug.c                          |  5 ++--
 include/linux/kernel.h                        |  2 +-
 init/initramfs.c                              |  8 ++++-
 kernel/module/main.c                          |  3 +-
 kernel/params.c                               |  3 +-
 lib/test_firmware.c                           |  3 +-
 mm/damon/lru_sort.c                           |  3 +-
 mm/damon/reclaim.c                            |  3 +-
 mm/page_table_check.c                         |  3 +-
 mm/usercopy.c                                 |  3 +-
 net/bluetooth/hci_debugfs.c                   |  3 +-
 51 files changed, 143 insertions(+), 90 deletions(-)

-- 
2.34.1
Re: [PATCH 00/30] Remove strtobool()
Posted by Andy Shevchenko 3 years, 5 months ago
On Tue, Nov 01, 2022 at 10:05:37PM +0100, Christophe JAILLET wrote:
> 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>)
> 
> 
> Each patch can be applied independently from the other ones.
> The last patch of the serie removes the definition of strtobool().
> 
> All patches have been compile tested, except the ones against
> arm64, s390 and irq-gic.

I got only a cover letter...
Luckily we have a powerful `b4` tool and available archives on lore.kernel.org :-)

> Finally, this gives the opportunity to remove some <linux/kernel.h> from
> a few drivers. This will be addressed later when/if the serie is applied.

-- 
With Best Regards,
Andy Shevchenko
Re: (subset) [PATCH 00/30] Remove strtobool()
Posted by Catalin Marinas 3 years, 2 months ago
On Tue, 01 Nov 2022 22:05:37 +0100, Christophe JAILLET wrote:
> 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>)
> 
> [...]

Applied to arm64 (for-next/misc), thanks!

[24/30] arm64: cpufeature: Use kstrtobool() instead of strtobool()
        https://git.kernel.org/arm64/c/1a920c92cd0c

-- 
Catalin