Re: [PATCH] platform/x86: thinkpad_acpi: Use bool for local is_thinkpad variable

From: Mark Pearson

Date: Tue Oct 06 2026 - 08:34:49 EST


On Mon, Oct 5, 2026, at 12:31 PM, Thorsten Blum wrote:
> Use bool for the local is_thinkpad variable because it only stores a
> boolean value.
>
> Signed-off-by: Thorsten Blum <blum@xxxxxxxxxx>
> ---
> drivers/platform/x86/lenovo/thinkpad_acpi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/lenovo/thinkpad_acpi.c
> b/drivers/platform/x86/lenovo/thinkpad_acpi.c
> index 1661f070c571..1082c36ae723 100644
> --- a/drivers/platform/x86/lenovo/thinkpad_acpi.c
> +++ b/drivers/platform/x86/lenovo/thinkpad_acpi.c
> @@ -11925,7 +11925,7 @@ static int __must_check __init
> get_thinkpad_model_data(
>
> static int __init probe_for_thinkpad(void)
> {
> - int is_thinkpad;
> + bool is_thinkpad;
>
> if (acpi_disabled)
> return -ENODEV;

Reviewed-by: Mark Pearson <mpearson-lenovo@xxxxxxxxx>
Mark