[PATCH u-boot-mvebu 2/3] arm: mvebu: a37xx: Map CCI-400 and AP BootROM address space

[Marek Behún]

On Tue, 15 Feb 2022 14:15:17 +0100
Marek Behún <marek.behun at nic.cz> wrote:

> > In _production version_ where is no debug capability and no access to
> > any memory (just ability to boot) is is probably not needed, but none of
> > A3720 board is building this kind of version (by default). And in case
> > BootROM is mapped also in these versions, is there any issue with it?  
> 
> My issue is that it isn't needed. You can just dump it once, publish it
> somewhere, and you are done. No need to keep that window mapped for
> everyone else.

BTW, I can imagine situation where mapping BootROM code can be useful.
For example if the BootROM code contains some cryptographic functions
(which are necessary for secure boot) and you know where they are and
their type, so you can use them if you want to save space in your own
code.

But mapping BootROM so that everyone, if they want, can dump it, is
unnecessary IMO, because you can simply do it once and then you have
the code.

Keep in mind that this is my opinion. If Stefan agrees with you, I have
no issue with merging this.

Marek