| Message ID | 20220111051842.25634-1-ajderossi@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | hwmon: (asus_wmi_ec_sensors) Support T_Sensor on Prime X570-Pro | expand |
Thank you, Anthony, for the fix. It was also included in the v3 patch for a new EC sensors driver that should replace this one. Perhaps I can enable the T_Sensor for each B550/X570 ASUS model that provides this connector? Seems like it's always found in the EC registers. Before you submitted this patch another user provided information for this board with no T_Sensor, and I thought this board was the exception. Best regards, Eugene
On Tue, Jan 11, 2022 at 9:24 AM Eugene Shalygin <eugene.shalygin@gmail.com> wrote: > > Thank you, Anthony, for the fix. It was also included in the v3 patch > for a new EC sensors driver that should replace this one. Perhaps I > can enable the T_Sensor for each B550/X570 ASUS model that provides > this connector? Seems like it's always found in the EC registers. > Before you submitted this patch another user provided information for > this board with no T_Sensor, and I thought this board was the > exception. If the header is present on a B550/X570 board I assume the sensor is in the EC registers, but I don't have any other models to test. Anthony
On Mon, Jan 10, 2022 at 09:18:42PM -0800, Anthony DeRossi wrote: > Asus Prime X570-Pro motherboards have a T_Sensor header that can be > connected to an optional temperature probe. > > Signed-off-by: Anthony DeRossi <ajderossi@gmail.com> Applied to hwmon-next. Guenter > --- > drivers/hwmon/asus_wmi_ec_sensors.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/hwmon/asus_wmi_ec_sensors.c b/drivers/hwmon/asus_wmi_ec_sensors.c > index 22a1459305a7..a3a2f014dec0 100644 > --- a/drivers/hwmon/asus_wmi_ec_sensors.c > +++ b/drivers/hwmon/asus_wmi_ec_sensors.c > @@ -112,7 +112,8 @@ struct asus_wmi_data { > /* boards with EC support */ > static struct asus_wmi_data sensors_board_PW_X570_P = { > .known_board_sensors = { > - SENSOR_TEMP_CHIPSET, SENSOR_TEMP_CPU, SENSOR_TEMP_MB, SENSOR_TEMP_VRM, > + SENSOR_TEMP_CHIPSET, SENSOR_TEMP_CPU, SENSOR_TEMP_MB, > + SENSOR_TEMP_T_SENSOR, SENSOR_TEMP_VRM, > SENSOR_FAN_CHIPSET, > SENSOR_MAX > },
diff --git a/drivers/hwmon/asus_wmi_ec_sensors.c b/drivers/hwmon/asus_wmi_ec_sensors.c index 22a1459305a7..a3a2f014dec0 100644 --- a/drivers/hwmon/asus_wmi_ec_sensors.c +++ b/drivers/hwmon/asus_wmi_ec_sensors.c @@ -112,7 +112,8 @@ struct asus_wmi_data { /* boards with EC support */ static struct asus_wmi_data sensors_board_PW_X570_P = { .known_board_sensors = { - SENSOR_TEMP_CHIPSET, SENSOR_TEMP_CPU, SENSOR_TEMP_MB, SENSOR_TEMP_VRM, + SENSOR_TEMP_CHIPSET, SENSOR_TEMP_CPU, SENSOR_TEMP_MB, + SENSOR_TEMP_T_SENSOR, SENSOR_TEMP_VRM, SENSOR_FAN_CHIPSET, SENSOR_MAX },
Asus Prime X570-Pro motherboards have a T_Sensor header that can be connected to an optional temperature probe. Signed-off-by: Anthony DeRossi <ajderossi@gmail.com> --- drivers/hwmon/asus_wmi_ec_sensors.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)