* [lm-sensors] [patch 0/6 v2] hwmon/pc87360 individual alarm files
@ 2008-06-29 17:52 Jim Cromie
0 siblings, 0 replies; 2+ messages in thread
From: Jim Cromie @ 2008-06-29 17:52 UTC (permalink / raw)
To: lm-sensors
in this 2nd rev of the patchset (against -rc8), the new voltage and temp
alarms
work as they should; setting min & max above and below input results in
alarm activation.
- My board has no thermistors, so I cannot test them.
- The temp units have temp[1-3]-fault files, 2 of them are active on my
soekris.
- The therm(istor) units temp[4-6] do not have one, since they use
voltage units 11-14,
which doesnt support OPEN bit (but does have an overtemp bit on
v-units 11-14)
- temp_alarms and in_alarms do *not* work.
They never did, IIRC, in any case theyre not touched in this patchset.
Are they properly subject to deprecation and eventual removal ?
Or do they work on lesser pc8736x chips ?
(theres some vague hint of diffs where x=[56] in sec 11.2.3)
- All patches pass scripts/checkpatch.pl
Jim Cromie (6):
hwmon/pc87360 separate alarm files - define some constants
hwmon/pc87360 separate alarm files - add in-min/max-alarms
hwmon/pc87360 separate alarm files - add
temp-min/max/crit/fault-alarms
hwmon/pc87360 separate alarm files - define LDNI_MAX const
hwmon/pc87360 separate alarm files - add dev_dbg help
hwmon/pc87360 separate alarm files - add therm-min/max/crit-alarms
drivers/hwmon/pc87360.c | 244
+++++++++++++++++++++++++++++++++++++++++++----
1 files changed, 223 insertions(+), 21 deletions(-)
Having worked thru the patches again, ISTM that 1st patch might have an
aesthetic issue - I put defines as close as possible to where theyll be
used,
not at the top. If you want this changed, please indicate if an additional
fixup patch to move them up is acceptable.
thanks
Jim Cromie
PS. The following is a copy-paste of alarms being toggled by setting
limits..
# sensors -v
sensors version 2.10.1 with libsensors version 2.10.1
# sensors
pc87366-isa-6620
Adapter: ISA a[ 3659.886392] pc87360 pc87360.26144: Data update
dapter
VCORE: +2.01 V (min = +0.00 V, max = +3.01 V)
VCC: +4.96 V (min = +0.00 V, max = +6.03 V)
VPWR: +13.55 V (min = +0.00 V, max = +60.56 V)
+12V: +11.87 V (min = +0.00 V, max = +14.55 V)
-12V: -11.90 V (min = -59.77 V, max = -1.92 V)
GND: +0.00 V (min = +0.00 V, max = +3.01 V)
Vsb: +3.31 V (min = +0.00 V, max = +6.03 V)
Vdd: +2.98 V (min = +0.00 V, max = +6.03 V)
Vbat: +0.00 V (min = +0.00 V, max = +0.00 V)
AVdd: +3.28 V (min = +0.00 V, max = +6.03 V)
temp1: +127 C (low = -55 C, high = +127 C) OPEN
temp1_crit:
+127 C ALARM
temp2: +127 C (low = -55 C, high = +127 C) OPEN
temp2_crit:
+127 C ALARM
Temp: +98 C (low = -55 C, high = +127 C)
Critical: +127 C
soekris:/sys/devices/platform/pc87360.26144# cat temp[123]*fault
[ 4044.030817] pc87360 pc87360.26144: Data update
1
1
0
the 2 faults agree with the 2 OPENs in sensors output
(though I suspect they are determined by other means)
soekris:/sys/devices/platform/pc87360.26144# cat temp[123]*max_alarm
[ 4302.257654] pc87360 pc87360.26144: Data update
1
1
0
soekris:/sys/devices/platform/pc87360.26144# ls temp3*
temp3_crit temp3_fault temp3_max temp3_min temp3_status
temp3_crit_alarm temp3_input temp3_max_alarm temp3_min_alarm
soekris:/sys/devices/platform/pc87360.26144# cat temp3*
[ 4321.958856] pc87360 pc87360.26144: Data update
127000
0
0
98000
127000
0
-55000
0
129
soekris:/sys/devices/platform/pc87360.26144# echo 97000 > temp3_max
soekris:/sys/devices/platform/pc87360.26144# cat temp3*
[ 4434.226746] pc87360 pc87360.26144: Data update
127000
0
0
98000
97000
1
-55000
0
133
soekris:/sys/devices/platform/pc87360.26144# echo 97000 > temp3_crit
soekris:/sys/devices/platform/pc87360.26144# cat temp3*
[ 4461.619010] pc87360 pc87360.26144: Data update
97000
1
0
98000
97000
1
-55000
0
141
soekris:/sys/devices/platform/pc87360.26144# echo 101000 > temp3_max
soekris:/sys/devices/platform/pc87360.26144# cat temp3*
[ 4489.270994] pc87360 pc87360.26144: Data update
97000
1
0
98000
101000
1
-55000
0
141
soekris:/sys/devices/platform/pc87360.26144#
soekris:/sys/devices/platform/pc87360.26144# cat temp3*
[ 4495.027270] pc87360 pc87360.26144: Data update
97000
1
0
98000
101000
0
-55000
0
137
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [lm-sensors] [patch 0/6 v2] hwmon/pc87360 individual alarm files
@ 2008-07-25 21:28 Jim Cromie
0 siblings, 0 replies; 2+ messages in thread
From: Jim Cromie @ 2008-07-25 21:28 UTC (permalink / raw)
To: lm-sensors
Hi Jean, Mark,
Im reminding, since its been a few weeks, and nobody has commented.
IIRC, patches like this have been considered bug-fixes in the past,
and have been integrated to linux-current.
I'll leave it to you to consider whether this patchset qualifies.
I could also fold them all into one if thats preferred.
thanks
JimC
On 6/29/08, Jim Cromie <jim.cromie@gmail.com> wrote:
>
> in this 2nd rev of the patchset (against -rc8), the new voltage and temp
> alarms
> work as they should; setting min & max above and below input results in
> alarm activation.
> - My board has no thermistors, so I cannot test them.
> - The temp units have temp[1-3]-fault files, 2 of them are active on my
> soekris.
> - The therm(istor) units temp[4-6] do not have one, since they use voltage
> units 11-14,
> which doesnt support OPEN bit (but does have an overtemp bit on v-units
> 11-14)
> - temp_alarms and in_alarms do *not* work. They never did, IIRC, in any
> case theyre not touched in this patchset.
> Are they properly subject to deprecation and eventual removal ?
> Or do they work on lesser pc8736x chips ?
> (theres some vague hint of diffs where x=[56] in sec 11.2.3)
> - All patches pass scripts/checkpatch.pl
>
> Jim Cromie (6):
> hwmon/pc87360 separate alarm files - define some constants
> hwmon/pc87360 separate alarm files - add in-min/max-alarms
> hwmon/pc87360 separate alarm files - add
> temp-min/max/crit/fault-alarms
> hwmon/pc87360 separate alarm files - define LDNI_MAX const
> hwmon/pc87360 separate alarm files - add dev_dbg help
> hwmon/pc87360 separate alarm files - add therm-min/max/crit-alarms
>
> drivers/hwmon/pc87360.c | 244
> +++++++++++++++++++++++++++++++++++++++++++----
> 1 files changed, 223 insertions(+), 21 deletions(-)
>
>
> Having worked thru the patches again, ISTM that 1st patch might have an
> aesthetic issue - I put defines as close as possible to where theyll be
> used,
> not at the top. If you want this changed, please indicate if an additional
> fixup patch to move them up is acceptable.
>
>
> thanks
> Jim Cromie
>
>
> PS. The following is a copy-paste of alarms being toggled by setting
> limits..
>
> # sensors -v
> sensors version 2.10.1 with libsensors version 2.10.1
>
> # sensors
> pc87366-isa-6620
> Adapter: ISA a[ 3659.886392] pc87360 pc87360.26144: Data update
> dapter
> VCORE: +2.01 V (min = +0.00 V, max = +3.01 V) VCC: +4.96
> V (min = +0.00 V, max = +6.03 V) VPWR: +13.55 V (min = +0.00
> V, max = +60.56 V) +12V: +11.87 V (min = +0.00 V, max = +14.55 V)
> -12V: -11.90 V (min = -59.77 V, max = -1.92 V) GND:
> +0.00 V (min = +0.00 V, max = +3.01 V) Vsb: +3.31 V (min > +0.00 V, max = +6.03 V) Vdd: +2.98 V (min = +0.00 V, max > +6.03 V) Vbat: +0.00 V (min = +0.00 V, max = +0.00 V)
> AVdd: +3.28 V (min = +0.00 V, max = +6.03 V) temp1: +127 C
> (low = -55 C, high = +127 C) OPEN
> temp1_crit:
> +127 C ALARM
> temp2: +127 C (low = -55 C, high = +127 C) OPEN
> temp2_crit:
> +127 C ALARM
> Temp: +98 C (low = -55 C, high = +127 C) Critical: +127 C
>
>
> soekris:/sys/devices/platform/pc87360.26144# cat
> temp[123]*fault
> [ 4044.030817] pc87360 pc87360.26144: Data update
> 1
> 1
> 0
>
> the 2 faults agree with the 2 OPENs in sensors output
> (though I suspect they are determined by other means)
>
>
> soekris:/sys/devices/platform/pc87360.26144# cat
> temp[123]*max_alarm
> [ 4302.257654] pc87360 pc87360.26144: Data update
> 1
> 1
> 0
> soekris:/sys/devices/platform/pc87360.26144# ls temp3*
>
> temp3_crit temp3_fault temp3_max temp3_min temp3_status
> temp3_crit_alarm temp3_input temp3_max_alarm temp3_min_alarm
>
> soekris:/sys/devices/platform/pc87360.26144# cat temp3*
> [ 4321.958856] pc87360 pc87360.26144: Data update
> 127000
> 0
> 0
> 98000
> 127000
> 0
> -55000
> 0
> 129
>
> soekris:/sys/devices/platform/pc87360.26144# echo 97000 >
> temp3_max soekris:/sys/devices/platform/pc87360.26144# cat
> temp3*
> [ 4434.226746] pc87360 pc87360.26144: Data update
> 127000
> 0
> 0
> 98000
> 97000
> 1
> -55000
> 0
> 133
>
> soekris:/sys/devices/platform/pc87360.26144# echo 97000 >
> temp3_crit
> soekris:/sys/devices/platform/pc87360.26144# cat temp3*
> [ 4461.619010] pc87360 pc87360.26144: Data update
> 97000
> 1
> 0
> 98000
> 97000
> 1
> -55000
> 0
> 141
>
> soekris:/sys/devices/platform/pc87360.26144# echo 101000 >
> temp3_max
> soekris:/sys/devices/platform/pc87360.26144# cat temp3*
> [ 4489.270994] pc87360 pc87360.26144: Data update
> 97000
> 1
> 0
> 98000
> 101000
> 1
> -55000
> 0
> 141
> soekris:/sys/devices/platform/pc87360.26144#
> soekris:/sys/devices/platform/pc87360.26144# cat temp3*
> [ 4495.027270] pc87360 pc87360.26144: Data update
> 97000
> 1
> 0
> 98000
> 101000
> 0
> -55000
> 0
> 137
>
>
>
>
>
>
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-07-25 21:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-29 17:52 [lm-sensors] [patch 0/6 v2] hwmon/pc87360 individual alarm files Jim Cromie
-- strict thread matches above, loose matches on Subject: below --
2008-07-25 21:28 Jim Cromie
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.