[coreboot-gerrit] Patch merged into coreboot/master: intel raminit: fix timB high adjust calculation

gerrit at coreboot.org gerrit at coreboot.org
Mon Jul 13 01:59:48 CEST 2015


the following patch was just integrated into master:
commit 6f7ce9b2172297b56c4b255a1c322b576a4583b8
Author: Patrick Rudolph <siro at das-labor.org>
Date:   Sun Jul 12 12:17:21 2015 +0200

    intel raminit: fix timB high adjust calculation
    
    Issue observed:
    Any memory DIMM placed in channel0 slots stops at "c320c discovery failed".
    The same memory DIMM works when placed in channel1 slots.
    
    Test system:
     * Intel Pentium CPU G2130
     * Gigabyte GA-B75M-D3H
     * DIMMs:
      * elixir 1GB 1Rx8 PC3-10600U M2Y1G64CB88A5N
      * crucial 2GB 256Mx64 CT2566aBA160BJ
      * corsair 8GB CMZ16GX3M2A1866C9
    
    Problem description:
    In case of good timmings (all bits are set) an offset of 3*64 was applied.
    The following test (c320c discovery) failed only on those byte-lanes.
    
    Problem solution:
    Don't modify timB in case of good timings measured.
    
    Final testing result:
    The system boots with every DIMM placed in channel 0 slots.
    
    Change-Id: Iea426ea4470640ce254f16e958a395644ff1a55c
    Signed-off-by: Patrick Rudolph <siro at das-labor.org>
    Reviewed-on: http://review.coreboot.org/10889
    Tested-by: build bot (Jenkins)
    Reviewed-by: Vladimir Serbinenko <phcoder at gmail.com>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Nicolas Reinecke <nr at das-labor.org>


See http://review.coreboot.org/10889 for details.

-gerrit



More information about the coreboot-gerrit mailing list