[coreboot-gerrit] Patch merged into coreboot/master: 9b800ae southbridge/amd: Change #if defined to #if IS_ENABLED

gerrit at coreboot.org gerrit at coreboot.org
Thu Jun 12 20:13:24 CEST 2014


the following patch was just integrated into master:
commit 9b800ae9547ed5890773fd6e12781a49cf3fcfc4
Author: Dave Frodin <dave.frodin at se-eng.com>
Date:   Wed Jun 11 13:15:56 2014 -0600

    southbridge/amd: Change #if defined to #if IS_ENABLED
    
    The IMC functions were being called and timing out when the
    CONFIG_SB800_IMC_FWM/CONFIG_HUDSON_IMC_FWM were defined as 0.
    Changing to a IS_ENABLED will keep the IMC handshake from
    occuring if the IMC firmware isn't running.
    
    Tested on a Persimmon platform which makes three calls to
    spi_claim_bus() with each call timing out after 500ms.
    
    Change-Id: I5d4bbcecf003b93704553b495a16bcd15f66763b
    Signed-off-by: Dave Frodin <dave.frodin at se-eng.com>
    Reviewed-on: http://review.coreboot.org/5974
    Tested-by: build bot (Jenkins)
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Reviewed-by: Marc Jones <marc.jones at se-eng.com>


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

-gerrit



More information about the coreboot-gerrit mailing list