[coreboot-gerrit] New patch to review for coreboot: arch: introduce architecture dependent common variables

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Thu Apr 21 21:07:18 CEST 2016


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14454

-gerrit

commit dd6625264e9c51db8706fddf118296e63e70b3f1
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Thu Apr 21 14:02:40 2016 -0500

    arch: introduce architecture dependent common variables
    
    Stefan and others have discussed their interest in only
    including options in Kconfig that are directly associated
    with building a coreboot image. There are variables that
    are architecture dependent that are utilized in the
    coreboot infrastructure. To meet that goal, introduce
    <arch/cbconfig.h> header file which defines variables
    for the coreboot infrastructure that are architecture
    dependent but utilized in common infrastructure.
    
    Change-Id: Ic4cb9e81bab042797539dce004db0f7ee8526ea6
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/arch/arm/include/arch/cbconfig.h    | 26 ++++++++++++++++++++++++++
 src/arch/arm64/include/arch/cbconfig.h  | 26 ++++++++++++++++++++++++++
 src/arch/mips/include/arch/cbconfig.h   | 26 ++++++++++++++++++++++++++
 src/arch/power8/include/arch/cbconfig.h | 26 ++++++++++++++++++++++++++
 src/arch/riscv/include/arch/cbconfig.h  | 26 ++++++++++++++++++++++++++
 src/arch/x86/include/arch/cbconfig.h    | 26 ++++++++++++++++++++++++++
 6 files changed, 156 insertions(+)

diff --git a/src/arch/arm/include/arch/cbconfig.h b/src/arch/arm/include/arch/cbconfig.h
new file mode 100644
index 0000000..7bf9452
--- /dev/null
+++ b/src/arch/arm/include/arch/cbconfig.h
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright 2016 Google Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _ARCH_CBCONFIG_H_
+#define _ARCH_CBCONFIG_H_
+
+/*
+ * Instead of using Kconfig variables for internal coreboot infrastructure
+ * variables that are architecture dependent land those things in this file.
+ * If it's not obvious all variables that are used in the common code need
+ * to have the same name across all architectures.
+ */
+
+#endif
diff --git a/src/arch/arm64/include/arch/cbconfig.h b/src/arch/arm64/include/arch/cbconfig.h
new file mode 100644
index 0000000..7bf9452
--- /dev/null
+++ b/src/arch/arm64/include/arch/cbconfig.h
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright 2016 Google Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _ARCH_CBCONFIG_H_
+#define _ARCH_CBCONFIG_H_
+
+/*
+ * Instead of using Kconfig variables for internal coreboot infrastructure
+ * variables that are architecture dependent land those things in this file.
+ * If it's not obvious all variables that are used in the common code need
+ * to have the same name across all architectures.
+ */
+
+#endif
diff --git a/src/arch/mips/include/arch/cbconfig.h b/src/arch/mips/include/arch/cbconfig.h
new file mode 100644
index 0000000..7bf9452
--- /dev/null
+++ b/src/arch/mips/include/arch/cbconfig.h
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright 2016 Google Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _ARCH_CBCONFIG_H_
+#define _ARCH_CBCONFIG_H_
+
+/*
+ * Instead of using Kconfig variables for internal coreboot infrastructure
+ * variables that are architecture dependent land those things in this file.
+ * If it's not obvious all variables that are used in the common code need
+ * to have the same name across all architectures.
+ */
+
+#endif
diff --git a/src/arch/power8/include/arch/cbconfig.h b/src/arch/power8/include/arch/cbconfig.h
new file mode 100644
index 0000000..7bf9452
--- /dev/null
+++ b/src/arch/power8/include/arch/cbconfig.h
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright 2016 Google Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _ARCH_CBCONFIG_H_
+#define _ARCH_CBCONFIG_H_
+
+/*
+ * Instead of using Kconfig variables for internal coreboot infrastructure
+ * variables that are architecture dependent land those things in this file.
+ * If it's not obvious all variables that are used in the common code need
+ * to have the same name across all architectures.
+ */
+
+#endif
diff --git a/src/arch/riscv/include/arch/cbconfig.h b/src/arch/riscv/include/arch/cbconfig.h
new file mode 100644
index 0000000..7bf9452
--- /dev/null
+++ b/src/arch/riscv/include/arch/cbconfig.h
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright 2016 Google Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _ARCH_CBCONFIG_H_
+#define _ARCH_CBCONFIG_H_
+
+/*
+ * Instead of using Kconfig variables for internal coreboot infrastructure
+ * variables that are architecture dependent land those things in this file.
+ * If it's not obvious all variables that are used in the common code need
+ * to have the same name across all architectures.
+ */
+
+#endif
diff --git a/src/arch/x86/include/arch/cbconfig.h b/src/arch/x86/include/arch/cbconfig.h
new file mode 100644
index 0000000..7bf9452
--- /dev/null
+++ b/src/arch/x86/include/arch/cbconfig.h
@@ -0,0 +1,26 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * Copyright 2016 Google Inc.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 2 of the License.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ */
+
+#ifndef _ARCH_CBCONFIG_H_
+#define _ARCH_CBCONFIG_H_
+
+/*
+ * Instead of using Kconfig variables for internal coreboot infrastructure
+ * variables that are architecture dependent land those things in this file.
+ * If it's not obvious all variables that are used in the common code need
+ * to have the same name across all architectures.
+ */
+
+#endif



More information about the coreboot-gerrit mailing list