[coreboot-gerrit] Patch merged into coreboot/master: skylake: clarify and fix gpio macros

gerrit at coreboot.org gerrit at coreboot.org
Fri Aug 14 15:21:29 CEST 2015


the following patch was just integrated into master:
commit ced995a89f39c55639a26dce7e2a3e7bf5c2f873
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Sat Aug 8 01:31:05 2015 -0500

    skylake: clarify and fix gpio macros
    
    The gpio pad configuration currently defaults to ACPI
    owned GPIs. A '0' was used which wasn't so clear. Add
    a comment and explicitly set it to ACPI. Also,
    PAD_CFG_GPI_ACPI_SMI wasn't using the _PAD_CFG_ATTRS
    macro which causes compliation errors if attempted
    to be instantiated. No piece of code tried to use
    it so the error was overlooked.
    
    Lastly, allow for soc/gpio.h to be included during
    ASL compilation. That allows for gpio_defs.h to be
    included and those macros utilized without needing
    to know the file name and where it lives; just use
    the generic gpio.h.
    
    BUG=chrome-os-partner:43778
    BRANCH=None
    TEST=Built and booted glados.
    
    Original-Change-Id: I9dbadb0b494683ab38babfc1ac5e13093ee37730
    Original-Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/291935
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    
    Change-Id: Id4fa8b65ec1e1537dbf09824c2155119a768807e
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: http://review.coreboot.org/11206
    Tested-by: build bot (Jenkins)
    Reviewed-by: Patrick Georgi <pgeorgi at google.com>


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

-gerrit



More information about the coreboot-gerrit mailing list