On Wed, 17 Jul 2024 22:03:32 +0200, Hans de Goede wrote: > info->max_cv is in millivolts, divide the microvolt value being written > to constant_charge_voltage by 1000 *before* clamping it to info->max_cv. > > Before this fix the code always tried to set constant_charge_voltage > to max_cv / 1000 = 4 millivolt, which ends up in setting it to 4.1V > which is the lowest supported value. > > [...] Applied, thanks! [1/2] power: supply: axp288_charger: Fix constant_charge_voltage writes commit: b34ce4a59cfe9cd0d6f870e6408e8ec88a964585 [2/2] power: supply: axp288_charger: Round constant_charge_voltage writes down commit: 81af7f2342d162e24ac820c10e68684d9f927663 Best regards, -- Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>