[coreboot] [PATCH]: AMD FAM10 MCT: Channel B only.
Bao, Zheng
Zheng.Bao at amd.com
Tue Aug 25 06:15:33 CEST 2009
Committed, r4569.
And the extra semicolon is removed.
-----Original Message-----
From: Stefan Reinauer [mailto:stepan at coresystems.de]
Sent: Monday, August 24, 2009 6:01 PM
To: Bao, Zheng
Cc: Coreboot
Subject: Re: [coreboot] [PATCH]: AMD FAM10 MCT: Channel B only.
On 8/24/09 5:41 AM, Bao, Zheng wrote:
> Without this patch, if we only got a DIMM in Channel B, memory can not
> be
> set up correctly. Now it can. Please test it.
>
> Moving "mct_AfterGetCLT(pMCTstat, pDCTstat, dct);" out of the "if" is
> the
> key point.
> Changing the Get_DIMMAddress_D(pDCTstat, i) to
> Get_DIMMAddress_D(pDCTstat, dct + i)
> doesnt seem to take any effect. But I believe this is what it should be.
>
> Zheng
>
> Signed-off-by: Zheng Bao <zheng.bao at amd.com>
>
Awesome finding!
Acked-by: Stefan Reinauer <stepan at coresystems.de>
> Index: src/northbridge/amd/amdmct/mct/mct_d.c
> ===================================================================
> --- src/northbridge/amd/amdmct/mct/mct_d.c (revision 4561)
> +++ src/northbridge/amd/amdmct/mct/mct_d.c (working copy)
> @@ -982,8 +982,8 @@
> if ( mctGet_NVbits(NV_MCTUSRTMGMODE) == 2)
> pDCTstat->Speed = mctGet_NVbits(NV_MemCkVal) +
> 1;
>
> - mct_AfterGetCLT(pMCTstat, pDCTstat, dct);
> }
> + mct_AfterGetCLT(pMCTstat, pDCTstat, dct);
>
> /* Gather all DIMM mini-max values for cycle timing data */
> Rows = 0;
> @@ -1001,7 +1001,7 @@
> for ( i = 0; i< MAX_DIMMS_SUPPORTED; i++) {
> LDIMM = i >> 1;
> if (pDCTstat->DIMMValid & (1 << i)) {
> - smbaddr = Get_DIMMAddress_D(pDCTstat, i);
> + smbaddr = Get_DIMMAddress_D(pDCTstat, dct + i);
> byte = mctRead_SPD(smbaddr, SPD_ROWSZ);
> if (Rows < byte)
> Rows = byte; /* keep track of largest
> row sz */
>
--
coresystems GmbH * Brahmsstr. 16 * D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 * Fax: +49 761 7664613
Email: info at coresystems.de * http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg * HRB 7656
Geschäftsführer: Stefan Reinauer * Ust-IdNr.: DE245674866
More information about the coreboot
mailing list