[coreboot-gerrit] New patch to review for coreboot: 7e0db61 northbridge/intel/nehalem/raminit.c: Remove redundant function

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Wed Jan 7 16:07:12 CET 2015


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8166

-gerrit

commit 7e0db61b839bc684017e1e09a2d9ee7fec7e399a
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Thu Jan 8 01:55:22 2015 +1100

    northbridge/intel/nehalem/raminit.c: Remove redundant function
    
    Remove 'read_mchbar8_bypass()' as 'read_mchbar8()' does the same
    thing and 'read_mchbar8_bypass()' is unused.
    
    Change-Id: I0dc2325e6205c38b63e5200c408ece65e4f938dc
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/northbridge/intel/nehalem/raminit.c | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/src/northbridge/intel/nehalem/raminit.c b/src/northbridge/intel/nehalem/raminit.c
index 9ca98d5..85c82c9 100644
--- a/src/northbridge/intel/nehalem/raminit.c
+++ b/src/northbridge/intel/nehalem/raminit.c
@@ -146,11 +146,6 @@ static inline u8 read_mchbar8(u32 addr)
 	return MCHBAR8(addr);
 }
 
-static inline u8 read_mchbar8_bypass(u32 addr)
-{
-	return MCHBAR8(addr);
-}
-
 static void clflush(u32 addr)
 {
 	asm volatile ("clflush (%0)"::"r" (addr));



More information about the coreboot-gerrit mailing list