Author: stepan
Date: Wed Apr 14 19:18:34 2010
New Revision: 5434
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5434

Log:
fix a case where the fam10 code would overwrite parts of a struct.

Signed-off-by: Stefan Reinauer <[email protected]>
Acked-by: Stefan Reinauer <[email protected]>

Modified:
   trunk/src/northbridge/amd/amdmct/mct/mctardk3.c
   trunk/src/northbridge/amd/amdmct/mct/mctardk4.c

Modified: trunk/src/northbridge/amd/amdmct/mct/mctardk3.c
==============================================================================
--- trunk/src/northbridge/amd/amdmct/mct/mctardk3.c     Wed Apr 14 19:11:47 
2010        (r5433)
+++ trunk/src/northbridge/amd/amdmct/mct/mctardk3.c     Wed Apr 14 19:18:34 
2010        (r5434)
@@ -18,7 +18,7 @@
  */
 
 
-static void Get_ChannelPS_Cfg0_D( u8 MAAdimms, u8 Speed, u8 MAAload,
+static void Get_ChannelPS_Cfg0_D(u8 MAAdimms, u8 Speed, u8 MAAload,
                                u8 DATAAload, u32 *AddrTmgCTL, u32 *ODC_CTL);
 
 

Modified: trunk/src/northbridge/amd/amdmct/mct/mctardk4.c
==============================================================================
--- trunk/src/northbridge/amd/amdmct/mct/mctardk4.c     Wed Apr 14 19:11:47 
2010        (r5433)
+++ trunk/src/northbridge/amd/amdmct/mct/mctardk4.c     Wed Apr 14 19:18:34 
2010        (r5434)
@@ -18,9 +18,9 @@
  */
 
 
-static void Get_ChannelPS_Cfg0_D( u8 MAAdimms, u8 Speed, u8 MAAload,
+static void Get_ChannelPS_Cfg0_D(u8 MAAdimms, u8 Speed, u8 MAAload,
                                u8 DATAAload, u32 *AddrTmgCTL, u32 *ODC_CTL,
-                               u32 *CMDmode);
+                               u8 *CMDmode);
 
 
 void mctGet_PS_Cfg_D(struct MCTStatStruc *pMCTstat,
@@ -102,7 +102,7 @@
 
 static void Get_ChannelPS_Cfg0_D( u8 MAAdimms, u8 Speed, u8 MAAload,
                                u8 DATAAload, u32 *AddrTmgCTL, u32 *ODC_CTL,
-                               u32 *CMDmode)
+                               u8 *CMDmode)
 {
        u8 *p;
 

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

Reply via email to