All of lore.kernel.org
 help / color / mirror / Atom feed
From: Kever Yang <kever.yang@rock-chips.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [U-Boot, 07/15] spl: atf: introduce spl_invoke_atf and make bl31_entry private
Date: Fri, 15 Dec 2017 11:14:06 +0800	[thread overview]
Message-ID: <4d678a4e-0e2c-f1ef-076a-40b346395027@rock-chips.com> (raw)
In-Reply-To: <E1eHsqh-0003mk-0a@mail.theobroma-systems.com>

Hi Philipp,

     This patch use fdt_addr as plat_params, break the compatible with 
upstream

ATF, and get error:

"ERROR:   not expected type found 6410029648624618960"

The ATF do have a requirement for plat_params structure, and fdt_addr 
does not match this:

/* common header for all plat parameter type */
struct bl31_plat_param {
 >-------uint64_t type;
 >-------void *next;
};


Thanks,
- Kever

On 11/23/2017 10:51 PM, Philipp Tomsich wrote:
>> This adds a new interface spl_invoke_atf() that takes a spl_image_info
>> argument and then derives the necessary parameters for the ATF entry.
>> Based on the additional information recorded (into /fit-images) from
>> the FIT loadables, we can now easily locate the next boot stage.
>>
>> We now pass a pointer to a FDT as the platform-specific parameter
>> pointer to ATF (so we don't run into the future headache of every
>> board/platform defining their own proprietary tag-structure), as
>> FDT access is already available in ATF.
>>
>> With the necessary infrastructure in place, we can now update the
>> support for the ARM Trusted Firmware to dispatch into the
>> spl_invoke_atf function only if a IH_OS_ARM_TRUSTED_FIRMWARE image is
>> loaded.
>>
>> Signed-off-by: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
>> Reviewed-by: Simon Glass <sjg@chromium.org>
>> ---
>>
>>   common/spl/spl.c     | 11 +++----
>>   common/spl/spl_atf.c | 84 +++++++++++++++++++++++++++++++++++++++++++++++-----
>>   2 files changed, 82 insertions(+), 13 deletions(-)
>>
> Applied to u-boot-rockchip, thanks!
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot

  reply	other threads:[~2017-12-15  3:14 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1505330989-25602-1-git-send-email-philipp.tomsich@theobroma-systems.com>
2017-09-13 19:29 ` [U-Boot] [PATCH 01/15] image: add IH_OS_ARM_TRUSTED_FIRMWARE for ARM Trusted Firmware Philipp Tomsich
2017-09-17 17:53   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 02/15] spl: add a fdt_addr field to spl_image_info Philipp Tomsich
2017-09-17 17:53   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 03/15] spl: change load_addr and entry_point to uintptr_t Philipp Tomsich
2017-09-17 17:53   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 04/15] spl: fit: simplify logic for FDT loading for non-OS boots Philipp Tomsich
2017-09-13 21:16   ` York Sun
2017-09-13 21:21     ` Dr. Philipp Tomsich
2017-09-13 21:24       ` York Sun
2017-09-13 22:11         ` Dr. Philipp Tomsich
2017-09-14 17:51           ` York Sun
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 05/15] spl: fit: implement fdt_record_loadable Philipp Tomsich
2017-09-17 17:53   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 06/15] spl: fit: implement recording of loadables into /fit-images Philipp Tomsich
2017-09-17 17:53   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2018-01-18 12:56   ` [U-Boot] [PATCH " Michal Simek
2018-01-18 13:17     ` Dr. Philipp Tomsich
2018-01-18 13:40       ` Michal Simek
2018-02-01  7:51       ` Michal Simek
2018-02-02 11:03         ` Dr. Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 07/15] spl: atf: introduce spl_invoke_atf and make bl31_entry private Philipp Tomsich
2017-09-17 17:53   ` Simon Glass
2017-11-07  9:30     ` Dr. Philipp Tomsich
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-12-15  3:14     ` Kever Yang [this message]
2017-09-13 19:29 ` [U-Boot] [PATCH 08/15] spl: rename config item SPL_ATF_SUPPORT to SPL_ATF Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 09/15] spl: atf: drop the SPL_ATF_TEXT_BASE configuration item Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 10/15] rockchip: dts: rk3399-puma: add /config/arm-trusted-firmware, reset-gpio property Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-07  9:40     ` Dr. Philipp Tomsich
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-11-23 15:52     ` Dr. Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 11/15] rockchip: defconfig: firefly-rk3399: sync up with SPL changes for ATF Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 12/15] rockchip: board: puma-rk3399: update .its file to use new features Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 13/15] rockchip: board: lion-rk3368: update .its file Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 14/15] rockchip: defconfig: puma-rk3399: sync up with SPL changes for ATF Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-13 19:29 ` [U-Boot] [PATCH 15/15] rockchip: defconfig: lion-rk3368: " Philipp Tomsich
2017-09-17 17:54   ` Simon Glass
2017-11-23 14:51   ` [U-Boot] [U-Boot, " Philipp Tomsich
2017-09-25  9:05 ` [U-Boot] [PATCH 00/15] spl: atf: update booting images via ATF to use info from FIT images Michal Simek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4d678a4e-0e2c-f1ef-076a-40b346395027@rock-chips.com \
    --to=kever.yang@rock-chips.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.