From 0a8da746c21818429f3799a1956958830bbf22f6 Mon Sep 17 00:00:00 2001
From: Julius Werner <jwerner@chromium.org>
Date: Fri, 2 Aug 2019 12:45:24 -0700
Subject: arm64: Rename arm_tf.c/h to bl31.c/h

This patch renames arm_tf.c and arm_tf.h to bl31.c and bl31.h,
respectively. That name is closer to the terminology used in most
functions related to Trusted Firmware, and it removes the annoying
auto-completion clash between arm64/arm_tf.c and arm64/armv8.

Change-Id: I2741e2bce9d079b1025f82ecb3bb78a02fe39ed5
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34677
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Hung-Te Lin <hungte@chromium.org>
---
 src/arch/arm64/Makefile.inc                |  2 +-
 src/arch/arm64/arm_tf.c                    | 90 ------------------------------
 src/arch/arm64/bl31.c                      | 90 ++++++++++++++++++++++++++++++
 src/arch/arm64/boot.c                      |  4 +-
 src/arch/arm64/include/arm_tf.h            | 28 ----------
 src/arch/arm64/include/bl31.h              | 28 ++++++++++
 src/soc/cavium/cn81xx/bl31_plat_params.c   |  2 +-
 src/soc/mediatek/mt8173/bl31_plat_params.c |  2 +-
 src/soc/nvidia/tegra210/arm_tf.c           |  2 +-
 src/soc/rockchip/rk3399/bl31_plat_params.c |  2 +-
 10 files changed, 125 insertions(+), 125 deletions(-)
 delete mode 100644 src/arch/arm64/arm_tf.c
 create mode 100644 src/arch/arm64/bl31.c
 delete mode 100644 src/arch/arm64/include/arm_tf.h
 create mode 100644 src/arch/arm64/include/bl31.h

(limited to 'src')

diff --git a/src/arch/arm64/Makefile.inc b/src/arch/arm64/Makefile.inc
index 5992a2a5e2..1aee6b83f7 100644
--- a/src/arch/arm64/Makefile.inc
+++ b/src/arch/arm64/Makefile.inc
@@ -131,7 +131,7 @@ ramstage-$(CONFIG_ARM64_USE_ARCH_TIMER) += arch_timer.c
 ramstage-y += memset.S
 ramstage-y += memcpy.S
 ramstage-y += memmove.S
-ramstage-$(CONFIG_ARM64_USE_ARM_TRUSTED_FIRMWARE) += arm_tf.c
+ramstage-$(CONFIG_ARM64_USE_ARM_TRUSTED_FIRMWARE) += bl31.c
 ramstage-y += transition.c transition_asm.S
 ramstage-$(CONFIG_PAYLOAD_FIT_SUPPORT) += fit_payload.c
 
diff --git a/src/arch/arm64/arm_tf.c b/src/arch/arm64/arm_tf.c
deleted file mode 100644
index 5059a89391..0000000000
--- a/src/arch/arm64/arm_tf.c
+++ /dev/null
@@ -1,90 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * 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.
- */
-
-#include <arch/cache.h>
-#include <arch/lib_helpers.h>
-#include <arch/mmu.h>
-#include <arch/transition.h>
-#include <arm_tf.h>
-#include <bootmem.h>
-#include <cbfs.h>
-#include <console/console.h>
-#include <program_loading.h>
-
-/*
- * TODO: Many of these structures are currently unused. Better not fill them out
- * to make future changes fail fast, rather than try to come up with content
- * that might turn out to not make sense. Implement later as required.
- *
-static image_info_t bl31_image_info;
-static image_info_t bl32_image_info;
-static image_info_t bl33_image_info;
- */
-static entry_point_info_t bl32_ep_info;
-static entry_point_info_t bl33_ep_info;
-static bl31_params_t bl31_params;
-
-void __weak *soc_get_bl31_plat_params(bl31_params_t *params)
-{
-	/* Default weak implementation. */
-	return NULL;
-}
-
-void arm_tf_run_bl31(u64 payload_entry, u64 payload_arg0, u64 payload_spsr)
-{
-	struct prog bl31 = PROG_INIT(PROG_BL31, CONFIG_CBFS_PREFIX"/bl31");
-	void (*bl31_entry)(bl31_params_t *params, void *plat_params) = NULL;
-
-	if (prog_locate(&bl31))
-		die("BL31 not found");
-
-	if (!selfload_check(&bl31, BM_MEM_BL31))
-		die("BL31 load failed");
-	bl31_entry = prog_entry(&bl31);
-
-	SET_PARAM_HEAD(&bl31_params, PARAM_BL31, VERSION_1, 0);
-
-	if (CONFIG(ARM64_USE_SECURE_OS)) {
-		struct prog bl32 = PROG_INIT(PROG_BL32,
-					     CONFIG_CBFS_PREFIX"/secure_os");
-
-		if (prog_locate(&bl32))
-			die("BL32 not found");
-
-		if (cbfs_prog_stage_load(&bl32))
-			die("BL32 load failed");
-
-		SET_PARAM_HEAD(&bl32_ep_info, PARAM_EP, VERSION_1,
-			       PARAM_EP_SECURE);
-		bl32_ep_info.pc = (uintptr_t)prog_entry(&bl32);
-		bl32_ep_info.spsr = SPSR_EXCEPTION_MASK |
-				get_eret_el(EL1, SPSR_USE_L);
-		bl31_params.bl32_ep_info = &bl32_ep_info;
-	}
-
-	bl31_params.bl33_ep_info = &bl33_ep_info;
-
-	SET_PARAM_HEAD(&bl33_ep_info, PARAM_EP, VERSION_1, PARAM_EP_NON_SECURE);
-	bl33_ep_info.pc = payload_entry;
-	bl33_ep_info.spsr = payload_spsr;
-	bl33_ep_info.args.arg0 = payload_arg0;
-
-	/* May update bl31_params if necessary. */
-	void *bl31_plat_params = soc_get_bl31_plat_params(&bl31_params);
-
-	/* MMU disable will flush cache, so passed params land in memory. */
-	raw_write_daif(SPSR_EXCEPTION_MASK);
-	mmu_disable();
-	bl31_entry(&bl31_params, bl31_plat_params);
-	die("BL31 returned!");
-}
diff --git a/src/arch/arm64/bl31.c b/src/arch/arm64/bl31.c
new file mode 100644
index 0000000000..fb24dff40e
--- /dev/null
+++ b/src/arch/arm64/bl31.c
@@ -0,0 +1,90 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * 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.
+ */
+
+#include <arch/cache.h>
+#include <arch/lib_helpers.h>
+#include <arch/mmu.h>
+#include <arch/transition.h>
+#include <bl31.h>
+#include <bootmem.h>
+#include <cbfs.h>
+#include <console/console.h>
+#include <program_loading.h>
+
+/*
+ * TODO: Many of these structures are currently unused. Better not fill them out
+ * to make future changes fail fast, rather than try to come up with content
+ * that might turn out to not make sense. Implement later as required.
+ *
+static image_info_t bl31_image_info;
+static image_info_t bl32_image_info;
+static image_info_t bl33_image_info;
+ */
+static entry_point_info_t bl32_ep_info;
+static entry_point_info_t bl33_ep_info;
+static bl31_params_t bl31_params;
+
+void __weak *soc_get_bl31_plat_params(bl31_params_t *params)
+{
+	/* Default weak implementation. */
+	return NULL;
+}
+
+void run_bl31(u64 payload_entry, u64 payload_arg0, u64 payload_spsr)
+{
+	struct prog bl31 = PROG_INIT(PROG_BL31, CONFIG_CBFS_PREFIX"/bl31");
+	void (*bl31_entry)(bl31_params_t *params, void *plat_params) = NULL;
+
+	if (prog_locate(&bl31))
+		die("BL31 not found");
+
+	if (!selfload_check(&bl31, BM_MEM_BL31))
+		die("BL31 load failed");
+	bl31_entry = prog_entry(&bl31);
+
+	SET_PARAM_HEAD(&bl31_params, PARAM_BL31, VERSION_1, 0);
+
+	if (CONFIG(ARM64_USE_SECURE_OS)) {
+		struct prog bl32 = PROG_INIT(PROG_BL32,
+					     CONFIG_CBFS_PREFIX"/secure_os");
+
+		if (prog_locate(&bl32))
+			die("BL32 not found");
+
+		if (cbfs_prog_stage_load(&bl32))
+			die("BL32 load failed");
+
+		SET_PARAM_HEAD(&bl32_ep_info, PARAM_EP, VERSION_1,
+			       PARAM_EP_SECURE);
+		bl32_ep_info.pc = (uintptr_t)prog_entry(&bl32);
+		bl32_ep_info.spsr = SPSR_EXCEPTION_MASK |
+				get_eret_el(EL1, SPSR_USE_L);
+		bl31_params.bl32_ep_info = &bl32_ep_info;
+	}
+
+	bl31_params.bl33_ep_info = &bl33_ep_info;
+
+	SET_PARAM_HEAD(&bl33_ep_info, PARAM_EP, VERSION_1, PARAM_EP_NON_SECURE);
+	bl33_ep_info.pc = payload_entry;
+	bl33_ep_info.spsr = payload_spsr;
+	bl33_ep_info.args.arg0 = payload_arg0;
+
+	/* May update bl31_params if necessary. */
+	void *bl31_plat_params = soc_get_bl31_plat_params(&bl31_params);
+
+	/* MMU disable will flush cache, so passed params land in memory. */
+	raw_write_daif(SPSR_EXCEPTION_MASK);
+	mmu_disable();
+	bl31_entry(&bl31_params, bl31_plat_params);
+	die("BL31 returned!");
+}
diff --git a/src/arch/arm64/boot.c b/src/arch/arm64/boot.c
index 58d7234710..c6df0ee02e 100644
--- a/src/arch/arm64/boot.c
+++ b/src/arch/arm64/boot.c
@@ -15,7 +15,7 @@
 #include <arch/lib_helpers.h>
 #include <arch/stages.h>
 #include <arch/transition.h>
-#include <arm_tf.h>
+#include <bl31.h>
 #include <program_loading.h>
 
 static void run_payload(struct prog *prog)
@@ -28,7 +28,7 @@ static void run_payload(struct prog *prog)
 	u64 payload_spsr = get_eret_el(EL2, SPSR_USE_L);
 
 	if (CONFIG(ARM64_USE_ARM_TRUSTED_FIRMWARE))
-		arm_tf_run_bl31((u64)doit, (u64)arg, payload_spsr);
+		run_bl31((u64)doit, (u64)arg, payload_spsr);
 	else
 		transition_to_el2(doit, arg, payload_spsr);
 }
diff --git a/src/arch/arm64/include/arm_tf.h b/src/arch/arm64/include/arm_tf.h
deleted file mode 100644
index 692a5beb22..0000000000
--- a/src/arch/arm64/include/arm_tf.h
+++ /dev/null
@@ -1,28 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * 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 __ARM_TF_H__
-#define __ARM_TF_H__
-
-#include <types.h>
-
-/* TODO: Pull in directly from ARM TF once its headers have been reorganized. */
-#include <arm_tf_temp.h>
-
-/* Load and enter BL31, set it up to exit to payload according to arguments. */
-void arm_tf_run_bl31(u64 payload_entry, u64 payload_arg0, u64 payload_spsr);
-
-/* Return platform-specific bl31_plat_params. May update bl31_params. */
-void *soc_get_bl31_plat_params(bl31_params_t *bl31_params);
-
-#endif /* __ARM_TF_H__ */
diff --git a/src/arch/arm64/include/bl31.h b/src/arch/arm64/include/bl31.h
new file mode 100644
index 0000000000..08855f64cf
--- /dev/null
+++ b/src/arch/arm64/include/bl31.h
@@ -0,0 +1,28 @@
+/*
+ * This file is part of the coreboot project.
+ *
+ * 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 __BL31_H__
+#define __BL31_H__
+
+#include <types.h>
+
+/* TODO: Pull in directly from ARM TF once its headers have been reorganized. */
+#include <arm_tf_temp.h>
+
+/* Load and enter BL31, set it up to exit to payload according to arguments. */
+void run_bl31(u64 payload_entry, u64 payload_arg0, u64 payload_spsr);
+
+/* Return platform-specific bl31_plat_params. May update bl31_params. */
+void *soc_get_bl31_plat_params(bl31_params_t *bl31_params);
+
+#endif /* __BL31_H__ */
diff --git a/src/soc/cavium/cn81xx/bl31_plat_params.c b/src/soc/cavium/cn81xx/bl31_plat_params.c
index 5d4dead71e..5002e146bc 100644
--- a/src/soc/cavium/cn81xx/bl31_plat_params.c
+++ b/src/soc/cavium/cn81xx/bl31_plat_params.c
@@ -14,8 +14,8 @@
  *
  */
 
-#include <arm_tf.h>
 #include <assert.h>
+#include <bl31.h>
 #include <soc/bl31_plat_params.h>
 
 static struct bl31_plat_param *plat_params;
diff --git a/src/soc/mediatek/mt8173/bl31_plat_params.c b/src/soc/mediatek/mt8173/bl31_plat_params.c
index 245866a104..44ef934ac2 100644
--- a/src/soc/mediatek/mt8173/bl31_plat_params.c
+++ b/src/soc/mediatek/mt8173/bl31_plat_params.c
@@ -12,7 +12,7 @@
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  */
-#include <arm_tf.h>
+#include <bl31.h>
 
 void *soc_get_bl31_plat_params(bl31_params_t *bl31_params)
 {
diff --git a/src/soc/nvidia/tegra210/arm_tf.c b/src/soc/nvidia/tegra210/arm_tf.c
index 38bb8bfc9e..1da351afb3 100644
--- a/src/soc/nvidia/tegra210/arm_tf.c
+++ b/src/soc/nvidia/tegra210/arm_tf.c
@@ -14,8 +14,8 @@
  */
 
 #include <arch/cache.h>
-#include <arm_tf.h>
 #include <assert.h>
+#include <bl31.h>
 #include <soc/addressmap.h>
 #include <soc/console_uart.h>
 #include <symbols.h>
diff --git a/src/soc/rockchip/rk3399/bl31_plat_params.c b/src/soc/rockchip/rk3399/bl31_plat_params.c
index 9c11552301..b8836c1576 100644
--- a/src/soc/rockchip/rk3399/bl31_plat_params.c
+++ b/src/soc/rockchip/rk3399/bl31_plat_params.c
@@ -14,7 +14,7 @@
  *
  */
 
-#include <arm_tf.h>
+#include <bl31.h>
 #include <cbmem.h>
 #include <soc/bl31_plat_params.h>
 
-- 
cgit v1.2.3