diff options
author | Henrik Rydberg <rydberg@euromail.se> | 2008-10-18 20:27:43 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-10-20 08:52:35 -0700 |
commit | 8c9398d1e9766e3659e277acb2e8ca1c17684139 (patch) | |
tree | 160520c76c2dfbe89398b6c53aa8813e212abed3 /drivers/hwmon/smsc47b397.c | |
parent | 07e8dbd3ebbdedc71335049dd4b0d542cb038d7d (diff) | |
download | op-kernel-dev-8c9398d1e9766e3659e277acb2e8ca1c17684139.zip op-kernel-dev-8c9398d1e9766e3659e277acb2e8ca1c17684139.tar.gz |
hwmon: applesmc: lighter wait mechanism, drastic improvement
The read fail ratio is sensitive to the delay between the first byte
written and the first byte read; apparently the sensors cannot be rushed.
Increasing the minimum wait time, without changing the total wait time,
improves the fail ratio from a 8% chance that any of the sensors fails in
one read, down to 0.4%, on a Macbook Air. On a Macbook Pro 3,1, the
effect is even more apparent. By reducing the number of status polls, the
ratio is further improved to below 0.1%. Finally, increasing the total
wait time brings the fail ratio down to virtually zero.
Signed-off-by: Henrik Rydberg <rydberg@euromail.se>
Tested-by: Bob McElrath <bob@mcelrath.org>
Cc: Nicolas Boichat <nicolas@boichat.ch>
Cc: "Mark M. Hoffman" <mhoffman@lightlink.com>
Cc: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/hwmon/smsc47b397.c')
0 files changed, 0 insertions, 0 deletions