[U-Boot] [PATCH] rockchip: rk3399-puma: Re-disable SPL_ATF_NO_PLATFORM_PARAM

[Christoph Müllner]

On 28.05.19 14:55, Kever Yang wrote:
> Hi Christoph and Mark,
> 
>     Does this patch ready for rk3399-puma?

Hi Kever,

yes, this commit is ready for mainline.
However, there are more (not Puma specific) commits needed to make get
mainline ATF working. I haven't got time recently to work on them.

If you want we can wait for the complete series.
But, as I said, the commit is ready for mainline now and
would just be merged later.

Thanks,
Christoph

> 
> 
> Thanks,
> - Kever
> On 05/07/2019 05:18 PM, Christoph Müllner wrote:
>> Hi Mark,
>>
>> On 07.05.19 10:57, Mark Kettenis wrote:
>>>> From: Christoph Muellner <christoph.muellner at theobroma-systems.com>
>>>> Date: Tue,  7 May 2019 10:39:54 +0200
>>>>
>>>> This reverts commit c869d63f2785ccde22582f96cf40cb8809f838ba.
>>>>
>>>> Mainline ATF already supports fdt-parsing, which allows us to
>>>> pass rk3399-puma specific data to setup things like the correct
>>>> UART. Therefore we can re-enable platform parameterss again.
>>> Hi Christoph,
>>>
>>> I tried this last weekend on my rk3399-puma board with the (then)
>>> latest ATF and that didn't work for me.  Looked like the board simply
>>> reset shortly after jumping into the ATF code.  Now I may very well
>>> have done something wrong, and I ran out of time debugging this.
>> There's a lot more needed to get mainline ATF up and running on RK3399:
>>
>> * Only mainline ATF will be able to use correct console (UART0)
>> * bouncebuf needs to be aware of SD DMA not being able to copy to SRAM/PMUSRAM
>> * bouncebuf needs static buffer to address heap limitation
>> * bugfix in ITS generatorscript
>>
>> I've already addressed all issues for SD card boot (Fedora boots fine).
>>
>> If you want to test, you can use my ATF tree:
>>
>> https://git.theobroma-systems.com/puma-u-boot.git/log/?h=puma-v2019.04-atf
>>
>> Status for booting from SPI NOR is, that ATF will be branched to,
>> but then the board hangs. However, I haven't analyzed the situation
>> in detail yet.
>>
>> BR
>> Christoph
>>
>>> Cheers,
>>>
>>> Mark
>>>
>>>>  configs/puma-rk3399_defconfig | 1 -
>>>>  1 file changed, 1 deletion(-)
>>>>
>>>> diff --git a/configs/puma-rk3399_defconfig b/configs/puma-rk3399_defconfig
>>>> index 964464ac0f..f4c5168011 100644
>>>> --- a/configs/puma-rk3399_defconfig
>>>> +++ b/configs/puma-rk3399_defconfig
>>>> @@ -33,7 +33,6 @@ CONFIG_SPL_I2C_SUPPORT=y
>>>>  CONFIG_SPL_POWER_SUPPORT=y
>>>>  CONFIG_SPL_SPI_LOAD=y
>>>>  CONFIG_SPL_ATF=y
>>>> -CONFIG_SPL_ATF_NO_PLATFORM_PARAM=y
>>>>  CONFIG_CMD_BOOTZ=y
>>>>  CONFIG_CMD_GPT=y
>>>>  CONFIG_CMD_I2C=y
>>>> -- 
>>>> 2.11.0
>>>>
>>>> _______________________________________________
>>>> U-Boot mailing list
>>>> U-Boot at lists.denx.de
>>>> https://lists.denx.de/listinfo/u-boot
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot at lists.denx.de
>> https://lists.denx.de/listinfo/u-boot
> 
> 
>