google/urara: use board ID information to set up hardware
The hardware initialization is now split in basic
initialization (MIPS and system PLL, system clock,
SPIM, UART), and initialization of other hardware
blocks (USB, I2C, ETH). The second part uses board ID
information to select setup that is board specific
(currently only I2C interface is selected through
board ID).
BRANCH=none
BUG=chrome-os-partner:37593
TEST=tested on bring up board for both Urara and Concerto;
to simulate the use of Concerto (I2C3) DIP SW17 was
set to 0.
it works with default settings on Urara
Change-Id: Ic5bbf28ab42545a4fb2aa6fd30592a02ecc15cb5
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Commit-Id: f2b3db2e7f9fa898214f974ca34ea427196d2e4e
Original-Change-Id: Iac9a082ad84444af1d9d9785a2d0cc3205140d15
Original-Signed-off-by: Ionela Voinescu <ionela.voinescu@imgtec.com>
Original-Reviewed-on: https://chromium-review.googlesource.com/257401
Original-Reviewed-by: Vadim Bendebury <vbendeb@chromium.org>
Original-Commit-Queue: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: http://review.coreboot.org/9888
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
diff --git a/src/arch/mips/bootblock_simple.c b/src/arch/mips/bootblock_simple.c
index 64bbae2..85f9336 100644
--- a/src/arch/mips/bootblock_simple.c
+++ b/src/arch/mips/bootblock_simple.c
@@ -21,11 +21,14 @@
#include <bootblock_common.h>
#include <console/console.h>
+#include <halt.h>
#include <program_loading.h>
void main(void)
{
bootblock_cpu_init();
+
+ /* Mainboard basic init */
bootblock_mainboard_init();
#if CONFIG_BOOTBLOCK_CONSOLE
@@ -34,5 +37,10 @@
bootblock_mmu_init();
- run_romstage();
+ if (init_extra_hardware()) {
+ printk(BIOS_ERR, "bootblock_simple: failed to init HW.\n");
+ } else {
+ run_romstage();
+ }
+ halt();
}
diff --git a/src/include/boardid.h b/src/include/boardid.h
index 41c21d6..ce8dc6c 100644
--- a/src/include/boardid.h
+++ b/src/include/boardid.h
@@ -22,6 +22,11 @@
#include <stdint.h>
+struct board_hw {
+ uint8_t i2c_interface;
+};
+
+const struct board_hw *board_get_hw(void);
uint8_t board_id(void);
uint32_t ram_code(void);
diff --git a/src/mainboard/google/urara/Makefile.inc b/src/mainboard/google/urara/Makefile.inc
index 1428912..62dc3e4 100644
--- a/src/mainboard/google/urara/Makefile.inc
+++ b/src/mainboard/google/urara/Makefile.inc
@@ -19,6 +19,7 @@
# MA 02110-1301 USA
#
+bootblock-y += boardid.c
ramstage-y += boardid.c
ramstage-$(CONFIG_CHROMEOS) += chromeos.c
ramstage-y += mainboard.c
diff --git a/src/mainboard/google/urara/boardid.c b/src/mainboard/google/urara/boardid.c
index 02cfa05..a95bc14 100644
--- a/src/mainboard/google/urara/boardid.c
+++ b/src/mainboard/google/urara/boardid.c
@@ -34,13 +34,14 @@
const struct bid_map {
const char *board_name;
uint8_t board_id;
+ struct board_hw hardware;
} board_id_map[] = {
- {"urara", URARA_BOARD_ID_BUB},
- {"buranku", URARA_BOARD_ID_BURANKU},
- {"derwent", URARA_BOARD_ID_DERWENT},
- {"jaguar", URARA_BOARD_ID_JAGUAR},
- {"kennet", URARA_BOARD_ID_KENNET},
- {"space", URARA_BOARD_ID_SPACE},
+ {"urara", URARA_BOARD_ID_BUB, {0} },
+ {"buranku", URARA_BOARD_ID_BURANKU, {3} },
+ {"derwent", URARA_BOARD_ID_DERWENT, {3} },
+ {"jaguar", URARA_BOARD_ID_JAGUAR, {3} },
+ {"kennet", URARA_BOARD_ID_KENNET, {3} },
+ {"space", URARA_BOARD_ID_SPACE, {3} },
};
static int cached_board_id = -1;
@@ -91,6 +92,19 @@
return 0;
}
+const struct board_hw *board_get_hw(void)
+{
+ int i;
+ uint8_t bid = board_id();
+
+ for (i = 0; i < ARRAY_SIZE(board_id_map); i++) {
+ if (bid == board_id_map[i].board_id)
+ return &(board_id_map[i].hardware);
+ }
+
+ return 0;
+}
+
uint8_t board_id(void)
{
if (cached_board_id == -1)
diff --git a/src/mainboard/google/urara/bootblock.c b/src/mainboard/google/urara/bootblock.c
index 4530c8c..3692802e 100644
--- a/src/mainboard/google/urara/bootblock.c
+++ b/src/mainboard/google/urara/bootblock.c
@@ -23,6 +23,7 @@
#include <stdint.h>
#include <soc/clocks.h>
#include <assert.h>
+#include <boardid.h>
#define PADS_FUNCTION_SELECT0_ADDR (0xB8101C00 + 0xC0)
@@ -172,15 +173,10 @@
write32(PADS_FUNCTION_SELECT0_ADDR, reg);
}
-static int init_clocks(void)
+static void bootblock_mainboard_init(void)
{
int ret;
- /*
- * Set up dividers for peripherals before setting up PLLs
- * in order to not over-clock them when enabling PLLs
- */
-
/* System PLL divided by 2 -> 400 MHz */
/* The same frequency will be the input frequency for the SPFI block */
system_clk_setup(1);
@@ -190,40 +186,57 @@
* the values set or not by the boot ROM code */
mips_clk_setup(0, 0);
- /* System clock divided by 8 -> 50 MHz */
- ret = usb_clk_setup(7, 2, 7);
+ /* Setup system PLL at 800 MHz */
+ ret = sys_pll_setup(2, 1);
if (ret != CLOCKS_OK)
- return ret;
+ return;
+ /* Setup MIPS PLL at 546 MHz */
+ ret = mips_pll_setup(2, 1, 1, 21);
+ if (ret != CLOCKS_OK)
+ return;
- /* System PLL divided by 7 divided by 62 -> 1.8433 Mhz */
+ /* Setup SPIM1 MFIOs */
+ spim1_mfio_setup();
+ /* Setup UART1 clock and MFIOs
+ * System PLL divided by 7 divided by 62 -> 1.8433 Mhz
+ */
uart1_clk_setup(6, 61);
+ uart1_mfio_setup();
+}
- /* System PLL divided by 4 divided by 3 -> 33.33 MHz */
- i2c_clk_setup(3, 2, 0);
+
+static int init_extra_hardware(void)
+{
+ const struct board_hw *hardware;
+
+ /* Obtain information about current board */
+ hardware = board_get_hw();
+ if (!hardware) {
+ printk(BIOS_ERR, "%s: Invalid hardware information.\n",
+ __func__);
+ return -1;
+ }
+
+ /* Setup USB clock
+ * System clock divided by 8 -> 50 MHz
+ */
+ if (usb_clk_setup(7, 2, 7) != CLOCKS_OK) {
+ printk(BIOS_ERR, "%s: Failed to set up USB clock.\n",
+ __func__);
+ return -1;
+ }
+
+ /* Setup I2C clocks and MFIOs
+ * System PLL divided by 4 divided by 3 -> 33.33 MHz
+ */
+ i2c_clk_setup(3, 2, hardware->i2c_interface);
+ i2c_mfio_setup(hardware->i2c_interface);
+
/* Ethernet clocks setup: ENET as clock source */
eth_clk_setup(0, 7);
-
/* ROM clock setup: system clock divided by 2 -> 200 MHz */
/* Hash accelerator is driven from the ROM clock */
rom_clk_setup(1);
- /* Setup system PLL at 800 MHz */
- ret = sys_pll_setup(2, 1);
- if (ret != CLOCKS_OK)
- return ret;
- /* Setup MIPS PLL at 546 MHz */
- ret = mips_pll_setup(2, 1, 1, 21);
- if (ret != CLOCKS_OK)
- return ret;
- return CLOCKS_OK;
-}
-
-static void bootblock_mainboard_init(void)
-{
- if (!init_clocks()) {
- /* Disable GPIO on the peripheral lines */
- uart1_mfio_setup();
- spim1_mfio_setup();
- i2c_mfio_setup(0);
- }
+ return 0;
}