All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <ijc@hellion.org.uk>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v6 00/15] Add PSCI support for Jetson TK1/Tegra124 + CNTFRQ fix
Date: Fri, 17 Apr 2015 15:04:36 +0100	[thread overview]
Message-ID: <1429279476.25195.333.camel@hellion.org.uk> (raw)
In-Reply-To: <55311148.3090309@wwwdotorg.org>

On Fri, 2015-04-17 at 07:57 -0600, Stephen Warren wrote:
> On 04/17/2015 12:47 AM, Jan Kiszka wrote:
> > On 2015-04-14 16:30, Ian Campbell wrote:
> >> Would enabling it in the compile but adding "bootm_boot_mode=sec" to the
> >> default environment (so it isn't used by default) be considered
> >> sufficiently backwards compatible?
> >
> > This turned out to not work as expected: booting in secure mode seems
> > to prevent that Linux can bring up CPUs 1-3. Not sure if this is to be
> > expected or a bug, but I will now take a different route:
> 
> That was the whole point of the environment variable suggestion; the 
> environment variable would default to off so nobody got new behaviour, 
> but anyone who wanted to boot in secure mode could simply set the 
> environment variable and get it. That way, nobody who doesn't want the 
> feature needs to co-ordinate U-Boot and kernel updates. Why doesn't that 
> work?

Right, if this doesn't work then it is certainly a bug somewhere on the
u-boot side.

I fixed one such issue a while back ("ARM: bootm: do not add PSCI to fdt
when booting in secure mode.") but that's ancient history, but I know
that at that time it was working for me (I'm reasonably sure that was on
Tegra with my earlier PSCI patches)

Ian.

  parent reply	other threads:[~2015-04-17 14:04 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-13  4:48 [U-Boot] [PATCH v6 00/15] Add PSCI support for Jetson TK1/Tegra124 + CNTFRQ fix Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 01/15] sun7i: Remove duplicate call to psci_arch_init Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 02/15] ARM: Factor out common psci_get_cpu_id Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 03/15] ARM: Factor out reusable psci_cpu_off_common Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 04/15] ARM: Factor out reusable psci_cpu_entry Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 05/15] ARM: Factor out reusable psci_get_cpu_stack_top Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 06/15] ARM: Put target PC for PSCI CPU_ON on per-CPU stack Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 07/15] tegra124: Add more registers to struct mc_ctlr Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 08/15] virt-dt: Allow reservation of secure region when in a RAM carveout Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 09/15] tegra: Make tegra_powergate_power_on public Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 10/15] tegra: Add ap_pm_init hook Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 11/15] tegra124: Add PSCI support for Tegra124 Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 12/15] tegra124: Reserve secure RAM using MC_SECURITY_CFG{0, 1}_0 Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 13/15] tegra: Set CNTFRQ for secondary CPUs Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 14/15] ARM: tegra: Enable SMMU when going non-secure Jan Kiszka
2015-04-13  4:48 ` [U-Boot] [PATCH v6 15/15] jetson-tk1: Add PSCI configuration options and reserve secure code Jan Kiszka
2015-04-14 13:46 ` [U-Boot] [PATCH v6 00/15] Add PSCI support for Jetson TK1/Tegra124 + CNTFRQ fix Tom Rini
2015-04-14 14:06   ` Stephen Warren
2015-04-14 14:12     ` Jan Kiszka
2015-04-14 14:21       ` Stephen Warren
2015-04-14 14:30       ` Ian Campbell
2015-04-14 14:33         ` Jan Kiszka
2015-04-14 14:40           ` Ian Campbell
2015-04-14 14:45             ` Jan Kiszka
2015-04-14 14:50               ` Stephen Warren
2015-04-14 15:05                 ` Jan Kiszka
2015-04-14 14:35         ` Stephen Warren
2015-04-17  6:47         ` Jan Kiszka
2015-04-17 13:57           ` Stephen Warren
2015-04-17 14:02             ` Jan Kiszka
2015-04-17 14:12               ` Stephen Warren
2015-04-17 14:20                 ` Jan Kiszka
2015-04-17 14:43                   ` Stephen Warren
2015-04-18 11:42                     ` Jan Kiszka
2015-04-17 14:04             ` Ian Campbell [this message]
2015-04-17  6:56 ` [U-Boot] [PATCH 14.5/15] tegra: Keep virt support disabled by default Jan Kiszka

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=1429279476.25195.333.camel@hellion.org.uk \
    --to=ijc@hellion.org.uk \
    --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.