Attention is currently required from: Christian Walter, Julius Werner, Yu-Ping Wu. Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59682 )
Change subject: cbfs: Remove deprecated APIs ......................................................................
Patch Set 3:
(1 comment)
Patchset:
PS2:
Loading a payload should probably not take over two seconds, unless it is ridiculously huge... how big is the payload you use? And do you happen to know the SPI bus speed for your platform?
I do have a linux kernel as a payload which is uncmpressed ~4,5 MB in size. And I have a SPI clock of 50 MHz while usung a standard SPI interface (so no dual or quad mode enabled).
Anyway, it should be pretty simple to figure out if this is it: I've uploaded a test CL CB:59767 which just loads the whole payload into RAM before doing anything else with it. Can you try that out and see what difference it makes for your boot times?
I will have access to the hardware tomorrow and will give this test patch a try.
(If that's not it, could you try narrowing down a bit further where exactly all this time is spent for you? E.g. just sprinkle some timestamp_add_now(1234) with increasing ID numbers throughout the code to see where the big delay comes from.)
Yes, I can try to narrow it down furter.