Changeset 2398

Timestamp:

03/28/04 15:01:34 (9 years ago)

Author:

khali

Message:

Change pcf8591 magnitude (libsensors only) so as to display

values in pseudo-volts, much like what the 2.6 driver handles.

Location:

lm-sensors/trunk

Files:

• lib/chips.c (modified) (1 diff)
• prog/sensors/chips.c (modified) (5 diffs)

lm-sensors/trunk/lib/chips.c

@@ -4084 +4084 @@
                       RW, PCF8591_SYSCTL_AIN_CONF, VALUE(1), 0 },
     { SENSORS_PCF8591_CH0, "ch0", NOMAP, NOMAP,
-                      R, PCF8591_SYSCTL_CH0, VALUE(1), 0, "in0_input", 3 },
+                      R, PCF8591_SYSCTL_CH0, VALUE(1), 2, "in0_input", 3 },
     { SENSORS_PCF8591_CH1, "ch1", NOMAP, NOMAP,
-                      R, PCF8591_SYSCTL_CH1, VALUE(1), 0, "in1_input", 3 },
+                      R, PCF8591_SYSCTL_CH1, VALUE(1), 2, "in1_input", 3 },
     { SENSORS_PCF8591_CH2, "ch2", NOMAP, NOMAP,
-                      R, PCF8591_SYSCTL_CH2, VALUE(1), 0, "in2_input", 3 },
+                      R, PCF8591_SYSCTL_CH2, VALUE(1), 2, "in2_input", 3 },
     { SENSORS_PCF8591_CH3, "ch3", NOMAP, NOMAP,
-                      R, PCF8591_SYSCTL_CH3, VALUE(1), 0, "in3_input", 3 },
+                      R, PCF8591_SYSCTL_CH3, VALUE(1), 2, "in3_input", 3 },
     { SENSORS_PCF8591_AOUT_ENABLE, "aout_enable", NOMAP, NOMAP,
                       RW, PCF8591_SYSCTL_AOUT_ENABLE, VALUE(1), 0, "out0_enable", 0 },
     { SENSORS_PCF8591_AOUT, "aout", NOMAP, NOMAP,
-                      RW, PCF8591_SYSCTL_AOUT, VALUE(1), 0, "out0_output", 3 },
+                      RW, PCF8591_SYSCTL_AOUT, VALUE(1), 2, "out0_output", 3 },
     { 0 }
   };

lm-sensors/trunk/prog/sensors/chips.c

@@ -3667 +3667 @@
 {
   char *label;
-  double ain_conf, ch0, ch1, ch2, ch3;
+  int ain_conf = -1;
+  double ain;
   double aout_enable, aout;
   int valid;
 
   if (!sensors_get_label_and_valid(*name,SENSORS_PCF8591_AIN_CONF,&label,&valid) &&
-      !sensors_get_feature(*name,SENSORS_PCF8591_AIN_CONF,&ain_conf)) {
+      !sensors_get_feature(*name, SENSORS_PCF8591_AIN_CONF, &ain)) {
+        ain_conf = (int)ain;
         if (valid) {
           print_label(label,10);
-          switch ((int)ain_conf)
+          switch (ain_conf)
           {
             case 0: printf("four single ended inputs\n");
@@ -3688 +3690 @@
         }
       }
-  else printf("ERROR: Can't read analog inputs configuration!\n");
+  /* display no error, 2.6 driver doesn't have that file */
   free_the_label(&label);
 
   if (!sensors_get_label_and_valid(*name,SENSORS_PCF8591_CH0,&label,&valid) &&
-      !sensors_get_feature(*name,SENSORS_PCF8591_CH0,&ch0)) {
+      !sensors_get_feature(*name, SENSORS_PCF8591_CH0, &ain)) {
         if (valid) {
           print_label(label,10);
-          printf("%0.0f\n", ch0);
+          printf("%6.2fV\n", ain);
         }
       }
@@ -3702 +3704 @@
 
   if (!sensors_get_label_and_valid(*name,SENSORS_PCF8591_CH1,&label,&valid) &&
-      !sensors_get_feature(*name,SENSORS_PCF8591_CH1,&ch1)) {
+      !sensors_get_feature(*name, SENSORS_PCF8591_CH1, &ain)) {
         if (valid) {
           print_label(label,10);
-          printf("%0.0f\n", ch1);
+          printf("%6.2fV\n", ain);
         }
       }
@@ -3713 +3715 @@
   if (ain_conf != 3) {
     if (!sensors_get_label_and_valid(*name,SENSORS_PCF8591_CH2,&label,&valid) &&
-        !sensors_get_feature(*name,SENSORS_PCF8591_CH2,&ch2)) {
+        !sensors_get_feature(*name, SENSORS_PCF8591_CH2, &ain)) {
           if (valid) {
             print_label(label,10);
-            printf("%0.0f\n", ch2);
+            printf("%6.2fV\n", ain);
           }
         }
-    else printf("ERROR: Can't read ch2!\n");
+    else if (ain_conf >= 0) /* hide error for 2.6 kernel driver */
+      printf("ERROR: Can't read ch2!\n");
     free_the_label(&label);
   }
 
-  if (ain_conf == 0) {
+  if (ain_conf <= 0) {
     if (!sensors_get_label_and_valid(*name,SENSORS_PCF8591_CH3,&label,&valid) &&
-        !sensors_get_feature(*name,SENSORS_PCF8591_CH3,&ch3)) {
+        !sensors_get_feature(*name, SENSORS_PCF8591_CH3, &ain)) {
           if (valid) {
             print_label(label,10);
-            printf("%0.0f\n", ch3);
+            printf("%6.2fV\n", ain);
           }
         }
-    else printf("ERROR: Can't read ch3!\n");
+    else if (ain_conf >= 0) /* hide error for 2.6 kernel driver */
+      printf("ERROR: Can't read ch3!\n");
     free_the_label(&label);
   }
@@ -3740 +3744 @@
         if (valid) {
           print_label(label,10);
-          printf("%0.0f (%s)\n", aout, aout_enable?"enabled":"disabled");
+          printf("%6.2fV (%s)\n", aout, aout_enable?"enabled":"disabled");
         }
       }