Changeset 20

Timestamp:

12/02/98 01:47:37 (14 years ago)

Author:

frodo

Message:

Bug-fixes: lm78.c is now working!

Yes, our first chip driver is now fully operative. Some parts of it can
still be moved to sensors.o and reused by other drivers, but this can be
done later.

Bug fixes:
* i2c-core.c: Corrected previous patch (a client was now unloaded twice,

which caused problems, as could be expected).

* lm78.c, sensors.c: some minor typos, which caused the crashes
* lm78.c: FAN_DIV only has two fields (fan 3 is not changable)
* sensors.c: ISA/SMBus addresses are prefixed with zeros in directory names

Location:

lm-sensors/trunk

Files:

• TODO (modified) (1 diff)
• i2c/i2c-core.c (modified) (2 diffs)
• kernel/chips/lm78.c (modified) (10 diffs)
• kernel/sensors.c (modified) (4 diffs)
• src/lm78.c (modified) (10 diffs)
• src/sensors.c (modified) (4 diffs)

lm-sensors/trunk/TODO

@@ -4 +4 @@
 * Make it SMP-safe: there are no spinlocks yet. They are needed at many
   places, probably (everywhere where global vars are accessed).
+* Extend sensors.c with a file /proc/sys/dev/sensors/chips, which lists
+  all directories under /proc/sys/dev/sensors with their SYSCTL id
+  (needed for good sysctl access).
 * Write smbus_access_i2c in smbus.c
 * Write piix4_* in piix4.c
-* Complete lm78 driver
-* Extend dev interface for SMBus
-* Create entries in /proc/bus: a list of all registered busses, and for
-  each bus, a file with registered clients (partially done)
+* Make lm78.c detect 'double hits', (same chip connected to both SMBus and
+  ISA).
+* Check whether some lm78 functionality is chip-generic and can be moved to
+  sensors.c.
+* Rename i2c-proc.c to i2c-user.c, and extend it with /dev entries;
+  or, create a better i2c-dev.c, that understands SMBus commands.
 * Write all other drivers
 * How to make drivers detect thing *not*; for example, if a Winbond is 
   present, the LM75 driver should not try to access its simulated
-  LM75 chips!
+  LM75 chips (in this case, simply disable them - but what if the LM75
+  driver was loaded first?).
 * Write a userland-library for SMBus/i2c access (through the /dev interface)
 * Write a userland detection program for busses (clients are less important;

lm-sensors/trunk/i2c/i2c-core.c

@@ -18 +18 @@
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.                */
 /* ------------------------------------------------------------------------- */
-#define RCSID "$Id: i2c-core.c,v 1.4 1998/11/27 22:02:09 frodo Exp $"
+#define RCSID "$Id: i2c-core.c,v 1.5 1998/12/01 21:11:32 frodo Exp $"
 /* ------------------------------------------------------------------------- */
 
@@ -249 +249 @@
                                 DEB2(printk("i2c:   detaching client %s:\n",
                                         client->name));
-                                i2c_detach_client(client);
+                                /* i2c_detach_client(client); */
                                 driver->detach_client(client);
                         }

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

@@ -92 +92 @@
 #define LM78_INIT_IN_6 -500
 
-#define LM78_INIT_IN_PERCENTAGE 100
+#define LM78_INIT_IN_PERCENTAGE 10
 
 #define LM78_INIT_IN_MIN_0 \
@@ -257 +257 @@
   { LM78_SYSCTL_IN6, "in6", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_FAN1, "fan1", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
-  { LM78_SYSCTL_FAN2, "fan1", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
-  { LM78_SYSCTL_FAN3, "fan1", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
+  { LM78_SYSCTL_FAN2, "fan2", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
+  { LM78_SYSCTL_FAN3, "fan3", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_TEMP, "temp", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_VID, "vid", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_FAN_DIV, "fan_div", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
-
-  { LM78_SYSCTL_ALARMS, "alarms", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl }
+  { LM78_SYSCTL_ALARMS, "alarms", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
+  { 0 }
 };
 
@@ -338 +338 @@
     /* Register a new directory entry with module sensors */
     if ((err = sensors_register_entry((struct i2c_client *) new_client,"lm78",
-                                      lm78_dir_table_template) < 0))
+                                      lm78_dir_table_template)) < 0)
       goto ERROR4;
     ((struct lm78_data *) (new_client->data)) -> sysctl_id = err;
@@ -381 +381 @@
   }
   lm78_remove_client((struct i2c_client *) client);
+  release_region(client->isa_addr,LM78_EXTENT);
   kfree(client);
-  release_region(client->isa_addr,LM78_EXTENT);
   return 0;
 }
@@ -423 +423 @@
     /* Register a new directory entry with module sensors */
     if ((err = sensors_register_entry(new_client,"lm78",
-                                      lm78_dir_table_template) < 0))
+                                      lm78_dir_table_template)) < 0)
       goto ERROR4;
     ((struct lm78_data *) (new_client->data))->sysctl_id = err;
@@ -593 +593 @@
 void lm78_update_client(struct i2c_client *client)
 {
-  struct lm78_data *data =  client->data;
+  struct lm78_data *data = client->data;
   int i;
 
@@ -601 +601 @@
       (jiffies < data->last_updated) || ! data->valid) {
 
+#ifdef DEBUG
+    printk("Starting lm78 update\n");
+#endif
     for (i = 0; i <= 6; i++) {
       data->in[i]     = lm78_read_value(client,LM78_REG_IN(i));
@@ -651 +654 @@
       mag=2;
       break;
-    case LM78_SYSCTL_FAN_DIV: case LM78_SYSCTL_TEMP:
+    case LM78_SYSCTL_TEMP:
       nrels=3;
       mag=0;
       break;
     case LM78_SYSCTL_FAN1: case LM78_SYSCTL_FAN2: case LM78_SYSCTL_FAN3:
+    case LM78_SYSCTL_FAN_DIV:
       nrels=2;
       mag=0;
@@ -737 +741 @@
     case LM78_SYSCTL_IN0: case LM78_SYSCTL_IN1: case LM78_SYSCTL_IN2: 
     case LM78_SYSCTL_IN3: case LM78_SYSCTL_IN4: case LM78_SYSCTL_IN5: 
-    case LM78_SYSCTL_IN6: case LM78_SYSCTL_TEMP: case LM78_SYSCTL_FAN_DIV:
+    case LM78_SYSCTL_IN6: case LM78_SYSCTL_TEMP: 
       nrels=3;
       break;
     case LM78_SYSCTL_FAN1: case LM78_SYSCTL_FAN2: case LM78_SYSCTL_FAN3:
+    case LM78_SYSCTL_FAN_DIV:
       nrels=2;
       break;
@@ -826 +831 @@
   struct lm78_data *data = client->data;
   results[0] = IN_FROM_REG(data->in_min[nr],nr);
-  results[1] = IN_FROM_REG(data->in_min[nr],nr);
-  results[2] = IN_FROM_REG(data->in_min[nr],nr);
+  results[1] = IN_FROM_REG(data->in_max[nr],nr);
+  results[2] = IN_FROM_REG(data->in[nr],nr);
 }
 

lm-sensors/trunk/kernel/sensors.c

@@ -63 +63 @@
   int id;
   if (i2c_is_isa_adapter(adapter)) 
-    sprintf(name_buffer,"%s-isa-%d",prefix,addr);
+    sprintf(name_buffer,"%s-isa-%04x",prefix,addr);
   else {
     if ((id = i2c_adapter_id(adapter)) < 0);
       return -ENOENT;
-    sprintf(name_buffer,"%s-i2c-%d-%d",prefix,id,addr);
+    sprintf(name_buffer,"%s-i2c-%d-%02x",prefix,id,addr);
   }
   *name = kmalloc(strlen(name_buffer)+1,GFP_KERNEL);
@@ -95 +95 @@
     return res;
 
-  for (id = 0; id < SENSORS_ENTRY_MAX; i++)
+  for (id = 0; id < SENSORS_ENTRY_MAX; id++)
     if (! sensors_entries[id]) {
       break;
@@ -130 +130 @@
   }
 
-  sensors_entries[i] = new_header;
+  sensors_entries[id-256] = new_header;
 
   return id;
@@ -142 +142 @@
     table = sensors_entries[id]->ctl_table;
     unregister_sysctl_table(sensors_entries[id]);
-    kfree((void *) (table->procname));
+    kfree((void *) (table[4].procname));
     kfree(table);
     sensors_entries[id] = 0;

lm-sensors/trunk/src/lm78.c

@@ -92 +92 @@
 #define LM78_INIT_IN_6 -500
 
-#define LM78_INIT_IN_PERCENTAGE 100
+#define LM78_INIT_IN_PERCENTAGE 10
 
 #define LM78_INIT_IN_MIN_0 \
@@ -257 +257 @@
   { LM78_SYSCTL_IN6, "in6", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_FAN1, "fan1", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
-  { LM78_SYSCTL_FAN2, "fan1", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
-  { LM78_SYSCTL_FAN3, "fan1", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
+  { LM78_SYSCTL_FAN2, "fan2", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
+  { LM78_SYSCTL_FAN3, "fan3", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_TEMP, "temp", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_VID, "vid", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
   { LM78_SYSCTL_FAN_DIV, "fan_div", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
-
-  { LM78_SYSCTL_ALARMS, "alarms", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl }
+  { LM78_SYSCTL_ALARMS, "alarms", NULL, 0, 0644, NULL, &lm78_proc, &lm78_sysctl },
+  { 0 }
 };
 
@@ -338 +338 @@
     /* Register a new directory entry with module sensors */
     if ((err = sensors_register_entry((struct i2c_client *) new_client,"lm78",
-                                      lm78_dir_table_template) < 0))
+                                      lm78_dir_table_template)) < 0)
       goto ERROR4;
     ((struct lm78_data *) (new_client->data)) -> sysctl_id = err;
@@ -381 +381 @@
   }
   lm78_remove_client((struct i2c_client *) client);
+  release_region(client->isa_addr,LM78_EXTENT);
   kfree(client);
-  release_region(client->isa_addr,LM78_EXTENT);
   return 0;
 }
@@ -423 +423 @@
     /* Register a new directory entry with module sensors */
     if ((err = sensors_register_entry(new_client,"lm78",
-                                      lm78_dir_table_template) < 0))
+                                      lm78_dir_table_template)) < 0)
       goto ERROR4;
     ((struct lm78_data *) (new_client->data))->sysctl_id = err;
@@ -593 +593 @@
 void lm78_update_client(struct i2c_client *client)
 {
-  struct lm78_data *data =  client->data;
+  struct lm78_data *data = client->data;
   int i;
 
@@ -601 +601 @@
       (jiffies < data->last_updated) || ! data->valid) {
 
+#ifdef DEBUG
+    printk("Starting lm78 update\n");
+#endif
     for (i = 0; i <= 6; i++) {
       data->in[i]     = lm78_read_value(client,LM78_REG_IN(i));
@@ -651 +654 @@
       mag=2;
       break;
-    case LM78_SYSCTL_FAN_DIV: case LM78_SYSCTL_TEMP:
+    case LM78_SYSCTL_TEMP:
       nrels=3;
       mag=0;
       break;
     case LM78_SYSCTL_FAN1: case LM78_SYSCTL_FAN2: case LM78_SYSCTL_FAN3:
+    case LM78_SYSCTL_FAN_DIV:
       nrels=2;
       mag=0;
@@ -737 +741 @@
     case LM78_SYSCTL_IN0: case LM78_SYSCTL_IN1: case LM78_SYSCTL_IN2: 
     case LM78_SYSCTL_IN3: case LM78_SYSCTL_IN4: case LM78_SYSCTL_IN5: 
-    case LM78_SYSCTL_IN6: case LM78_SYSCTL_TEMP: case LM78_SYSCTL_FAN_DIV:
+    case LM78_SYSCTL_IN6: case LM78_SYSCTL_TEMP: 
       nrels=3;
       break;
     case LM78_SYSCTL_FAN1: case LM78_SYSCTL_FAN2: case LM78_SYSCTL_FAN3:
+    case LM78_SYSCTL_FAN_DIV:
       nrels=2;
       break;
@@ -826 +831 @@
   struct lm78_data *data = client->data;
   results[0] = IN_FROM_REG(data->in_min[nr],nr);
-  results[1] = IN_FROM_REG(data->in_min[nr],nr);
-  results[2] = IN_FROM_REG(data->in_min[nr],nr);
+  results[1] = IN_FROM_REG(data->in_max[nr],nr);
+  results[2] = IN_FROM_REG(data->in[nr],nr);
 }
 

lm-sensors/trunk/src/sensors.c

@@ -63 +63 @@
   int id;
   if (i2c_is_isa_adapter(adapter)) 
-    sprintf(name_buffer,"%s-isa-%d",prefix,addr);
+    sprintf(name_buffer,"%s-isa-%04x",prefix,addr);
   else {
     if ((id = i2c_adapter_id(adapter)) < 0);
       return -ENOENT;
-    sprintf(name_buffer,"%s-i2c-%d-%d",prefix,id,addr);
+    sprintf(name_buffer,"%s-i2c-%d-%02x",prefix,id,addr);
   }
   *name = kmalloc(strlen(name_buffer)+1,GFP_KERNEL);
@@ -95 +95 @@
     return res;
 
-  for (id = 0; id < SENSORS_ENTRY_MAX; i++)
+  for (id = 0; id < SENSORS_ENTRY_MAX; id++)
     if (! sensors_entries[id]) {
       break;
@@ -130 +130 @@
   }
 
-  sensors_entries[i] = new_header;
+  sensors_entries[id-256] = new_header;
 
   return id;
@@ -142 +142 @@
     table = sensors_entries[id]->ctl_table;
     unregister_sysctl_table(sensors_entries[id]);
-    kfree((void *) (table->procname));
+    kfree((void *) (table[4].procname));
     kfree(table);
     sensors_entries[id] = 0;