Depthcharge: Boot power check unnecessarily waits for 3 seconds |
||
Issue descriptionCurrently, Depthcharge waits for 3 seconds before it decides to quit booting if the condition isn't met: https://github.com/rockchip-linux/depthcharge/blob/a979856e235caa5b20303b2ea65d2acda8e1448e/src/vboot/callbacks/ec.c#L228 This wait is only meaningful if the AC power is supplied (for PD power contract). If AC power isn't present, Depthcharge should immediately quit booting.
,
Jul 6
There are two (separate) thresholds: one for EC and the other for Depthcharge. I guess EC's thresholds intended to be more permissive so that Depthcharge can show the battery low screen? The range is pretty slim: the power supply has to be enough to boot the AP and show a firmware screen but not enough to run the OS. Some boards have the same thresholds for AP and EC. So, the range doesn't exist and the necessity of this range is questionable.
,
Jul 7
Depthcharge is never showing a low battery screen. That's only done in the OS. There's not point in having a different threshold for powering up and for continuing to boot once we're in depthcharge. They both lead to the same result (shutdown) if they're not met. If there's a difference there today, that's probably something we should fix. (Also, the whole point of doing this in depthcharge is to determine whether we have enough power to show a screen. We have not enabled the display yet at that point, and if we're below the threshold we'll shut down without every showing anything.) |
||
►
Sign in to add a comment |
||
Comment 1 by jwer...@chromium.org
, Jul 6