From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Wed, 22 May 2019 13:39:52 -0600
Subject: [U-Boot]
	=?utf-8?q?=5BPATCH_v2_2/3=5D_spl=3A_add_relocation_suppo?=
	=?utf-8?b?cnTjgJDor7fms6jmhI/vvIzpgq7ku7bnlLFzamdAZ29vZ2xlLmNvbQ==?=
	=?utf-8?b?5Luj5Y+R44CR?=
In-Reply-To: <CANbgqATzo5Z0td=G1Azed_bbNW+OpNPd=30gjLExnYhF8sbGqA@mail.gmail.com>
References: <20190516121926.31927-1-andy.yan@rock-chips.com>
 <20190516122150.32163-1-andy.yan@rock-chips.com>
 <CAPnjgZ1f-DOEj+rd4Fi=X8xPUkT_cbvSGcXgUFuY0DB+=Z5Obg@mail.gmail.com>
 <99b9d595-7d1e-4cb5-63bf-12826570a0d7@rock-chips.com>
 <20190520165858.532e6f1e@donnerap.cambridge.arm.com>
 <CANbgqATzo5Z0td=G1Azed_bbNW+OpNPd=30gjLExnYhF8sbGqA@mail.gmail.com>
Message-ID: <CAPnjgZ3K=tEpoRMME6r_Kxu2CQ7NLLRTWfScwtWy883O2uKP_A@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
To: u-boot@lists.denx.de

Hi Andy,

On Tue, 21 May 2019 at 19:43, Andy Yan <andyshrk@gmail.com> wrote:
>
> Hi Andre:
>
> Andre Przywara <andre.przywara@arm.com> 于2019年5月20日周一 下午11:59写道：
>
> > On Mon, 20 May 2019 14:34:01 +0800
> > Andy Yan <andy.yan@rock-chips.com> wrote:
> >
> > Hi,
> >
> > > On 2019/5/19 上午12:26, Simon Glass wrote:
> > > > Hi Andy,
> > > >
> > > > Instead of this could you:
> > > >
> > > > - move ATF?
> > >
> > > All rockchip based arm64 ATF run from the start 64KB of dram as this
> > > will give convenient for kernel manage the memory.
> >
> > This is just BL31 of ATF, right?
> > ATF recently gained PIE support for BL31 [1], so by just enabling this in
> > platform.mk you will get a relocatable BL31 image, with a very minimal
> > runtime linker. Worked out of the box on Allwinner for me, but YMMV.
> > So you could load newer ATF builds everywhere.
> >
> >
> This is not the root case, actually we want put ATF as close as possible to
> the start of dram, this give linux kernel convenient to manage the memory.

But instead of 64KB you could put it at 32KB or 128KB. It's still in
the first 1MB. Linux won't care, right?

>
>
>
> > Does that help you?
> >
> > > On the other hand, change the ATF load address will break the
> > > compatibility of the exiting firmware.
> >
> > I am not sure what you mean with "compatibility of existing firmware"?
> > Surely you combine all the firmware components (SPL/TPL/ATF/U-Boot proper)
> > into one image? And there would be no real mix and match, with older
> > pre-compiled builds? So by changing the ATF base address and the load
> > address in TPL at the same time you won't have issues?
> >
>
> I mean older pre-compiled builds published by rockchip rkbin [1], many
> projects and popular boards directly use this , for example Armbian. How to
> change the base address of the pre-build binary?
>  [1] https://github.com/rockchip-linux/rkbin

Perhaps I am misunderstanding your intent here, but mainline U-Boot
should not be bound to the design decisions of old closed-source
binaries.

[...]

Regards,
Simon