All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/3] arm: socfpga: Convert reset manager from struct to defines
Date: Wed, 28 Aug 2019 15:39:30 +0200	[thread overview]
Message-ID: <8a6cfb10-7e62-9f39-2409-0691fbc512c2@denx.de> (raw)
In-Reply-To: <CAFiDJ59m8LUjn-xCyJg3Eu0cv4ra2xyPtn=cYMoHP_Mh--O4Gg@mail.gmail.com>

On 8/28/19 3:28 PM, Ley Foon Tan wrote:

[...]

>>>    Tested this on Agilex, look okay. But, pending test on other device
>>> families, like Gen5 or A10. Not sure any side effect with this change.
>>> 2. Secure (for PSCI) and non-secure functions share same functions,
>>> these functions can't use global data (for base address). Secure
>>> functions have its own secure data region.
>>
>> There was a discussion about this before, instead of hacking up this
>> part constantly, maybe we should create a separate build of U-Boot (like
>> SPL/TPL) and use that for the "secure" mode. But that's for another
>> patchset and another discussion.
> Yes, we have discussion here internally regarding this secure firmware.
> We plan change the boot flow to include ATF. But, our downstream git
> still need to maintain this "secure" region for some period (so
> customer can migrate to new boot flow).
> If change to DT address, we need patch our downstream git for these
> secure functions. :)
> Anyway, I will try change for upstream. I'm trying to find where is
> the proper place to get DT address in Uboot. SPL is more straight
> forward, get the DT address in board_init_f.

I have to admit, I have many things to say about ATF and none of them
are positive. So it's real sad this is starting to move onto S10 too.

Anyway, one of the fdt_ functions should allow you to fish out the
address from DT.

  reply	other threads:[~2019-08-28 13:39 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-20  2:35 [U-Boot] [PATCH 0/3] arm: socfpga: Convert drivers from struct to defines Ley Foon Tan
2019-08-20  2:35 ` [U-Boot] [PATCH 1/3] arm: socfpga: Convert reset manager " Ley Foon Tan
2019-08-20  9:37   ` Marek Vasut
2019-08-20  9:55     ` Simon Goldschmidt
2019-08-20 10:15       ` Marek Vasut
2019-08-20 10:29         ` Simon Goldschmidt
2019-08-20 11:24           ` Marek Vasut
2019-08-21  1:51             ` Ley Foon Tan
2019-08-21  1:50     ` Ley Foon Tan
2019-08-23  8:57       ` Ley Foon Tan
2019-08-23  9:22         ` Marek Vasut
2019-08-23  9:52           ` Ley Foon Tan
2019-08-23  9:56             ` Marek Vasut
2019-08-26  7:28               ` Ley Foon Tan
2019-08-26  8:50                 ` Marek Vasut
2019-08-26 10:58                   ` Simon Goldschmidt
2019-08-28  8:13               ` Ley Foon Tan
2019-08-28 11:14                 ` Marek Vasut
2019-08-28 13:03                   ` Simon Goldschmidt
2019-08-28 13:28                   ` Ley Foon Tan
2019-08-28 13:39                     ` Marek Vasut [this message]
2019-08-28 14:07                       ` Ley Foon Tan
2019-08-20  2:35 ` [U-Boot] [PATCH 2/3] arm: socfpga: Convert system " Ley Foon Tan
2019-08-20  2:35 ` [U-Boot] [PATCH 3/3] arm: socfpga: Convert clock " Ley Foon Tan
2019-08-20  8:33 ` [U-Boot] [PATCH 0/3] arm: socfpga: Convert drivers " Simon Goldschmidt

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=8a6cfb10-7e62-9f39-2409-0691fbc512c2@denx.de \
    --to=marex@denx.de \
    --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.