On 13/10/21, Jorge Ramirez-Ortiz wrote:

Output the secure boot configuration to the console.

Hi, Michal was there any reason for not merging this patch? thanks Jorge

Signed-off-by: Jorge Ramirez-Ortiz jorge@foundries.io

v2: Michal review 12 Aug 2021 print information on SPL and UBOOT improve the print command add macros to mask the status

arch/arm/mach-zynqmp/include/mach/hardware.h | 6 +++++- board/xilinx/zynqmp/zynqmp.c | 14 ++++++++++++++ 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mach-zynqmp/include/mach/hardware.h b/arch/arm/mach-zynqmp/include/mach/hardware.h index eebf38551c..c6b1464a9f 100644 --- a/arch/arm/mach-zynqmp/include/mach/hardware.h +++ b/arch/arm/mach-zynqmp/include/mach/hardware.h @@ -141,8 +141,12 @@ struct apu_regs { #define ZYNQMP_SILICON_VER_MASK 0xF #define ZYNQMP_SILICON_VER_SHIFT 0

+#define ZYNQMP_CSU_STATUS_AUTHENTICATED BIT(0) +#define ZYNQMP_CSU_STATUS_ENCRYPTED BIT(1)

struct csu_regs {

• u32 reserved0[4];

• u32 status;
• u32 reserved0[3]; u32 multi_boot; u32 reserved1[11]; u32 idcode;

diff --git a/board/xilinx/zynqmp/zynqmp.c b/board/xilinx/zynqmp/zynqmp.c index 000a7cde8d..3e7ca2e9c6 100644 --- a/board/xilinx/zynqmp/zynqmp.c +++ b/board/xilinx/zynqmp/zynqmp.c @@ -358,6 +358,18 @@ static int multi_boot(void) return multiboot; }

+static void print_secure_boot(void) +{

• u32 status = 0;
• if (zynqmp_mmio_read((ulong)&csu_base->status, &status))

• return;
  

• printf("Secure Boot:\t%sauthenticated, %sencrypted\n",

•       status & ZYNQMP_CSU_STATUS_AUTHENTICATED ? "" : "not ",
  

•       status & ZYNQMP_CSU_STATUS_ENCRYPTED ? "" : "not ");
  

+}

#define PS_SYSMON_ANALOG_BUS_VAL 0x3210 #define PS_SYSMON_ANALOG_BUS_REG 0xFFA50914

@@ -394,6 +406,8 @@ int board_init(void) fpga_add(fpga_xilinx, &zynqmppl); #endif

• /* display secure boot information */
• print_secure_boot(); if (current_el() == 3) printf("Multiboot:\t%d\n", multi_boot());

-- 2.31.1