New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 820335 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

ec: charger/rt946x: Consider sending EC_RES_IN_PROGRESS for VBUS measurement

Project Member Reported by philipchen@chromium.org, Mar 9 2018

Issue description

For now we set ADC timeout as 50ms to avoid host command timeout.
But this is not ideal - the ADC may take up to 200ms to finish conversion according to the datasheet.

We had a provision for handling long transactions; we can report things like EC_RES_INPROGRESS (not sure of spelling), and that triggers the AP to poll with subsequent commands (10-11ms backoffs, with up to 50 tries = about 500 ms).
 
Components: OS>Firmware>EC
Labels: OS-Chrome
Passing through old stuff...I believe EC_RES_IN_PROGRESS is not something we want to make heavy use of. It was only designed for flash erase, and then we added FLASH_ERASE_SECTOR_ASYNC to avoid even that.

Sign in to add a comment