lkml.org 
[lkml]   [2013]   [Feb]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCH v6 00/12] hwmon: Driver for Nuvoton NCT6775F, NCT6776F, and NCT6779D
Date
This driver will replace the w83627ehf driver for NCT6775F and NCT6776F,
and provides support for NCT6779D.

This is a resend for most of the series. Patch 04/12 was updated, and patch
12/12 was added.

The driver has been available as standalone driver (https://github.com/groeck/nct6775)
since March 2012. It has been available as Archlinux package since July 2012
(https://aur.archlinux.org/packages.php?ID=61127).
At least one inofficial Ubuntu/Debian build is available at Launchpad
(http://ppa.launchpad.net/foresto/extradrivers/ubuntu/dists/quantal/main/source/Sources).
A bug has been submitted in Debian against not having this driver upstream
(http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=699140).

So there is some interest to get the driver upstream, and it does have at least
some level of test coverage. The only missing detail is someone with the time
to review the code. Expanding the audience in the hope of finding one.

v6:
- Spelling fix
- Fix fan register initialization for chips with no fan divisor (NCT6776,
NCT6779)
- Don't report VID if disabled (NCT6775, NCT6776) or not supported (NCT6779)

v5:
- Review fedback
- Documentation fixes
- Split patch into several parts to simplify review.
- Updated attribute names
- Improved support for thermal and speed cruise modes
- Dropped pwmX_interpolate (or pwmX_step_enable) since I could not get it
to work on any of the chips.
- Added power management support
- Added support for fanX_pulses attribute
- Improved fan divider selection
- Removed unused defines
- 'valid' is now bool
- Removed __devinit/__devexit

v4:
- Fix "temp" array size (reported by smatch)
- Fix whitespace errors reported by "git am" (but not by checkpatch).
- Order register definition data by chip type
- Improve parameter alignment

v3:
- Variables pwm_temp, REG_TEMP_MON, and NCT6775_REG_TEMP_MON are not used,
drop it.

v2:
- superio region access protection
- Add support for pwmX_step_enable attribute
- Improved documentation


\
 
 \ /
  Last update: 2013-02-26 20:01    [W:0.049 / U:1.576 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site