public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 4/5] ti_omap5_common: mark region of DRAM protected on HS parts
Date: Fri, 2 Sep 2016 10:54:47 -0400	[thread overview]
Message-ID: <20160902145447.GX4990@bill-the-cat> (raw)
In-Reply-To: <1472794824-6032-5-git-send-email-d-allred@ti.com>

On Fri, Sep 02, 2016 at 12:40:23AM -0500, Daniel Allred wrote:

> If the ending portion of the DRAM is reserved for secure
> world use, then u-boot cannot use this memory for its relocation
> purposes. To prevent issues, we mark this memory as PRAM and this
> prevents it from being used by u-boot at all.
> 
> Signed-off-by: Daniel Allred <d-allred@ti.com>
> ---
>  include/configs/ti_omap5_common.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/include/configs/ti_omap5_common.h b/include/configs/ti_omap5_common.h
> index e42c88e..70fdc6e 100644
> --- a/include/configs/ti_omap5_common.h
> +++ b/include/configs/ti_omap5_common.h
> @@ -146,6 +146,14 @@
>   */
>  #define TI_OMAP5_SECURE_BOOT_RESV_SRAM_SZ	0x1000
>  #define CONFIG_SPL_TEXT_BASE	0x40301350
> +/* If no specific start address is specified then the secure EMIF
> + * region will be placed at the end of the DDR space. In order to prevent
> + * the main u-boot relocation from clobbering that memory and causing a
> + * firewall violation, we tell u-boot that memory is protected RAM (PRAM)
> + */
> +#if (CONFIG_TI_SECURE_EMIF_REGION_START == 0)
> +#define CONFIG_PRAM (CONFIG_TI_SECURE_EMIF_TOTAL_REGION_SIZE) >> 10
> +#endif
>  #else
>  /*
>   * For all booting on GP parts, the flash loader image is

Can you move PRAM to Kconfig please?  Thinking about this more, I think
what you want for the size option is CONFIG_PRAM and then to use that to
expand out the size.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160902/020b314d/attachment.sig>

  reply	other threads:[~2016-09-02 14:54 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-02  5:40 [U-Boot] [PATCH 0/5] Secure EMIF firewall and memory reservation on DRA7xx/AM57x devices Daniel Allred
2016-09-02  5:40 ` [U-Boot] [PATCH 1/5] ti: omap5: Add Kconfig options for secure EMIF reservations Daniel Allred
2016-09-02 14:54   ` Tom Rini
2016-09-06 20:23   ` Andrew F. Davis
2016-09-07  4:17     ` Allred, Daniel
2016-10-03 13:37   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-02  5:40 ` [U-Boot] [PATCH 2/5] arm: omap5: secure API for EMIF memory reservations Daniel Allred
2016-09-02 14:54   ` Tom Rini
2016-10-03 13:37   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-02  5:40 ` [U-Boot] [PATCH 3/5] ARM: DRA7: Add secure emif setup calls Daniel Allred
2016-09-02 14:54   ` Tom Rini
2016-09-07  3:50     ` Allred, Daniel
2016-09-06 20:41   ` Andrew F. Davis
2016-09-07  3:57     ` Allred, Daniel
2016-10-03 13:37   ` [U-Boot] [U-Boot,3/5] " Tom Rini
2016-09-02  5:40 ` [U-Boot] [PATCH 4/5] ti_omap5_common: mark region of DRAM protected on HS parts Daniel Allred
2016-09-02 14:54   ` Tom Rini [this message]
2016-09-06 20:54   ` Andrew F. Davis
2016-09-06 23:25     ` Tom Rini
2016-09-07  3:48     ` Allred, Daniel
2016-10-03 13:38   ` [U-Boot] [U-Boot, " Tom Rini
2016-09-02  5:40 ` [U-Boot] [PATCH 5/5] ARM: omap5: add fdt secure dram reservation fixup Daniel Allred
2016-09-02 14:54   ` Tom Rini
2016-09-07 19:48   ` Andrew F. Davis
2016-10-03 13:38   ` [U-Boot] [U-Boot, " Tom Rini

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=20160902145447.GX4990@bill-the-cat \
    --to=trini@konsulko.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox