Hi, On 4/29/24 11:34 AM, Hans de Goede wrote: > acpi_send_ev is a bool and everywhere else true/false is used > to set it. Replace the one instance using 0 with false. > > Suggested-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx> > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> I've added this to my review-hans (soon to be for-next) branch now. Regards, Hans > --- > drivers/platform/x86/thinkpad_acpi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/platform/x86/thinkpad_acpi.c b/drivers/platform/x86/thinkpad_acpi.c > index 7e2288009063..e2e6c4d3dff7 100644 > --- a/drivers/platform/x86/thinkpad_acpi.c > +++ b/drivers/platform/x86/thinkpad_acpi.c > @@ -3912,7 +3912,7 @@ static void hotkey_notify(struct ibm_struct *ibm, u32 event) > if (tp_features.hotkey_wlsw && > hkey == TP_HKEY_EV_RFKILL_CHANGED) { > tpacpi_send_radiosw_update(); > - send_acpi_ev = 0; > + send_acpi_ev = false; > known_ev = true; > } > break;