README: Add dkms instructions for #95

Signed-off-by: Burt P <pburt0@gmail.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>

auto-adjust loop threshold to changes in the array size

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

replaced two spaces with one

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

fixing indents in #defines: replacing spaces by tabs

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Enable all temperature registers for IT8655E and IT8665E

Those chips always have all 6 temperature registers enabled.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix temperature type for peci/amdtsi if diode/thermistor is also set

PECI / AMDTSI has higher priority than thermal diode/thermistor settings.
Always report PECI/AMDTSI if it is enabled on a given port.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Revert "Provide vid_from_reg and vid_which_vrm if needed"

This reverts commit 905a3a095d49ef9f6eea5ec1b0bc21e3c6901b9e.

This didn't work and results in duplicate symbols.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Provide vid_from_reg and vid_which_vrm if needed

Some oddball distributions do not have CONFIG_HWMON_VID enabled.
Work around the problem by providing dummy functions if this is the case.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Support Arch

Arch uses /proc/kallsyms and /proc/config.gz by default instead of /boot/System.map* and /boot/config*

Fix fan control for IT8736F

The 4th fan control does not have a separate set of registers,
so don't bother with it. PWM control is the old fashined method,
not the new one.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add preliminary support for IT8736F and IT8738E

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix scaling for IT8732F

3.3V channels report the wrong voltages on IT8732F since the scaling bit
is not set.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Save and restore bank around envmon register accesses if needed

If an external EC accesses the chip through SMBus registers
and if the chip's environmental registers are paged, it is likely
that the EC modifies the page register. If so, it likely will not
update the page on each access but assume that it "owns" the chip.
If the driver then modifies the page register, subsequent accesses
from the EC will likely have unpredictable results.
To avoid that problem, let's save the page register value after
disabling SMBus accesses and restore the original value when done.
This is only necessary if accesses are not handled through MMIO
since we don't touch the page register in that case.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix up fan / pwm detection for IT8625/IT8665

Hope it is correct this time around.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Makefile: DKMS fix

Allow DKMS (or anyone) to specify TARGET in Makefile.

Signed-off-by: Burt P <pburt0@gmail.com>

Add optional DKMS target to Makefile (v2)

Using DKMS, the driver will be automatically rebuilt when
the kernel is updated.

* `make dkms` to install via DKMS
* `make dkms_clean` to remove from DKMS

Signed-off-by: Burt P <pburt0@gmail.com>

README: Fix typo

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Revert "Add optional DKMS target to Makefile"

This reverts commit 6c1e48b4f16b65b6e2ff95d950b250581b0cbe91.

See comments:

"This is indeed not working, at least on Fedora. I tried specifying
MAKE and CLEAN but the outcome is the same (which makes sense, since
the defaults should already cover this)."

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add comment explaining why smbus_disable is needed in probe

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add more known problems to ISSUES file

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add ISSUES file

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add optional DKMS target to Makefile

Using DKMS, the driver will be automatically rebuilt when
the kernel is updated.

* `make dkms` to install via DKMS
* `make dkms_clean` to remove from DKMS

Signed-off-by: Burt P <pburt0@gmail.com>

Fix PECI/AMDTSI selection

We can not use register 0x98 since it is typically not programmed
on systems selecting PECI. Try register 0x0a (Interface Selection)
which should be a better fit anyway.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Do not disable SMBs unless really necessary

Only disable SMBus if we are going to read/write data.
Disabling it on each attribute read can result in system
instabilities.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Report actual chip name, not its ID

For some recent chips (at least IT8792E/IT8795E), the value of the chip ID
register does not match the chip name. Display the textual, not the value
of the ID register, in the kernel log.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Experimental support for IT8606E

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Makefile: Versioning, take three

Make sure the build passes if the source is not in a git repository.
Again, that means that version information won't be available, but there
is only so much we can do about that.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Checkpatch cleanup: Line length

Try to stay below the 80-character-per-line limit.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Checkpatch cleanup: Opening brackets

Checkpatch nowadays is allergic against '{' in continuation lines.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Checkpatch cleanup: Use octal file permissions

Symbolic file permissions ran out of favor and result in a checkpatch
warning. Use octal permissions instead.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Checkpatch cleanup: Use octal permissions for module parameters

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Checkpatch cleanup: Double semicolon, missing space

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Drop version.h

Some distributions use their own Makefile and thus won't create version.h,
resulting in a build failure. Pass the version as define instead.
That won't help for generating the driver version with those distributions,
but at least the driver will build.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add support for 4th temperature sensor on IT8622

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Improve AMDTSI temp type detection, and temp 4 type detection on IT8622

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add SMBus bitmap for IT8622

IT8622 supports two external SMBus channels. Add bitmask to disable them
while accessing the chip.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix mmio resource request

We have to request a memory resource with IORESOURCE_MEM.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

IT8655E supports MMIO

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Print MMIO address into kernel log

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Update README to describe new module parameters

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add MMIO support

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Disable SMBus access while accessing Enviromnental Controller registers

Some chips support Environmental Controller access through SMBus.
On those chips, it is possible that an Embedded Controller accesses
Environmental Controller chip registers at any time. There is no real
means for synchronization. On banked chips, this can and will result in
access errors with unpredictable result.

Disable SMBus access while reading or writing environmental controller
registers to work around the problem.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix FAN_TAC5 detection for IT8665E

FAN_TAC5 is connected if 26h[4]=0.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Remove version.h when running clean, and clean does not depend on version.h

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Ignore version.h

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

it87: Display driver version

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Makefile: Generate version.h

Update version after each change in it87.c.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Simplify and fix temperature sensor type detection

Temperature sensor types won't change, so we only need to read it once.
Also fix temperature register calculation.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Always restore previous bank register after reading/writing data

There is some indication that other entities (BIOS ? EC ?) may access
the chip asynchronously. Restore the bank selection after read/write
operations to limit any potenial impact.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Improve temperature sensor type detection for IT8686

Introduce two new type values, 7 for 'virtual' and 8 for 'other'.
This will need to be added to the ABI once approved.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix reworked temperature mapping

With the reworked temperature mapping it was no longer possible to set
manual mode since the 'automatic' configuration bit was never cleared.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Update README

Add a note on sensors-detect. Remove some obsolete information.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Remove TODO

It is obsolete.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Try to add support for automatically detected build directories

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add 'ignore_resource_conflict' error

It appears that some BIOSes reserve ACPI resources without using them,
and acpi_enable_resources=lax seems to result in failures to load
certain drivers.

Introduce a 'ignore_resource_conflict' module parameter as alternate
means to ignore ACPI resource conflicts.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Makefile: Add support for compressed modules

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Rework pwm temperature mapping

Various chips have different numbers of valid values for pwm temperature
mapping. Rework the code to take this into account.

No longer keep map as bitmap, but maintain it as number 1..X. Do not try to
define a separate mapping entry for pwm4..6 on chips where the temperature
map for those pwm controls is taken from a different temperature input.
Introduce helpers to convert the control register contents to a map and
vice versa.

Reflect that IT8607 uses the new temperature map in bit 3..5 of the pwm
control register.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Update pwm cached control registers before writing pwm enable

If pwmX_enable is written prior to displaying any attributes,
t87_update_pwm_ctrl() is never called, and the cached pwm registers
are never read before written. This results in bad values written
into various registers, and can result in operational failures.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Create the hwmon folder if it does not exist

IT8655E and IT8665E both support internal register scaling

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Force IT8790E and IT8792E into configuration mode

IT8790E and IT8792E need to be in configuration mode to avoid LPC bus errors.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Drop ACPI mutex code

The observed problems with IT8792E were after all not caused by ACPI
mutex problems, so drop that code for now.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix accesses to IT8792E

If IT8792E or a compatible chip such as IT8795 (which has the same chip ID,
0x8733) resides at SIO address 0x4e/0x4f, and another Super-IO chip is at
SIO address 0x2e/0x2f, the IT8792E causes a cycle decode problem when exiting
its configuration mode. This can result in access errors for both chips, and
may result in system hangs.

Never exit configuration mode for IT8792E to avoid the problem.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

README missing necessary build instructions

README missing necessary build instructions

IT8607 does not support the 1st fan, only fan2-3

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Drop FEAT_TEMP_OFFSET

Since we now count the number of temperature offset registers directly,
there is no need for the temperature offset feature flag anymore.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add support for 6 temperature limit registers on IT8628E

IT8628E has 6 temperature limit registers, overlayed with VIN7..VIN9
limit registers, but only 3 temperature offset registers. Given that,
introduce separate variables for the number of temperature limit
registers and the number of temperature offset registers.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Declare dummy vid functions if needed

Some kernel configurations may have CONFIG_HWMON_VID disabled.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Provide blacklist module parameter to override blacklist.

Load module with blacklist=0 to override blacklist.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Mark 2nd Super-IO chip on AB350M-D3H as broken

Configuration register access of the second Super-IO chip on AB350M-D3H
is just as broken as the others and can result in the known system hang-up.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Merge pull request #27 from jmaggard10/master

Fix up support for IT8625E

Fix up support for IT8625E

Add missing scaling feature and fix broken temp type detection on temp
1-3 sensors.

Add support for IT8625E

This is mostly identical to IT8665E, with the exception of the
configuration bits for fans 4 and 5.

Signed-off-by: Justin Maggard <jmaggard@netgear.com>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix whitespace issues

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Initialize register pointers before using them

Register pointers have to be initialized before they can be used in
it87_check_pwm().

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add Gigabyte AX370-Gaming 5 to IT8792E blacklist

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Introduce FEAT_NEW_TEMPMAP

Several recent chips have a different register definition for
temperature to pwm assignment. Introduce FEAT_NEW_TEMPMAP to reflect
this and assign to affected chips.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix up support for IT8613E

Feature detection and pwm temperature mapping differs from other chips.
Also disable pwm1, fan1, vin3, and vin6.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add support for IT8613E

[groeck: Fix README]

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Blacklist IT8792E on AX370-Gaming K7

Accessing it may result in a system hang.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Disable access to 2nd Super-IO chip

It is known to be broken.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

FEAT_TEMP_PECI does not apply to multi-bank chips.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add support for 11mV ADC

The ADC LSB on some newer chips such as IT8613E and IT8625E is
officially 11mV, not 10.9mV. Add support for it.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix up temp offset register addresses

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Various improvements

- Acquire acpi mutex (not supported in upstream kernel)
- Try to handle 0x4e access failures on Gigabyte boards
- Improve temperature register / limit / type support for recent chips

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add limit support for temp 4-6

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Register map as CSV

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add register map

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add feature flag FEAT_FANCTL_ONOFF

Several recent chips don't support configuration bits to turn fan control
off entirely for the first three fans. Handle all of them with a
configuration flag.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Improve AVCC3 support

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Preliminary support for IT8655E

Assume it is similar to IT8665E with three instead of six fans.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Update TODO and README files

Add sections about preliminary support and what to report for unsupported
chips to README.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add support for chip specific register sets

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Added preliminary support for IT8665E

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Separate internal scaling from 12mV ADC

Some chips have internal voltage scaling but 10.9mV ADC,
so we can no longer associate the 12mV ADC with internal scaling
but need a different flag to distinguish scaling support.

Also fix ADC resolution for IT8790E and IT8792E.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add multi-bank and initial IT8686E support

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Force chip ID only if a chip has been found

If the reported chip ID is 0xffff, there is no chip, and forcing the chip
type does not add value.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Update TODO: 4th fan control for IT8732E is now supported

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Add 4th fan control and measurement for IT8732E

Signed-off-by: Guenter Roeck <linux@roeck-us.net>

Fix feature mask for IT8792E (IT8733E)

The chip features match those of IT8790E. Most important is the
ADC resolution, which is 12 mV, not 10.9 mV.

Signed-off-by: Guenter Roeck <linux@roeck-us.net>