[coreboot] New patch to review for coreboot: 5ea7eba tyan s8226: move pcie training and other cimx wrappers to romstage
Siyuan Wang (wangsiyuanbuaa@gmail.com)
gerrit at coreboot.org
Fri Oct 19 15:23:25 CEST 2012
Siyuan Wang (wangsiyuanbuaa at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1594
-gerrit
commit 5ea7eba71dc91a4924decb52318388f6d56e7500
Author: Siyuan Wang <wangsiyuanbuaa at gmail.com>
Date: Fri Oct 19 21:57:15 2012 +0800
tyan s8226: move pcie training and other cimx wrappers to romstage
We move s8226's 5 cimx wrappers to romstage due to last commit.
The changes are include Pcie_Early_Init, Pcie_Late_Init
Early_Post_Init, Mid_Post_Init and Late_Post_Init.
In order to config rd890 in romstage, we modify rd890_cfg.c too.
We find the rd890 device in device list and get the config.
Change-Id: I4ff70d0e6a0235b87806a64376f3ecee6487fdee
Signed-off-by: Siyuan Wang <SiYuan.Wang at amd.com>
Signed-off-by: Siyuan Wang <wangsiyuanbuaa at gmail.com>
---
src/mainboard/tyan/s8226/rd890_cfg.c | 22 +++++++++++++---------
src/mainboard/tyan/s8226/rd890_cfg.h | 1 +
src/mainboard/tyan/s8226/romstage.c | 15 +++++++++++++++
3 files changed, 29 insertions(+), 9 deletions(-)
diff --git a/src/mainboard/tyan/s8226/rd890_cfg.c b/src/mainboard/tyan/s8226/rd890_cfg.c
index 7a947b3..d097e4c 100644
--- a/src/mainboard/tyan/s8226/rd890_cfg.c
+++ b/src/mainboard/tyan/s8226/rd890_cfg.c
@@ -23,25 +23,32 @@
#include "nbInitializer.h"
#include <string.h>
#include <arch/ioapic.h>
-
-#ifndef __PRE_RAM__
#include <device/device.h>
+
+extern ROMSTAGE_CONST struct device dev_root;
extern void set_pcie_reset(void *config);
extern void set_pcie_dereset(void *config);
/**
* Platform dependent configuration at ramstage
*/
-static void nb_platform_config(device_t nb_dev, AMD_NB_CONFIG *NbConfigPtr)
+void nb_platform_config(AMD_NB_CONFIG_BLOCK *pConfig)
{
u16 i;
+ struct device *pdev;
+ AMD_NB_CONFIG *NbConfigPtr = &(pConfig->Northbridges[0]);
PCIE_CONFIG *pPcieConfig = NbConfigPtr->pPcieConfig;
- //AMD_NB_CONFIG_BLOCK *ConfigPtr = GET_BLOCK_CONFIG_PTR(NbConfigPtr);
struct northbridge_amd_cimx_rd890_config *rd890_info = NULL;
DEFAULT_PLATFORM_CONFIG(platform_config);
+ /* find rd890 in device list */
+ for (pdev = &dev_root; pdev; pdev=pdev->next) {
+ if ((pdev->path.type == DEVICE_PATH_PCI) && (pdev->path.pci.devfn == 0))
+ break;
+ }
+ rd890_info = pdev->chip_info;
+
/* update the platform depentent configuration by devicetree */
- rd890_info = nb_dev->chip_info;
platform_config.PortEnableMap = rd890_info->port_enable;
if (rd890_info->gpp1_configuration == 0) {
platform_config.Gpp1Config = GFX_CONFIG_AAAA;
@@ -97,7 +104,6 @@ static void nb_platform_config(device_t nb_dev, AMD_NB_CONFIG *NbConfigPtr)
}
}
}
-#endif // __PRE_RAM__
/**
* @brief Entry point of Northbridge CIMx callout/CallBack
@@ -160,9 +166,6 @@ static u32 rd890_callout_entry(u32 func, u32 data, void *config)
break;
case CB_AmdSetPcieEarlyConfig:
-#ifndef __PRE_RAM__
- nb_platform_config(nb_dev, nbConfigPtr);
-#endif
break;
case CB_AmdSetEarlyPostConfig:
@@ -244,6 +247,7 @@ void rd890_cimx_config(AMD_NB_CONFIG_BLOCK *pConfig, NB_CONFIG *nbConfig, HT_CON
pConfig->Northbridges[0].NbHtPath.NodeID = sbNode;
pConfig->Northbridges[0].NbHtPath.LinkID = sbLink;
//TODO: other NBs
+ pConfig->Northbridges[0].pNbConfig->IoApicBaseAddress = IO_APIC_ADDR;
#ifndef __PRE_RAM__
/* If temporrary MMIO enable set up CPU MMIO */
diff --git a/src/mainboard/tyan/s8226/rd890_cfg.h b/src/mainboard/tyan/s8226/rd890_cfg.h
index 8f45019..b11928f 100644
--- a/src/mainboard/tyan/s8226/rd890_cfg.h
+++ b/src/mainboard/tyan/s8226/rd890_cfg.h
@@ -170,5 +170,6 @@ typedef struct {
* Bridge CIMx configuration
*/
void rd890_cimx_config(AMD_NB_CONFIG_BLOCK *pConfig, NB_CONFIG *nbConfig, HT_CONFIG *htConfig, PCIE_CONFIG *pcieConfig);
+void nb_platform_config(AMD_NB_CONFIG_BLOCK *pConfigPtr);
#endif //_RD890_CFG_H_
diff --git a/src/mainboard/tyan/s8226/romstage.c b/src/mainboard/tyan/s8226/romstage.c
index 7076eed..b784c25 100644
--- a/src/mainboard/tyan/s8226/romstage.c
+++ b/src/mainboard/tyan/s8226/romstage.c
@@ -45,6 +45,10 @@ extern void disable_cache_as_ram(void); /* cache_as_ram.inc */
void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
{
u32 val;
+ NB_CONFIG nb_cfg[MAX_NB_COUNT];
+ HT_CONFIG ht_cfg[MAX_NB_COUNT];
+ PCIE_CONFIG pcie_cfg[MAX_NB_COUNT];
+ AMD_NB_CONFIG_BLOCK gConfig;
post_code(0x30);
agesawrapper_amdinitmmio();
@@ -139,6 +143,17 @@ void cache_as_ram_main(unsigned long bist, unsigned long cpu_init_detectedx)
post_code(0x51);
setup_i8259 ();
setup_i8254 ();
+
+ /* config fore pcie init and post init */
+ rd890_cimx_config(&gConfig, &nb_cfg[0], &ht_cfg[0], &pcie_cfg[0]);
+ nb_platform_config(&gConfig);
+
+ nb_Pcie_Early_Init(&gConfig);
+ nb_Pcie_Late_Init(&gConfig);
+ nb_Early_Post_Init(&gConfig);
+ nb_Mid_Post_Init(&gConfig);
+ nb_Late_Post_Init(&gConfig);
+
copy_and_run(0);
/* We will not return, Should never see this message and post code. */
More information about the coreboot
mailing list