Changeset 4758 for lm-sensors/branches/lm-sensors-3.0.0/lib/sysfs.c

Timestamp:

09/05/07 10:17:22 (6 years ago)

Author:

khali

Message:

The compute mapping value is either SENSORS_NO_MAPPING or the same
feature number as the logical mapping. This means that the compute
mapping can be turned into a simple boolean flag. Doing so makes
struct sensors_feature_data smaller, which saves some memory (about
17 kB in my tests.)

Files:

• lm-sensors/branches/lm-sensors-3.0.0/lib/sysfs.c (modified) (1 diff)

lm-sensors/branches/lm-sensors-3.0.0/lib/sysfs.c

@@ -143 +143 @@
                         /* main feature */
                         feature.data.mapping = SENSORS_NO_MAPPING;
-                        feature.data.compute_mapping = SENSORS_NO_MAPPING;
-                } else if (type & 0x10) {
-                        /* sub feature without compute mapping */
+                } else {
+                        /* sub feature */
                         feature.data.mapping = i - i % MAX_SUB_FEATURES;
-                        feature.data.compute_mapping = SENSORS_NO_MAPPING;
-                } else {
-                        feature.data.mapping = i - i % MAX_SUB_FEATURES;
-                        feature.data.compute_mapping = feature.data.mapping;
+                        if (!(type & 0x10))
+                                feature.data.flags |= SENSORS_COMPUTE_MAPPING;
                 }