Re: [PATCH] i8k: Add support for temperature sensor labels

Jean Delvare
2014-12-03T09:09:36Z

Hi Guenter,

On Tue, 02 Dec 2014 06:26:29 -0800, Guenter Roeck wrote:
> On 12/02/2014 05:23 AM, Jean Delvare wrote:
> > Ultimately /proc/i8k should be killed altogether and every function
> > should use the appropriate standard interface. But I don't have any Dell
> > laptop around so I won't go into that myself.
>
> The only function not covered by hwmon, as far as I can see, is reading
> the "Fn key" status, whatever that is, and reporting the power status
> (AC or battery).
>
> The first seems to be covered by the already existing dell drivers in
> platform/x86. The latter must be covered as well; Linux does
> display the correct power status on all my Dell laptops.

How old are they? The i8k driver is ancestral, so I could imagine that
i8k is the only way to report power status on very old Dell laptop
models. Or maybe not, I just don't know.

> Changing the ABI is not supposed to happen, so I am not sure if we can
> just drop the /proc interface.

Dropping deprecated interfaces after a transition period is not so
uncommon. AFAIK the proc interface of i8k is only used by the i8kutils
user-space package, which FWIW is no longer part of openSUSE. The
dell-laptop and dell-wmi drivers are almost 6 years old, that looks
like an already very comfortable transition period to me.

That being said, that's only my opinion and I'm not going to fight for
this. If the hwmon interface is moved to dell-laptop and i8k is killed,
I'm happy. If the i8k driver is moved to drivers/hwmon and stripped
down, I'm happy. But if none of this happens, I don't really care, to
be honest.

--
Jean Delvare
SUSE L3 Support