Hello Patrick Rudolph, Hung-Te Lin, Patrick Georgi,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/34677
to review the following change.
Change subject: arm64: Rename arm_tf.c/h to bl31.c/h ......................................................................
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 --- M src/arch/arm64/Makefile.inc R src/arch/arm64/bl31.c M src/arch/arm64/boot.c R src/arch/arm64/include/bl31.h M src/soc/mediatek/mt8173/bl31_plat_params.c M src/soc/nvidia/tegra210/arm_tf.c M src/soc/rockchip/rk3399/bl31_plat_params.c 7 files changed, 12 insertions(+), 12 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/34677/1
diff --git a/src/arch/arm64/Makefile.inc b/src/arch/arm64/Makefile.inc index 6bb7196..2f862da 100644 --- a/src/arch/arm64/Makefile.inc +++ b/src/arch/arm64/Makefile.inc @@ -135,7 +135,7 @@ 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/bl31.c similarity index 96% rename from src/arch/arm64/arm_tf.c rename to src/arch/arm64/bl31.c index 291e4ee..0a793ee 100644 --- a/src/arch/arm64/arm_tf.c +++ b/src/arch/arm64/bl31.c @@ -17,7 +17,7 @@ #include <arch/lib_helpers.h> #include <arch/mmu.h> #include <arch/transition.h> -#include <arm_tf.h> +#include <bl31.h> #include <bootmem.h> #include <cbfs.h> #include <console/console.h> @@ -42,7 +42,7 @@ return NULL; }
-void arm_tf_run_bl31(u64 payload_entry, u64 payload_arg0, u64 payload_spsr) +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; diff --git a/src/arch/arm64/boot.c b/src/arch/arm64/boot.c index 7fbc525..2c3d243 100644 --- a/src/arch/arm64/boot.c +++ b/src/arch/arm64/boot.c @@ -17,7 +17,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) @@ -30,7 +30,7 @@ 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/bl31.h similarity index 85% rename from src/arch/arm64/include/arm_tf.h rename to src/arch/arm64/include/bl31.h index ea5a0f4..460286b 100644 --- a/src/arch/arm64/include/arm_tf.h +++ b/src/arch/arm64/include/bl31.h @@ -13,8 +13,8 @@ * GNU General Public License for more details. */
-#ifndef __ARM_TF_H__ -#define __ARM_TF_H__ +#ifndef __BL31_H__ +#define __BL31_H__
#include <types.h>
@@ -22,9 +22,9 @@ #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); +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 /* __ARM_TF_H__ */ +#endif /* __BL31_H__ */ diff --git a/src/soc/mediatek/mt8173/bl31_plat_params.c b/src/soc/mediatek/mt8173/bl31_plat_params.c index 245866a..44ef934 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 38bb8bf..1da351a 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 9c11552..b8836c1 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>