Sven Schnelle (sv...@stackframe.org) just uploaded a new patch set to gerrit, 
which you can find at http://review.coreboot.org/149

-gerrit

commit 071f7502bf104035e6c42d1b3f0281d5607bbeea
Author: Sven Schnelle <sv...@stackframe.org>
Date:   Wed Aug 10 22:47:20 2011 +0200

    X60: use new acpi code
    
    Change-Id: Id2d7e8fe9e8218eccaef6ad426f40209506a345a
    Signed-off-by: Sven Schnelle <sv...@stackframe.org>
---
 src/mainboard/lenovo/x60/acpi_tables.c |  193 +++-----------------------------
 src/mainboard/lenovo/x60/mainboard.c   |    4 +
 2 files changed, 19 insertions(+), 178 deletions(-)

diff --git a/src/mainboard/lenovo/x60/acpi_tables.c 
b/src/mainboard/lenovo/x60/acpi_tables.c
index 3742c20..af214fb 100644
--- a/src/mainboard/lenovo/x60/acpi_tables.c
+++ b/src/mainboard/lenovo/x60/acpi_tables.c
@@ -32,9 +32,6 @@
 #include "dmi.h"
 
 extern const unsigned char AmlCode[];
-#if CONFIG_HAVE_ACPI_SLIC
-unsigned long acpi_create_slic(unsigned long current);
-#endif
 
 #include "southbridge/intel/i82801gx/nvs.h"
 static void acpi_create_gnvs(global_nvs_t *gnvs)
@@ -56,38 +53,6 @@ static void acpi_create_gnvs(global_nvs_t *gnvs)
        gnvs->did[4] = 0x00000005;
 }
 
-static void acpi_create_intel_hpet(acpi_hpet_t * hpet)
-{
-#define HPET_ADDR  0xfed00000ULL
-       acpi_header_t *header = &(hpet->header);
-       acpi_addr_t *addr = &(hpet->addr);
-
-       memset((void *) hpet, 0, sizeof(acpi_hpet_t));
-
-       /* fill out header fields */
-       memcpy(header->signature, "HPET", 4);
-       memcpy(header->oem_id, OEM_ID, 6);
-       memcpy(header->oem_table_id, "COREBOOT", 8);
-       memcpy(header->asl_compiler_id, ASLC, 4);
-
-       header->length = sizeof(acpi_hpet_t);
-       header->revision = 1;
-
-       /* fill out HPET address */
-       addr->space_id = 0;     /* Memory */
-       addr->bit_width = 64;
-       addr->bit_offset = 0;
-       addr->addrl = HPET_ADDR & 0xffffffff;
-       addr->addrh = HPET_ADDR >> 32;
-
-       hpet->id = 0x8086a201;  /* Intel */
-       hpet->number = 0x00;
-       hpet->min_tick = 0x0080;
-
-       header->checksum =
-           acpi_checksum((void *) hpet, sizeof(acpi_hpet_t));
-}
-
 unsigned long acpi_fill_madt(unsigned long current)
 {
        /* Local APICs */
@@ -136,129 +101,31 @@ unsigned long acpi_fill_srat(unsigned long current)
 
 void smm_setup_structures(void *gnvs, void *tcg, void *smi1);
 
-#define ALIGN_CURRENT current = ((current + 0x0f) & -0x10)
-unsigned long write_acpi_tables(unsigned long start)
+
+void acpi_patch_dsdt(acpi_header_t *dsdt, unsigned long *current)
 {
-       unsigned long current;
        int i;
-       acpi_rsdp_t *rsdp;
-       acpi_rsdt_t *rsdt;
-       acpi_xsdt_t *xsdt;
-       acpi_hpet_t *hpet;
-       acpi_madt_t *madt;
-       acpi_mcfg_t *mcfg;
-       acpi_fadt_t *fadt;
-       acpi_facs_t *facs;
-#if CONFIG_HAVE_ACPI_SLIC
-       acpi_header_t *slic;
-#endif
-       acpi_header_t *ssdt;
-       acpi_header_t *dsdt;
        void *gnvs;
 
-       current = start;
-
-       /* Align ACPI tables to 16byte */
-       ALIGN_CURRENT;
-
-       printk(BIOS_INFO, "ACPI: Writing ACPI tables at %lx.\n", start);
-
-       /* We need at least an RSDP and an RSDT Table */
-       rsdp = (acpi_rsdp_t *) current;
-       current += sizeof(acpi_rsdp_t);
-       ALIGN_CURRENT;
-       rsdt = (acpi_rsdt_t *) current;
-       current += sizeof(acpi_rsdt_t);
-       ALIGN_CURRENT;
-       xsdt = (acpi_xsdt_t *) current;
-       current += sizeof(acpi_xsdt_t);
-       ALIGN_CURRENT;
-
-       /* clear all table memory */
-       memset((void *) start, 0, current - start);
-
-       acpi_write_rsdp(rsdp, rsdt, xsdt);
-       acpi_write_rsdt(rsdt);
-       acpi_write_xsdt(xsdt);
-
-       /*
-        * We explicitly add these tables later on:
-        */
-       printk(BIOS_DEBUG, "ACPI:    * HPET\n");
-
-       hpet = (acpi_hpet_t *) current;
-       current += sizeof(acpi_hpet_t);
-       ALIGN_CURRENT;
-       acpi_create_intel_hpet(hpet);
-       acpi_add_table(rsdp, hpet);
-
-       /* If we want to use HPET Timers Linux wants an MADT */
-       printk(BIOS_DEBUG, "ACPI:    * MADT\n");
-
-       madt = (acpi_madt_t *) current;
-       acpi_create_madt(madt);
-       current += madt->header.length;
-       ALIGN_CURRENT;
-       acpi_add_table(rsdp, madt);
-
-       printk(BIOS_DEBUG, "ACPI:    * MCFG\n");
-       mcfg = (acpi_mcfg_t *) current;
-       acpi_create_mcfg(mcfg);
-       current += mcfg->header.length;
-       ALIGN_CURRENT;
-       acpi_add_table(rsdp, mcfg);
-
-       printk(BIOS_DEBUG, "ACPI:     * FACS\n");
-       facs = (acpi_facs_t *) current;
-       current += sizeof(acpi_facs_t);
-       ALIGN_CURRENT;
-       acpi_create_facs(facs);
-
-       dsdt = (acpi_header_t *) current;
-       memcpy(dsdt, &AmlCode, sizeof(acpi_header_t));
-       current += dsdt->length;
-       memcpy(dsdt, &AmlCode, dsdt->length);
-
-       /* Fix up global NVS region for SMI handler. The GNVS region lives
-        * in the (high) table area. The low memory map looks like this:
-        *
-        * 0x00000000 - 0x000003ff      Real Mode IVT
-        * 0x00000020 - 0x0000019c      Low MP Table (XXX conflict?)
-        * 0x00000400 - 0x000004ff      BDA (somewhat unused)
-        * 0x00000500 - 0x0000052f      Moved GDT
-        * 0x00000530 - 0x00000b64      coreboot table
-        * 0x0007c000 - 0x0007dfff      OS boot sector (unused?)
-        * 0x0007e000 - 0x0007ffff      free to use (so no good for acpi+smi)
-        * 0x00080000 - 0x0009fbff      usable ram
-        * 0x0009fc00 - 0x0009ffff      EBDA (unused?)
-        * 0x000a0000 - 0x000bffff      VGA memory
-        * 0x000c0000 - 0x000cffff      VGA option rom
-        * 0x000d0000 - 0x000dffff      free for other option roms?
-        * 0x000e0000 - 0x000fffff      SeaBIOS? (conflict with low tables:)
-        * 0x000f0000 - 0x000f03ff      PIRQ table
-        * 0x000f0400 - 0x000f66??      ACPI tables
-        * 0x000f66?? - 0x000f????      DMI tables
-        */
-
-       ALIGN_CURRENT;
+       *current = ALIGN(*current, 64);
 
        /* Pack GNVS into the ACPI table area */
        for (i=0; i < dsdt->length; i++) {
                if (*(u32*)(((u32)dsdt) + i) == 0xC0DEBABE) {
-                       printk(BIOS_DEBUG, "ACPI: Patching up global NVS in 
DSDT at offset 0x%04x -> 0x%08x\n", i, (u32)current);
-                       *(u32*)(((u32)dsdt) + i) = current; // 0x92 bytes
+                       printk(BIOS_DEBUG, "ACPI: Patching up global NVS in 
DSDT at offset 0x%04x -> 0x%08x\n", i, (u32)*current);
+                       *(u32*)(((u32)dsdt) + i) = *current; // 0x92 bytes
                        break;
                }
        }
 
        /* And fill it */
-       acpi_create_gnvs((global_nvs_t *)current);
+       acpi_create_gnvs((global_nvs_t *)*current);
 
        /* Keep pointer around */
-       gnvs = (void *)current;
+       gnvs = (void *)*current;
 
-       current += 0x100;
-       ALIGN_CURRENT;
+       *current += 0x100;
+       *current = ALIGN(*current, 64);
 
        /* And tell SMI about it */
        smm_setup_structures(gnvs, NULL, NULL);
@@ -266,48 +133,18 @@ unsigned long write_acpi_tables(unsigned long start)
        /* We patched up the DSDT, so we need to recalculate the checksum */
        dsdt->checksum = 0;
        dsdt->checksum = acpi_checksum((void *)dsdt, dsdt->length);
+}
 
-       printk(BIOS_DEBUG, "ACPI:     * DSDT @ %p Length %x\n", dsdt,
-                    dsdt->length);
-
-#if CONFIG_HAVE_ACPI_SLIC
-       printk(BIOS_DEBUG, "ACPI:     * SLIC\n");
-       slic = (acpi_header_t *)current;
-       current += acpi_create_slic(current);
-       ALIGN_CURRENT;
-       acpi_add_table(rsdp, slic);
-#endif
-
-       printk(BIOS_DEBUG, "ACPI:     * FADT\n");
-       fadt = (acpi_fadt_t *) current;
-       current += sizeof(acpi_fadt_t);
-       ALIGN_CURRENT;
-
-       acpi_create_fadt(fadt, facs, dsdt);
-       acpi_add_table(rsdp, fadt);
-
-       printk(BIOS_DEBUG, "ACPI:     * SSDT\n");
-       ssdt = (acpi_header_t *)current;
-       acpi_create_ssdt_generator(ssdt, "COREBOOT");
-       current += ssdt->length;
-       acpi_add_table(rsdp, ssdt);
-       ALIGN_CURRENT;
-
-       printk(BIOS_DEBUG, "current = %lx\n", current);
-
+void acpi_dmi_workaround(unsigned long *current)
+{
        printk(BIOS_DEBUG, "ACPI:     * DMI (Linux workaround)\n");
        memcpy((void *)0xfff80, dmi_table, DMI_TABLE_SIZE);
 #if CONFIG_WRITE_HIGH_TABLES == 1
-       memcpy((void *)current, dmi_table, DMI_TABLE_SIZE);
-       current += DMI_TABLE_SIZE;
-       ALIGN_CURRENT;
+       memcpy((void *)*current, dmi_table, DMI_TABLE_SIZE);
+       *current += DMI_TABLE_SIZE;
+       *current = ALIGN(*current, 64);
 #endif
-
        printk(BIOS_INFO, "ACPI: done.\n");
 
-       /* Enable Dummy DCC ON# for DVI */
-       printk(BIOS_DEBUG, "Laptop handling...\n");
-       outb(inb(0x60f) & ~(1 << 5), 0x60f);
-
        return current;
 }
diff --git a/src/mainboard/lenovo/x60/mainboard.c 
b/src/mainboard/lenovo/x60/mainboard.c
index bf3d8d3..53a2937 100644
--- a/src/mainboard/lenovo/x60/mainboard.c
+++ b/src/mainboard/lenovo/x60/mainboard.c
@@ -80,6 +80,10 @@ static void mainboard_enable(device_t dev)
                set_option("volume", &(u8[]){ 0x03 });
                set_option("cmos_defaults_loaded", &(u8[]){ 0x01 });
        }
+
+       /* Enable Dummy DCC ON# for DVI */
+       printk(BIOS_DEBUG, "Laptop handling...\n");
+       outb(inb(0x60f) & ~(1 << 5), 0x60f);
 }
 
 struct chip_operations mainboard_ops = {

-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to