github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-09-22 12:44:03 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

power: supply: Add registration init callback

Browse Source 
Some battery drivers need to consume monitored-battery data before their
power_supply is visible. That lets them prepare hardware configuration from
parsed battery information without racing userspace exposure.

power_supply_get_battery_info() already runs in
__power_supply_register() for battery devices before device_add(). Add an
optional descriptor init callback after driver data and battery info are
available. The callback runs in sleepable process context while the power
supply is still unpublished.

Keep the callback synchronous: it must not publish changes or start
asynchronous activity that can access the power supply before registration
completes.

Require callbacks to return zero or a negative errno. Defensively reject
positive returns so registration cannot return an invalid error pointer.

Assisted-by: OpenCode:gpt-5.6-sol
Signed-off-by: Vincent Cloutier <vincent@cloutier.co>
Link: https://patch.msgid.link/20260727011319.621794-2-vincent.cloutier@icloud.com
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
This commit is contained in:
Vincent Cloutier 2026-07-26 21:12:59 -04:00 committed by Sebastian Reichel
parent ab1112df8f
commit c1eb5905fd
2 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/power/supply/power_supply_core.c
View File

@ -1777,6 +1777,14 @@ __power_supply_register(struct device *parent,
init_rwsem(&psy->extensions_sem);
INIT_LIST_HEAD(&psy->extensions);
if (desc->init) {
rc = desc->init(psy);
if (WARN_ON_ONCE(rc > 0))
rc = -EINVAL;
if (rc)
goto check_supplies_failed;
}
rc = device_add(dev);
if (rc)
goto device_add_failed;

9
include/linux/power_supply.h
View File

@ -285,6 +285,15 @@ struct power_supply_desc {
int (*property_is_writeable)(struct power_supply *psy,
enum power_supply_property psp);
void (*external_power_changed)(struct power_supply *psy);
/*
* Optional registration-time initialization. This runs in sleepable
* process context after driver data and any battery info are available,
* but before the device is added. The callback must not publish changes or
* start asynchronous activity that can access the power supply before
* registration completes. Return 0 on success or a negative errno on
* failure.
*/
int (*init)(struct power_supply *psy);
/*
* Set if thermal zone should not be created for this power supply.