Messages in this thread Patch in this message | | | From | Dejin Zheng <> | Subject | [PATCH v2 2/2] regmap: Simplify implementation of the regmap_field_read_poll_timeout() macro | Date | Mon, 20 Apr 2020 21:46:47 +0800 |
| |
Simplify the implementation of the macro regmap_field_read_poll_timeout() by using the macro read_poll_timeout().
Signed-off-by: Dejin Zheng <zhengdejin5@gmail.com> --- v1 -> v2: - modify the commit comments by Markus's suggestion .
include/linux/regmap.h | 23 ++++------------------- 1 file changed, 4 insertions(+), 19 deletions(-)
diff --git a/include/linux/regmap.h b/include/linux/regmap.h index 299c1f6a03b4..78ddf224f988 100644 --- a/include/linux/regmap.h +++ b/include/linux/regmap.h @@ -194,25 +194,10 @@ struct reg_sequence { */ #define regmap_field_read_poll_timeout(field, val, cond, sleep_us, timeout_us) \ ({ \ - u64 __timeout_us = (timeout_us); \ - unsigned long __sleep_us = (sleep_us); \ - ktime_t timeout = ktime_add_us(ktime_get(), __timeout_us); \ - int pollret; \ - might_sleep_if(__sleep_us); \ - for (;;) { \ - pollret = regmap_field_read((field), &(val)); \ - if (pollret) \ - break; \ - if (cond) \ - break; \ - if (__timeout_us && ktime_compare(ktime_get(), timeout) > 0) { \ - pollret = regmap_field_read((field), &(val)); \ - break; \ - } \ - if (__sleep_us) \ - usleep_range((__sleep_us >> 2) + 1, __sleep_us); \ - } \ - pollret ?: ((cond) ? 0 : -ETIMEDOUT); \ + int __ret, __tmp; \ + __tmp = read_poll_timeout(regmap_field_read, __ret, __ret || (cond), \ + sleep_us, timeout_us, false, (field), &(val)); \ + __ret ?: __tmp; \ }) #ifdef CONFIG_REGMAP -- 2.25.0
| |