lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <260a3e55-7072-4298-8fff-b5c2c674c65b@t-8ch.de>
Date: Tue, 27 Aug 2024 22:50:17 +0200
From: Thomas Weißschuh <linux@...ssschuh.net>
To: Hans de Goede <hdegoede@...hat.com>
Cc: Andres Salomon <dilinger@...ued.net>, linux-kernel@...r.kernel.org, 
	Pali Rohár <pali@...nel.org>, platform-driver-x86@...r.kernel.org, 
	Matthew Garrett <mjg59@...f.ucam.org>, Sebastian Reichel <sre@...nel.org>, 
	Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>, linux-pm@...r.kernel.org, Dell.Client.Kernel@...l.com
Subject: Re: [PATCH v4 1/2] platform/x86:dell-laptop: Add knobs to change
 battery charge settings

On 2024-08-26 16:44:35+0000, Hans de Goede wrote:
> [..]

> Yes not being able to see the supported values is annoying I actually
> wrote an email about that earlier today:
> 
> https://lore.kernel.org/linux-pm/49993a42-aa91-46bf-acef-4a089db4c2db@redhat.com/
> 
> but we need to make sure that the output is consistent between drivers otherwise
> userspace can never know how to use the API, so for charge_type the dell
> driver should only output the active type, not all the options.
> 
> This reminds me that there was a patch-series to allow battery extension drivers
> like this one to actually use the power-supply core code for show()/store()
> Thomas IIRC that series was done by you ?  What is the status of that ?

Yes, that is from me, [0].
At least I'll need to implement the feedback from Sebastian and a proper
locking scheme.
I still intend to work on it, but if there are other people waiting for
it, I'll try to get on with it.

In the meantime maybe the psy core could export a helper function for
formatting and parsing? Similar to the charge_behaviour helpers.

[0] https://lore.kernel.org/lkml/20240608-power-supply-extensions-v2-0-2dcd35b012ad@weissschuh.net/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ