diff options
author | Arnd Bergmann <arnd@arndb.de> | 2020-05-27 15:51:57 +0200 |
---|---|---|
committer | Guenter Roeck <linux@roeck-us.net> | 2020-05-27 09:18:13 -0700 |
commit | fff2d0f701e6753591609739f8ab9be1c8e80ebb (patch) | |
tree | ee91b58d6b14c7e435e4550025c6fc1b4668f0ee /kernel/gcov/clang.c | |
parent | 156ad7f9e0b39cfa0ed02e42c6e1c258c4e81e05 (diff) |
hwmon: (applesmc) avoid overlong udelay()
Building this driver with "clang -O3" produces a link error
after the compiler partially unrolls the loop and 256ms
becomes a compile-time constant that triggers the check
in udelay():
ld.lld: error: undefined symbol: __bad_udelay
>>> referenced by applesmc.c
>>> hwmon/applesmc.o:(read_smc) in archive drivers/built-in.a
I can see no reason against using a sleeping function here,
as no part of the driver runs in atomic context, so instead use
usleep_range() with a wide range and use jiffies for the
end condition.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20200527135207.1118624-1-arnd@arndb.de
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'kernel/gcov/clang.c')
0 files changed, 0 insertions, 0 deletions