Kever,

We use a fork of the ATF repository and have the M3 firmware completely 
factored out and have changed parts of the SRAM setup.
In other words: there will be two separate .bin files created that need to be 
loaded to two separate locations.
Please refer to our board’s README for the steps to get it set up and booted.

Note that there will be a structural update to our ATF in the next couple of 
weeks that is expected to change things around quite a bit.

Regards,
Philipp.

> On 02 Aug 2017, at 05:10, Kever Yang <kever.y...@rock-chips.com> wrote:
> 
> Hi Philipp,
> 
>    rk3399 atf has 3 parts, sram.data, sram.text, pmu_fw for m0, and now the 
> ATF project output
> 
> with all 3 section in one bl31.elf, I use to abstract them into 3 separate 
> bin file and package them
> 
> with fit.
> 
>    Could you share some more detail about how you use the ATF in puma-rk3399? 
> You only have
> 
> two section in its description. How many steps do you need for rk3399 to get 
> uboot, atf and the image
> 
> can be flash to eMMC?
> 
> Thanks,
> - Kever
> On 07/29/2017 03:22 AM, Philipp Tomsich wrote:
>> The ITS file generated warnings due to @<num> designations in the naming
>> which cause DTC to complain as follows:
>>   Warning (unit_address_vs_reg): Node /images/uboot@1 has a unit name, but 
>> no reg property
>>   Warning (unit_address_vs_reg): Node /images/atf@1 has a unit name, but no 
>> reg property
>>   Warning (unit_address_vs_reg): Node /images/pmu@1 has a unit name, but no 
>> reg property
>>   Warning (unit_address_vs_reg): Node /images/fdt@1 has a unit name, but no 
>> reg property
>>   Warning (unit_address_vs_reg): Node /configurations/conf@1 has a unit 
>> name, but no reg property
>> 
>> This removes the @<num> part from the names, as we only have a single
>> image for each payload aspect (and only a single configuration) anyway.
>> 
>> Signed-off-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com>
>> 
>> Reviewed-by: Simon Glass <s...@chromium.org>
>> ---
>> 
>> Changes in v3: None
>> Changes in v2: None
>> 
>>  board/theobroma-systems/puma_rk3399/fit_spl_atf.its | 18 +++++++++---------
>>  1 file changed, 9 insertions(+), 9 deletions(-)
>> 
>> diff --git a/board/theobroma-systems/puma_rk3399/fit_spl_atf.its 
>> b/board/theobroma-systems/puma_rk3399/fit_spl_atf.its
>> index f93c251..0519fad 100644
>> --- a/board/theobroma-systems/puma_rk3399/fit_spl_atf.its
>> +++ b/board/theobroma-systems/puma_rk3399/fit_spl_atf.its
>> @@ -13,7 +13,7 @@
>>      #address-cells = <1>;
>>      images {
>> -            uboot@1 {
>> +            uboot {
>>                      description = "U-Boot (64-bit)";
>>                      data = /incbin/("../../../u-boot-nodtb.bin");
>>                      type = "standalone";
>> @@ -21,7 +21,7 @@
>>                      compression = "none";
>>                      load = <0x00200000>;
>>              };
>> -            atf@1 {
>> +            atf {
>>                      description = "ARM Trusted Firmware";
>>                      data = /incbin/("../../../bl31.bin");
>>                      type = "firmware";
>> @@ -30,14 +30,14 @@
>>                      load = <0x00001000>;
>>                      entry = <0x00001000>;
>>              };
>> -            pmu@1 {
>> +            pmu {
>>                      description = "Cortex-M0 firmware";
>>                      data = /incbin/("../../../rk3399m0.bin");
>>                      type = "pmu-firmware";
>>                      compression = "none";
>>                      load = <0xff8c0000>;
>>                  };
>> -            fdt@1 {
>> +            fdt {
>>                      description = "RK3399-Q7 (Puma) flat device-tree";
>>                      data = /incbin/("../../../u-boot.dtb");
>>                      type = "flat_dt";
>> @@ -46,12 +46,12 @@
>>      };
>>      configurations {
>> -            default = "conf@1";
>> -            conf@1 {
>> +            default = "conf";
>> +            conf {
>>                      description = "Theobroma Systems RK3399-Q7 (Puma) SoM";
>> -                    firmware = "uboot@1";
>> -                    loadables = "atf@1";
>> -                    fdt = "fdt@1";
>> +                    firmware = "uboot";
>> +                    loadables = "atf";
>> +                    fdt = "fdt";
>>              };
>>      };
>>  };
> 
> 

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to