x86: Move table csum into separate header
authorAlexander Graf <agraf@suse.de>
Thu, 18 Aug 2016 23:23:22 +0000 (01:23 +0200)
committerBin Meng <bmeng.cn@gmail.com>
Tue, 18 Oct 2016 07:58:50 +0000 (15:58 +0800)
We need the checksum function without all the other table functionality
soon, so let's split it out into its own header file.

Signed-off-by: Alexander Graf <agraf@suse.de>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
arch/x86/include/asm/tables.h
arch/x86/lib/tables.c
include/tables_csum.h [new file with mode: 0644]

index ae9f0d0d626a2920595d3225bc7e16e4a96f917a..81f98f27745381993f009409dc0536e80026b6c3 100644 (file)
@@ -7,6 +7,8 @@
 #ifndef _X86_TABLES_H_
 #define _X86_TABLES_H_
 
+#include <tables_csum.h>
+
 /*
  * All x86 tables happen to like the address range from 0xf0000 to 0x100000.
  * We use 0xf0000 as the starting address to store those tables, including
index f92111e4c8a075caaf4368c52c8623e6c3369d68..9ee6b5e9249d28c8a9609b8cc37492ae270073e6 100644 (file)
@@ -38,18 +38,6 @@ static table_write table_write_funcs[] = {
 #endif
 };
 
-u8 table_compute_checksum(void *v, int len)
-{
-       u8 *bytes = v;
-       u8 checksum = 0;
-       int i;
-
-       for (i = 0; i < len; i++)
-               checksum -= bytes[i];
-
-       return checksum;
-}
-
 void table_fill_string(char *dest, const char *src, size_t n, char pad)
 {
        int start, len;
diff --git a/include/tables_csum.h b/include/tables_csum.h
new file mode 100644 (file)
index 0000000..27d147b
--- /dev/null
@@ -0,0 +1,22 @@
+/*
+ * Copyright (C) 2015, Bin Meng <bmeng.cn@gmail.com>
+ *
+ * SPDX-License-Identifier:    GPL-2.0+
+ */
+
+#ifndef _TABLES_CSUM_H_
+#define _TABLES_CSUM_H_
+
+static inline u8 table_compute_checksum(void *v, int len)
+{
+       u8 *bytes = v;
+       u8 checksum = 0;
+       int i;
+
+       for (i = 0; i < len; i++)
+               checksum -= bytes[i];
+
+       return checksum;
+}
+
+#endif