Paul Menzel has posted comments on this change by Federico Amedeo Izzo. ( https://review.coreboot.org/c/coreboot/+/82010?usp=email )
Change subject: mb/aoostar: Add Alder Lake based AOOSTAR R1 (WTR_R1) ......................................................................
Patch Set 17:
(1 comment)
Patchset:
PS17:
I uploaded the two logs for two different commits. […]
Thank you very much.
1. https://review.coreboot.org/plugins/gitiles/board-status/+/d2df631729544b9a6... 2. https://review.coreboot.org/plugins/gitiles/board-status/+/73bab87c09851459a...
$ git grep "Total Time" aoostar/wtr_r1/ aoostar/wtr_r1/24.05-171-gc9302d5d39/2024-06-07T06_16_27Z/coreboot_timestamps.txt:Total Time: 2,292,748 aoostar/wtr_r1/24.05-384-g0486f2b3bb/2024-06-28T22_11_31Z/coreboot_timestamps.txt:Total Time: 2,070,133
``` $ git grep CONFIG_CONSOLE_SERIAL= aoostar/wtr_r1/ aoostar/wtr_r1/24.05-171-gc9302d5d39/2024-06-07T06_16_27Z/config.short.txt:CONFIG_CONSOLE_SERIAL=y aoostar/wtr_r1/24.05-171-gc9302d5d39/2024-06-07T06_16_27Z/config.txt:CONFIG_CONSOLE_SERIAL=y aoostar/wtr_r1/24.05-384-g0486f2b3bb/2024-06-28T22_11_31Z/config.short.txt:CONFIG_CONSOLE_SERIAL=y aoostar/wtr_r1/24.05-384-g0486f2b3bb/2024-06-28T22_11_31Z/config.txt:CONFIG_CONSOLE_SERIAL=y ```
Two seconds is quite long and probably caused by the serial console. Sorry for not writing it earlier. If you find time, it’d be great if you did both things with serial console disable so the lowest boot time is documented. I am marking this comment as resolved nevertheless. Big thanks again for the port and the uploads.