From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peng Fan <van.freenix@gmail.com>
Date: Mon, 15 Jan 2018 12:00:16 +0800
Subject: [U-Boot] [PATCH 0/9] Add new OPTEE bootm support to u-boot
In-Reply-To: <1515768744-25246-1-git-send-email-bryan.odonoghue@linaro.org>
References: <1515768744-25246-1-git-send-email-bryan.odonoghue@linaro.org>
Message-ID: <20180115040016.GA14281@shlinux2>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Bryan,
On Fri, Jan 12, 2018 at 02:52:15PM +0000, Bryan O'Donoghue wrote:
>This series adds a new OPTEE bootable image type to u-boot, which is
>directly bootable with the bootm command.
>
>There is already a TEE image type but, in this case the TEE firmware is
>loaded into RAM, jumped into and then back out of. This image type is a
>directly bootable image as described here :
>http://mrvan.github.io/optee-imx6ul
>
>Instead of reusing the Linux bootable image type instead a new image type
>is defined, which allows us to perform additional image verification, prior
>to handing off control via bootm.
>
>OPTEE images get linked to a specific address at compile time and must be
>loaded to this address too. This series extends out mkimage with a new
>image type that allows the OPTEE binary link location to be validated
>against CONFIG_OPTEE_TZDRAM_BASE and CONFIG_OPTEE_TZDRAM_SIZE respectively
>prior to proceeding through the bootm phase.
>
>Once applied you can generate a bootable OPTEE image like this
>
>mkimage -A arm -T optee -C none -d ./out/arm-plat-imx/core/tee.bin uTee.optee
>
>That image can then be booted directly by bootm. bootm will verify the
>header contents of the OPTEE binary against the DRAM area carved out in
>u-boot. If the defined DRAM area does not match the link address specified
>we refuse to boot.
>
>Kever - I'd like to suggest that your OPTEE SPL image takes a different
>image type IH_TYPE_OPTEE_SPL ? to indicate the different behavior your
>image type has versus a directly bootable bootm image.
>
>Bryan O'Donoghue (9):
>  optee: Add lib entries for sharing OPTEE code across ports
>  optee: Add CONFIG_OPTEE_TZDRAM_SIZE
>  optee: Make OPTEE_TZDRAM_BASE a mandatory define
>  optee: Add optee_image_get_entry_point()
>  optee: Add optee_image_get_load_addr()
>  tools: mkimage: add optee image type
>  optee: Add optee_verify_bootm_image()
>  optee: Improve error printout
>  bootm: optee: Add mechanism to validate an OPTEE image before boot
>
> common/bootm.c        | 11 +++++++-
> common/image.c        |  1 +
> include/image.h       |  1 +
> include/tee/optee.h   | 41 ++++++++++++++++++++++++++++++
> lib/Kconfig           |  1 +
> lib/Makefile          |  1 +
> lib/optee/Kconfig     | 16 ++++++++++++
> lib/optee/Makefile    |  7 ++++++
> lib/optee/optee.c     | 69 +++++++++++++++++++++++++++++++++++++++++++++++++++
> tools/default_image.c | 25 ++++++++++++++-----
> 10 files changed, 166 insertions(+), 7 deletions(-)
> create mode 100644 lib/optee/Kconfig
> create mode 100644 lib/optee/Makefile
> create mode 100644 lib/optee/optee.c

Thanks for patchset. I gave a test on i.MX7D-SDB and it works well.

Tested-by: Peng Fan <peng.fan@nxp.com>

>
>-- 
>2.7.4
>
>_______________________________________________
>U-Boot mailing list
>U-Boot at lists.denx.de
>https://lists.denx.de/listinfo/u-boot

--