summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDenis Brockus <dbrockus@google.com>2020-11-09 12:38:36 -0700
committerCommit Bot <commit-bot@chromium.org>2020-11-10 01:05:42 +0000
commit63b45bb3c5bac197b281c419a3e0c3e3c9327ea7 (patch)
tree1bbaceeaf3c65dc17f7ca0ef086b69aea1f878ec
parent3b4177ddd41a6f39725b6fe80b0ad01bf1bf7cad (diff)
downloadchrome-ec-63b45bb3c5bac197b281c419a3e0c3e3c9327ea7.tar.gz
genvif: Guard Manufacturer_Info_PID_Port
Adding CONFIG_USB_PD_MANUFACTURING_INFO guard BUG=b:172842062 BRANCH=none TEST=verify XML output Signed-off-by: Denis Brockus <dbrockus@google.com> Change-Id: Ia048ead9060f11dc9bc943ef6dd3b935c7d5cf5f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2527701 Tested-by: Denis Brockus <dbrockus@chromium.org> Reviewed-by: Denis Brockus <dbrockus@chromium.org> Reviewed-by: Keith Short <keithshort@chromium.org> Commit-Queue: Denis Brockus <dbrockus@chromium.org>
-rw-r--r--util/genvif.c14
1 files changed, 9 insertions, 5 deletions
diff --git a/util/genvif.c b/util/genvif.c
index caa7e29a89..88e19502b0 100644
--- a/util/genvif.c
+++ b/util/genvif.c
@@ -1356,13 +1356,13 @@ static void init_vif_component_general_pd_fields(
"Unchunked_Extended_Messages_Supported",
false);
- set_vif_field_b(&vif_fields[Manufacturer_Info_Supported_Port],
- "Manufacturer_Info_Supported_Port",
- IS_ENABLED(CONFIG_USB_PD_MANUFACTURER_INFO));
-
- {
+ if (IS_ENABLED(CONFIG_USB_PD_MANUFACTURER_INFO)) {
char hex_str[10];
+ set_vif_field_b(&vif_fields[Manufacturer_Info_Supported_Port],
+ "Manufacturer_Info_Supported_Port",
+ true);
+
#if defined(CONFIG_USB_PID)
sprintf(hex_str, "%04X", CONFIG_USB_PID);
set_vif_field_itss(&vif_fields[
@@ -1376,6 +1376,10 @@ static void init_vif_component_general_pd_fields(
"Manufacturer_Info_PID_Port",
DEFAULT_MISSING_PID, hex_str);
#endif
+ } else {
+ set_vif_field_b(&vif_fields[Manufacturer_Info_Supported_Port],
+ "Manufacturer_Info_Supported_Port",
+ false);
}
set_vif_field_b(&vif_fields[Security_Msgs_Supported_SOP],