Hi Jonas,
On Wed, 31 Jul 2024 at 09:03, Jonas Karlman jonas@kwiboo.se wrote:
Hi Simon,
On 2024-07-31 16:49, Simon Glass wrote:
The code here is confusing due to large blocks which are #ifdefed out. Add a function phase_sdram_init() which returns whether SDRAM init should happen in the current phase, using that as needed to control the code flow.
This increases code size by about 500 bytes in SPL when the cache is on, since it must call the rather large rockchip_sdram_size() function.
I am guessing this size increase only is related to kevin/bob since all other RK3399 boards already use rockchip_sdram_size() in SPL.
That's not what I am seeing:
buildman -b bugs8 rk3399 -sS Summary of 9 commits for 29 boards (29 threads, 2 jobs per thread) 01: Merge patch series "led: implement software blinking" aarch64: w+ roc-pc-mezzanine-rk3399 roc-pc-rk3399 chromebook_bob chromebook_kevin pinebook-pro-rk3399 pinephone-pro-rk3399 rockpro64-rk3399 rock-4c-plus-rk3399 rock-4se-rk3399 rock-pi-4-rk3399 rock-pi-4c-rk3399 eaidk-610-rk3399 evb-rk3399 firefly-rk3399 khadas-edge-captain-rk3399 khadas-edge-rk3399 khadas-edge-v-rk3399 leez-rk3399 nanopc-t4-rk3399 nanopi-m4-2gb-rk3399 nanopi-m4-rk3399 nanopi-m4b-rk3399 nanopi-neo4-rk3399 nanopi-r4s-rk3399 orangepi-rk3399 rock-pi-n10-rk3399pro puma-rk3399 ficus-rk3399 rock960-rk3399 02: mkeficapsule: Add a --version argument 03: binman: Collect the version number for mkeficapsule 04: binman: Deal with mkeficapsule being missing 05: binman: Return failure when a usage() message is generated 06: binman: Keep the efi_capsule input file 07: fdt: Correct condition for bloblist existing 08: rockchip: Ensure memory size is available in RK3399 SPL aarch64: (for 29/29 boards) spl/u-boot-spl:all +33.1 spl/u-boot-spl:text +33.1 tpl/u-boot-tpl:all +446.9 tpl/u-boot-tpl:text +446.9 09: rockchip: Avoid #ifdefs in RK3399 SPL aarch64: (for 29/29 boards) all +69.2 bss +5.2 spl/u-boot-spl:all +24.0 spl/u-boot-spl:text +24.0 text +64.0 tpl/u-boot-tpl:all +21.5 tpl/u-boot-tpl:text +21.5 ✔ ~/u [bugs8 ↓·18↑·8|…2⚑ 1001] 15:48 $ buildman -b bugs8 rk3399 -sSB Summary of 9 commits for 29 boards (29 threads, 2 jobs per thread) 01: Merge patch series "led: implement software blinking" aarch64: w+ roc-pc-mezzanine-rk3399 roc-pc-rk3399 chromebook_bob chromebook_kevin pinebook-pro-rk3399 pinephone-pro-rk3399 rockpro64-rk3399 rock-4c-plus-rk3399 rock-4se-rk3399 rock-pi-4-rk3399 rock-pi-4c-rk3399 eaidk-610-rk3399 evb-rk3399 firefly-rk3399 khadas-edge-captain-rk3399 khadas-edge-rk3399 khadas-edge-v-rk3399 leez-rk3399 nanopc-t4-rk3399 nanopi-m4-2gb-rk3399 nanopi-m4-rk3399 nanopi-m4b-rk3399 nanopi-neo4-rk3399 nanopi-r4s-rk3399 orangepi-rk3399 rock-pi-n10-rk3399pro puma-rk3399 ficus-rk3399 rock960-rk3399 02: mkeficapsule: Add a --version argument 03: binman: Collect the version number for mkeficapsule 04: binman: Deal with mkeficapsule being missing 05: binman: Return failure when a usage() message is generated 06: binman: Keep the efi_capsule input file 07: fdt: Correct condition for bloblist existing 08: rockchip: Ensure memory size is available in RK3399 SPL aarch64: (for 29/29 boards) spl/u-boot-spl:all +33.1 spl/u-boot-spl:text +33.1 tpl/u-boot-tpl:all +446.9 tpl/u-boot-tpl:text +446.9 roc-pc-mezzanine-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4120 4168 +48 roc-pc-rk3399 : tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4120 4168 +48 chromebook_bob : spl/u-boot-spl:all +480 spl/u-boot-spl:text +480 spl-u-boot-spl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 chromebook_kevin: spl/u-boot-spl:all +480 spl/u-boot-spl:text +480 spl-u-boot-spl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 pinebook-pro-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 pinephone-pro-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 rockpro64-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 rock-4c-plus-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 rock-4se-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 rock-pi-4-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 rock-pi-4c-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 eaidk-610-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 evb-rk3399 : tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 firefly-rk3399 : tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 khadas-edge-captain-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 khadas-edge-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 khadas-edge-v-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 leez-rk3399 : tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 nanopc-t4-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 nanopi-m4-2gb-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 nanopi-m4-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 nanopi-m4b-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 nanopi-neo4-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 nanopi-r4s-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4164 4212 +48 orangepi-rk3399: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 rock-pi-n10-rk3399pro: tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4248 4296 +48 puma-rk3399 : tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 ficus-rk3399 : tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 rock960-rk3399 : tpl/u-boot-tpl:all +480 tpl/u-boot-tpl:text +480 tpl-u-boot-tpl: add: 1/0, grow: 1/0 bytes: 480/0 (480) function old new delta rockchip_sdram_size - 432 +432 rk3399_dmc_probe 4304 4352 +48 09: rockchip: Avoid #ifdefs in RK3399 SPL aarch64: (for 29/29 boards) all +69.2 bss +5.2 spl/u-boot-spl:all +24.0 spl/u-boot-spl:text +24.0 text +64.0 tpl/u-boot-tpl:all +21.5 tpl/u-boot-tpl:text +21.5 rock-4c-plus-rk3399: all +120 bss +56 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 nanopc-t4-rk3399: all +120 bss +56 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 nanopi-r4s-rk3399: all +120 bss +56 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 rockpro64-rk3399: all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 nanopi-m4-2gb-rk3399: all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 nanopi-m4-rk3399: all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 nanopi-m4b-rk3399: all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 puma-rk3399 : all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 eaidk-610-rk3399: all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 firefly-rk3399 : all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 khadas-edge-captain-rk3399: all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 khadas-edge-rk3399: all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 khadas-edge-v-rk3399: all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 leez-rk3399 : all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 orangepi-rk3399: all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 rock-pi-n10-rk3399pro: all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +16 tpl/u-boot-tpl:text +16 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 16/0 (16) function old new delta rk3399_dmc_probe 4296 4312 +16 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 ficus-rk3399 : all +64 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 pinebook-pro-rk3399: all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 pinephone-pro-rk3399: all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 rock-4se-rk3399: all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 rock-pi-4-rk3399: all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 rock-pi-4c-rk3399: all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4212 4236 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 evb-rk3399 : all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 nanopi-neo4-rk3399: all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 rock960-rk3399 : all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +24 tpl/u-boot-tpl:text +24 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 roc-pc-mezzanine-rk3399: all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +16 tpl/u-boot-tpl:text +16 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 16/0 (16) function old new delta rk3399_dmc_probe 4168 4184 +16 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 roc-pc-rk3399 : all +56 bss -8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 tpl/u-boot-tpl:all +16 tpl/u-boot-tpl:text +16 tpl-u-boot-tpl: add: 0/0, grow: 1/0 bytes: 16/0 (16) function old new delta rk3399_dmc_probe 4168 4184 +16 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 1/0, grow: 0/0 bytes: 24/0 (24) function old new delta rk3399_dmc_of_to_plat - 24 +24 chromebook_bob : all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24 chromebook_kevin: all +72 bss +8 spl/u-boot-spl:all +24 spl/u-boot-spl:text +24 text +64 u-boot: add: 1/0, grow: 1/0 bytes: 40/0 (40) function old new delta rk3399_dmc_of_to_plat - 32 +32 rk3399_dmc_probe 64 72 +8 spl-u-boot-spl: add: 0/0, grow: 1/0 bytes: 24/0 (24) function old new delta rk3399_dmc_probe 4352 4376 +24
- Drop the non-dcache optimisation, since the cache should normally be on
Is this referencing to something that is still done in this patch?
No, it is attached to v5, but I had a blank line in the way. Will fix.
Signed-off-by: Simon Glass sjg@chromium.org
(no changes since v5)
Changes in v5:
- Move setting of pmugrf into the probe() function
- Drop patches previously applied
Changes in v4:
- Fix 'stating' typo
- Move Binman size feature to a separate series
Changes in v3:
- Split out the refactoring into a separate patch
Changes in v2:
- Drop patch "regulator: rk8xx: Fix incorrect parameter"
- Rewrite boneblack patch to onstead drop the target and update docs
drivers/ram/rockchip/sdram_rk3399.c | 33 ++++++++++++++--------------- 1 file changed, 16 insertions(+), 17 deletions(-)
diff --git a/drivers/ram/rockchip/sdram_rk3399.c b/drivers/ram/rockchip/sdram_rk3399.c index bc79c034808..8a7cbb1849f 100644 --- a/drivers/ram/rockchip/sdram_rk3399.c +++ b/drivers/ram/rockchip/sdram_rk3399.c @@ -13,6 +13,7 @@ #include <log.h> #include <ram.h> #include <regmap.h> +#include <spl.h> #include <syscon.h> #include <asm/arch-rockchip/clock.h> #include <asm/arch-rockchip/cru.h> @@ -63,8 +64,6 @@ struct chan_info { };
struct dram_info { -#if defined(CONFIG_TPL_BUILD) || \
(!defined(CONFIG_TPL) && defined(CONFIG_SPL_BUILD)) u32 pwrup_srefresh_exit[2]; struct chan_info chan[2]; struct clk ddr_clk;@@ -75,7 +74,6 @@ struct dram_info { struct rk3399_pmusgrf_regs *pmusgrf; struct rk3399_ddr_cic_regs *cic; const struct sdram_rk3399_ops *ops; -#endif struct ram_info info; struct rk3399_pmugrf_regs *pmugrf; }; @@ -92,9 +90,6 @@ struct sdram_rk3399_ops { struct rk3399_sdram_params *params); };
-#if defined(CONFIG_TPL_BUILD) || \
(!defined(CONFIG_TPL) && defined(CONFIG_SPL_BUILD))struct rockchip_dmc_plat { #if CONFIG_IS_ENABLED(OF_PLATDATA) struct dtd_rockchip_rk3399_dmc dtplat; @@ -191,6 +186,17 @@ struct io_setting { }, };
+/**
- phase_sdram_init() - Check if this is the phase where SDRAM init happens
- Returns: true to do SDRAM init in this phase, false to not
- */
+static bool phase_sdram_init(void) +{
return spl_phase() == PHASE_TPL ||(!IS_ENABLED(CONFIG_TPL) && !spl_in_proper());This is still wrong, please consider ROCKCHIP_EXTERNAL_TPL or this function fail to do what you document it to do.
See our prior thread about this: https://lore.kernel.org/u-boot/CAFLszThg5xohR-9AgSMHVrW5Dp_tN39nC7rq1Z4ZMnqO...
Oh yes, I was looking at that earlier and then forgot about it.
Regards, Simon
Regards, Jonas
+}
static struct io_setting * lpddr4_get_io_settings(const struct rk3399_sdram_params *params, u32 mr5) { @@ -3024,7 +3030,7 @@ static int rk3399_dmc_of_to_plat(struct udevice *dev) struct rockchip_dmc_plat *plat = dev_get_plat(dev); int ret;
if (!CONFIG_IS_ENABLED(OF_REAL))
if (!CONFIG_IS_ENABLED(OF_REAL) || !phase_sdram_init()) return 0; ret = dev_read_u32_array(dev, "rockchip,sdram-params",@@ -3093,7 +3099,6 @@ static int rk3399_dmc_init(struct udevice *dev) priv->cic = syscon_get_first_range(ROCKCHIP_SYSCON_CIC); priv->grf = syscon_get_first_range(ROCKCHIP_SYSCON_GRF); priv->pmu = syscon_get_first_range(ROCKCHIP_SYSCON_PMU);
priv->pmugrf = syscon_get_first_range(ROCKCHIP_SYSCON_PMUGRF); priv->pmusgrf = syscon_get_first_range(ROCKCHIP_SYSCON_PMUSGRF); priv->pmucru = rockchip_get_pmucru(); priv->cru = rockchip_get_cru();@@ -3138,19 +3143,16 @@ static int rk3399_dmc_init(struct udevice *dev)
return 0;} -#endif
static int rk3399_dmc_probe(struct udevice *dev) { struct dram_info *priv = dev_get_priv(dev);
-#if defined(CONFIG_TPL_BUILD) || \
(!defined(CONFIG_TPL) && defined(CONFIG_SPL_BUILD))if (rk3399_dmc_init(dev))return 0;-#endif priv->pmugrf = syscon_get_first_range(ROCKCHIP_SYSCON_PMUGRF); debug("%s: pmugrf = %p\n", __func__, priv->pmugrf);
if (phase_sdram_init() && rk3399_dmc_init(dev))return 0;priv->info.base = CFG_SYS_SDRAM_BASE; priv->info.size = rockchip_sdram_size((phys_addr_t)&priv->pmugrf->os_reg2);@@ -3181,10 +3183,7 @@ U_BOOT_DRIVER(dmc_rk3399) = { .id = UCLASS_RAM, .of_match = rk3399_dmc_ids, .ops = &rk3399_dmc_ops, -#if defined(CONFIG_TPL_BUILD) || \
(!defined(CONFIG_TPL) && defined(CONFIG_SPL_BUILD)) .of_to_plat = rk3399_dmc_of_to_plat,-#endif .probe = rk3399_dmc_probe, .priv_auto = sizeof(struct dram_info), #if defined(CONFIG_TPL_BUILD) || \