[U-Boot] [PATCH 2/3] serial: ns16550: use DIV_ROUND_CLOSEST macro to compute the divisor
Masahiro Yamada
yamada.m at jp.panasonic.com
Fri Jul 11 13:29:03 CEST 2014
The function still returns the same value.
The comment block is no longer necessary because our intention is
clear enough by using DIV_ROUND_CLOSEST() macro.
Signed-off-by: Masahiro Yamada <yamada.m at jp.panasonic.com>
---
drivers/serial/serial_ns16550.c | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/drivers/serial/serial_ns16550.c b/drivers/serial/serial_ns16550.c
index 056ef2a..49e2c1f 100644
--- a/drivers/serial/serial_ns16550.c
+++ b/drivers/serial/serial_ns16550.c
@@ -130,13 +130,9 @@ static int calc_divisor (NS16550_t port)
#endif
#define MODE_X_DIV 16
- /* Compute divisor value. Normally, we should simply return:
- * CONFIG_SYS_NS16550_CLK) / MODE_X_DIV / gd->baudrate
- * but we need to round that value by adding 0.5.
- * Rounding is especially important at high baud rates.
- */
- return (CONFIG_SYS_NS16550_CLK + (gd->baudrate * (MODE_X_DIV / 2))) /
- (MODE_X_DIV * gd->baudrate);
+
+ return DIV_ROUND_CLOSEST(CONFIG_SYS_NS16550_CLK,
+ MODE_X_DIV * gd->baudrate);
}
void
--
1.9.1
More information about the U-Boot
mailing list