servo: dut-power queries Sweetberry for EC readings |
|
Issue descriptionIn the current dut-power, dut-power cannot tell between sweetberry and other servo devices, thus it queries sweetberry for EC readings as well. And the banner says "onboard INA" instead of "sweetberry." This is not top priority, but is helpful for UX.
,
Aug 14
I'm more thinking along the lines of querying servod to determine what servo device is currently in use. If it's Sweetberry then there's no need to add the EC Power tracker, since it doesn't exist. Users who know that Sweetberry does not provide EC data won't care, but users who don't know that would not be surprised by an error message.
,
Aug 14
Makes sense. (This comment is just to keep track of a thought) Maybe it makes sense to query if the device has ec_pty & if it doesn't, then assume it's a servo device that doesn't support ec querying. That way we don't have to hard-code a servo type etc.
,
Jan 16
|
|
►
Sign in to add a comment |
|
Comment 1 by coconutruben@chromium.org
, Aug 14