Hi Peter,
On 03/27/2018 07:55 PM, Peter Robinson wrote:
> On Tue, Mar 27, 2018 at 10:29 AM, Kever Yang <kever.y...@rock-chips.com>
> wrote:
>> Use "uboot,spl-boot-orde" instead of "uboot,boot0".
> I think these should be in *-u-boot.dtsi files,
Yes, you are right about this, and move all this u-boot configs into
*-u-boot.dtsi files is also one of my TODO clean up jobs.
I didn't do this now because of the frame work do not fit what we
already have now.
@Simon, @Philipp,
Here is what I want to do:
- I think in most case, all the boards with same SoC shares the U-Boot
configs,
eg. it's better to have one rk3288-u-boot.dtsi instead of 12
rk3288-xxboard-u-boot.dtsi
- The build system supports to automatic add x-u-boot.dtsi(feature add
by Simon),
the x could be:
<dts>-u-boot.dtsi where <dts> is the base name of the .dts
file
<CONFIG_SYS_SOC>-u-boot.dtsi
<CONFIG_SYS_CPU>-u-boot.dtsi
<CONFIG_SYS_VENDOR>-u-boot.dtsi
So use CONFIG_SYS_SOC-u-boot.dtsi should be the best choice.
- But we can't use it because the value is always "rockchip"
- we can not change this into a real soc name because:
There is a symbolic link "arch/${ARCH}/include/asm/arch" for header
file pointing to
the SoC/CPU specific header directory; and all rockchip headers are
locate at
arch/arm/include/asm/arch-rockchip/, including many common header
files and
soc specific files.
- Are we going to add arch-rk3xxx folder for each soc like arch-mxx and
arch-tegraxxx,
and copy the common header files into each soc directory? Is there
other solution?
Thanks,
- Kever
> that enables easy sync
> between the linux kernel dts files and u-boot like done on a number of
> other configs, a couple of the rk3368 devices seem to already use this
> as does devices like AllWinner
>
>> Signed-off-by: Kever Yang <kever.y...@rock-chips.com>
>> ---
>>
>> arch/arm/dts/rk3288-phycore-rdk.dts | 6 +-----
>> arch/arm/dts/rk3288-veyron.dtsi | 6 +-----
>> 2 files changed, 2 insertions(+), 10 deletions(-)
>>
>> diff --git a/arch/arm/dts/rk3288-phycore-rdk.dts
>> b/arch/arm/dts/rk3288-phycore-rdk.dts
>> index f2bb7b5..3bf8efb 100644
>> --- a/arch/arm/dts/rk3288-phycore-rdk.dts
>> +++ b/arch/arm/dts/rk3288-phycore-rdk.dts
>> @@ -53,11 +53,7 @@
>>
>> chosen {
>> stdout-path = &uart2;
>> - };
>> -
>> - config {
>> - u-boot,dm-pre-reloc;
>> - u-boot,boot0 = &emmc;
>> + u-boot,spl-boot-order = &emmc;
>> };
>>
>> user_buttons: user-buttons {
>> diff --git a/arch/arm/dts/rk3288-veyron.dtsi
>> b/arch/arm/dts/rk3288-veyron.dtsi
>> index a314058..77b9bf8 100644
>> --- a/arch/arm/dts/rk3288-veyron.dtsi
>> +++ b/arch/arm/dts/rk3288-veyron.dtsi
>> @@ -17,11 +17,7 @@
>>
>> chosen {
>> stdout-path = &uart2;
>> - };
>> -
>> - config {
>> - u-boot,dm-pre-reloc;
>> - u-boot,boot0 = &spi_flash;
>> + u-boot,spl-boot-order = &spi_flash;
>> };
>>
>> firmware {
>> --
>> 1.9.1
>>
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot@lists.denx.de
>> https://lists.denx.de/listinfo/u-boot
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
