Update the string to take account of UARTs which are connected on I/O
ports, as on x86.

Fix a typo in an error message in the same command, while we are here.

Signed-off-by: Simon Glass <s...@chromium.org>
---

(no changes since v3)

Changes in v3:
- Add new patch to support IO UARTs for earlycon and console

 boot/bootflow.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/boot/bootflow.c b/boot/bootflow.c
index 58a1afa7a75..4054a966af8 100644
--- a/boot/bootflow.c
+++ b/boot/bootflow.c
@@ -942,8 +942,9 @@ int bootflow_cmdline_auto(struct bootflow *bflow, const 
char *arg)
        *buf = '\0';
        if (!strcmp("earlycon", arg) && info.type == 
SERIAL_CHIP_16550_COMPATIBLE) {
                snprintf(buf, sizeof(buf),
-                        "uart8250,mmio32,%#lx,%dn8", info.addr,
-                        info.baudrate);
+                        "uart8250,%s,%#lx,%dn8",
+                        info.addr_space == SERIAL_ADDRESS_SPACE_IO ? "io" :
+                        "mmio", info.addr, info.baudrate);
        } else if (!strcmp("earlycon", arg) && info.type == SERIAL_CHIP_PL01X) {
                snprintf(buf, sizeof(buf),
                         "pl011,mmio32,%#lx,%dn8", info.addr,
@@ -954,7 +955,7 @@ int bootflow_cmdline_auto(struct bootflow *bflow, const 
char *arg)
        }
 
        if (!*buf) {
-               printf("Unknown param '%s\n", arg);
+               printf("Unknown param '%s'\n", arg);
                return -ENOENT;
        }
 
-- 
2.43.0

Reply via email to