[PATCH 02/12] rockchip: rk3399-gru: Use SYS_SPI_U_BOOT_OFFS value in offset prop
Jonas Karlman
jonas at kwiboo.se
Sun Feb 9 02:27:14 CET 2025
Hi Simon,
On 2025-02-09 00:04, Simon Glass wrote:
> Hi Jonas,
>
> On Thu, 12 Dec 2024 at 16:57, Jonas Karlman <jonas at kwiboo.se> wrote:
>>
>> Use the offset configured with SYS_SPI_U_BOOT_OFFS Kconfig option
>> instead of a hardcoded 0x40000 for the FIT payload offset.
>>
>> This has no intended impact as SYS_SPI_U_BOOT_OFFS=0x40000.
>
> When you add CONFIG options into a devicetree, it makes it difficult
> to build it. We have to individually build U-Boot for each target just
> to get the right values.
This only aligned everything with the simple-bin-spi image and prepared
for moving the SPI flash payload offset to 0xE0000.
If you are looking at this now, please also take a look at [1]. There
where some questions directed at you there :-)
[1] https://patchwork.ozlabs.org/project/uboot/patch/20241212235735.164272-7-jonas@kwiboo.se/
>
> For rk3399-generic, this means I have to drop gru.
>
> Can we not use a binman symbols instead?
All RK3399 targets having support for building a SPI flash image now use
CONFIG_SYS_SPI_U_BOOT_OFFS=0xE0000 after these changes where merged.
Regards,
Jonas
>
>>
>> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
>> ---
>> arch/arm/dts/rk3399-gru-u-boot.dtsi | 2 +-
>> arch/arm/dts/rk3399-u-boot.dtsi | 2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> Regards,
> SImon
More information about the U-Boot
mailing list