Raul Rangel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58955 )
Change subject: lib/hardwaremain: Run timers more frequently
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58955/comment/a7e3466f_9c108754
PS2, Line 14: see SPI transactions progress faster
Basically, `bs_run_timers(0)` calls `timers_run()` once, which checks if the timer queue's first ele […]
Yep exactly. That's why I said progress faster. The current SPI DMA transactions complete before the data is required, so it doesn't actually affect boot time. What this does allow though is more opportunities to enqueue more transactions in the same time, so it's possible to add more preloads.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/58955
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9508e7777d52fe934cc09d486abc0dab5cf7dad8
Gerrit-Change-Number: 58955
Gerrit-PatchSet: 2
Gerrit-Owner: Raul Rangel
rrangel@chromium.org
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Karthik Ramasubramanian
kramasub@google.com
Gerrit-Reviewer: Raul Rangel
rrangel@chromium.org
Gerrit-Reviewer: Rob Barnes
robbarnes@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: 9elements QA
hardwaretestrobot@gmail.com
Gerrit-CC: Angel Pons
th3fanbus@gmail.com
Gerrit-CC: Paul Menzel
paulepanter@mailbox.org
Gerrit-Comment-Date: Fri, 19 Nov 2021 16:22:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Raul Rangel
rrangel@chromium.org
Comment-In-Reply-To: Paul Menzel
paulepanter@mailbox.org
Comment-In-Reply-To: Angel Pons
th3fanbus@gmail.com
Gerrit-MessageType: comment