mbox series

[v3,0/2] drivers: thermal: intel: Use model-specific bitmasks for temperature registers

Message ID 20240614211606.5896-1-ricardo.neri-calderon@linux.intel.com (mailing list archive)
Headers show
Series drivers: thermal: intel: Use model-specific bitmasks for temperature registers | expand

Message

Ricardo Neri June 14, 2024, 9:16 p.m. UTC
Hi,

Here is v3 of the patchset to use model-specific bitmasks to read TCC
offset and the digital temperature readout of IA32_[PACKAGE]_THERM_STATUS.

You can read the details and motivation in the cover letter of v1[1]. V2
is here [2].

Changes since v2:
 * Dropped patch 3/3 ("hwmon: (coretemp) Extend the bitmask to read
   temperature to 0xff") as it has been merged in v6.10-rc1.
 * Used the new X86_MATCH_VFM() macro. (Rafael)
 * Added Reviewed-by tags from Rui. Thanks!
 * Rebased on Rafael's testing branch (based on v6.10-rc3).

I have tested these patches on Alder Lake, Meteor Lake, and Grand Ridge
systems.

These patches apply cleanly on top of the `testing` branch of the linux-pm
repository.

Thanks and BR,
Ricardo

[1]. https://lore.kernel.org/linux-pm/20240406010416.4821-1-ricardo.neri-calderon@linux.intel.com/
[2]. https://lore.kernel.org/all/20240425171311.19519-1-ricardo.neri-calderon@linux.intel.com/

Ricardo Neri (2):
  thermal: intel: intel_tcc: Add model checks for temperature registers
  thermal: intel: intel_tcc_cooling: Use a model-specific bitmask for
    TCC offset

 drivers/thermal/intel/intel_tcc.c         | 177 +++++++++++++++++++++-
 drivers/thermal/intel/intel_tcc_cooling.c |   2 +-
 include/linux/intel_tcc.h                 |   1 +
 3 files changed, 174 insertions(+), 6 deletions(-)

Comments

Rafael J. Wysocki June 21, 2024, 12:53 p.m. UTC | #1
Hi,

On Fri, Jun 14, 2024 at 11:10 PM Ricardo Neri
<ricardo.neri-calderon@linux.intel.com> wrote:
>
> Hi,
>
> Here is v3 of the patchset to use model-specific bitmasks to read TCC
> offset and the digital temperature readout of IA32_[PACKAGE]_THERM_STATUS.
>
> You can read the details and motivation in the cover letter of v1[1]. V2
> is here [2].
>
> Changes since v2:
>  * Dropped patch 3/3 ("hwmon: (coretemp) Extend the bitmask to read
>    temperature to 0xff") as it has been merged in v6.10-rc1.
>  * Used the new X86_MATCH_VFM() macro. (Rafael)
>  * Added Reviewed-by tags from Rui. Thanks!
>  * Rebased on Rafael's testing branch (based on v6.10-rc3).
>
> I have tested these patches on Alder Lake, Meteor Lake, and Grand Ridge
> systems.
>
> These patches apply cleanly on top of the `testing` branch of the linux-pm
> repository.
>
> Thanks and BR,
> Ricardo
>
> [1]. https://lore.kernel.org/linux-pm/20240406010416.4821-1-ricardo.neri-calderon@linux.intel.com/
> [2]. https://lore.kernel.org/all/20240425171311.19519-1-ricardo.neri-calderon@linux.intel.com/
>
> Ricardo Neri (2):
>   thermal: intel: intel_tcc: Add model checks for temperature registers
>   thermal: intel: intel_tcc_cooling: Use a model-specific bitmask for
>     TCC offset
>
>  drivers/thermal/intel/intel_tcc.c         | 177 +++++++++++++++++++++-
>  drivers/thermal/intel/intel_tcc_cooling.c |   2 +-
>  include/linux/intel_tcc.h                 |   1 +
>  3 files changed, 174 insertions(+), 6 deletions(-)
>
> --

Both patches applied as 6.11 material, thanks!