Changeset 2428

Timestamp:

04/09/04 20:50:43 (5 years ago)

Author:

khali

Message:

Drop PWM support for as99127f. Update AS99127F docs.

Files:

• lm-sensors/trunk/doc/chips/SUMMARY (modified) (1 diff)
• lm-sensors/trunk/doc/chips/w83781d (modified) (7 diffs)
• lm-sensors/trunk/kernel/chips/w83781d.c (modified) (7 diffs)

lm-sensors/trunk/doc/chips/SUMMARY

@@ -183 +183 @@
-
-w83781d
-2 pwm?  
+-      
-        w83781d         3       7       3       -       yes     yes
-        w83782d         3       9       3       2-4 pwm yes     yes

lm-sensors/trunk/doc/chips/w83781d

@@ -189 +189 @@
------------------
-The as99127f support was developed without the benefit of a datasheet.
-
+
+
-This support will be BETA until a datasheet is released.
-One user has reported problems with fans stopping
@@ -200 +201 @@
-Known problems:
-        - Problems with diode/thermistor settings (supported?)
-        - fan_div register may not really be fan_div.
-        - One user reports fans stopping under high server load.
-
-
-
-
-
+
+
-
-These will not be fixed unless we get a datasheet.
@@ -525 +522 @@
-Additional entries not in the lm78:
-
-
+ nor as99127f
-          Controls the speed of the fans with PWM (Pulse Width Modulation)
-          Valid values:
@@ -531 +528 @@
-        Second value is PWM enable 0/1.
-
-
+ nor as99127f
-          Controls the sensor type. To change to a different
-          sensor type, for example, do 'echo 2 > sensor1'.
@@ -542 +539 @@
-          after correspondence with Winbond. See below.
-
-
+w83
-          32-byte tables controlling the conversion from
-          resistance to temperature for thermistors.
@@ -581 +578 @@
-Data sheet updates:
-------------------
-        - Mask off lower bit of all ID registers.
-
-        - PWM clock registers:
-
@@ -740 +735 @@
-in5=255
-in6=255
+
+
+# PWM
+
+Additional info about PWM on the AS99127F (may apply to other Asus
+chips as well) by Jean Delvare as of 2004-04-09:
+
+AS99127F revision 2 seems to have two PWM registers at 0x59 and 0x5A,
+and a temperature sensor type selector at 0x5B (which basically means
+that they swapped registers 0x59 and 0x5B when you compare with Winbond
+chips).
+Revision 1 of the chip also has the temperature sensor type selector at
+0x5B, but PWM registers have no effect.
+
+We don't know exactly how the temperature sensor type selection works.
+Looks like bits 1-0 are for temp1, bits 3-2 for temp2 and bits 5-4 for
+temp3, although it is possible that only the most significant bit matters
+each time. So far, values other than 0 always broke the readings.
+
+PWM registers seem to be split in two parts: bit 7 is a mode selector,
+while the other bits seem to define a value or threshold.
+
+When bit 7 is clear, bits 6-0 seem to hold a threshold value. If the value
+is below a given limit, the fan runs at low speed. If the value is above
+the limit, the fan runs at full speed. We have no clue as to what the limit
+represents. Note that there seem to be some inertia in this mode, speed
+changes may need some time to trigger. Also, an hysteresis mechanism is
+suspected since walking through all the values increasingly and then
+decreasingly led to slighlty different limits.
+
+When bit 7 is set, bits 3-0 seem to hold a threshold value, while bits 6-4
+would not be significant. If the value is below a given limit, the fan runs
+at full speed, while if it is above the limit it runs at low speed (so this
+is the contrary of the other mode, in a way). Here again, we don't know
+what the limit is supposed to represent.
+
+One remarkable thing is that the fans would only have two or three
+different speeds (transitional states left apart), not a whole range as
+you usually get with PWM.
+
+As a conclusion, you can write 0x00 or 0x8F to the PWM registers to make
+fans run at low speed, and 0x7F or 0x80 to make them run at full speed.
+
+Please contact us if you can figure out how it is supposed to work. As
+long as we don't know more, the w83781d driver doesn't handle PWM on
+AS99127F chips at all.

lm-sensors/trunk/kernel/chips/w83781d.c

@@ -25 +25 @@
-
-    Chip        #vin    #fanin  #pwm    #temp   wchipid vendid  i2c     ISA
-1?
+0
-    as99127f rev.2 (type name = as99127f)       0x31    0x5ca3  yes     no
-    w83627hf    9       3       2       3       0x21    0x5ca3  yes     yes(LPC)
@@ -443 +443 @@
-        {W83781D_SYSCTL_BEEP, "beep", NULL, 0, 0644, NULL, &i2c_proc_real,
-         &i2c_sysctl_real, NULL, &w83781d_beep},
-        {W83781D_SYSCTL_PWM1, "pwm1", NULL, 0, 0644, NULL, &i2c_proc_real,
-         &i2c_sysctl_real, NULL, &w83781d_pwm},
-        {W83781D_SYSCTL_PWM2, "pwm2", NULL, 0, 0644, NULL, &i2c_proc_real,
-         &i2c_sysctl_real, NULL, &w83781d_pwm},
-        {0}
-};
@@ -1062 +1058 @@
-        }
-        data->sysctl_id = i;
+
+        /* Only PWM2 can be disabled */
+        for(i = 0; i < 4; i++)
+                data->pwmenable[i] = 1;
-
-        /* Initialize the chip */
@@ -1380 +1380 @@
-                                w83781d_write_value(client, W83781D_REG_IRQ,
-                                                    i | 0x40);
-
-                        for(i = 0; i < 3; i++)
-                                data->pwmenable[i] = 1;
-                }
-        }
@@ -1443 +1440 @@
-                       if (data->type != w83791d && i == 3) break;
-               }
-
+ && data->type != as99127f
-                       for (i = 1; i <= 4; i++) {
-                               data->pwm[i - 1] =
@@ -1450 +1447 @@
-                               if (((data->type == w83783s)
-                                    || (data->type == w83627hf)
-                                    || (data->type == as99127f)
-                                    || (data->type == w83697hf)
-                                    || ((data->type == w83782d)
@@ -1457 +1453 @@
-                                       break;
-                       }
+                        /* Only PWM2 can be disabled */
+                        data->pwmenable[1] = (w83781d_read_value(client,
+                                              W83781D_REG_PWMCLK12) & 0x08) >> 3;
-               }
-