diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2019-01-03 11:38:39 +0200 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2019-01-04 17:20:24 +0000 |
commit | c0b1be0ba1d0c87d9670e695c60eeb6ec901a2d8 (patch) | |
tree | 5441907a48042d21e459c470a5a514527162fa8c /src/mainboard/msi/ms9652_fam10 | |
parent | a2cfe9e90001fe11f7c7653c9917ea75dd082333 (diff) |
amdfam10 boards: Call get_bus_conf() just once
It has to be called once before PIRQ and MP table generation.
Change-Id: I238c6b4810404d320b36d4f6b4a161c1ff11c8d3
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/30630
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/mainboard/msi/ms9652_fam10')
-rw-r--r-- | src/mainboard/msi/ms9652_fam10/acpi_tables.c | 1 | ||||
-rw-r--r-- | src/mainboard/msi/ms9652_fam10/get_bus_conf.c | 8 | ||||
-rw-r--r-- | src/mainboard/msi/ms9652_fam10/irq_tables.c | 1 | ||||
-rw-r--r-- | src/mainboard/msi/ms9652_fam10/mptable.c | 1 |
4 files changed, 0 insertions, 11 deletions
diff --git a/src/mainboard/msi/ms9652_fam10/acpi_tables.c b/src/mainboard/msi/ms9652_fam10/acpi_tables.c index 5b00374abc..6e759739c9 100644 --- a/src/mainboard/msi/ms9652_fam10/acpi_tables.c +++ b/src/mainboard/msi/ms9652_fam10/acpi_tables.c @@ -39,7 +39,6 @@ unsigned long acpi_fill_madt(unsigned long current) struct resource *res; struct device *dev; - get_bus_conf(); sbdn = sysconf.sbdn; m = sysconf.mb; diff --git a/src/mainboard/msi/ms9652_fam10/get_bus_conf.c b/src/mainboard/msi/ms9652_fam10/get_bus_conf.c index 887110f717..c2183a3b0f 100644 --- a/src/mainboard/msi/ms9652_fam10/get_bus_conf.c +++ b/src/mainboard/msi/ms9652_fam10/get_bus_conf.c @@ -55,8 +55,6 @@ static unsigned hcdnx[] = { 0x20202020, 0x20202020, }; -static unsigned get_bus_conf_done = 0; - void get_bus_conf(void) { unsigned apicid_base; @@ -65,12 +63,6 @@ void get_bus_conf(void) struct device *dev; int i; - printk(BIOS_SPEW, "get_bus_conf()\n"); - - if (get_bus_conf_done == 1) - return; //do it only once - - get_bus_conf_done = 1; sysconf.mb = &mb_sysconf; diff --git a/src/mainboard/msi/ms9652_fam10/irq_tables.c b/src/mainboard/msi/ms9652_fam10/irq_tables.c index 513fd57c58..97245ec292 100644 --- a/src/mainboard/msi/ms9652_fam10/irq_tables.c +++ b/src/mainboard/msi/ms9652_fam10/irq_tables.c @@ -57,7 +57,6 @@ unsigned long write_pirq_routing_table(unsigned long addr) uint8_t sum = 0; int i; - get_bus_conf(); // it will find out all bus num and apic that share with mptable.c and mptable.c and acpi_tables.c sbdn = sysconf.sbdn; m = sysconf.mb; diff --git a/src/mainboard/msi/ms9652_fam10/mptable.c b/src/mainboard/msi/ms9652_fam10/mptable.c index a4362f40de..8cf5b3d507 100644 --- a/src/mainboard/msi/ms9652_fam10/mptable.c +++ b/src/mainboard/msi/ms9652_fam10/mptable.c @@ -36,7 +36,6 @@ static void *smp_write_config_table(void *v) smp_write_processors(mc); - get_bus_conf(); sbdn = sysconf.sbdn; m = sysconf.mb; |