From 6c4038cf114eb47232ca6c13ae54e0d597110fb8 Mon Sep 17 00:00:00 2001 From: Mary Ruthven Date: Tue, 1 Mar 2022 13:02:13 -0600 Subject: ccd_config: add more options to why_denied Some boards don't use battery presence for ccd. They just have a chassis_open signal. Update the why_denied message to add this option. BUG=b:197974058 TEST=check ccd open denial message Change-Id: I92254b35cc98492709ec14a26a71cecc7d273a6b Signed-off-by: Mary Ruthven Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3498701 Reviewed-by: Vadim Sukhomlinov --- common/ccd_config.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/common/ccd_config.c b/common/ccd_config.c index 684a747f8a..bd1b0a8b6e 100644 --- a/common/ccd_config.c +++ b/common/ccd_config.c @@ -967,7 +967,8 @@ static enum vendor_cmd_rc ccd_open(struct vendor_cmd_params *p) * - Battery is present * - Either not in developer mode or the command came from USB */ - why_denied = "open from AP in devmode or remove batt"; + why_denied = "run from AP in devmode, remove batt, or short " + "chassis_open"; goto denied; } -- cgit v1.2.1