* [U-Boot] [PATCH] configs: ti_armv7_keystone2: make SYS_TEXT_BASE configurable at build time
@ 2016-03-28 19:15 Vitaly Andrianov
2016-03-28 19:32 ` Tom Rini
2016-04-02 1:59 ` [U-Boot] " Tom Rini
0 siblings, 2 replies; 4+ messages in thread
From: Vitaly Andrianov @ 2016-03-28 19:15 UTC (permalink / raw)
To: u-boot
U-boot for general purpose KS2 devices is loaded to the beginning of the
internal memory (0x0c000000). Secure devices uses this memory and
CONFIG_SYS_TEXT_BASE has to be different for those devices.
This commit make this configurable at build time by giving
CONFIG_SYS_TEXT_BASE as a command line definition to make command.
Signed-off-by: Vitaly Andrianov <vitalya@ti.com>
---
include/configs/ti_armv7_keystone2.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/configs/ti_armv7_keystone2.h b/include/configs/ti_armv7_keystone2.h
index a7206f4..76b901bf 100644
--- a/include/configs/ti_armv7_keystone2.h
+++ b/include/configs/ti_armv7_keystone2.h
@@ -20,7 +20,9 @@
/* SoC Configuration */
#define CONFIG_ARCH_CPU_INIT
#define CONFIG_SYS_ARCH_TIMER
+#ifndef CONFIG_SYS_TEXT_BASE
#define CONFIG_SYS_TEXT_BASE 0x0c000000
+#endif
#define CONFIG_SPL_TARGET "u-boot-spi.gph"
#define CONFIG_SYS_DCACHE_OFF
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] configs: ti_armv7_keystone2: make SYS_TEXT_BASE configurable at build time
2016-03-28 19:15 [U-Boot] [PATCH] configs: ti_armv7_keystone2: make SYS_TEXT_BASE configurable at build time Vitaly Andrianov
@ 2016-03-28 19:32 ` Tom Rini
2016-03-29 11:55 ` Vitaly Andrianov
2016-04-02 1:59 ` [U-Boot] " Tom Rini
1 sibling, 1 reply; 4+ messages in thread
From: Tom Rini @ 2016-03-28 19:32 UTC (permalink / raw)
To: u-boot
On Mon, Mar 28, 2016 at 03:15:59PM -0400, Vitaly Andrianov wrote:
> U-boot for general purpose KS2 devices is loaded to the beginning of the
> internal memory (0x0c000000). Secure devices uses this memory and
> CONFIG_SYS_TEXT_BASE has to be different for those devices.
>
> This commit make this configurable at build time by giving
> CONFIG_SYS_TEXT_BASE as a command line definition to make command.
I'm interested in getting secure device support going, but it seems like
we should need more than that, ie something to keep the chain of trust
going. I also don't like that we're just overriding this in the command
line, especially as I expect we'll need other changes too once this gets
moving forward. Thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160328/3ce4d0fe/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] [PATCH] configs: ti_armv7_keystone2: make SYS_TEXT_BASE configurable at build time
2016-03-28 19:32 ` Tom Rini
@ 2016-03-29 11:55 ` Vitaly Andrianov
0 siblings, 0 replies; 4+ messages in thread
From: Vitaly Andrianov @ 2016-03-29 11:55 UTC (permalink / raw)
To: u-boot
On 03/28/2016 03:32 PM, Tom Rini wrote:
> On Mon, Mar 28, 2016 at 03:15:59PM -0400, Vitaly Andrianov wrote:
>
>> U-boot for general purpose KS2 devices is loaded to the beginning of the
>> internal memory (0x0c000000). Secure devices uses this memory and
>> CONFIG_SYS_TEXT_BASE has to be different for those devices.
>>
>> This commit make this configurable at build time by giving
>> CONFIG_SYS_TEXT_BASE as a command line definition to make command.
>
> I'm interested in getting secure device support going, but it seems like
> we should need more than that, ie something to keep the chain of trust
> going. I also don't like that we're just overriding this in the command
> line, especially as I expect we'll need other changes too once this gets
> moving forward. Thanks!
>
Tom,
I reworked boot-monitor for KS2 secure devices and CONFIG_SYS_TEXT_BASE
is only change required to support KS2 secure devices.
Thanks,
Vitaly
^ permalink raw reply [flat|nested] 4+ messages in thread
* [U-Boot] configs: ti_armv7_keystone2: make SYS_TEXT_BASE configurable at build time
2016-03-28 19:15 [U-Boot] [PATCH] configs: ti_armv7_keystone2: make SYS_TEXT_BASE configurable at build time Vitaly Andrianov
2016-03-28 19:32 ` Tom Rini
@ 2016-04-02 1:59 ` Tom Rini
1 sibling, 0 replies; 4+ messages in thread
From: Tom Rini @ 2016-04-02 1:59 UTC (permalink / raw)
To: u-boot
On Mon, Mar 28, 2016 at 03:15:59PM -0400, Vitaly Andrianov wrote:
> U-boot for general purpose KS2 devices is loaded to the beginning of the
> internal memory (0x0c000000). Secure devices uses this memory and
> CONFIG_SYS_TEXT_BASE has to be different for those devices.
>
> This commit make this configurable at build time by giving
> CONFIG_SYS_TEXT_BASE as a command line definition to make command.
>
> Signed-off-by: Vitaly Andrianov <vitalya@ti.com>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160401/99c6e795/attachment.sig>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-04-02 1:59 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-03-28 19:15 [U-Boot] [PATCH] configs: ti_armv7_keystone2: make SYS_TEXT_BASE configurable at build time Vitaly Andrianov
2016-03-28 19:32 ` Tom Rini
2016-03-29 11:55 ` Vitaly Andrianov
2016-04-02 1:59 ` [U-Boot] " Tom Rini
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.