[coreboot-gerrit] Patch merged into coreboot/master: 8a1d11f libpayload: Expand setbits_le32() and fix readl() const-ness

gerrit at coreboot.org gerrit at coreboot.org
Mon Jan 12 05:56:04 CET 2015


the following patch was just integrated into master:
commit 8a1d11f7973f3b4d01f2d9e7c57a4a0c5e7c3959
Author: Julius Werner <jwerner at chromium.org>
Date:   Thu Jul 17 10:43:15 2014 -0700

    libpayload: Expand setbits_le32() and fix readl() const-ness
    
    setbits_le32() is not really arch-specific... the arch-specific part of
    accessing memory is wrapped by readl() and writel(), and the endianness
    can be accounted for with the right macros. Generalize the definitions,
    add a be32 version and move them to endian.h so that all platforms can
    use them. Also include endian.h from libpayload.h so we won't update any
    payload's old use of the macros (endianness is something useful enough
    to always have avalable anyway, and shouldn't clash with other things).
    This also fixes a bug where these macros would only be available if
    libpayload-config.h had been independently included before.
    
    Also fix a bug with readl() macros on all archs where they refused to
    work on const pointers (which they should).
    
    CQ-DEPEND=CL:208712
    BUG=None
    TEST=Stuff still compiles. Built and booted on Storm.
    
    Original-Change-Id: I01a7fbadbb5d740675657d95c1e969027562ba8c
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/208713
    Original-Reviewed-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-by: David Hendricks <dhendrix at chromium.org>
    (cherry picked from commit 951f8a6d77bc21bd793bf4f228a0965ade586f00)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I51c25f01b200b91abbe32c879905349bb05dc9c8
    Reviewed-on: http://review.coreboot.org/8129
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Tested-by: build bot (Jenkins)


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

-gerrit



More information about the coreboot-gerrit mailing list