Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31793 )
Change subject: drivers/spi: Instrument erase operation
......................................................................
Patch Set 1:
Like it says, bootblock runs out of space. From what I remember, SMAUG had some 150 bytes room, failed during those console loglevel changes in similar fashion.
BTW: There is code adding timestamps around every AmdXX() call. Besides, I don't think there is much you can do here, SPI erase is pretty much a busy wait-poll-loop and that W25Q part may have 45ms(typ) to 400ms(max) sectore erase time.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/31793
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4eb8a850d7263bc3c800d97655f0b37623b78562
Gerrit-Change-Number: 31793
Gerrit-PatchSet: 1
Gerrit-Owner: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Kyösti Mälkki
kyosti.malkki@gmail.com
Gerrit-CC: Nico Huber
nico.h@gmx.de
Gerrit-Comment-Date: Thu, 07 Mar 2019 14:39:22 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment