Project

General

Profile

Feature #13193

pchtemp could support Intel 400 series chipset

Added by Robert Mustacchi 6 months ago. Updated 6 months ago.

Status:
Closed
Priority:
Normal
Category:
driver - device drivers
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

While looking around at docs, I found Intel has documented the 400 series chipset and they support the pch thermal sensor. We should add this alias to the pchtemp(7D) driver.

#1

Updated by Electric Monk 6 months ago

  • Gerrit CR set to 954
#2

Updated by Robert Mustacchi 6 months ago

I tested this by doing a quick sanity test by passing through an instance of this to a VM on my laptop and verifying that it showed up in topo. I can't test this against newer hardware, since I don't have it.

rm@nienor:~$ pfexec /usr/lib/fm/fmd/fmtopo -V *sensor*
TIME                 UUID
Oct 03 08:21:57 4525fca4-cff3-e9a2-ad07-90a00c203550

hc://:product-id=Standard-PC-(i440FX-+-PIIX,-1996):server-id=nienor/motherboard=0/chipset=0?sensor=temp
  group: protocol                       version: 1   stability: Private/Private
    resource          fmri      hc://:product-id=Standard-PC-(i440FX-+-PIIX,-1996):server-id=nienor/motherboard=0/chipset=0?sensor=temp
  group: authority                      version: 1   stability: Private/Private
    product-id        string    Standard-PC-(i440FX-+-PIIX,-1996)
    server-id         string    nienor
  group: facility                       version: 1   stability: Private/Private
    sensor-class      string    threshold
    type              uint32    0x1 (TEMP)
    units             uint32    0x1 (DEGREES_C)
    reading           double    54.000000
#3

Updated by Electric Monk 6 months ago

  • Status changed from New to Closed
  • % Done changed from 80 to 100

git commit 60de61ff53a6483ad00524b59031b99fcc10d46d

commit  60de61ff53a6483ad00524b59031b99fcc10d46d
Author: Robert Mustacchi <rm@fingolfin.org>
Date:   2020-10-05T15:36:14.000Z

    13193 pchtemp could support Intel 400 series chipset
    Reviewed by: Paul Winder <paul@winder.uk.net>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF