|
From: Hans de G. <hde...@re...> - 2025-05-17 11:54:13
|
Hi, On 17-May-25 04:33, Mark Pearson wrote: > If user modifies the battery charge threshold an ACPI event is generated. > Confirmed with Lenovo FW team this is only generated on user event. As no > action is needed, ignore the event and prevent spurious kernel logs. > > Reported-by: Derek Barbosa <deb...@re...> > Closes: https://lore.kernel.org/platform-driver-x86/7e9...@ap.../T/#m5f5b9ae31d3fbf30d7d9a9d76c15fb3502dfd903 > Signed-off-by: Mark Pearson <mpe...@sq...> Thanks, patch looks good to me: Reviewed-by: Hans de Goede <hde...@re...> Regards, Hans > --- > drivers/platform/x86/thinkpad_acpi.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c > index 92b21e49faf6..657625dd60a0 100644 > --- a/drivers/platform/x86/thinkpad_acpi.c > +++ b/drivers/platform/x86/thinkpad_acpi.c > @@ -231,6 +231,7 @@ enum tpacpi_hkey_event_t { > /* Thermal events */ > TP_HKEY_EV_ALARM_BAT_HOT = 0x6011, /* battery too hot */ > TP_HKEY_EV_ALARM_BAT_XHOT = 0x6012, /* battery critically hot */ > + TP_HKEY_EV_ALARM_BAT_LIM_CHANGE = 0x6013, /* battery charge limit changed*/ > TP_HKEY_EV_ALARM_SENSOR_HOT = 0x6021, /* sensor too hot */ > TP_HKEY_EV_ALARM_SENSOR_XHOT = 0x6022, /* sensor critically hot */ > TP_HKEY_EV_THM_TABLE_CHANGED = 0x6030, /* windows; thermal table changed */ > @@ -3777,6 +3778,10 @@ static bool hotkey_notify_6xxx(const u32 hkey, bool *send_acpi_ev) > pr_alert("THERMAL EMERGENCY: battery is extremely hot!\n"); > /* recommended action: immediate sleep/hibernate */ > break; > + case TP_HKEY_EV_ALARM_BAT_LIM_CHANGE: > + pr_debug("Battery Info: battery charge threshold changed\n"); > + /* User changed charging threshold. No action needed */ > + return true; > case TP_HKEY_EV_ALARM_SENSOR_HOT: > pr_crit("THERMAL ALARM: a sensor reports something is too hot!\n"); > /* recommended action: warn user through gui, that */ |
