From ac8805e7e9bd40c03baf44e37b26f28d9b763ab5 Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Sat, 16 Mar 2013 04:03:40 -0700 Subject: Get rid of some crufty macros. These were macros that were never used, or that were only set to one thing and could be substituted up front. I left in code guarded by the HAVE_ENDIAN_H and HAVE_LITTLE_ENDIAN macros even though those are never defined because they guard a reportedly significantly faster implementation of some functionality, at least according to a comment in the source. It would be a good idea to enable that code path and see if it really does make a big difference before removing it entirely. BUG=None TEST=Built for Link, Daisy, and the host with FEATURES=test. Built depthcharge for Link and booted in normal mode. BRANCH=None Change-Id: I934a4dd0da169ac018ba07350d56924ab88b1acc Signed-off-by: Gabe Black Reviewed-on: https://gerrit.chromium.org/gerrit/45687 Reviewed-by: Randall Spangler Commit-Queue: Gabe Black Tested-by: Gabe Black --- utility/load_kernel_test.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'utility/load_kernel_test.c') diff --git a/utility/load_kernel_test.c b/utility/load_kernel_test.c index f5585c38..307b8ee2 100644 --- a/utility/load_kernel_test.c +++ b/utility/load_kernel_test.c @@ -131,12 +131,13 @@ int main(int argc, char* argv[]) { fprintf(stderr, "usage: %s [options] []\n", argv[0]); fprintf(stderr, "\noptions:\n"); + /* These cases are because uint64_t isn't necessarily the same as ULL. */ fprintf(stderr, " -b NUM boot flag bits (default %" PRIu64 "):\n", - BOOT_FLAG_RECOVERY); + (uint64_t)BOOT_FLAG_RECOVERY); fprintf(stderr, " %" PRIu64 " = developer mode on\n", - BOOT_FLAG_DEVELOPER); + (uint64_t)BOOT_FLAG_DEVELOPER); fprintf(stderr, " %" PRIu64 " = recovery mode on\n", - BOOT_FLAG_RECOVERY); + (uint64_t)BOOT_FLAG_RECOVERY); return 1; } -- cgit v1.2.1