summaryrefslogtreecommitdiff
path: root/src/mainboard/hp
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2014-06-19 22:19:20 +0300
committerKyösti Mälkki <kyosti.malkki@gmail.com>2014-07-03 09:48:57 +0200
commitef40ca57ebd4de746eafaa1e5a1cae035337f285 (patch)
tree5425f163dfe47117f9c8e0062c415b2400788792 /src/mainboard/hp
parent005028e0a952b00b6184cdddf5905a1637029585 (diff)
downloadcoreboot-ef40ca57ebd4de746eafaa1e5a1cae035337f285.tar.xz
AGESA: Call get_bus_conf() just once
Instead of calling get_bus_conf() three times from write_tables() and executing it once, just make one call before entering write_tables(). Change-Id: I818e37128cb0fb5eaded3c1e00b6b146c1267647 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: http://review.coreboot.org/6133 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Diffstat (limited to 'src/mainboard/hp')
-rw-r--r--src/mainboard/hp/pavilion_m6_1035dx/acpi_tables.c2
-rw-r--r--src/mainboard/hp/pavilion_m6_1035dx/get_bus_conf.c7
-rw-r--r--src/mainboard/hp/pavilion_m6_1035dx/irq_tables.c2
-rw-r--r--src/mainboard/hp/pavilion_m6_1035dx/mptable.c2
4 files changed, 0 insertions, 13 deletions
diff --git a/src/mainboard/hp/pavilion_m6_1035dx/acpi_tables.c b/src/mainboard/hp/pavilion_m6_1035dx/acpi_tables.c
index 01281ea2de..dbe550c634 100644
--- a/src/mainboard/hp/pavilion_m6_1035dx/acpi_tables.c
+++ b/src/mainboard/hp/pavilion_m6_1035dx/acpi_tables.c
@@ -131,8 +131,6 @@ unsigned long write_acpi_tables(unsigned long start)
acpi_header_t *ivrs;
acpi_hest_t *hest;
- get_bus_conf(); /* it will get sblk, pci1234, hcdn, and sbdn */
-
/* Align ACPI tables to 16 bytes */
start = ALIGN(start, 16);
current = start;
diff --git a/src/mainboard/hp/pavilion_m6_1035dx/get_bus_conf.c b/src/mainboard/hp/pavilion_m6_1035dx/get_bus_conf.c
index 34afc171cc..78193ebede 100644
--- a/src/mainboard/hp/pavilion_m6_1035dx/get_bus_conf.c
+++ b/src/mainboard/hp/pavilion_m6_1035dx/get_bus_conf.c
@@ -47,7 +47,6 @@ u32 pci1234x[] = {
u32 bus_type[256];
u32 sbdn_hudson;
-static u32 get_bus_conf_done = 0;
void get_bus_conf(void)
{
@@ -56,12 +55,6 @@ void get_bus_conf(void)
device_t dev;
int i, j;
- if (get_bus_conf_done == 1)
- return; /* do it only once */
-
- get_bus_conf_done = 1;
-
-
sbdn_hudson = 0;
memset(bus_hudson, 0, sizeof(bus_hudson));
diff --git a/src/mainboard/hp/pavilion_m6_1035dx/irq_tables.c b/src/mainboard/hp/pavilion_m6_1035dx/irq_tables.c
index 076edc58ff..1d99b4c0db 100644
--- a/src/mainboard/hp/pavilion_m6_1035dx/irq_tables.c
+++ b/src/mainboard/hp/pavilion_m6_1035dx/irq_tables.c
@@ -56,8 +56,6 @@ unsigned long write_pirq_routing_table(unsigned long addr)
u8 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 */
-
/* Align the table to be 16 byte aligned. */
addr += 15;
addr &= ~15;
diff --git a/src/mainboard/hp/pavilion_m6_1035dx/mptable.c b/src/mainboard/hp/pavilion_m6_1035dx/mptable.c
index 2d7646a5fd..d90a453f1c 100644
--- a/src/mainboard/hp/pavilion_m6_1035dx/mptable.c
+++ b/src/mainboard/hp/pavilion_m6_1035dx/mptable.c
@@ -84,8 +84,6 @@ static void *smp_write_config_table(void *v)
smp_write_processors(mc);
- get_bus_conf();
-
//mptable_write_buses(mc, NULL, &bus_isa);
my_smp_write_bus(mc, 0, "PCI ");
my_smp_write_bus(mc, 1, "PCI ");