[coreboot-gerrit] Patch set updated for coreboot: 1dcef53 northbridge/intel/i945/raminit.c: Remove set but unused variable `reg16`

Ronald G. Minnich (rminnich@gmail.com) gerrit at coreboot.org
Tue Oct 22 02:21:01 CEST 2013


Ronald G. Minnich (rminnich at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3984

-gerrit

commit 1dcef53d9db49a589d564ea1b18f091c181ed395
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Mon Oct 21 21:22:09 2013 +0200

    northbridge/intel/i945/raminit.c: Remove set but unused variable `reg16`
    
    
    Change-Id: Id4a758644a7b799e7662113c07d395e053525934
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/northbridge/intel/i945/raminit.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/northbridge/intel/i945/raminit.c b/src/northbridge/intel/i945/raminit.c
index b50f1d8..512d8e9 100644
--- a/src/northbridge/intel/i945/raminit.c
+++ b/src/northbridge/intel/i945/raminit.c
@@ -1952,8 +1952,6 @@ static void sdram_set_channel_mode(struct sys_info *sysinfo)
 
 static void sdram_program_pll_settings(struct sys_info *sysinfo)
 {
-	volatile u16 reg16;
-
 	MCHBAR32(PLLMON) = 0x80800000;
 
 	sysinfo->fsb_frequency = fsbclk();
@@ -1970,7 +1968,7 @@ static void sdram_program_pll_settings(struct sys_info *sysinfo)
 
 	MCHBAR16(CPCTL) &= ~(1 << 11);
 
-	reg16 = MCHBAR16(CPCTL); /* Read back register to activate settings */
+	MCHBAR16(CPCTL); /* Read back register to activate settings */
 }
 
 static void sdram_program_graphics_frequency(struct sys_info *sysinfo)



More information about the coreboot-gerrit mailing list