[coreboot-gerrit] Patch merged into coreboot/master: 5a0fdb4 exynos5420: minor clean-up memory related stuff

gerrit at coreboot.org gerrit at coreboot.org
Tue Aug 12 22:20:25 CEST 2014


the following patch was just integrated into master:
commit 5a0fdb4565dc6baf89155268dd0f52e6885197d8
Author: David Hendricks <dhendrix at chromium.org>
Date:   Thu Aug 29 13:12:56 2013 -0700

    exynos5420: minor clean-up memory related stuff
    
    This cleans up a few minor things (mostly #defines) of the memory code
    for exynos5420, pit, and kirby. Specifically:
    
    - CONCONTROL.empty is read-only, so don't try to set it and also
      get rid of the unneeded DMC_CONCONTROL_EMPTY_ENABLE #define.
    - MEMBASECONFIG* overlaps members of the mem_timings struct and
      are mainboard-dependent anyway, so get rid of 'em.
    - DMC_MEMCONTROL_TP_DISABLE corresponds to a reserved bit. It may
      have been deprecated.
    - Same with TIMING* #defines.
    - Clarify DDR_MODE_* usage and use mem->mem_type when appropriate.
    
    Signed-off-by: David Hendricks <dhendrix at chromium.org>
    
    Change-Id: Ideb21efcc97b24f7e115e90051c20daef4480f17
    Reviewed-on: https://chromium-review.googlesource.com/167500
    Commit-Queue: David Hendricks <dhendrix at chromium.org>
    Tested-by: David Hendricks <dhendrix at chromium.org>
    Reviewed-by: ron minnich <rminnich at chromium.org>
    (cherry picked from commit 650dba32cb217414c422907398f68e784e5720e8)
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
    Reviewed-on: http://review.coreboot.org/6614
    Tested-by: build bot (Jenkins)
    Reviewed-by: David Hendricks <dhendrix at chromium.org>


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

-gerrit



More information about the coreboot-gerrit mailing list