[coreboot-gerrit] Patch merged into coreboot/master: 430363a acpigen: Add new function acpigen_pop_len

gerrit at coreboot.org gerrit at coreboot.org
Sat Nov 8 22:28:45 CET 2014


the following patch was just integrated into master:
commit 430363ace7338b9708feefb5917d667f1733944d
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Sun Nov 2 21:51:22 2014 +0100

    acpigen: Add new function acpigen_pop_len
    
    acpigen_patch_len doesn't really need its argument: length always includes
    everything from length bytes to current pointer and never bytes before it.
    Hence just infer all the info implicitly.
    
    Argument is wrong in several places through the codebase but ACPI parsing
    is lax enough to swallow incorrect SSDT. After this function is used throughout
    the codebase, these issues will be fixed.
    
    Change-Id: I9fa536a614c5595146a7a1cd71f2676d8a8d9c2f
    Signed-off-by: Vladimir Serbinenko <phcoder at gmail.com>
    Reviewed-on: http://review.coreboot.org/7325
    Reviewed-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list