aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBin Meng <bmeng.cn@gmail.com>2021-02-03 22:42:25 +0800
committerSimon Glass <sjg@chromium.org>2021-03-03 19:17:25 -0700
commit384b62c073f3aaccba0c917a8da7701a82441aec (patch)
tree34016a954bbcba79b78713790480b5072828d29b
parent09bd08401a5ff2cdbf40fdc27a8add809ae11075 (diff)
downloadu-boot-384b62c073f3aaccba0c917a8da7701a82441aec.zip
u-boot-384b62c073f3aaccba0c917a8da7701a82441aec.tar.gz
u-boot-384b62c073f3aaccba0c917a8da7701a82441aec.tar.bz2
serial: ns16550: Handle zero <clock-frequency> value
A working device tree node of ns16550 should never be populated with value zero for the <clock-frequency> property. Unfortunately this is the case for the QEMU ppce500 target. Let's try to assign plat->clock to CONFIG_SYS_NS16550_CLK as the last resort to handle such case. This commit should be reverted when: - The following QEMU patch [1] is merged, and - U-Boot CI has upgraded its QEMU version that contains the fix [1] http://patchwork.ozlabs.org/project/qemu-devel/patch/1612362288-22216-2-git-send-email-bmeng.cn@gmail.com/ Signed-off-by: Bin Meng <bmeng.cn@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org>
-rw-r--r--drivers/serial/ns16550.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/serial/ns16550.c b/drivers/serial/ns16550.c
index 1377c9a..cc121ee 100644
--- a/drivers/serial/ns16550.c
+++ b/drivers/serial/ns16550.c
@@ -564,6 +564,8 @@ int ns16550_serial_of_to_plat(struct udevice *dev)
if (!plat->clock)
plat->clock = dev_read_u32_default(dev, "clock-frequency",
CONFIG_SYS_NS16550_CLK);
+ if (!plat->clock)
+ plat->clock = CONFIG_SYS_NS16550_CLK;
if (!plat->clock) {
debug("ns16550 clock not defined\n");
return -EINVAL;