linux.git/drivers/char/i8k.c, branch master

hwmon: Rename i8k driver to dell-smm-hwmon and move it to hwmon tree

2015-05-24T19:48:12+00:00

This commit moves i8k driver to hwmon tree under name dell-smm-hwmon which is
better name then abbreviation i8k. For backward compatibility is added macro
MODULE_ALIAS("i8k") so modprobe will load driver also old name i8k. CONFIG_I8K
compile option was not changed.

This commit also adds me as maintainer of this new dell-smm-hwmon driver and
remove Guenter Roeck from list who is implicit maintainer all hwmon drivers.

Signed-off-by: Pali Rohár 
Signed-off-by: Greg Kroah-Hartman

i8k: Remove use of seq_printf return value

2015-03-25T14:12:28+00:00

The seq_printf return value, because it's frequently misused,
will eventually be converted to void.

See: commit 1f33c41c03da ("seq_file: Rename seq_overflow() to
     seq_has_overflowed() and make public")

Signed-off-by: Joe Perches 
Acked-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

i8k: Add support for fan labels

2015-01-25T17:17:59+00:00

This patch adds labels support for fans if SMM function with EAX register
0x03a3 reports it. This information was taken from DOS binary NBSVC.MDM.

Additionally this patch change detection of fan presece. Instead reading fan
status now detection is based on new label SMM function. Dell DOS binary
NBSVC.MDM is doing similar checks, so we should do that too.

This patch also remove I8K_FAN_LEFT and I8K_FAN_RIGHT usage from hwmon driver
part because that names does not make sense anymore. So numeric constants are
used instead. Original /proc/i8k ioctl part was not changed for compatibility
reasons.

Signed-off-by: Pali Rohár 
Reviewed-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

i8k: Remove DMI config data for Latitude E6440 and E6540

2015-01-25T17:17:59+00:00

Both Dell Latitude machines were tested with new fan autodetection code and they
are working fine. We already have DMI_MATCH data for generic Latitude machines
which match also E6440 and E6540 models. So we do not need to maintain DMI data
for those specific machines anymore.

Signed-off-by: Pali Rohár 
Tested-by: Pali Rohár 
Tested-by: Steven Honeyman 
Acked-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

i8k: Autodetect fan RPM multiplier

2015-01-25T17:17:59+00:00

This patch adds new function i8k_get_fan_nominal_speed() for doing SMM call
which will return nominal fan RPM for specified fan speed. It returns nominal
RPM value at which fan operate when speed (0, 1, 2, 3) is set. It looks like
RPM value is not accurate, but still provides very useful information.

New function i8k_get_fan_nominal_speed() is used for determinate if fan
multiplier is 1 or 30. If function for maximal fan value success and returned
RPM value too high (above 30000) then fan multiplier is set to 1. Otherwise
multiplier is not changed and default value 30 is used.

Signed-off-by: Pali Rohár 
Tested-by: Pali Rohár 
Tested-by: Gabriele Mazzotta 
Reviewed-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

i8k: Make fan module parameters an unsigned

2015-01-25T17:17:59+00:00

Setting negative fan multiplier or maximal fan speed does make any sense and
can cause problems. So ensure that negative values will not be accepted.

Signed-off-by: Pali Rohár 
Reviewed-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

i8k: Add support for Dell XPS 13

2015-01-25T17:17:58+00:00

XPS 13 does not support turbo speed, so its initialization data
matches that of XPS M140. Make XPS initialization data generic,
and add support for XPS 13.

Signed-off-by: Guenter Roeck 
Reviewed-by: Jean Delvare 
Signed-off-by: Greg Kroah-Hartman

i8k: Rework error retries

2015-01-25T17:17:58+00:00

Instead of returning a previous value if the SMM code returns
an error when trying to read a temperature, retry once.
If that fails again, return -ENODATA. Also return -ENODATA if an
attempt is made to read the GPU temperature but the GPU is
currently turned off.

Drop the I8K_TEMPERATURE_BUG definition and handle the related bug
unconditionally.

Signed-off-by: Guenter Roeck 
Tested-by: Pali Rohár 
Signed-off-by: Greg Kroah-Hartman

i8k: Return -ENODATA for invalid temperature

2015-01-25T17:17:58+00:00

Guenter Roeck suggested to return -ENODATA instead -ERANGE or -EINVAL when BIOS
reports invalid temperature value.

Signed-off-by: Pali Rohár 
Reviewed-by: Guenter Roeck 
Tested-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman

i8k: Register only temperature sensors which have labels

2015-01-25T17:17:58+00:00

Detect presense of sensor by calling type function instead trying to read
temperature value. Type function is working also for sensors which are temporary
turned off (e.g on GPU which is turned off). Dell DOS binary NBSVC.MDM is doing
similar checks, so we should do that too.

Signed-off-by: Pali Rohár 
Reviewed-by: Guenter Roeck 
Tested-by: Guenter Roeck 
Signed-off-by: Greg Kroah-Hartman