Hi Quentin, On 2025-02-11 11:29, Quentin Schulz wrote: > From: Quentin Schulz <quentin.sch...@cherry.de> > > This was only used on RK3288 Chromebooks and the EVB. > > If it follows the same pattern as for RK3399 Chromebooks where their > maintainer (Simon) agreed[1] to removal of u-boot.rom on the basis that > the generic u-boot-rockchip-spi.bin is now enough, let's do the same for > RK3288 and remove the last Rockchip users of u-boot.rom (and HAS_ROM > symbol). > > At the same time, remove HAS_ROM symbol from the RK3288 Chromebooks and > EVB configs since they were used only for that. > > SYS_SPI_U_BOOT_OFFS offset in rockchip-u-boot.dtsi for the u-boot-img > node of simple-bin-spi binman image matches the one used in u-boot.rom > except for the EVB. > The EVB doesn't have ROCKCHIP_SPI_IMAGE symbol enabled, so HAS_ROM had > no effect anyway. Even if it had, this would not have been enough > considering that SPL_SPI_LOAD symbol is not set, so U-Boot proper could > not be loaded from SPI even if SPL/TPL does. > > [1] > https://lore.kernel.org/u-boot/caflszth-sewfod8deof3+e-wce1qff0cyxxr8cbqwy3brw3...@mail.gmail.com/ > Signed-off-by: Quentin Schulz <quentin.sch...@cherry.de>
Thanks for this cleanup :-) Reviewed-by: Jonas Karlman <jo...@kwiboo.se> Regards, Jonas > --- > arch/arm/dts/rk3288-u-boot.dtsi | 24 ------------------------ > arch/arm/mach-rockchip/rk3288/Kconfig | 5 ----- > 2 files changed, 29 deletions(-) > [snip]
