Add a wrapper function of LoadKernel()

include/chromeos/load_util.h

Index: include/chromeos/load_util.h
diff --git a/include/chromeos/load_util.h b/include/chromeos/load_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..03be4a0ae9412894100f41082217ad3ce76d96a6
--- /dev/null
+++ b/include/chromeos/load_util.h
@@ -0,0 +1,27 @@
+/*
+ * Copyright (c) 2011 The Chromium OS Authors. All rights reserved.
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#ifndef __CHROMEOS_LOAD_UTIL__
+#define __CHROMEOS_LOAD_UTIL__
+
+#include <linux/types.h>
+#include <load_kernel_fw.h>
+
+/*
+ * Wrapper of LoadKernel() function. Returns the return value of LoadKernel().
+ *
+ * See vboot_reference/firmware/include/load_kernel_fw.h for documentation.
+ *
+ * When LoadKernel() returns LOAD_KERNEL_SUCCESS, the wrapper will malloc a
+ * space for storing gbb_data, it is caller's decision and responsibility to
+ * free this space.
+ *
+ * On other return values, the wrapper will set gbb_data to NULL and
+ * gbb_size to zero.
+ */
+int load_kernel_wrapper(LoadKernelParams *params, uint64_t boot_flags);

[Tom Wai-Hong Tam, 2011/03/14 09:37:27]

It seems confused that we mix loading GBB with loading kernel. Better to
separate these two things into 2 functions: one is load_gbb(void *gbb_data), and
other is load_kernel_wrapper(LoadKernelParams *params, void *gbb_data, uint64_t
boot_flags). Such that it is clear to see when to free the gbb_data.

[Che-Liang Chiou, 2011/03/14 10:39:35]

Good point! Will do.

+
+#endif /* __CHROMEOS_LOAD_UTIL__ */