Re: [U-Boot] [PATCH v1 7/7] imx: add status reporting for HAB status

2013-07-12 Thread Stefano Babic
Hi Fabio, On 11/07/2013 19:17, Fabio Estevam wrote: Hi Stefano, On Thu, Jul 11, 2013 at 10:06 AM, Stefano Babic sba...@denx.de wrote: --- a/arch/arm/cpu/armv7/mx6/Makefile +++ b/arch/arm/cpu/armv7/mx6/Makefile @@ -27,7 +27,7 @@ include $(TOPDIR)/config.mk LIB= $(obj)lib$(SOC).o

Re: [U-Boot] [PATCH v1 7/7] imx: add status reporting for HAB status

2013-07-12 Thread Stefano Babic
Hi Marek, On 11/07/2013 20:31, Marek Vasut wrote: +void display_event(uint8_t *event_data, size_t bytes) +{ +uint32_t i; + +if ((event_data) (bytes 0)) { if (!cond) return; The loop goes here Fix in V2, thanks. +if (is_hab_enabled()) +

[U-Boot] [PATCH v1 7/7] imx: add status reporting for HAB status

2013-07-11 Thread Stefano Babic
Add functions to report the HAB (High Assurance Boot) status of e.g. i.MX6 CPUs. This is taken from git://git.freescale.com/imx/uboot-imx.git branch imx_v2009.08_3.0.35_4.0.0 cpu/arm_cortexa8/mx6/generic.c include/asm-arm/arch-mx6/mx6_secure.h Signed-off-by: Stefano Babic sba...@denx.de ---

Re: [U-Boot] [PATCH v1 7/7] imx: add status reporting for HAB status

2013-07-11 Thread Fabio Estevam
Hi Stefano, On Thu, Jul 11, 2013 at 10:06 AM, Stefano Babic sba...@denx.de wrote: --- a/arch/arm/cpu/armv7/mx6/Makefile +++ b/arch/arm/cpu/armv7/mx6/Makefile @@ -27,7 +27,7 @@ include $(TOPDIR)/config.mk LIB= $(obj)lib$(SOC).o Whole series looks good. Only a minor comment: -COBJS

Re: [U-Boot] [PATCH v1 7/7] imx: add status reporting for HAB status

2013-07-11 Thread Marek Vasut
Dear Stefano Babic, Add functions to report the HAB (High Assurance Boot) status of e.g. i.MX6 CPUs. This is taken from git://git.freescale.com/imx/uboot-imx.git branch imx_v2009.08_3.0.35_4.0.0 cpu/arm_cortexa8/mx6/generic.c include/asm-arm/arch-mx6/mx6_secure.h Signed-off-by: