[coreboot-gerrit] New patch to review for coreboot: glados: Remove functions from acpi_tables.c

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Sep 7 18:43:29 CET 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11579

-gerrit

commit 9edc04998c3b376d337b4d3285ef272d7ad5fc34
Author: Duncan Laurie <dlaurie at chromium.org>
Date:   Fri Sep 4 14:15:36 2015 -0700

    glados: Remove functions from acpi_tables.c
    
    Remove the acpi_tables.c functions so these functions can move to
    SOC init code.  The file itself is included by x86/arch code and
    must exist for the build to succeed.
    
    BUG=chrome-os-partner:40635
    BRANCH=none
    TEST=emerge-glados coreboot
    
    Change-Id: I18e6a0be5eac053598b613b30b622c4963417919
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: af04eb112adf58578c8d2c9d3d182d4c2024abb2
    Original-Change-Id: Ibe026d493c25d771357ea39e4b956629fbb799ac
    Original-Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/297758
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 src/mainboard/google/glados/acpi_tables.c | 22 ----------------------
 1 file changed, 22 deletions(-)

diff --git a/src/mainboard/google/glados/acpi_tables.c b/src/mainboard/google/glados/acpi_tables.c
index 6da1e1d..6b6e038 100644
--- a/src/mainboard/google/glados/acpi_tables.c
+++ b/src/mainboard/google/glados/acpi_tables.c
@@ -17,25 +17,3 @@
  * along with this program; if not, write to the Free Software
  * Foundation, Inc.
  */
-
-#include <arch/acpi.h>
-#include <arch/ioapic.h>
-#include <soc/acpi.h>
-#include <soc/nvs.h>
-
-void acpi_create_gnvs(global_nvs_t *gnvs)
-{
-	acpi_init_gnvs(gnvs);
-}
-
-unsigned long acpi_fill_madt(unsigned long current)
-{
-	/* Local APICs */
-	current = acpi_create_madt_lapics(current);
-
-	/* IOAPIC */
-	current += acpi_create_madt_ioapic((acpi_madt_ioapic_t *) current,
-				2, IO_APIC_ADDR, 0);
-
-	return acpi_madt_irq_overrides(current);
-}



More information about the coreboot-gerrit mailing list