Re: [PATCH v2 0/2] platform/x86: thinkpad_acpi: cleanup conversions
From: Ilpo Järvinen
Date: Mon Oct 05 2026 - 14:52:18 EST
On Tue, 29 Sep 2026 21:59:09 -0700, Dmitry Torokhov wrote:
> This series converts thinkpad_acpi resource management to use helpers
> from linux/cleanup.h:
> - Convert conditional mutex lock calls to ACQUIRE() and ACQUIRE_ERR()
> - Use __free(kfree) for local pointer allocations
>
> Changes in v2:
> - Dropped patch 1 ("convert mutex_lock() to guard(mutex)") as it was
> already applied upstream.
> - Preserved 0 return value on failure in brightness_get().
> - Fixed auxmac_init() to avoid setting auxmac to "unavailable" before
> evaluating the MACA ACPI method.
> - Collected Reviewed-by and Tested-by trailers.
>
> [...]
Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.
FYI [if applicable to your patch], as per Linus' policy change, also
fixes are mostly routed through for-next unless the fix is for a
commit introduced in the most recent cycle or is clearly a regression
fix.
The list of commits applied:
[1/2] platform/x86: thinkpad_acpi: convert conditional mutex locks to ACQUIRE_ERR()
commit: 86cab001301c4c65e6763282ca9a477dc1fc4e72
[2/2] platform/x86: thinkpad_acpi: use __free(kfree) for automatic cleanup
commit: 40da5fbf461e4cefa45aa259b0771449597d3135
--
i.