[U-Boot] [PATCH 0/5] arm: k3: Print cpu and board names during boot

This series adds support for printing CPU name and board name for all k3 devices. Also a misc update for getting tisci handle
Lokesh Vutla (5): arm: k3: Add support for printing CPUINFO board: am65x: Print board name and version during boot configs: j721e_evm_a72: Enable DISPLAY_CPUINFO configs: am65x_evm_a53: Enable DISPLAY_CPUINFO arm: k3: Use driver_name to get ti_sci handle
arch/arm/mach-k3/common.c | 46 +++++++++++++++++++++++- arch/arm/mach-k3/common.h | 15 ++++++++ arch/arm/mach-k3/include/mach/hardware.h | 24 +++++++++++++ board/ti/am65x/evm.c | 15 +++++++- configs/am65x_evm_a53_defconfig | 1 - configs/j721e_evm_a72_defconfig | 1 - 6 files changed, 98 insertions(+), 4 deletions(-)

Add support for printing CPU info for all K3 devices.
Signed-off-by: Lokesh Vutla lokeshvutla@ti.com --- arch/arm/mach-k3/common.c | 43 ++++++++++++++++++++++++ arch/arm/mach-k3/common.h | 15 +++++++++ arch/arm/mach-k3/include/mach/hardware.h | 24 +++++++++++++ 3 files changed, 82 insertions(+)
diff --git a/arch/arm/mach-k3/common.c b/arch/arm/mach-k3/common.c index 3e36d90ace..7339d7398d 100644 --- a/arch/arm/mach-k3/common.c +++ b/arch/arm/mach-k3/common.c @@ -14,6 +14,22 @@ #include <linux/soc/ti/ti_sci_protocol.h> #include <fdt_support.h> #include <asm/arch/sys_proto.h> +#include <asm/hardware.h> +#include <asm/io.h> + +#if defined(CONFIG_DISPLAY_CPUINFO) +static const char soc_name[K3_MAX + 1][SOC_NAME_MAX_LENGTH] = { + [AM654] = "AM654", + [J721E] = "J721E", + [K3_MAX] = "UNKNOWN" +}; + +static const char soc_revision[REV_PG_MAX + 1][SOC_REVISION_MAX_LENGTH] = { + [REV_PG1_0] = "1.0", + [REV_PG2_0] = "2.0", + [REV_PG_MAX] = "NULL" +}; +#endif
struct ti_sci_handle *get_ti_sci_handle(void) { @@ -144,3 +160,30 @@ void reset_cpu(ulong ignored) { } #endif + +#if defined(CONFIG_DISPLAY_CPUINFO) +int print_cpuinfo(void) +{ + k3_soc_rev rev; + k3_soc soc; + + soc = (readl(CTRLMMR_WKUP_JTAG_DEVICE_ID) & + DEVICE_ID_FAMILY_MASK) >> DEVICE_ID_FAMILY_SHIFT; + rev = (readl(CTRLMMR_WKUP_JTAG_ID) & + JTAG_ID_VARIANT_MASK) >> JTAG_ID_VARIANT_SHIFT; + + printf("SoC: "); + if (soc > K3_MAX || !soc_name[soc]) + printf("Unknown Silicon %d ", soc); + else + printf("%s ", soc_name[soc]); + + printf("PG "); + if (rev > REV_PG_MAX) + printf("Unknown revision %d\n", rev); + else + printf("%s\n", soc_revision[rev]); + + return 0; +} +#endif diff --git a/arch/arm/mach-k3/common.h b/arch/arm/mach-k3/common.h index ac7e80d9af..9f8fbb1241 100644 --- a/arch/arm/mach-k3/common.h +++ b/arch/arm/mach-k3/common.h @@ -8,4 +8,19 @@
#include <asm/armv7_mpu.h>
+#define SOC_NAME_MAX_LENGTH 10 +#define SOC_REVISION_MAX_LENGTH 5 + +typedef enum { + AM654 = 2, + J721E = 4, + K3_MAX +} k3_soc; + +typedef enum { + REV_PG1_0, + REV_PG2_0, + REV_PG_MAX +} k3_soc_rev; + void setup_k3_mpu_regions(void); diff --git a/arch/arm/mach-k3/include/mach/hardware.h b/arch/arm/mach-k3/include/mach/hardware.h index 4e629822aa..b888dbf0b2 100644 --- a/arch/arm/mach-k3/include/mach/hardware.h +++ b/arch/arm/mach-k3/include/mach/hardware.h @@ -13,4 +13,28 @@ #ifdef CONFIG_SOC_K3_J721E #include "j721e_hardware.h" #endif + +/* Assuming these addresses and definitions stay common across K3 devices */ +#define CTRLMMR_WKUP_JTAG_DEVICE_ID 0x43000018 +#define DEVICE_ID_FAMILY_SHIFT 26 +#define DEVICE_ID_FAMILY_MASK (0x3f << 26) +#define DEVICE_ID_BASE_SHIFT 11 +#define DEVICE_ID_BASE_MASK (0x1fff << 11) +#define DEVICE_ID_SPEED_SHIFT 6 +#define DEVICE_ID_SPEED_MASK (0x1f << 6) +#define DEVICE_ID_TEMP_SHIFT 3 +#define DEVICE_ID_TEMP_MASK (0x7 << 3) + +#define CTRLMMR_WKUP_JTAG_ID 0x43000014 +#define JTAG_ID_VARIANT_SHIFT 28 +#define JTAG_ID_VARIANT_MASK (0xf << 28) +#define JTAG_ID_PARTNO_SHIFT 12 +#define JTAG_ID_PARTNO_MASK (0x7ff << 1) + +#define CTRLMMR_WKUP_DIE_ID0 0x43000020 +#define CTRLMMR_WKUP_DIE_ID1 0x43000024 +#define CTRLMMR_WKUP_DIE_ID2 0x43000028 +#define CTRLMMR_WKUP_DIE_ID3 0x4300002c +#define CTRLMMR_WKUP_BOOTCFG 0x43000034 + #endif /* _ASM_ARCH_HARDWARE_H_ */

Print the board name and ver along with the DT Model. While at it print the ver for all the detected daughter cards.
Signed-off-by: Lokesh Vutla lokeshvutla@ti.com --- board/ti/am65x/evm.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/board/ti/am65x/evm.c b/board/ti/am65x/evm.c index e01adcd642..4de1eb4de0 100644 --- a/board/ti/am65x/evm.c +++ b/board/ti/am65x/evm.c @@ -116,6 +116,19 @@ int do_board_detect(void) return ret; }
+int checkboard(void) +{ + struct ti_am6_eeprom *ep = TI_AM6_EEPROM_DATA; + + if (do_board_detect()) + /* EEPROM not populated */ + printf("Board: %s ver %s\n", "AM6-COMPROCEVM", "E3"); + else + printf("Board: %s ver %s\n", ep->name, ep->version); + + return 0; +} + static void setup_board_eeprom_env(void) { char *name = "am65x"; @@ -261,7 +274,7 @@ static int probe_daughtercards(void) if (strncmp(ep.name, cards[i].card_name, sizeof(ep.name))) continue;
- printf("detected %s\n", cards[i].card_name); + printf("Detected: %s ver %s\n", ep.name, ep.version);
/* * Populate any MAC addresses from daughtercard into the U-Boot

Enable CONFIG_DISPLAY_CPUINFO so that cpuinfo is printed during boot.
Signed-off-by: Lokesh Vutla lokeshvutla@ti.com --- configs/j721e_evm_a72_defconfig | 1 - 1 file changed, 1 deletion(-)
diff --git a/configs/j721e_evm_a72_defconfig b/configs/j721e_evm_a72_defconfig index 46667da36e..cc8f99d21d 100644 --- a/configs/j721e_evm_a72_defconfig +++ b/configs/j721e_evm_a72_defconfig @@ -20,7 +20,6 @@ CONFIG_DISTRO_DEFAULTS=y # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set CONFIG_SPL_LOAD_FIT=y CONFIG_BOOTCOMMAND="run findfdt; run envboot; run init_${boot}; run get_kern_${boot}; run get_fdt_${boot}; run get_overlay_${boot}; run run_kern" -# CONFIG_DISPLAY_CPUINFO is not set CONFIG_SPL_SYS_MALLOC_SIMPLE=y CONFIG_SPL_STACK_R=y CONFIG_SPL_SEPARATE_BSS=y

Enable CONFIG_DISPLAY_CPUINFO so that cpuinfo is printed during boot.
Signed-off-by: Lokesh Vutla lokeshvutla@ti.com --- configs/am65x_evm_a53_defconfig | 1 - 1 file changed, 1 deletion(-)
diff --git a/configs/am65x_evm_a53_defconfig b/configs/am65x_evm_a53_defconfig index 17065a3813..cb7140a574 100644 --- a/configs/am65x_evm_a53_defconfig +++ b/configs/am65x_evm_a53_defconfig @@ -19,7 +19,6 @@ CONFIG_DISTRO_DEFAULTS=y CONFIG_SPL_LOAD_FIT=y CONFIG_OF_BOARD_SETUP=y CONFIG_BOOTCOMMAND="run findfdt; run envboot; run init_${boot}; run get_kern_${boot}; run get_fdt_${boot}; run get_overlay_${boot}; run run_kern" -# CONFIG_DISPLAY_CPUINFO is not set CONFIG_SPL_SYS_MALLOC_SIMPLE=y CONFIG_SPL_STACK_R=y CONFIG_SPL_SEPARATE_BSS=y

Use the driver name to get ti_sci handle rather than relying on just the FIRMWARE uclass.
Signed-off-by: Lokesh Vutla lokeshvutla@ti.com --- arch/arm/mach-k3/common.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/mach-k3/common.c b/arch/arm/mach-k3/common.c index 7339d7398d..32712e5883 100644 --- a/arch/arm/mach-k3/common.c +++ b/arch/arm/mach-k3/common.c @@ -36,7 +36,8 @@ struct ti_sci_handle *get_ti_sci_handle(void) struct udevice *dev; int ret;
- ret = uclass_get_device(UCLASS_FIRMWARE, 0, &dev); + ret = uclass_get_device_by_driver(UCLASS_FIRMWARE, + DM_GET_DRIVER(ti_sci), &dev); if (ret) panic("Failed to get SYSFW (%d)\n", ret);
participants (1)
-
Lokesh Vutla