Update README

The driver supports additional chips, and up to 6 fans and fan controls.

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

Improve fan6/pwm6 support

Improve fan6/pwm6 detection. Add support for fan pulses for fans 4..6
and fan min limits for fan6.

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

Add debug file

Provide debug file to report temperature register settings and values.

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

Used bits in temp_mask start with 1

Used bits in temperature source registers, and thus temp_mask, start with 1.
Valid sources in reg_temp_alternate[] start with 0. Add missing offset.

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

Use bitops

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

Sync with upstream

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

Improve NCT6795 support

Improve fan and pwm pin detection for both NCT6793D and NCT6795D.
Add temperature sensor labels for NCT6795D.

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

Rework temperature sensor detection

We don't know temperature sensor assignment on NCT6795D.
Separate sensor presence detection from sensor label display,
and disable sensor label display for NCT6795D.

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

Add basic support for NCT6795D

Temperature sensor labels are known to be wrong.

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

nct6775: Add support for multiple virtual temperature sources

For virtual temperatures, the actual temperature values are written
by software, presumably by the BIOS. This functionality is (as of
right now) supported on NCT6791D, NCT6792D, and NCT6793D. On those chips,
the temperatures are written into registers 0xea..0xef on page 0.
This is known to be used on some Asus motherboards. Report the 'virtual'
temperatures for all monotoring sources to address this situation.

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

hwmon: (nct6775) constify sensor_template_group structures

The sensor_template_group structures are never modified, so declare them as
const.

Done with the help of Coccinelle.

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>

hwmon: (nct6775) Do not accept force_id unless chip is found

Since commit 698a7c24a544 ("hwmon: (nct6775) Support two SuperIO chips
in the same system"), the driver supports two Super-IO chips. This has
the undesirable side effect that force_id always detects a second chip
at address 0xfff8, even if no chip exists at that address.

nct6775: Found NCT6793D or compatible chip at 0x4e:0xfff8

If no chip at all is found at a given SIO address, it does not make sense
to instantiate it. Limit force_id to only work if some chip is found,
that is if the chip ID returns a value other than 0xffff.

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

compat.h: Define sysfs_attr_init if not defined

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

Merge pull request #36 from airbjorn/master

nct6775: fix compile errors on RHEL7

fix compile errors on RHEL7

hwmon: (nct6775) Introduce separate temperature labels for NCT6792 and NCT6793

NCT6792 and NCT6793 are mostly register compatible to NCT6791, but
temperature sources are different and difficult to manage with a single
temperature label array. Introduce separate temperature label arrays
for those chips to reflect the differences.

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

Add missing sysfs_attr_init()

Also synchronize STEP_UP_TIME and STREP_DOWN_TIME handling
with upstream code.

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

Fix typo in NCT6792D version ID

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

Move nct6775_sio_names closer to enum to simplify adding new chips

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

Add missing NCT6793D string

Thanks to Eric Swanson for the report.

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

Swap SETP_UP_TIME and STEP_DOWN_TIME for all chips but NCT6775

The register definition was changed (swapped) after NCT6776.

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

Fix NCT6793D device ID

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

Add support for NCT6793D

NCT6793D is register compatible with NCT6792D.

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

NCT6791D and NCT6792D have an additional temperature source

Both NCT6791D and NCT6792D permit selection of a 'virtual' temperature
register as temperature source. The virtual temperature registers are
registers 0xea to 0xef in bank 0 and can be written by software.

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

Merge pull request #34 from graysky2/master

Merge pull request #1 from groeck/master

Merge pull request #32 from pjperry/master

nct6775: fix compile errors on RHEL6.7

nct6775: fix compile errors on RHEL6.7

Signed-off-by: Philip J Perry <phil@elrepo.org>

Enable auxiliary fan monitoring on ASRock Z77 Pro4-M

Auxiliary fan monitoring is not enabled on ASRock Z77 Pro4-M
with BIOS version 2.00 if booted in UEFI Ultra-FastBoot mode.

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

Restore hardware monitoring logical device status on resume

After a suspend/resume cycle it is not guaranteed that the hardware monitoring
device is still enabled. Ensure that this is the case after resume.

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

Convert to use SIMPLE_DEV_PM_OPS

Get rid of #ifdef CONFIG_PM by using SIMPLE_DEV_PM_OPS and declaring suspend
and resume functions with __maybe_unused.

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

Revert "Restore hardware monitoring logical device status on resume"

This reverts commit 8ec9699a1bc4e90f4642efcb68127800ced79591.

Revert "Enable fan 3-5 on ASRock Z77 Pro4-M"

This reverts commit ad00077d680fba0530befe9e9ebb94ea090ce9ed.

Enable fan 3-5 on ASRock Z77 Pro4-M

ASRock Z77 Pro4-M BIOS version 2.00 is known to not enable fan 3 to 5
if booted in UEFI Ultra-FastBoot mode.

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

Restore hardware monitoring logical device status on resume

After a suspend/resume cycle it is not guaranteed that the hardware monitoring
device is still enabled. Ensure that this is the case after resume.

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

Update README to reflect all supported chips and minimum kernel version

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

Update minimum kernel version to 2.6.32

The driver needs is_visible support from the kernel core.
Working qround this for older kernels would be too difficult,
so set the minimum kernel version to 2.6.32, which is also
the oldest still supported longterm kernel.

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

Merge pull request #1 from groeck/master

sync with upstream

compat.h: Make kstrtol definition conditional

strict_strtol and strict_strtoul are no longer defined in the latest kernel,
so don't use those functions unless really needed.

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

Synchronize code with upstream version

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

Drop obsolete pwmX_step_enable from README file

THis attribute was removed a long time ago,

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

Add new beep registers for nct6792d

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

Initial support for NCT6792D - untested

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

Merge pull request #18 from karlvoigtland/master

compat.h:  define pr_warn and pr_warn_ratelimited macros if not already ...

compat.h:  define pr_warn and pr_warn_ratelimited macros if not already defined

pr_warn and pr_warn_ratelimited not added until kernel 2.6.35

make define release_region conditional

release_region is defined on RHEL6 although request_muxed_region isn't so generates a warning:

warning: "release_region" redefined

Making it conditional eliminates the warning.

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

compat.h: Don't inlude i2C compatibility code

This is not an I2C driver, so don't include any I2C compatibility code.

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

README: Update to reflect that the driver is available upstream

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

Declare vid_from_reg and vid_which_vrm locally if HWMON_VID is not enabled

This prevents compilation errors if HWMON_VID is not enabled in the system
configuration.

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

Add declaration of PTR_ERR_OR_ZERO for kernels prior to 3.12

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

Add known problem about fan control settings after resume to README file

Fan control settings are not restored in a resume cycle.

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

sync with latest upstream version

Use devm_hwmon_device_register_with_groups() and sysfs_create_groups() if
possible.

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

Add support for hibernate

Hibernate uses its own set of callbacks, even if they are the same as
suspen/resume. Add support for it. Also synchronize NCT6791 suspend/resume
fix with upstream code.

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

Re-enable logical device mapping for NCT6791 after restore

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

nct6775: NCT6791 supports weight control only for CPUFAN

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

nct6775: Monitor additional temperature registers

The number of SMIOVT registers on NCT6779 and NCT6791 is limited to 2.
As result, the driver may not report some of the temperatures used for fan
control. Fortunately, the chip has registers to monitor those temperatures.
Add them to the list of temperatures to report.

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

README: Add comment about ASUS desktop boards

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

Update compat.h to improve support of older kernels.

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

temp[] array needs to have an entry for temp_lcrit

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

hwmon: (nct6775) Avoid using device platform data outside probe function

Plan going forward is to attach all device attributes to the hwmon device and no
longer to the platform device. With that change, accessing platform data outside
the probe function will be more difficult. To avoid the problem, change code
to no longer rely on it.

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

Add support for NCT6791D.

Signed-off-by: David Bartley <andareed@gmail.com>

Add beep support

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

Fix temperature alarms

Which temperatures generate alarms is not fixed but determined by
the temperature source mapping.

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

Do not assign non-existing over-temperature and hysteresis registers

Overtemperature and hysteresis registers only exist for primary
temperature registers, not for alternates.

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

Replace SENSORS_LIMIT in lm75.h with clamp_val

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

Synchronize with upstream version

Coding style impreovements
Improve sensor template definitions

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

Add LICENSE file

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

Fix reported use of unassigned variable

In some conditions, the compiler reports the use of address as returned
from nct6775_find to be unassigned. Fix by returning it as return value
together with the error.

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

Update README

Signed-off-by: Guenter <groeck@desktop.roeck-us.net>

Foermatting changes: Fix checkpatch problems

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

Add support for critical low/high temperature limits on NCT6106

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

Add support for NCT6102D/6106D

Basic testing.

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

Use ARRAY_SIZE for loops where possible

This ensures that the loop iterations are correct even if/when
the number of elements in an array changes.

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

Enable both VIN3 and AUXTIN

Both are reported at the same time on SuperMicro C7H61.

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

Support up to two SuperIO chips in the same system

Support both SuperIO chips on SuperMicro C7H61-O.

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

nct6775: Allocate and create attributes from templates

This patch significantly reduces the amount of boiler-plate code.
Module size is reduced by more than 20k.

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

clarify supported chips

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

Drop read/write lock

The read/write lock is acquired for each read/write operation from/to the chip.
This occurs either during initialization, when it is not needed, or during
updates, when the update_lock is held as well, and it is not needed either.

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

Improve temperature register array declarations

Specify explicit sizes for temperature register arrays.

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

nct6775: Use clamp_val instead of SENSORS_LIMIT

SENSORS_LIMIT has been deprecated and was removed from the upstream kernel.

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

nct6775: Don't report VID if not supported

VID support is configured with CR2a on NCT6775 and NCT6776, so we can detect
if it is enabled. NCT6779 does not support VID at all.

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

README: Add note about testing and review feedback

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

Add README.ASRock

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

Minor bug fixes; include compat.h

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

Spelling fix in README

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

Latest code; sync with proposed kernel version

Many attribute changes.

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

Add backward compatibility to nct6775.c

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

Add backward compatibility support

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

Cleanup

Sort register definitions by chip type. Improve parameter alignment.

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

hwmon: (nct6775) Fix temp array overflow

With the addition of tempX_crit, there are four sets of temperature values
in the temp[][] array, but it was declared with a size of three. Fix it.

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

Update README

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

REG_PWM_TEMP is not used, drop it.

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

cosmetic: Formatting change

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

Add missing break statement

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

Improve error return from clear_caseopen

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

implement pwmX_step_enable

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

Implement superio access locking

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

Replace __initdata with __initconst

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

Clean up probe function: Use devm_request_region

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

Fix various registers for NCT6779

Also always display all VINx and AUXTINx channels, since we can not be sure
what is really enabled/used.

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

Remove temp11..temp15, and modify pwm temp source selection

It does not provide real value as temperatures are already reported.
Select pwm* temperature source index values based on tempX attributes
instead of chip values to make the pwm temperature source selection
more generic.

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