On Mon, Apr 29, 2024 at 5:58 AM Anastasia Klimchuk aklm@chromium.org wrote:
The patch saves 1s for every flashrom run, for almost all users, so it's a good thing. If it can be unblocked now just by adding 1-line condition, let's unblock it now.
I've updated the patch to approximately this suggestion.
Perhaps there is a bigger question of Parallel/LPC/FWH flash chips which need special treatment, but rarely used nowadays. I want to discuss, but as a topic by itself.
Honestly, I don't know how to move forward on assertions about "know[ing] for sure whether this delay is no longer needed". We have no actionable information about the original problem, so we can never be sure.
So for my purposes, I'll just drop the subject after this patch.
I don't think we need to resolve all of the problems in one patch :)
Great!