From patchwork Wed Feb 19 21:34:49 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Patchwork-Id: 13983046 Received: from todd.t-8ch.de (todd.t-8ch.de [159.69.126.157]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F07072580DD for ; Wed, 19 Feb 2025 21:36:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=159.69.126.157 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740000965; cv=none; b=rVLWQXB+G5b5k409ilUmeDQhdQDhzEMvjFsMTkWomc/hRyTjwTEnP4f3Ndq1N+ixZ2Z16g9O1TUM71ga86IpkkVeO1vXeFL6eycBuN9nx6DUY73ZmMdUqOYv/bubkhee+HxEDNeAj93SYuY2YaixqcZ6C51RPdOCq4Fcl8Rrzi8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740000965; c=relaxed/simple; bh=NwdYX80AoZVeXVazJE3UJmo3FYdczQC745wtVt8RgYM=; h=From:Subject:Date:Message-Id:MIME-Version:Content-Type:To:Cc; b=Tphgskc2W6yfCAWxp8eE2Eb9U5ZEqYrCuGrzj2RYo8r9vnEf6IAD7gX55aslTRncV/HDiETvuPJji3+2r3WZhGcVQa+4ZstvhG+WjU1OA//ya9Aj8DZ+Ese+Kis3IpM1SRb4jW3GMKP0IaNh03oWNAZ7Ov01VUfYhcfI0T+rsc8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=weissschuh.net; spf=pass smtp.mailfrom=weissschuh.net; dkim=pass (1024-bit key) header.d=weissschuh.net header.i=@weissschuh.net header.b=I0rwc7Ox; arc=none smtp.client-ip=159.69.126.157 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=weissschuh.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=weissschuh.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=weissschuh.net header.i=@weissschuh.net header.b="I0rwc7Ox" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=weissschuh.net; s=mail; t=1740000960; bh=NwdYX80AoZVeXVazJE3UJmo3FYdczQC745wtVt8RgYM=; h=From:Subject:Date:To:Cc:From; b=I0rwc7OxZO/A8+eM2K9ehjgODO4sXhrHw8dr4Lay6/FCtNjpWn/ZcMbqBavBCLbsE ZlLY9ZfiPQVJzpnfwGRc6zG2HThzo+X69tYn51AR+rrh8ozHoKic/9lJC+vejVX5wb w7fA8CI8F08P2wxeBvUxT4rqwK3N+UB+NV9iGNqc= From: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= Subject: [PATCH 0/6] sysfs: finalize the constification of 'struct bin_attribute' Date: Wed, 19 Feb 2025 22:34:49 +0100 Message-Id: <20250219-sysfs-const-bin_attr-final-v1-0-02828d86af3c@weissschuh.net> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAHlOtmcC/x3MUQqDMAwA0KtIvg20ZTLdVYZI16YaGHE0RRTx7 pZ9vp93glJmUng1J2TaWHmVCts2EBYvMyHHanDGdcbZJ+qhSTGsogU/LJMvJWNi8V+MAz2iCan 3NkINfpkS7//8PV7XDR+Bcv1sAAAA X-Change-ID: 20250217-sysfs-const-bin_attr-final-d9e4d0cf8a1d To: Greg Kroah-Hartman , "Rafael J. Wysocki" Cc: linux-kernel@vger.kernel.org, =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1740000960; l=10357; i=linux@weissschuh.net; s=20221212; h=from:subject:message-id; bh=NwdYX80AoZVeXVazJE3UJmo3FYdczQC745wtVt8RgYM=; b=rSZvM5P0jctPdSfRG6AD0XXuCnHbFqe+V1pK62DZ8JiCsQj3k4+uZ5n+qDz88INevjihWZT/e cKMhqu0V71nBanjEDVm+L4LgQcLXaXmw12sMshI+TWZ76RimRimZNYR X-Developer-Key: i=linux@weissschuh.net; a=ed25519; pk=KcycQgFPX2wGR5azS7RhpBqedglOZVgRPfdFSPB1LNw= All users have been migrated to the new const types. Change the types of the normal fields, switch all users over and then get rid of the transition machinery. This depends on the following patches which are not mainline yet: https://lore.kernel.org/r/20250114-sysfs-const-bin_attr-cxl-v1-1-5afa23fe2a52@weissschuh.net https://lore.kernel.org/r/20250125-sysfs-const-bin_attr-dmi-v2-0-ece1895936f4@weissschuh.net https://lore.kernel.org/r/20241216-sysfs-const-bin_attr-drm-v1-0-210f2b36b9bf@weissschuh.net https://lore.kernel.org/r/20241215-sysfs-const-bin_attr-fsi-v1-1-b717f76a0146@weissschuh.net https://lore.kernel.org/r/20241216-sysfs-const-bin_attr-habanalabs-v1-1-b35463197efb@weissschuh.net https://lore.kernel.org/r/20241222-sysfs-const-bin_attr-input-v1-1-1229dbe5ae71@weissschuh.net https://lore.kernel.org/r/20241215-sysfs-const-bin_attr-mokvar-v1-1-d5a3d1fff8d1@weissschuh.net https://lore.kernel.org/r/20241215-sysfs-const-bin_attr-pcmcia-v1-1-ebb82e47d834@weissschuh.net https://lore.kernel.org/r/20241216-sysfs-const-bin_attr-powerpc-v1-0-bbed8906f476@weissschuh.net https://lore.kernel.org/r/20250114-sysfs-const-bin_attr-qemu_fw_cfg-v1-1-76f525a3ee72@weissschuh.net https://lore.kernel.org/r/20241216-sysfs-const-bin_attr-rapidio-v1-1-0f47f4719683@weissschuh.net https://lore.kernel.org/r/20241122-sysfs-const-bin_attr-rci2-v1-1-3db1ec9aa203@weissschuh.net The treewide changes have been done with sed/grep. This series and prerequisites have passed 0day on top of 2408a807bfc3 ("Merge tag 'vfs-6.14-rc4.fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs"). I'm only sending this to the sysfs maintainers. The treewide changes are purely mechanical. Signed-off-by: Thomas Weißschuh --- Thomas Weißschuh (6): sysfs: constify bin_attribute argument of bin_attribute::read/write() sysfs: constify attribute_group::bin_attrs sysfs: treewide: switch back to bin_attribute::read()/write() sysfs: treewide: switch back to attribute_group::bin_attrs sysfs: remove bin_attribute::read_new/write_new() sysfs: remove attribute_group::bin_attrs_new arch/powerpc/kernel/secvar-sysfs.c | 2 +- arch/powerpc/perf/hv-24x7.c | 2 +- arch/powerpc/platforms/powernv/opal-core.c | 4 +-- arch/powerpc/platforms/powernv/opal-dump.c | 2 +- arch/powerpc/platforms/powernv/opal-elog.c | 2 +- arch/powerpc/platforms/powernv/opal-flash.c | 2 +- arch/powerpc/platforms/powernv/opal-msglog.c | 2 +- arch/powerpc/platforms/powernv/opal.c | 2 +- arch/powerpc/platforms/powernv/ultravisor.c | 2 +- arch/s390/kernel/cpacf.c | 2 +- arch/s390/kernel/ipl.c | 18 +++++------ arch/s390/pci/pci_sysfs.c | 2 +- arch/x86/kernel/ksysfs.c | 8 ++--- drivers/accel/habanalabs/common/sysfs.c | 4 +-- drivers/acpi/bgrt.c | 2 +- drivers/acpi/sysfs.c | 4 +-- drivers/base/devcoredump.c | 2 +- drivers/base/firmware_loader/sysfs.c | 6 ++-- drivers/base/node.c | 2 +- drivers/base/topology.c | 2 +- drivers/cxl/port.c | 2 +- drivers/firmware/efi/mokvar-table.c | 2 +- drivers/firmware/google/cbmem.c | 2 +- drivers/firmware/google/gsmi.c | 2 +- drivers/firmware/google/memconsole.c | 2 +- drivers/firmware/google/vpd.c | 4 +-- drivers/firmware/qemu_fw_cfg.c | 2 +- drivers/fsi/fsi-core.c | 4 +-- drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c | 6 ++-- drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 2 +- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_hdcp.c | 4 +-- drivers/gpu/drm/drm_sysfs.c | 4 +-- drivers/gpu/drm/i915/i915_gpu_error.c | 4 +-- drivers/gpu/drm/i915/i915_sysfs.c | 8 ++--- drivers/gpu/drm/lima/lima_drv.c | 4 +-- drivers/hid/hid-core.c | 2 +- drivers/hid/hid-roccat-arvo.c | 2 +- drivers/hid/hid-roccat-common.h | 8 ++--- drivers/hid/hid-roccat-isku.c | 10 +++--- drivers/hid/hid-roccat-kone.c | 6 ++-- drivers/hid/hid-roccat-koneplus.c | 14 ++++----- drivers/hid/hid-roccat-konepure.c | 2 +- drivers/hid/hid-roccat-kovaplus.c | 12 ++++---- drivers/hid/hid-roccat-lua.c | 4 +-- drivers/hid/hid-roccat-pyra.c | 14 ++++----- drivers/hid/hid-roccat-ryos.c | 2 +- drivers/hid/hid-roccat-savu.c | 2 +- drivers/i2c/i2c-slave-eeprom.c | 4 +-- drivers/iio/imu/bno055/bno055.c | 2 +- drivers/infiniband/hw/hfi1/sysfs.c | 2 +- drivers/infiniband/hw/qib/qib_sysfs.c | 2 +- drivers/input/touchscreen/goodix_berlin_core.c | 2 +- drivers/leds/led-class.c | 2 +- drivers/media/pci/solo6x10/solo6x10-core.c | 2 +- drivers/misc/c2port/core.c | 2 +- drivers/misc/cxl/sysfs.c | 4 +-- drivers/misc/ds1682.c | 4 +-- drivers/misc/eeprom/max6875.c | 2 +- drivers/misc/ocxl/sysfs.c | 2 +- drivers/misc/pch_phub.c | 4 +-- drivers/misc/sram.c | 4 +-- drivers/mtd/spi-nor/sysfs.c | 2 +- .../net/ethernet/qlogic/netxen/netxen_nic_main.c | 10 +++--- drivers/net/ethernet/qlogic/qlcnic/qlcnic_sysfs.c | 34 ++++++++++---------- drivers/net/phy/spi_ks8995.c | 4 +-- drivers/net/wireless/ti/wlcore/sysfs.c | 2 +- drivers/nvmem/core.c | 14 ++++----- drivers/of/kobj.c | 2 +- drivers/pci/hotplug/acpiphp_ibm.c | 2 +- drivers/pci/p2pdma.c | 2 +- drivers/pci/pci-sysfs.c | 12 ++++---- drivers/pci/vpd.c | 2 +- drivers/pcmcia/cistpl.c | 4 +-- drivers/platform/chrome/cros_ec_vbc.c | 2 +- drivers/platform/mellanox/mlxbf-bootctl.c | 2 +- drivers/platform/x86/amd/hsmp/acpi.c | 4 +-- drivers/platform/x86/amd/hsmp/plat.c | 4 +-- drivers/platform/x86/dell/dcdbas.c | 2 +- drivers/platform/x86/dell/dell_rbu.c | 2 +- drivers/platform/x86/intel/pmt/class.c | 2 +- drivers/platform/x86/intel/sdsi.c | 2 +- drivers/platform/x86/wmi-bmof.c | 2 +- drivers/power/supply/ds2760_battery.c | 2 +- drivers/power/supply/ds2780_battery.c | 10 +++--- drivers/power/supply/ds2781_battery.c | 10 +++--- drivers/power/supply/olpc_battery.c | 4 +-- drivers/ptp/ptp_ocp.c | 2 +- drivers/rapidio/rio-sysfs.c | 6 ++-- drivers/s390/char/sclp_config.c | 2 +- drivers/s390/char/sclp_sd.c | 2 +- drivers/s390/cio/chp.c | 2 +- drivers/s390/crypto/pkey_sysfs.c | 8 ++--- drivers/scsi/3w-sas.c | 4 +-- drivers/scsi/arcmsr/arcmsr_attr.c | 6 ++-- drivers/scsi/esas2r/esas2r_main.c | 6 ++-- drivers/scsi/ibmvscsi/ibmvfc.c | 2 +- drivers/scsi/ipr.c | 10 +++--- drivers/scsi/lpfc/lpfc_attr.c | 8 ++--- drivers/scsi/qedf/qedf_attr.c | 4 +-- drivers/scsi/qla2xxx/qla_attr.c | 28 ++++++++--------- drivers/scsi/qla4xxx/ql4_attr.c | 4 +-- drivers/scsi/scsi_sysfs.c | 6 ++-- drivers/usb/core/sysfs.c | 2 +- drivers/usb/usbip/vudc_sysfs.c | 2 +- drivers/video/fbdev/aty/radeon_base.c | 4 +-- drivers/video/fbdev/udlfb.c | 4 +-- drivers/w1/slaves/w1_ds2406.c | 4 +-- drivers/w1/slaves/w1_ds2408.c | 2 +- drivers/w1/slaves/w1_ds2413.c | 2 +- drivers/w1/slaves/w1_ds2430.c | 2 +- drivers/w1/slaves/w1_ds2431.c | 2 +- drivers/w1/slaves/w1_ds2433.c | 12 ++++---- drivers/w1/slaves/w1_ds2438.c | 2 +- drivers/w1/slaves/w1_ds2780.c | 2 +- drivers/w1/slaves/w1_ds2781.c | 2 +- drivers/w1/slaves/w1_ds2805.c | 4 +-- drivers/w1/slaves/w1_ds28e04.c | 2 +- drivers/w1/w1.c | 2 +- drivers/zorro/zorro-sysfs.c | 4 +-- fs/sysfs/file.c | 22 +++---------- fs/sysfs/group.c | 6 ++-- include/linux/sysfs.h | 36 ++++------------------ kernel/bpf/btf.c | 2 +- kernel/bpf/sysfs_btf.c | 2 +- kernel/module/sysfs.c | 14 ++++----- mm/page_idle.c | 2 +- net/bridge/br_sysfs_br.c | 2 +- 127 files changed, 291 insertions(+), 327 deletions(-) --- base-commit: 16fbcf2511cee77492408f12b31eb8557c24bee8 change-id: 20250217-sysfs-const-bin_attr-final-d9e4d0cf8a1d Best regards,