[PATCH u-boot-marvell 00/18] Upgrade A38x DDR3 training to version 14.0.0

[Chris Packham]

On 9/02/21 10:15 pm, Pali Rohár wrote:
> On Tuesday 09 February 2021 01:08:54 Chris Packham wrote:
>> On 9/02/21 1:16 pm, Chris Packham wrote:
>>> On 9/02/21 9:18 am, Marek Behun wrote:
>>>> On Mon, 8 Feb 2021 20:11:06 +0000
>>>> Chris Packham <Chris.Packham at alliedtelesis.co.nz> wrote:
>>>>
>>>>> Hi Marek,
>>>>>
>>>>> Do you have this in a repo I can pull from? I've got a couple of boards
>>>>> I can give this a spin on.
>>>> https://gitlab.nic.cz/turris/turris-omnia-uboot/
>>>> branch v2021.04-rc-mv-ddr-14.0.0
>>>>
>>>> also please test branch v2021.04-rc-mv-ddr-14.0.0-samsung-ddr-fix, that
>>>> one contains one more commit that is needed for Omnia with Samsung DDR
>>>> chips.
>>> I've tested the dm-88f6820-amc board. Training completed without
>>> issue, as does memtester running from Linux.
>>>
>>> Hit a bit of a snag on the x530 because the changes pushed it over the
>>> SPL size (it was already pretty close). I'll look to see if there's
>>> anything I can drop out or maybe bump the SPL size (I never did get a
>>> clear answer from Marvell as to what the size limit actually is).
>> I can temporarily work around the size issue by disabling watchdog
>> support in SPL (I really don't want that to be the long term solution).
> If you need to decrease size of U-Boot binary, can you try enabling
> following two options?
Technically it's the size of the spl not u-boot proper.
> CONFIG_CC_OPTIMIZE_FOR_SIZE=y
This is already set it's default y.
> CONFIG_OPTIMIZE_INLINING=y
Setting both CONFIG_OPTIMIZE_INLINING=y and 
CONFIG_SPL_OPTIMIZE_INLINING=y don't help (they actually make the SPL a 
tiny bit bigger).