Kyösti Mälkki ([email protected]) just uploaded a new patch set to 
gerrit, which you can find at http://review.coreboot.org/640

-gerrit

commit f0d584a489fa69bd8c49154132ec4742d7a74415
Author: Kyösti Mälkki <[email protected]>
Date:   Thu Jun 28 12:16:12 2012 +0300

    Intel cpus: delete dead CAR code and whitespace fixes
    
    A diff from model_6fx to model_106cx suggests there is little
    CORE2 specific code that was once considered useful to have.
    In its current status however, sockets supporting model_6fx use
    model_6ex CAR init, so that specific code is actually
    never used.
    
    Deletes file:
        model_6fx/cache_as_ram.inc
    
    Change-Id: I6c0204446fa98207e31f91895e1cf30fde42382c
    Signed-off-by: Kyösti Mälkki <[email protected]>
---
 src/cpu/intel/model_106cx/cache_as_ram.inc |   14 +--
 src/cpu/intel/model_6ex/cache_as_ram.inc   |   16 +--
 src/cpu/intel/model_6fx/cache_as_ram.inc   |  272 ----------------------------
 3 files changed, 3 insertions(+), 299 deletions(-)

diff --git a/src/cpu/intel/model_106cx/cache_as_ram.inc 
b/src/cpu/intel/model_106cx/cache_as_ram.inc
index 74d0a99..c626cd8 100644
--- a/src/cpu/intel/model_106cx/cache_as_ram.inc
+++ b/src/cpu/intel/model_106cx/cache_as_ram.inc
@@ -83,7 +83,7 @@ clear_mtrrs:
        wrmsr
 
        /* Enable cache (CR0.CD = 0, CR0.NW = 0). */
-        movl   %cr0, %eax
+       movl    %cr0, %eax
        andl    $(~(CR0_CacheDisable | CR0_NoWriteThrough)), %eax
        invd
        movl    %eax, %cr0
@@ -164,18 +164,6 @@ clear_mtrrs:
        post_code(0x31)
 
        invd
-#if 0
-       xorl    %eax, %eax
-       xorl    %edx, %edx
-       movl    $MTRRphysBase_MSR(0), %ecx
-       wrmsr
-       movl    $MTRRphysMask_MSR(0), %ecx
-       wrmsr
-       movl    $MTRRphysBase_MSR(1), %ecx
-       wrmsr
-       movl    $MTRRphysMask_MSR(1), %ecx
-       wrmsr
-#endif
 
        post_code(0x33)
 
diff --git a/src/cpu/intel/model_6ex/cache_as_ram.inc 
b/src/cpu/intel/model_6ex/cache_as_ram.inc
index 6f13cd9..bdefb21 100644
--- a/src/cpu/intel/model_6ex/cache_as_ram.inc
+++ b/src/cpu/intel/model_6ex/cache_as_ram.inc
@@ -83,7 +83,7 @@ clear_mtrrs:
        wrmsr
 
        /* Enable cache (CR0.CD = 0, CR0.NW = 0). */
-        movl   %cr0, %eax
+       movl    %cr0, %eax
        andl    $(~(CR0_CacheDisable | CR0_NoWriteThrough)), %eax
        invd
        movl    %eax, %cr0
@@ -164,18 +164,6 @@ clear_mtrrs:
        post_code(0x31)
 
        invd
-#if 0
-       xorl    %eax, %eax
-       xorl    %edx, %edx
-       movl    $MTRRphysBase_MSR(0), %ecx
-       wrmsr
-       movl    $MTRRphysMask_MSR(0), %ecx
-       wrmsr
-       movl    $MTRRphysBase_MSR(1), %ecx
-       wrmsr
-       movl    $MTRRphysMask_MSR(1), %ecx
-       wrmsr
-#endif
 
        post_code(0x33)
 
@@ -193,7 +181,7 @@ clear_mtrrs:
 
        post_code(0x38)
 
-       /* Enable Write Back and Speculative Reads for the first 1MB. */
+       /* Enable Write Back and Speculative Reads for low RAM. */
        movl    $MTRRphysBase_MSR(0), %ecx
        movl    $(0x00000000 | MTRR_TYPE_WRBACK), %eax
        xorl    %edx, %edx
diff --git a/src/cpu/intel/model_6fx/cache_as_ram.inc 
b/src/cpu/intel/model_6fx/cache_as_ram.inc
deleted file mode 100644
index d8d25a0..0000000
--- a/src/cpu/intel/model_6fx/cache_as_ram.inc
+++ /dev/null
@@ -1,272 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2000,2007 Ronald G. Minnich <[email protected]>
- * Copyright (C) 2007-2008 coresystems GmbH
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
- */
-
-#include <cpu/x86/stack.h>
-#include <cpu/x86/mtrr.h>
-#include <cpu/x86/cache.h>
-#include <cpu/x86/post_code.h>
-
-#define CPU_PHYSMASK_HI  (1 << (CONFIG_CPU_ADDR_BITS - 32) - 1)
-
-#define CACHE_AS_RAM_SIZE CONFIG_DCACHE_RAM_SIZE
-#define CACHE_AS_RAM_BASE CONFIG_DCACHE_RAM_BASE
-
-       /* Save the BIST result. */
-       movl    %eax, %ebp
-
-cache_as_ram:
-       post_code(0x20)
-
-       /* Send INIT IPI to all excluding ourself. */
-       movl    $0x000C4500, %eax
-       movl    $0xFEE00300, %esi
-       movl    %eax, (%esi)
-
-       /* Disable prefetchers */
-       movl    $0x01a0, %ecx
-       rdmsr
-       orl     $((1 << 9) | (1 << 19)), %eax
-       orl     $((1 << 5) | (1 << 7)), %edx
-       wrmsr
-
-       /* Zero out all fixed range and variable range MTRRs. */
-       movl    $mtrr_table, %esi
-       movl    $((mtrr_table_end - mtrr_table) / 2), %edi
-       xorl    %eax, %eax
-       xorl    %edx, %edx
-clear_mtrrs:
-       movw    (%esi), %bx
-       movzx   %bx, %ecx
-       wrmsr
-       add     $2, %esi
-       dec     %edi
-       jnz     clear_mtrrs
-
-       /* Configure the default memory type to uncacheable. */
-       movl    $MTRRdefType_MSR, %ecx
-       rdmsr
-       andl    $(~0x00000cff), %eax
-       wrmsr
-
-       /* Set Cache-as-RAM base address. */
-       movl    $(MTRRphysBase_MSR(0)), %ecx
-       movl    $(CACHE_AS_RAM_BASE | MTRR_TYPE_WRBACK), %eax
-       xorl    %edx, %edx
-       wrmsr
-
-       /* Set Cache-as-RAM mask. */
-       movl    $(MTRRphysMask_MSR(0)), %ecx
-       movl    $(~(CACHE_AS_RAM_SIZE - 1) | MTRRphysMaskValid), %eax
-       movl    $CPU_PHYSMASK_HI, %edx
-       wrmsr
-
-       /* Enable MTRR. */
-       movl    $MTRRdefType_MSR, %ecx
-       rdmsr
-       orl     $MTRRdefTypeEn, %eax
-       wrmsr
-
-       /* Enable L2 cache. */
-       movl    $0x11e, %ecx
-       rdmsr
-       orl     $(1 << 8), %eax
-       wrmsr
-
-       /* Enable cache (CR0.CD = 0, CR0.NW = 0). */
-        movl   %cr0, %eax
-       andl    $(~(CR0_CacheDisable | CR0_NoWriteThrough)), %eax
-       invd
-       movl    %eax, %cr0
-
-       /* Clear the cache memory reagion. */
-       movl    $CACHE_AS_RAM_BASE, %esi
-       movl    %esi, %edi
-       movl    $(CACHE_AS_RAM_SIZE / 4), %ecx
-       // movl $0x23322332, %eax
-       xorl    %eax, %eax
-       rep     stosl
-
-       /* Enable Cache-as-RAM mode by disabling cache. */
-       movl    %cr0, %eax
-       orl     $CR0_CacheDisable, %eax
-       movl    %eax, %cr0
-
-#if CONFIG_XIP_ROM_SIZE
-       /* Enable cache for our code in Flash because we do XIP here */
-       movl    $MTRRphysBase_MSR(1), %ecx
-       xorl    %edx, %edx
-       /*
-        * IMPORTANT: The following calculation _must_ be done at runtime. See
-        * http://www.coreboot.org/pipermail/coreboot/2010-October/060855.html
-        */
-       movl    $copy_and_run, %eax
-       andl    $(~(CONFIG_XIP_ROM_SIZE - 1)), %eax
-       orl     $MTRR_TYPE_WRBACK, %eax
-       wrmsr
-
-       movl    $MTRRphysMask_MSR(1), %ecx
-       movl    $CPU_PHYSMASK_HI, %edx
-       movl    $(~(CONFIG_XIP_ROM_SIZE - 1) | MTRRphysMaskValid), %eax
-       wrmsr
-#endif /* CONFIG_XIP_ROM_SIZE */
-
-       /* Enable cache. */
-       movl    %cr0, %eax
-       andl    $(~(CR0_CacheDisable | CR0_NoWriteThrough)), %eax
-       movl    %eax, %cr0
-
-       /* Set up the stack pointer. */
-#if CONFIG_USBDEBUG
-       /* Leave some space for the struct ehci_debug_info. */
-       movl    $(CACHE_AS_RAM_BASE + CACHE_AS_RAM_SIZE - 4 - 128), %eax
-#else
-       movl    $(CACHE_AS_RAM_BASE + CACHE_AS_RAM_SIZE - 4), %eax
-#endif
-       movl    %eax, %esp
-
-       /* Restore the BIST result. */
-       movl    %ebp, %eax
-       movl    %esp, %ebp
-       pushl   %eax
-
-       post_code(0x23)
-
-       /* Call romstage.c main function. */
-       call    main
-
-       post_code(0x2f)
-
-       post_code(0x30)
-
-       /* Disable cache. */
-       movl    %cr0, %eax
-       orl     $CR0_CacheDisable, %eax
-       movl    %eax, %cr0
-
-       post_code(0x31)
-
-       /* Disable MTRR. */
-       movl    $MTRRdefType_MSR, %ecx
-       rdmsr
-       andl    $(~MTRRdefTypeEn), %eax
-       wrmsr
-
-       post_code(0x31)
-
-       invd
-#if 0
-       xorl    %eax, %eax
-       xorl    %edx, %edx
-       movl    $MTRRphysBase_MSR(0), %ecx
-       wrmsr
-       movl    $MTRRphysMask_MSR(0), %ecx
-       wrmsr
-       movl    $MTRRphysBase_MSR(1), %ecx
-       wrmsr
-       movl    $MTRRphysMask_MSR(1), %ecx
-       wrmsr
-#endif
-
-       post_code(0x33)
-
-       /* Enable cache. */
-       movl    %cr0, %eax
-       andl    $~(CR0_CacheDisable | CR0_NoWriteThrough), %eax
-       movl    %eax, %cr0
-
-       post_code(0x36)
-
-       /* Disable cache. */
-       movl    %cr0, %eax
-       orl     $CR0_CacheDisable, %eax
-       movl    %eax, %cr0
-
-       post_code(0x38)
-
-       /* Enable Write Back and Speculative Reads for the first 1MB. */
-       movl    $MTRRphysBase_MSR(0), %ecx
-       movl    $(0x00000000 | MTRR_TYPE_WRBACK), %eax
-       xorl    %edx, %edx
-       wrmsr
-       movl    $MTRRphysMask_MSR(0), %ecx
-       movl    $(~(1024 * 1024 - 1) | MTRRphysMaskValid), %eax
-       movl    $CPU_PHYSMASK_HI, %edx
-       wrmsr
-
-       post_code(0x39)
-
-       /* And enable cache again after setting MTRRs. */
-       movl    %cr0, %eax
-       andl    $~(CR0_CacheDisable | CR0_NoWriteThrough), %eax
-       movl    %eax, %cr0
-
-       post_code(0x3a)
-
-       /* Enable MTRR. */
-       movl    $MTRRdefType_MSR, %ecx
-       rdmsr
-       orl     $MTRRdefTypeEn, %eax
-       wrmsr
-
-       post_code(0x3b)
-
-       /* Enable prefetchers */
-       movl    $0x01a0, %ecx
-       rdmsr
-       andl    $~((1 << 9) | (1 << 19)), %eax
-       andl    $~((1 << 5) | (1 << 7)), %edx
-       wrmsr
-
-       /* Invalidate the cache again. */
-       invd
-
-       post_code(0x3c)
-
-       /* Clear boot_complete flag. */
-       xorl    %ebp, %ebp
-__main:
-       post_code(POST_PREPARE_RAMSTAGE)
-       cld                     /* Clear direction flag. */
-
-       movl    %ebp, %esi
-
-       movl    $ROMSTAGE_STACK, %esp
-       movl    %esp, %ebp
-       pushl   %esi
-       call    copy_and_run
-
-.Lhlt:
-       post_code(POST_DEAD_CODE)
-       hlt
-       jmp     .Lhlt
-
-mtrr_table:
-       /* Fixed MTRRs */
-       .word 0x250, 0x258, 0x259
-       .word 0x268, 0x269, 0x26A
-       .word 0x26B, 0x26C, 0x26D
-       .word 0x26E, 0x26F
-       /* Variable MTRRs */
-       .word 0x200, 0x201, 0x202, 0x203
-       .word 0x204, 0x205, 0x206, 0x207
-       .word 0x208, 0x209, 0x20A, 0x20B
-       .word 0x20C, 0x20D, 0x20E, 0x20F
-mtrr_table_end:
-

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to