x86: fsp: Remove the call to set up internal uart in fsp_init()
authorBin Meng <bmeng.cn@gmail.com>
Thu, 1 Jun 2017 10:41:14 +0000 (03:41 -0700)
committerBin Meng <bmeng.cn@gmail.com>
Mon, 5 Jun 2017 00:55:22 +0000 (08:55 +0800)
First of all, it's inappropriate to call setup_internal_uart() in a
generic API fsp_init(), as CONFIG_INTERNAL_UART is an option that
is only available on BayTrail platform. Secondly even for BayTrail,
there is no need to call setup_internal_uart() at all, as Intel FSP
will do this for us.

Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Stefan Roese <sr@denx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
arch/x86/lib/fsp/fsp_support.c

index a48036121119a27699f6dc2415c017044f4cf4be..ab8340c8715ac411c4ee246d86db69059ce1da27 100644 (file)
@@ -110,10 +110,6 @@ void fsp_init(u32 stack_top, u32 boot_mode, void *nvs_buf)
        struct upd_region *fsp_upd;
 #endif
 
-#ifdef CONFIG_INTERNAL_UART
-       setup_internal_uart(1);
-#endif
-
        fsp_hdr = find_fsp_header();
        if (fsp_hdr == NULL) {
                /* No valid FSP info header was found */