[coreboot-gerrit] Patch merged into coreboot/master: e97b683 tegra: Change how tegra124 and tegra include files from each other.

gerrit at coreboot.org gerrit at coreboot.org
Mon Aug 18 19:11:00 CEST 2014


the following patch was just integrated into master:
commit e97b6835f4dcd70ebf23941fb2f56c0d9d1d3e97
Author: Gabe Black <gabeblack at google.com>
Date:   Sun Oct 6 06:13:24 2013 -0700

    tegra: Change how tegra124 and tegra include files from each other.
    
    A problem with including the tegra124 directory directly in the include path
    is that it makes all headers in that directory first level headers available
    everywhere including places that have nothing to do with the SOC, even headers
    which were only intended for local use by tegra124 code. This change modifies
    things a bit to be more like the way the arch headers are chosen. In the
    tegra124 directory, there's an include directory which has an soc subdirectory
    in it. That include directory is added to the include path, making it possible
    to have headers private to the tegra124. When files specific to whatever tegra
    is being built for are needed, you can include <soc/foo.h> and get the version
    specific to that particular soc.
    
    Also, the soc.h header file was overhauled to use enums instead of defines, to
    consistently name things as far as their prefix (the less cryptic TEGRA instead
    of NV_PA) and suffixes like "BASE", and to get rid of values which were
    specific to U-Boot which we don't need. Since the only thing in the file were
    address constants, I also renamed the file addressmap.h. It would be included
    as:
    
    <soc/addressmap.h>
    
    which I think is easy to remember, does what you'd think it does from the
    name, and won't conflict with other header files just minding their own
    business in some other directory.
    
    Change-Id: I6a1be1ba28417b7103ad8584e6ec5024a7ff4e55
    Signed-off-by: Gabe Black <gabeblack at google.com>
    Reviewed-on: https://chromium-review.googlesource.com/172080
    Reviewed-by: Julius Werner <jwerner at chromium.org>
    Reviewed-by: Ronald Minnich <rminnich at chromium.org>
    Tested-by: Gabe Black <gabeblack at chromium.org>
    Commit-Queue: Gabe Black <gabeblack at chromium.org>
    (cherry picked from commit 2c554f58f9ee18e151e824f01c03eb3f0e907858)
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
    Reviewed-on: http://review.coreboot.org/6659
    Tested-by: build bot (Jenkins)
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>


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

-gerrit



More information about the coreboot-gerrit mailing list