[coreboot-gerrit] Patch merged into coreboot/master: device/dram/ddr3: fix debug output

gerrit at coreboot.org gerrit at coreboot.org
Mon Mar 21 18:31:13 CET 2016


the following patch was just integrated into master:
commit 66a98ee9675d73dc4e5343782d42f80a09c2425e
Author: Patrick Rudolph <siro at das-labor.org>
Date:   Sun Mar 13 13:02:16 2016 +0100

    device/dram/ddr3: fix debug output
    
    Add missing punctuation and align output.
    No functionality is changed.
    
    Old logging output:
      Revision: 11
      Type    : b
      Key     : 2
      Banks   : 8
      Capacity: 4 Gb
      Supported voltages: 1.5V
      SDRAM width       : 8
      Bus extension     : 0 bits
      Bus width         : 64
      Optional features : DLL-Off_mode RZQ/7 RZQ/6
      Thermal features  : ASR ext_temp_range
      Thermal sensor    : no
      Standard SDRAM    : yes
      DIMM Rank1 Address bits mirrored!!!
      DIMM Reference card B
      DIMM Manufacturer ID cd04
      DIMM Part number F3-1866C9-8GSR
      XMP Profile 1
      Max DIMMs per channel: 4
      XMP Revision: 1.3
      Requested voltage: 1500 mV
    
    New logging output:
      Revision           : 11
      Type               : b
      Key                : 2
      Banks              : 8
      Capacity           : 4 Gb
      Supported voltages : 1.5V
      SDRAM width        : 8
      Bus extension      : 0 bits
      Bus width          : 64
      Optional features  : DLL-Off_mode RZQ/7 RZQ/6
      Thermal features   : ASR ext_temp_range
      Thermal sensor     : no
      Standard SDRAM     : yes
      Rank1 Address bits : mirrored
      DIMM Reference card: B
      Manufacturer ID    : cd04
      Part number        : F3-1866C9-8GSR
      XMP Profile        : 1
      Max DIMMs/channel  : 4
      XMP Revision       : 1.3
      Requested voltage  : 1500 mV
    
    Change-Id: Iee4d7a7c0e1070706fb60d7316fad49813963b51
    Signed-off-by: Patrick Rudolph <siro at das-labor.org>
    Reviewed-on: https://review.coreboot.org/14083
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Martin Roth <martinroth at google.com>


See https://review.coreboot.org/14083 for details.

-gerrit



More information about the coreboot-gerrit mailing list