[PATCH 1/3] arm: mach-k3: Move R5 specific code into new r5/ directory

Neha Malcom Francis n-francis at ti.com
Wed Nov 15 07:01:18 CET 2023


Hi Andrew

On 14/11/23 21:29, Andrew Davis wrote:
> This makes it clear these are only to be used by the R5 builds of SPL.
> And this will be used to later more cleanly split the two builds.
> 
> Signed-off-by: Andrew Davis <afd at ti.com>
> ---
>   arch/arm/mach-k3/Makefile                         |  6 +-----
>   arch/arm/mach-k3/r5/Makefile                      | 13 +++++++++++++
>   arch/arm/mach-k3/{ => r5}/am62ax/Makefile         |  0
>   arch/arm/mach-k3/{ => r5}/am62ax/am62a_qos_data.c |  0
>   arch/arm/mach-k3/{ => r5}/am62ax/clk-data.c       |  0
>   arch/arm/mach-k3/{ => r5}/am62ax/dev-data.c       |  0
>   arch/arm/mach-k3/{ => r5}/am62x/Makefile          |  0
>   arch/arm/mach-k3/{ => r5}/am62x/clk-data.c        |  0
>   arch/arm/mach-k3/{ => r5}/am62x/dev-data.c        |  0
>   arch/arm/mach-k3/{ => r5}/j7200/Makefile          |  0
>   arch/arm/mach-k3/{ => r5}/j7200/clk-data.c        |  0
>   arch/arm/mach-k3/{ => r5}/j7200/dev-data.c        |  0
>   arch/arm/mach-k3/{ => r5}/j721e/Makefile          |  0
>   arch/arm/mach-k3/{ => r5}/j721e/clk-data.c        |  0
>   arch/arm/mach-k3/{ => r5}/j721e/dev-data.c        |  0
>   arch/arm/mach-k3/{ => r5}/j721s2/Makefile         |  0
>   arch/arm/mach-k3/{ => r5}/j721s2/clk-data.c       |  0
>   arch/arm/mach-k3/{ => r5}/j721s2/dev-data.c       |  0
>   arch/arm/mach-k3/{ => r5}/lowlevel_init.S         |  0
>   arch/arm/mach-k3/{ => r5}/r5_mpu.c                |  2 +-
>   20 files changed, 15 insertions(+), 6 deletions(-)
>   create mode 100644 arch/arm/mach-k3/r5/Makefile
>   rename arch/arm/mach-k3/{ => r5}/am62ax/Makefile (100%)
>   rename arch/arm/mach-k3/{ => r5}/am62ax/am62a_qos_data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/am62ax/clk-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/am62ax/dev-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/am62x/Makefile (100%)
>   rename arch/arm/mach-k3/{ => r5}/am62x/clk-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/am62x/dev-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/j7200/Makefile (100%)
>   rename arch/arm/mach-k3/{ => r5}/j7200/clk-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/j7200/dev-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/j721e/Makefile (100%)
>   rename arch/arm/mach-k3/{ => r5}/j721e/clk-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/j721e/dev-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/j721s2/Makefile (100%)
>   rename arch/arm/mach-k3/{ => r5}/j721s2/clk-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/j721s2/dev-data.c (100%)
>   rename arch/arm/mach-k3/{ => r5}/lowlevel_init.S (100%)
>   rename arch/arm/mach-k3/{ => r5}/r5_mpu.c (98%)
> 
> diff --git a/arch/arm/mach-k3/Makefile b/arch/arm/mach-k3/Makefile
> index c7ca0fdce56..215c755c5dc 100644
> --- a/arch/arm/mach-k3/Makefile
> +++ b/arch/arm/mach-k3/Makefile
> @@ -3,12 +3,8 @@
>   # Copyright (C) 2017-2018 Texas Instruments Incorporated - https://www.ti.com/
>   #	Lokesh Vutla <lokeshvutla at ti.com>
>   
> -obj-$(CONFIG_SOC_K3_J721E) += j721e/ j7200/
> -obj-$(CONFIG_SOC_K3_J721S2) += j721s2/
> -obj-$(CONFIG_SOC_K3_AM625) += am62x/
> -obj-$(CONFIG_SOC_K3_AM62A7) += am62ax/
> +obj-$(CONFIG_CPU_V7R) += r5/
>   obj-$(CONFIG_ARM64) += arm64-mmu.o
> -obj-$(CONFIG_CPU_V7R) += r5_mpu.o lowlevel_init.o
>   obj-$(CONFIG_ARM64) += cache.o
>   obj-$(CONFIG_OF_LIBFDT) += common_fdt.o
>   ifeq ($(CONFIG_OF_LIBFDT)$(CONFIG_OF_SYSTEM_SETUP),yy)
> diff --git a/arch/arm/mach-k3/r5/Makefile b/arch/arm/mach-k3/r5/Makefile
> new file mode 100644
> index 00000000000..8a6af73a44e
> --- /dev/null
> +++ b/arch/arm/mach-k3/r5/Makefile
> @@ -0,0 +1,13 @@
> +# SPDX-License-Identifier: GPL-2.0+
> +#
> +# Copyright (C) 2023 Texas Instruments Incorporated - https://www.ti.com/
> +#	Andrew Davis <afd at ti.com>
> +
> +obj-$(CONFIG_SOC_K3_J721E) += j721e/
> +obj-$(CONFIG_SOC_K3_J721E) += j7200/
> +obj-$(CONFIG_SOC_K3_J721S2) += j721s2/
> +obj-$(CONFIG_SOC_K3_AM625) += am62x/
> +obj-$(CONFIG_SOC_K3_AM62A7) += am62ax/
> +
> +obj-y += lowlevel_init.o
> +obj-y += r5_mpu.o
> diff --git a/arch/arm/mach-k3/am62ax/Makefile b/arch/arm/mach-k3/r5/am62ax/Makefile
> similarity index 100%
> rename from arch/arm/mach-k3/am62ax/Makefile
> rename to arch/arm/mach-k3/r5/am62ax/Makefile
> diff --git a/arch/arm/mach-k3/am62ax/am62a_qos_data.c b/arch/arm/mach-k3/r5/am62ax/am62a_qos_data.c
> similarity index 100%
> rename from arch/arm/mach-k3/am62ax/am62a_qos_data.c
> rename to arch/arm/mach-k3/r5/am62ax/am62a_qos_data.c
> diff --git a/arch/arm/mach-k3/am62ax/clk-data.c b/arch/arm/mach-k3/r5/am62ax/clk-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/am62ax/clk-data.c
> rename to arch/arm/mach-k3/r5/am62ax/clk-data.c
> diff --git a/arch/arm/mach-k3/am62ax/dev-data.c b/arch/arm/mach-k3/r5/am62ax/dev-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/am62ax/dev-data.c
> rename to arch/arm/mach-k3/r5/am62ax/dev-data.c
> diff --git a/arch/arm/mach-k3/am62x/Makefile b/arch/arm/mach-k3/r5/am62x/Makefile
> similarity index 100%
> rename from arch/arm/mach-k3/am62x/Makefile
> rename to arch/arm/mach-k3/r5/am62x/Makefile
> diff --git a/arch/arm/mach-k3/am62x/clk-data.c b/arch/arm/mach-k3/r5/am62x/clk-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/am62x/clk-data.c
> rename to arch/arm/mach-k3/r5/am62x/clk-data.c
> diff --git a/arch/arm/mach-k3/am62x/dev-data.c b/arch/arm/mach-k3/r5/am62x/dev-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/am62x/dev-data.c
> rename to arch/arm/mach-k3/r5/am62x/dev-data.c
> diff --git a/arch/arm/mach-k3/j7200/Makefile b/arch/arm/mach-k3/r5/j7200/Makefile
> similarity index 100%
> rename from arch/arm/mach-k3/j7200/Makefile
> rename to arch/arm/mach-k3/r5/j7200/Makefile
> diff --git a/arch/arm/mach-k3/j7200/clk-data.c b/arch/arm/mach-k3/r5/j7200/clk-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/j7200/clk-data.c
> rename to arch/arm/mach-k3/r5/j7200/clk-data.c
> diff --git a/arch/arm/mach-k3/j7200/dev-data.c b/arch/arm/mach-k3/r5/j7200/dev-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/j7200/dev-data.c
> rename to arch/arm/mach-k3/r5/j7200/dev-data.c
> diff --git a/arch/arm/mach-k3/j721e/Makefile b/arch/arm/mach-k3/r5/j721e/Makefile
> similarity index 100%
> rename from arch/arm/mach-k3/j721e/Makefile
> rename to arch/arm/mach-k3/r5/j721e/Makefile
> diff --git a/arch/arm/mach-k3/j721e/clk-data.c b/arch/arm/mach-k3/r5/j721e/clk-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/j721e/clk-data.c
> rename to arch/arm/mach-k3/r5/j721e/clk-data.c
> diff --git a/arch/arm/mach-k3/j721e/dev-data.c b/arch/arm/mach-k3/r5/j721e/dev-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/j721e/dev-data.c
> rename to arch/arm/mach-k3/r5/j721e/dev-data.c
> diff --git a/arch/arm/mach-k3/j721s2/Makefile b/arch/arm/mach-k3/r5/j721s2/Makefile
> similarity index 100%
> rename from arch/arm/mach-k3/j721s2/Makefile
> rename to arch/arm/mach-k3/r5/j721s2/Makefile
> diff --git a/arch/arm/mach-k3/j721s2/clk-data.c b/arch/arm/mach-k3/r5/j721s2/clk-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/j721s2/clk-data.c
> rename to arch/arm/mach-k3/r5/j721s2/clk-data.c
> diff --git a/arch/arm/mach-k3/j721s2/dev-data.c b/arch/arm/mach-k3/r5/j721s2/dev-data.c
> similarity index 100%
> rename from arch/arm/mach-k3/j721s2/dev-data.c
> rename to arch/arm/mach-k3/r5/j721s2/dev-data.c
> diff --git a/arch/arm/mach-k3/lowlevel_init.S b/arch/arm/mach-k3/r5/lowlevel_init.S
> similarity index 100%
> rename from arch/arm/mach-k3/lowlevel_init.S
> rename to arch/arm/mach-k3/r5/lowlevel_init.S
> diff --git a/arch/arm/mach-k3/r5_mpu.c b/arch/arm/mach-k3/r5/r5_mpu.c
> similarity index 98%
> rename from arch/arm/mach-k3/r5_mpu.c
> rename to arch/arm/mach-k3/r5/r5_mpu.c
> index 605f7931518..e409fdcfea8 100644
> --- a/arch/arm/mach-k3/r5_mpu.c
> +++ b/arch/arm/mach-k3/r5/r5_mpu.c
> @@ -9,7 +9,7 @@
>   #include <common.h>
>   #include <asm/io.h>
>   #include <linux/kernel.h>
> -#include "common.h"
> +#include <asm/armv7_mpu.h>
>   
>   struct mpu_region_config k3_mpu_regions[16] = {
>   	/*

I think a cover-letter with the motivation behind the series could have helped 
me understand quicker (plus if in case there were any discussion links earlier 
about doing this)

-- 
Thanking You
Neha Malcom Francis


More information about the U-Boot mailing list