Re: [PATCHv2] arm64: zynqmp: Print the secure boot status information

["Jorge Ramirez-Ortiz, Foundries" <jorge@foundries.io>]

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
>