* i5k_amb temp_mid replacement
@ 2021-09-30 22:13 Armin Wolf
2021-10-02 14:04 ` Guenter Roeck
0 siblings, 1 reply; 7+ messages in thread
From: Armin Wolf @ 2021-09-30 22:13 UTC (permalink / raw)
To: Jean Delvare, Guenter Roeck; +Cc: linux-hwmon
Hello,
while trying to convert i5k_temp to the new hwmon API to resolve
https://bugzilla.kernel.org/show_bug.cgi?id=208693,
i was asking myself whether or not temp_mid could be replaced with a
standard sysfs attribute?
Since afaik temp_mid and temp_max are both temperature limits, they
could potentially be replaced with
temp_max and temp_crit.
Armin Wolf.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: i5k_amb temp_mid replacement
2021-09-30 22:13 i5k_amb temp_mid replacement Armin Wolf
@ 2021-10-02 14:04 ` Guenter Roeck
2021-10-07 14:20 ` Armin Wolf
0 siblings, 1 reply; 7+ messages in thread
From: Guenter Roeck @ 2021-10-02 14:04 UTC (permalink / raw)
To: Armin Wolf; +Cc: Jean Delvare, linux-hwmon
On Fri, Oct 01, 2021 at 12:13:53AM +0200, Armin Wolf wrote:
> Hello,
>
> while trying to convert i5k_temp to the new hwmon API to resolve
> https://bugzilla.kernel.org/show_bug.cgi?id=208693,
> i was asking myself whether or not temp_mid could be replaced with a
> standard sysfs attribute?
> Since afaik temp_mid and temp_max are both temperature limits, they
> could potentially be replaced with
> temp_max and temp_crit.
>
Quite likely. Unfortunately, the chipset documentation is not available
to the public, so it is difficult to determine what those temperatures
actually mean.
Guenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: i5k_amb temp_mid replacement
2021-10-02 14:04 ` Guenter Roeck
@ 2021-10-07 14:20 ` Armin Wolf
2021-10-07 16:30 ` Guenter Roeck
0 siblings, 1 reply; 7+ messages in thread
From: Armin Wolf @ 2021-10-07 14:20 UTC (permalink / raw)
To: Guenter Roeck; +Cc: Jean Delvare, linux-hwmon
Am 02.10.21 um 16:04 schrieb Guenter Roeck:
> On Fri, Oct 01, 2021 at 12:13:53AM +0200, Armin Wolf wrote:
>> Hello,
>>
>> while trying to convert i5k_temp to the new hwmon API to resolve
>> https://bugzilla.kernel.org/show_bug.cgi?id=208693,
>> i was asking myself whether or not temp_mid could be replaced with a
>> standard sysfs attribute?
>> Since afaik temp_mid and temp_max are both temperature limits, they
>> could potentially be replaced with
>> temp_max and temp_crit.
>>
> Quite likely. Unfortunately, the chipset documentation is not available
> to the public, so it is difficult to determine what those temperatures
> actually mean.
>
> Guenter
Darrick Wong says the chipset documentation would support such a change.
However, he fears that the changed meaning of tempX_max could confuse
userspace
programs.
But i think the current not using standard attribute names will confuse
even more
programms.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: i5k_amb temp_mid replacement
2021-10-07 14:20 ` Armin Wolf
@ 2021-10-07 16:30 ` Guenter Roeck
2021-10-07 17:59 ` Armin Wolf
0 siblings, 1 reply; 7+ messages in thread
From: Guenter Roeck @ 2021-10-07 16:30 UTC (permalink / raw)
To: Armin Wolf; +Cc: Jean Delvare, linux-hwmon
On 10/7/21 7:20 AM, Armin Wolf wrote:
> Am 02.10.21 um 16:04 schrieb Guenter Roeck:
>> On Fri, Oct 01, 2021 at 12:13:53AM +0200, Armin Wolf wrote:
>>> Hello,
>>>
>>> while trying to convert i5k_temp to the new hwmon API to resolve
>>> https://bugzilla.kernel.org/show_bug.cgi?id=208693,
>>> i was asking myself whether or not temp_mid could be replaced with a
>>> standard sysfs attribute?
>>> Since afaik temp_mid and temp_max are both temperature limits, they
>>> could potentially be replaced with
>>> temp_max and temp_crit.
>>>
>> Quite likely. Unfortunately, the chipset documentation is not available
>> to the public, so it is difficult to determine what those temperatures
>> actually mean.
>>
>> Guenter
> Darrick Wong says the chipset documentation would support such a change.
> However, he fears that the changed meaning of tempX_max could confuse
> userspace
> programs.
> But i think the current not using standard attribute names will confuse
> even more
> programms.
I still don't know what temperature limits temp_mid and temp_max actually
reflect, so I still can not really comment.
Guenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: i5k_amb temp_mid replacement
2021-10-07 16:30 ` Guenter Roeck
@ 2021-10-07 17:59 ` Armin Wolf
2021-10-07 19:24 ` Guenter Roeck
0 siblings, 1 reply; 7+ messages in thread
From: Armin Wolf @ 2021-10-07 17:59 UTC (permalink / raw)
To: Guenter Roeck; +Cc: Jean Delvare, linux-hwmon
Am 07.10.21 um 18:30 schrieb Guenter Roeck:
> On 10/7/21 7:20 AM, Armin Wolf wrote:
>> Am 02.10.21 um 16:04 schrieb Guenter Roeck:
>>> On Fri, Oct 01, 2021 at 12:13:53AM +0200, Armin Wolf wrote:
>>>> Hello,
>>>>
>>>> while trying to convert i5k_temp to the new hwmon API to resolve
>>>> https://bugzilla.kernel.org/show_bug.cgi?id=208693,
>>>> i was asking myself whether or not temp_mid could be replaced with a
>>>> standard sysfs attribute?
>>>> Since afaik temp_mid and temp_max are both temperature limits, they
>>>> could potentially be replaced with
>>>> temp_max and temp_crit.
>>>>
>>> Quite likely. Unfortunately, the chipset documentation is not available
>>> to the public, so it is difficult to determine what those temperatures
>>> actually mean.
>>>
>>> Guenter
>> Darrick Wong says the chipset documentation would support such a change.
>> However, he fears that the changed meaning of tempX_max could confuse
>> userspace
>> programs.
>> But i think the current not using standard attribute names will confuse
>> even more
>> programms.
>
> I still don't know what temperature limits temp_mid and temp_max actually
> reflect, so I still can not really comment.
>
> Guenter
"Intel 6400/6402 Advanced Memory Buffer" seems to document that.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: i5k_amb temp_mid replacement
2021-10-07 17:59 ` Armin Wolf
@ 2021-10-07 19:24 ` Guenter Roeck
2021-10-07 19:39 ` Guenter Roeck
0 siblings, 1 reply; 7+ messages in thread
From: Guenter Roeck @ 2021-10-07 19:24 UTC (permalink / raw)
To: Armin Wolf; +Cc: Jean Delvare, linux-hwmon
On 10/7/21 10:59 AM, Armin Wolf wrote:
> Am 07.10.21 um 18:30 schrieb Guenter Roeck:
>> On 10/7/21 7:20 AM, Armin Wolf wrote:
>>> Am 02.10.21 um 16:04 schrieb Guenter Roeck:
>>>> On Fri, Oct 01, 2021 at 12:13:53AM +0200, Armin Wolf wrote:
>>>>> Hello,
>>>>>
>>>>> while trying to convert i5k_temp to the new hwmon API to resolve
>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=208693,
>>>>> i was asking myself whether or not temp_mid could be replaced with a
>>>>> standard sysfs attribute?
>>>>> Since afaik temp_mid and temp_max are both temperature limits, they
>>>>> could potentially be replaced with
>>>>> temp_max and temp_crit.
>>>>>
>>>> Quite likely. Unfortunately, the chipset documentation is not available
>>>> to the public, so it is difficult to determine what those temperatures
>>>> actually mean.
>>>>
>>>> Guenter
>>> Darrick Wong says the chipset documentation would support such a change.
>>> However, he fears that the changed meaning of tempX_max could confuse
>>> userspace
>>> programs.
>>> But i think the current not using standard attribute names will confuse
>>> even more
>>> programms.
>>
>> I still don't know what temperature limits temp_mid and temp_max actually
>> reflect, so I still can not really comment.
>>
>> Guenter
> "Intel 6400/6402 Advanced Memory Buffer" seems to document that.
Ah, thanks.
From the description, it should be:
TEMPLO -> temp1_max
TEMPMID -> temp1_crit
TEMPHI -> temp1_emergency
The respective _alarm attributes should be supported as well (via TEMPSTAT),
but that would be a separate patch.
Guenter
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: i5k_amb temp_mid replacement
2021-10-07 19:24 ` Guenter Roeck
@ 2021-10-07 19:39 ` Guenter Roeck
0 siblings, 0 replies; 7+ messages in thread
From: Guenter Roeck @ 2021-10-07 19:39 UTC (permalink / raw)
To: Armin Wolf; +Cc: Jean Delvare, linux-hwmon
On 10/7/21 12:24 PM, Guenter Roeck wrote:
> On 10/7/21 10:59 AM, Armin Wolf wrote:
>> Am 07.10.21 um 18:30 schrieb Guenter Roeck:
>>> On 10/7/21 7:20 AM, Armin Wolf wrote:
>>>> Am 02.10.21 um 16:04 schrieb Guenter Roeck:
>>>>> On Fri, Oct 01, 2021 at 12:13:53AM +0200, Armin Wolf wrote:
>>>>>> Hello,
>>>>>>
>>>>>> while trying to convert i5k_temp to the new hwmon API to resolve
>>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=208693,
>>>>>> i was asking myself whether or not temp_mid could be replaced with a
>>>>>> standard sysfs attribute?
>>>>>> Since afaik temp_mid and temp_max are both temperature limits, they
>>>>>> could potentially be replaced with
>>>>>> temp_max and temp_crit.
>>>>>>
>>>>> Quite likely. Unfortunately, the chipset documentation is not available
>>>>> to the public, so it is difficult to determine what those temperatures
>>>>> actually mean.
>>>>>
>>>>> Guenter
>>>> Darrick Wong says the chipset documentation would support such a change.
>>>> However, he fears that the changed meaning of tempX_max could confuse
>>>> userspace
>>>> programs.
>>>> But i think the current not using standard attribute names will confuse
>>>> even more
>>>> programms.
>>>
>>> I still don't know what temperature limits temp_mid and temp_max actually
>>> reflect, so I still can not really comment.
>>>
>>> Guenter
>> "Intel 6400/6402 Advanced Memory Buffer" seems to document that.
>
> Ah, thanks.
>
> From the description, it should be:
>
> TEMPLO -> temp1_max
> TEMPMID -> temp1_crit
> TEMPHI -> temp1_emergency
>
Scratch that. I am misreading the document.
TEMPLO -> temp1_min
TEMPMID -> temp1_max
TEMPHI -> temp1_crit
is fine.
Guenter
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-10-07 19:39 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-30 22:13 i5k_amb temp_mid replacement Armin Wolf
2021-10-02 14:04 ` Guenter Roeck
2021-10-07 14:20 ` Armin Wolf
2021-10-07 16:30 ` Guenter Roeck
2021-10-07 17:59 ` Armin Wolf
2021-10-07 19:24 ` Guenter Roeck
2021-10-07 19:39 ` Guenter Roeck
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.