--Signature_Thu__6_Oct_2005_21_06_14_-0400_yRF8kjVPI+BYUZ_x
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

Hello,

>   | There's some support for Analog Devices ADT7467 and ADM1030 in the
>   | tree. ( 'some support' as in sensors can be read via envsys,
>   | thresholds can be set via sysctl but there's no way to change
>   | things like hysteresis, temperature/rpm slope or the more complex
>   | innards of the ADT 7467 )
>=20
> Arguably the envsys(4) API could be used instead of sysctl(3) for
> setting stuff like thresholds on a sensor, if the envsys API was
> slightly extended.

Agreed, this belongs into envsys. Sysctl was just the easiest way to do
it and so far nobody protested.

> I have a proposal for an ENVSYS_STREDATA which would support this.
> E.g, set the min & max members of envsys_tre_data and use the
> proposed ENVSYS_STREDATA to frob it to define the threshold. =20
> We use a similar technique abusing the rpms field of envsys_basic_info
> and ENVSYS_STREINFO to reset a sensor's alarm.

This should be generic enough to control the inner workings of more
complex fan controllers too ( the ADT 7467 for instance ), like which
fan responds to which sensors, slope, hysteresis and so on. Not that
most people would ever (need to) touch this but since the controller is
programmable in this way the driver should export it somehow.

have fun
Michael

--Signature_Thu__6_Oct_2005_21_06_14_-0400_yRF8kjVPI+BYUZ_x
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.0 (NetBSD)

iQEVAwUBQ0XKBspnzkX8Yg2nAQLj3Af/SjseFr1DfcQm8S0nu8k5teuGUp20FI+S
JZLF9+ZpEToMKxeDN9QJJ3KePdG5WXCo7kWE0a+rhUzfO6s2/DgzNmucIJXfFKnh
E6ovX9tRdwucqoWeSGIcXFmZlEBdqeZNK3VyhyZvbmsWBCYtN8wWIw1m2HmZElD2
/4lj7pMoiNBbr0JbPs9d/230VVt6DzdoCoHKIAPx3T7hbBYSd3UgVerl03/gV3SY
XILYYNLpW9mY7Y23zA9WXOLJBltnJvQf3R2VvNI2Zd7Fgf7z7283zn307zT8dUlW
KZOO569JFe/vkoLiVbUSqXFE8MYGMFy5ivW1prK8CFbS7cgmXW/4yA==
=BIbT
-----END PGP SIGNATURE-----

--Signature_Thu__6_Oct_2005_21_06_14_-0400_yRF8kjVPI+BYUZ_x--