linux-kernel - Re: [PATCH v5 00/11] hwmon: (w83627ehf) Add support for NCT6775F and NCT6776F

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <AANLkTinNrOG69Kzy=ZfRjurOedsz2Oazt-FT-m3Ro6Gg@mail.gmail.com>
Date:	Fri, 4 Mar 2011 13:50:41 -0500
From:	Andrew Lutomirski <luto@....edu>
To:	Guenter Roeck <guenter.roeck@...csson.com>
Cc:	Jean Delvare <khali@...ux-fr.org>,
	Ian Dobson <i.dobson@...net-ian.com>,
	Randy Dunlap <rdunlap@...otime.net>, binximeng@...il.com,
	lmsensors@...anitarvainen.fi, andrea.rizzolo@...il.com,
	jeff.sadowski@...il.com, lm-sensors@...sensors.org,
	linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5 00/11] hwmon: (w83627ehf) Add support for NCT6775F and NCT6776F

On Fri, Mar 4, 2011 at 12:57 PM, Guenter Roeck
<guenter.roeck@...csson.com> wrote:
> The following patch series adds support for NCT6775F and NCT6776F to the
> w83627ehf driver. It also includes some cleanup and adds support for
> the fourth temperature sensor on W83677HG-B.
>
> No code changes since v4. Resubmitting and widening audience in the hope that
> someone may find the time to review the changes prior to the next commit window.

I'm not sure my review is worth much, but the driver seems to work.  I've tested
fan speed measurement, changing pwm2_enable to 1, changing the fan speed,
and changing back to 5.  Everything looks good.

The only weird thing I noticed is that, the first time I ran sensors, I got:
fan2:                    0 RPM  (min =    0 RPM, div = 64)
and thereafter I get
fan2:                  136 RPM  (min =    0 RPM, div = 64)  ALARM

I suspect (although I haven't checked) that sensors is just reading
div after it's already changed to 64.

Thanks,
Andy

>
> v5:
> - Update Kconfig to reference W83667HG, NCT6775F, and NCT6776F.
>
> v4:
> - Store rpm instead of raw fan speed readings
> - For NCT6775F, increase fan divisor if the fan speed reads 0
>  [ On NCT6775F, fan speed readings can return 0 instead of 0xff if the
>    fan divisor value is too low ]
> - Separate changes into more patches to simplify review
>
> v3:
> - Documentation: Remove references to datasheets which no longer exist
> - Documentation: Add information about limits of SmartFan IV support.
> - SmartFan III mode is not supported on NCT6776F, so remove related attributes
>  and refuse to configure it.
> - (additional patch) Improve support for chips with 16-bit fan count registers
>
> v2:
> - W83677HG-B does share the AUXTIN/VIN3 pin, so we can not skip this check.
> - Cosmetic changes to fix a couple of checkpatch errors and to undo some
>  unnecessary formatting changes in patch 5.
> - Patch 1 introduced a bug in temp_to_reg which was fixed in a later patch.
>  Modified code to not introduce the bug in the first place.
> - Two dev_dbg messages were changed to dev_info for testing. Changed back
>  to dev_dbg.
>
> The patched driver can be downloaded as stand-alone driver from
>        http://www.roeck-us.net/linux/drivers/w83627ehf/
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/