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 <gabeblack@google.com>
Reviewed-on: https://gerrit.chromium.org/gerrit/45687
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Commit-Queue: Gabe Black <gabeblack@chromium.org>
Tested-by: Gabe Black <gabeblack@chromium.org>

1 files changed, 4 insertions, 3 deletions

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] <drive_image> [<sign_key>]\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;
   }