Re: [ibm-acpi-devel] 2.6.33-rc2: regression: gkrellm no longer shows all the temperatures on thinkpad x60

[Didier Spaier <didier.spaier@epsm.fr>]

Pavel Machek wrote:
> On Sun 2009-12-27 23:32:59, Didier Spaier wrote:
>> Pavel Machek wrote:
>>> On Sun 2009-12-27 23:24:15, Didier Spaier wrote:
>>>> Pavel Machek wrote:
>>>>> ...I'll have to find out where gkrell got that info. It worked in
>>>>> 2.6.32.
>>>>>
>>>>> 									Pavel
>>>> Only to make sure... What is the output of cat /proc/acpi/ibm/thermal?
>>>>
>>>> I ask that question because I noticed battery temperature 0° displayed  by conky after updating to 2.6.32.2.
>>>> Looking at my .conkyrc I saw I tried to display ${smapi_bat_temp 0}
>>> but I had forgotten to re-install the tp_smapi module.
>>>
>>> Uhuh, there's no /proc/acpi/ibm:
>>>
>>> ls /proc/acpi/ibm         ls: cannot access /proc/acpi/ibm: No
>>> such file or directory
>>>
>>> . I guess that's the problem :-(. Config is:
>>>
>>> CONFIG_THINKPAD_ACPI=y
>>> # CONFIG_THINKPAD_ACPI_DEBUGFACILITIES is not set
>>> # CONFIG_THINKPAD_ACPI_DEBUG is not set
>>> CONFIG_THINKPAD_ACPI_UNSAFE_LEDS=y
>>> CONFIG_THINKPAD_ACPI_VIDEO=y
>>> # CONFIG_THINKPAD_ACPI_HOTKEY_POLL is not set
>>>
>>> ...so I do not see "enable legacy interface" option.
> 
>> bash-3.1$ grep THINKPAD_ACPI /boot/config-2.6.32.2strip
>> CONFIG_THINKPAD_ACPI=m
>> CONFIG_THINKPAD_ACPI_DEBUGFACILITIES=y
>> CONFIG_THINKPAD_ACPI_DEBUG=y
>> CONFIG_THINKPAD_ACPI_UNSAFE_LEDS=y
>> CONFIG_THINKPAD_ACPI_VIDEO=y
>> CONFIG_THINKPAD_ACPI_HOTKEY_POLL=y
>> bash-3.1$
>>
>> I suggest you modularize THINKPAD_ACPI.
> 
> How would modular version help?
> 
I can't tell you how exactly.

But I can tell that is *does* work as a module, at least with 2.6.32.2.

Plus, reading Documentation/laptops/thinkpad-acpi make obvious IMHO,
though not explicitly stated, that it is supposed to be built as a module.

For instance 'grep module thinkpad-acpi|wc -l'  returns 17 here ;)

And how the 'modprobe' command (founded twice in the same file) is supposed
to work with a driver built in the kernel image?

But of course deciding to try that or not is up to you :-)

Best regards,

Didier