GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Exynos5420: Clean up console code 

- Don't initialize console twice in the bootblock
 - remove printk in memory init that would mess up the UART
 - unconditionally run console_init() in romstage, as it is
   also unconditionally run in the bootblock.

Change-Id: I983d011c6ca602445f447d17799c1b2a33e8bd1d
Signed-off-by: Stefan Reinauer <reinauer@chromium.org>
Signed-off-by: Gabe Black <gabeblack@chromium.org>
Reviewed-on: http://review.coreboot.org/3656
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
Stefan Reinauer 2013-05-20 12:29:37 -07:00 committed by Stefan Reinauer
parent 045222f312
commit 998ab0d5f9
3 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/cpu/samsung/exynos5420/bootblock.c
View File

@ -17,7 +17,6 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#include <console/console.h>
#include "clk.h" #include "clk.h"
#include "wakeup.h" #include "wakeup.h"
@ -38,6 +37,4 @@ void bootblock_cpu_init(void)
* (ex, SPI, SD/MMC, or eMMC) now; but for Exynos platform, that is * (ex, SPI, SD/MMC, or eMMC) now; but for Exynos platform, that is
* already handled by iROM so there's no need to setup again. * already handled by iROM so there's no need to setup again.
*/ */
console_init();
} }

1
src/mainboard/google/pit/memory.c
View File

@ -490,7 +490,6 @@ static int board_get_config(void)
id1 = gpio_read_mvl3(BOARD_ID1_GPIO); id1 = gpio_read_mvl3(BOARD_ID1_GPIO);
if (id0 < 0 || id1 < 0) if (id0 < 0 || id1 < 0)
return -1; return -1;
printk(BIOS_DEBUG, "%s: id0: %u, id1: %u\n", __func__, id0, id1);
for (i = 0; i < ARRAY_SIZE(id_map); i++) { for (i = 0; i < ARRAY_SIZE(id_map); i++) {
if (id0 == id_map[i].id0 && id1 == id_map[i].id1) { if (id0 == id_map[i].id0 && id1 == id_map[i].id1) {

5
src/mainboard/google/pit/romstage.c
View File

@ -129,7 +129,7 @@ static void setup_gpio(void)
static void setup_memory(struct mem_timings *mem, int is_resume) static void setup_memory(struct mem_timings *mem, int is_resume)
{ {
printk(BIOS_SPEW, "man: 0x%x type: 0x%x, div: 0x%x, mhz: 0x%x\n", printk(BIOS_SPEW, "man: 0x%x type: 0x%x, div: 0x%x, mhz: %d\n",
mem->mem_manuf, mem->mem_manuf,
mem->mem_type, mem->mem_type,
mem->mpll_mdiv, mem->mpll_mdiv,
@ -168,8 +168,9 @@ void main(void)
* to re-initialize serial console drivers again. */ * to re-initialize serial console drivers again. */
mem = setup_clock(); mem = setup_clock();
console_init();
if (!is_resume) { if (!is_resume) {
console_init();
setup_power(); setup_power();
} }