[U-Boot] [PATCH 2/2] rockchip: firefly-rk3399: enable SPL_ATF_NO_PLATFORM_PARAM

[Kever Yang]

Michal,


On 01/04/2018 03:56 PM, Michal Simek wrote:
> Hi Kever,
>
> On 3.1.2018 03:34, Kever Yang wrote:
>> Hi Philipp,
>>
>>
>> On 01/03/2018 04:16 AM, Philipp Tomsich wrote:
>>> The Rockchip-released ATF for the Firefly apparently (i.e. Kever
>>> reported this) does not tolerate a FDT being passed as the platform
>>> parameter and will run into a hard stop.
>>>
>>> To work around this limitation in the ATF parameter handling, we
>>> enable SPL_ATF_NO_PLATFORM_PARAM (which will force passing NULL for
>>> the platform parameters).
>>>
>>> Note that this only affects this platform, as the ATF releases for the
>>> RK3368 and RK3399 have always either ignored the platform parameter
>>> (i.e. before the FDT-based parameters were supported) or support
>>> receiving a pointer to a FDT.
>> Upstream ATF of Rockchip platform support a vendor defined
>> " structbl31_plat_param *bl2_param", will add support for FDT later,
>> see params_early_setup() in
>> https://github.com/ARM-software/arm-trusted-firmware/blob/master/plat/rockchip/common/params_setup.c
>   What exactly do you want to configure in ATF from FDT?
> Do you have any code somewhere to take a look?

Sorry, no code for it now.
I think we will use and append reserved memory for it.
SPL set available memory region, and ATF reserved for its TA so that 
U-Boot won't use it,
we are using the hard code for this feature now.

Thanks,
- Kever
>
>
> Thanks,
> Michal
>