| Message ID | 20241209204051.8786-1-hdegoede@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | power: supply: Add new "charge_types" property | expand |
On Mon, 09 Dec 2024 21:40:47 +0100, Hans de Goede wrote: > As first discussed here: > https://lore.kernel.org/linux-pm/49993a42-aa91-46bf-acef-4a089db4c2db@redhat.com/ > > Some power_supply devices have a writable charge_type property, but > userspace cannot know which charge_type values are valid to write. > > This series adds a new "charge_types" property, which is identical to > "charge_type" but reading returns a list of supported charge-types with > the currently active type surrounded by square brackets, e.g.: > > [...] Applied, thanks! [1/4] power: supply: power_supply_show_enum_with_available(): Replace spaces with '_' commit: cc4c34577dbc17729b7e3f1266cf12d043e8c170 Best regards,
Hi, As first discussed here: https://lore.kernel.org/linux-pm/49993a42-aa91-46bf-acef-4a089db4c2db@redhat.com/ Some power_supply devices have a writable charge_type property, but userspace cannot know which charge_type values are valid to write. This series adds a new "charge_types" property, which is identical to "charge_type" but reading returns a list of supported charge-types with the currently active type surrounded by square brackets, e.g.: Fast [Standard] Long_Life this allows userspace to find out which charge-types are supported. Patch 1/4 does some prep work and patch 2/4 is the core implementation of "charge_types" property support. Patch 3/4 is a standard psy-driver user of the new "charge_types" property support. Patch 4/4 shows an ACPI battery extension driver with support for this property using the new show()/store() helpers. Changes in v3: - Rebase on top of v6.13-rc1 Changes in v2: - Replace spaces with '_' instead of surrounding the text-value by "" - Add "Check charge_types to get the values supported by the battery." to Documentation/ABI/testing/sysfs-class-power - Add a note about labels with spaces having these replaced by '_' in charge_types output to Documentation/ABI/testing/sysfs-class-power - Use power_supply_match_string() in power_supply_charge_types_parse() Regards, Hans Hans de Goede (4): power: supply: power_supply_show_enum_with_available(): Replace spaces with '_' power: supply: core: Add new "charge_types" property power: supply: bq24190_charger: Add support for "charge_types" property platform/x86: dell-laptop: Use power_supply_charge_types_show/_parse() helpers Documentation/ABI/testing/sysfs-class-power | 20 ++++++ drivers/platform/x86/dell/dell-laptop.c | 54 ++++++++-------- drivers/power/supply/bq24190_charger.c | 7 ++ drivers/power/supply/power_supply_sysfs.c | 71 +++++++++++++++++++-- include/linux/power_supply.h | 23 ++++++- 5 files changed, 141 insertions(+), 34 deletions(-)