| Message ID | 20241005-power-supply-no-wakeup-source-v1-0-1d62bf9bcb1d@weissschuh.net |
|---|---|
| Headers | show |
| Series | power: supply: core: remove {,devm_}power_supply_register_no_ws() | expand |
On Sat, 05 Oct 2024 12:05:02 +0200, Thomas Weißschuh wrote: > The meaning of the _no_ws() suffix is not immediately clear when reading > the code. Furthermore there is already a dedicated API to configure the > details of the created power supply in "struct power_supply_config". > Migrate the functionality to "struct power_supply_config" and remove the > special functions. > > > [...] Applied, thanks! [1/8] power: supply: core: add wakeup source inhibit by power_supply_config commit: 49000fee9e639f62ba1f965ed2ae4c5ad18d19e2 [2/8] ACPI: battery: Register power supply with power_supply_register() commit: 2de244e91ddf5cfcf7d88275ad2dbe7f0367e57e [3/8] power: supply: acer_a500_battery: register power supply with devm_power_supply_register() commit: 8c3985bb4e78fb487621dce733fae387fce174a8 [4/8] power: supply: bq27xxx_battery: register power supply with power_supply_register() commit: 1011646d4bd6c2eedb68d131ba7ddaf9ecd2aa9e [5/8] power: supply: cros_usbpd-charger: register power supply with devm_power_supply_register() commit: e8d2dfa6d246b2b879287c79098a1960b3856533 [6/8] power: supply: lenovo_yoga_c630_battery: register power supplies with power_supply_register() commit: 705833305a7a68d6ff163626abd81735940b7751 [7/8] power: supply: max77976_charger: register power supply with devm_power_supply_register() commit: 79b9630e6fc80f225e1564b42cae154970f31136 [8/8] power: supply: core: remove {,devm_}power_supply_register_no_ws() commit: 85d319e14f301e1c68131b74c1dceabae73d1e81 Best regards,
The meaning of the _no_ws() suffix is not immediately clear when reading the code. Furthermore there is already a dedicated API to configure the details of the created power supply in "struct power_supply_config". Migrate the functionality to "struct power_supply_config" and remove the special functions. Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> --- Thomas Weißschuh (8): power: supply: core: add wakeup source inhibit by power_supply_config ACPI: battery: Register power supply with power_supply_register() power: supply: acer_a500_battery: register power supply with devm_power_supply_register() power: supply: bq27xxx_battery: register power supply with power_supply_register() power: supply: cros_usbpd-charger: register power supply with devm_power_supply_register() power: supply: lenovo_yoga_c630_battery: register power supplies with power_supply_register() power: supply: max77976_charger: register power supply with devm_power_supply_register() power: supply: core: remove {,devm_}power_supply_register_no_ws() drivers/acpi/battery.c | 3 +- drivers/power/supply/acer_a500_battery.c | 7 +-- drivers/power/supply/bq27xxx_battery.c | 3 +- drivers/power/supply/cros_usbpd-charger.c | 4 +- drivers/power/supply/lenovo_yoga_c630_battery.c | 14 ++--- drivers/power/supply/max77976_charger.c | 3 +- drivers/power/supply/power_supply_core.c | 68 ++----------------------- include/linux/power_supply.h | 10 +--- 8 files changed, 26 insertions(+), 86 deletions(-) --- base-commit: 27cc6fdf720183dce1dbd293483ec5a9cb6b595e change-id: 20240922-power-supply-no-wakeup-source-45cd19b68a9a Best regards,