[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAJZ5v0jPdfut_QHz9f0x+SSexD_i8xEb5bhkzMv_m=a598Hqxw@mail.gmail.com>
Date: Fri, 7 Nov 2025 20:48:21 +0100
From: "Rafael J. Wysocki" <rafael@...nel.org>
To: Slawomir Rosek <srosek@...gle.com>
Cc: "Rafael J . Wysocki" <rafael@...nel.org>, Alex Hung <alexhung@...il.com>,
Hans de Goede <hansg@...nel.org>, Ilpo Jarvinen <ilpo.jarvinen@...ux.intel.com>,
AceLan Kao <acelan.kao@...onical.com>, Daniel Lezcano <daniel.lezcano@...aro.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Zhang Rui <rui.zhang@...el.com>,
Srinivas Pandruvada <srinivas.pandruvada@...ux.intel.com>, Tomasz Nowicki <tnowicki@...gle.com>,
Stanislaw Kardach <skardach@...gle.com>, Michal Krawczyk <mikrawczyk@...gle.com>,
linux-kernel@...r.kernel.org, linux-acpi@...r.kernel.org,
platform-driver-x86@...r.kernel.org, linux-pm@...r.kernel.org
Subject: Re: [PATCH v4 0/2] ACPI: DPTF: Move INT340X enumeration from DPTF
scan handler to ACPI core
On Mon, Nov 3, 2025 at 5:25 PM Slawomir Rosek <srosek@...gle.com> wrote:
>
> The Intel Dynamic Platform and Thermal Framework (DPTF) relies on
> the INT340X ACPI device objects. The temperature information and
> cooling ability are exposed to the userspace via those objects.
>
> Since kernel v3.17 the ACPI bus scan handler is introduced to prevent
> enumeration of INT340X ACPI device objects on the platform bus unless
> related thermal drivers are enabled. However, using the IS_ENABLED()
> macro in the ACPI scan handler forces the kernel to be recompiled
> when thermal drivers are enabled or disabled, which is a significant
> limitation of its modularity. The IS_ENABLED() macro is particularly
> problematic for the Android Generic Kernel Image (GKI) project which
> uses unified core kernel while SoC/board support is moved to loadable
> vendor modules.
>
> The DPTF requires thermal drivers to be loaded at runtime, thus
> ACPI bus scan handler is not needed and acpi_default_enumeration()
> may create all platform devices, regardless of the actual setting
> of CONFIG_INT340X_THERMAL.
>
> Link to v1: https://lore.kernel.org/all/20250830053404.763995-1-srosek@google.com/
> Link to v2: https://lore.kernel.org/all/20250917120719.2390847-1-srosek@google.com/
> Link to v3: https://lore.kernel.org/all/20251002113404.3117429-1-srosek@google.com/
>
> In v4 the SoC DTS thermal explicitly depends on the X86_64 and NET,
> so the INT340X driver may safely be selected by the SoC DTS thermal
> driver. In addition most of previously submitted patches are dropped
> as they are not necessary, instead the ACPI bus scan handler is simply
> removed from the kernel, thus all platform devices are enumerated by
> the acpi_default_enumeration().
>
> Slawomir Rosek (2):
> ACPI: DPTF: Ignore SoC DTS thermal while scanning
> ACPI: DPTF: Remove int340x thermal scan handler
Both patches applied as 6.19 material, but the subject of the first
patch has been changed to "thermal: intel: Select INT340X_THERMAL from
INTEL_SOC_DTS_THERMAL".
Thanks!
Powered by blists - more mailing lists