Patchwork :The Errata350 is fixed as the way it should be

login
register
about
Submitter Bao, Zheng
Date 2009-08-19 02:39:34
Message ID <DD1CC71B621B004FA76856E5129D6B1702F5F88D@sbjgexmb1.amd.com>
Download mbox | patch
Permalink /patch/139/
State Accepted
Headers show

Comments

Bao, Zheng - 2009-08-19 02:39:34
The Errata350 is "Write 0000_8000h to register F2x[1, 0]9C_xD080F0C.",
instead of
F2x[1, 0]9C_x0C. It is a obvious bug. Some typos are also fixed.


 	/* 4. Write 0000_0000h to register F2x[1, 0]9C_xD080F0C. */
-	Set_NB32_index_wait(u32DctDev, 0x098, 0x0c, 0x00000000);
+	Set_NB32_index_wait(u32DctDev, 0x098, 0xD080F0C, 0x00000000);
 
 	if(!pDCTstat->GangedMode) {
-		print_t("vErrara350: step 4b\n");
-		Set_NB32_index_wait(u32DctDev, 0x198, 0x0c, 0x00000000);
+		print_t("vErrata350: step 4b\n");
+		Set_NB32_index_wait(u32DctDev, 0x198, 0xD080F0C,
0x00000000);
 	}
 
-	print_t("vErrara350: step 5\n");
+	print_t("vErrata350: step 5\n");
 	/* 5. Wait at least 2 microseconds. */
 	coreDelay();
Carl-Daniel Hailfinger - 2009-08-19 07:03:33
On 19.08.2009 04:39, Bao, Zheng wrote:
> The Errata350 is "Write 0000_8000h to register F2x[1, 0]9C_xD080F0C.",
> instead of
> F2x[1, 0]9C_x0C. It is a obvious bug. Some typos are also fixed.
>   

Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>

Regards,
Carl-Daniel
Bao, Zheng - 2009-08-19 07:08:38
Committed to r4553.

-----Original Message-----
From: Carl-Daniel Hailfinger [mailto:c-d.hailfinger.devel.2006@gmx.net] 
Sent: Wednesday, August 19, 2009 3:04 PM
To: Bao, Zheng
Cc: Coreboot
Subject: Re: [coreboot] [PATCH]:The Errata350 is fixed as the way it
should be

On 19.08.2009 04:39, Bao, Zheng wrote:
> The Errata350 is "Write 0000_8000h to register F2x[1, 0]9C_xD080F0C.",
> instead of
> F2x[1, 0]9C_x0C. It is a obvious bug. Some typos are also fixed.
>   

Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>

Regards,
Carl-Daniel

Patch

Index: src/northbridge/amd/amdmct/wrappers/mcti_d.c
===================================================================
--- src/northbridge/amd/amdmct/wrappers/mcti_d.c	(revision 4521)
+++ src/northbridge/amd/amdmct/wrappers/mcti_d.c	(working copy)
@@ -347,45 +347,45 @@ 
 			u32Addr = mct_GetRcvrSysAddr_D(pMCTstat,
pDCTstat, u8Channel, u8Receiver, &u8Valid);
 
 			if(!u8Valid) {	/* Address not supported on
current CS */
-				print_t("vErrara350: Address not
supported on current CS\n");
+				print_t("vErrata350: Address not
supported on current CS\n");
 				continue;
 			}
-			print_t("vErrara350: dummy read \n");
+			print_t("vErrata350: dummy read \n");
 			read32_fs(u32Addr);
 		}
 	}
 
-	print_t("vErrara350: step 2a\n");
+	print_t("vErrata350: step 2a\n");
 
 	/* 2. Write 0000_8000h to register F2x[1, 0]9C_xD080F0C. */
 	u32DctDev = pDCTstat->dev_dct;
-	Set_NB32_index_wait(u32DctDev, 0x098, 0x0c, 0x00008000);
+	Set_NB32_index_wait(u32DctDev, 0x098, 0xD080F0C, 0x00008000);
 	/*                                                ^--- value
-	                                        ^---F2x[1, 0]9C_x0C DRAM
Phy Miscellaneous Register
+	                                        ^---F2x[1,
0]9C_x0D080F0C, No description in BKDG.
 	                                 ^----F2x[1, 0]98 DRAM
Controller Additional Data Offset Register */
 
 	if(!pDCTstat->GangedMode) {
-		print_t("vErrara350: step 2b\n");
-		Set_NB32_index_wait(u32DctDev, 0x198, 0x0c, 0x00008000);
+		print_t("vErrata350: step 2b\n");
+		Set_NB32_index_wait(u32DctDev, 0x198, 0xD080F0C,
0x00008000);
 		/*                                                ^---
value
-		                                        ^---F2x[1,
0]9C_x0C DRAM Phy Miscellaneous Register
+		                                        ^---F2x[1,
0]9C_x0D080F0C, No description in BKDG
 		                                ^----F2x[1, 0]98 DRAM
Controller Additional Data Offset Register */
 	}
 
-	print_t("vErrara350: step 3\n");
+	print_t("vErrata350: step 3\n");
 	/* 3. Wait at least 300 nanoseconds. */
 	coreDelay();
 
-	print_t("vErrara350: step 4\n");
+	print_t("vErrata350: step 4\n");