the following patch was just integrated into master: commit 8a1d11f7973f3b4d01f2d9e7c57a4a0c5e7c3959 Author: Julius Werner jwerner@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@chromium.org Original-Reviewed-on: https://chromium-review.googlesource.com/208713 Original-Reviewed-by: Vadim Bendebury vbendeb@chromium.org Original-Reviewed-by: David Hendricks dhendrix@chromium.org (cherry picked from commit 951f8a6d77bc21bd793bf4f228a0965ade586f00) Signed-off-by: Marc Jones marc.jones@se-eng.com
Change-Id: I51c25f01b200b91abbe32c879905349bb05dc9c8 Reviewed-on: http://review.coreboot.org/8129 Reviewed-by: Stefan Reinauer stefan.reinauer@coreboot.org Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net Tested-by: build bot (Jenkins)
See http://review.coreboot.org/8129 for details.
-gerrit