[lm-sensors] [patch 1/1] w83627hf hoist nr-1 offset out of show-store-temp-X
Jim Cromie
jim.cromie at gmail.com
Sat Oct 13 00:50:05 CEST 2007
This hoists nr-1 offset out of (show|store)_temp_*(.*) callbacks, and into
SENSOR_DEVICE_ATTRs for sysfs tempN_X files. It also combines
temp[1] and temp_add[2] (array) fields in w83627hf_data into 3 elem arrays,
which simplifies special-case handling of nr, allowing simplification
of callback
bodies and rerolling a flattened loop in w83627hf_update_device(struct
device *dev). The array conversion changes temp[1] from u8 to u16,
but this was happening
implicitly via the helper functions anyway.
Signed-off-by: Jim Cromie <jim.cromie at gmail.com>
---
$ diffstat diff.hwmon-w83627hf-hoist-temp-offsets
drivers/hwmon/w83627hf.c | 121 +++++++++++++++++------------------------------
1 files changed, 45 insertions(+), 76 deletions(-)
size shrinks too:
12982 2652 36 15670 3d36 hwmon-demacro/drivers/hwmon/w83627hf.ko
12850 2652 36 15538 3cb2 hwmon-hf-2/drivers/hwmon/w83627hf.ko
and the u8->u16 doesnt seem to cost anything in the data section. (padding?)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: diff.hwmon-w83627hf-hoist-temp-offsets
Type: application/octet-stream
Size: 7144 bytes
Desc: not available
Url : http://lists.lm-sensors.org/pipermail/lm-sensors/attachments/20071012/2b7e221e/attachment-0001.obj
More information about the lm-sensors
mailing list