commit 6c94dde22691a9234a05c16902d177ec8808088b
author RD Babiera <rdbabiera@google.com> Wed Nov 09 22:55:47 2022 +0000
committer RD Babiera <rdbabiera@google.com> Mon Jan 09 21:06:18 2023 +0000
tree 512406e778c735124e40eddb74439fba6550d5d2
parent bab42594b21401f537f86485464bbe8d6f9c161f

Usb DisplayPort alt mode aidl interfaces

Adds enums for DisplayPortAltModeStatus and DisplayPortAltModePinAssignment.
Adds AltModeData and DisplayPortAltModeData.
Updates PortStatus to reflect these new enums and parcelables.

Test: atest VtsAidlUsbTargetTest
Bug: 253534975
Change-Id: I0fc62601dfc162b909e796586110686beed137ea

usb/aidl/vts/VtsAidlUsbTargetTest.cpp

diff --git a/usb/aidl/vts/VtsAidlUsbTargetTest.cpp b/usb/aidl/vts/VtsAidlUsbTargetTest.cpp
index d87f1f4..7b86d68 100644
--- a/usb/aidl/vts/VtsAidlUsbTargetTest.cpp
+++ b/usb/aidl/vts/VtsAidlUsbTargetTest.cpp
@@ -42,10 +42,14 @@
 
 #define TIMEOUT_PERIOD 10
 
+using ::aidl::android::hardware::usb::AltModeData;
 using ::aidl::android::hardware::usb::BnUsbCallback;
 using ::aidl::android::hardware::usb::ComplianceWarning;
+using ::aidl::android::hardware::usb::DisplayPortAltModePinAssignment;
+using ::aidl::android::hardware::usb::DisplayPortAltModeStatus;
 using ::aidl::android::hardware::usb::IUsb;
 using ::aidl::android::hardware::usb::IUsbCallback;
+using ::aidl::android::hardware::usb::PlugOrientation;
 using ::aidl::android::hardware::usb::PortDataRole;
 using ::aidl::android::hardware::usb::PortMode;
 using ::aidl::android::hardware::usb::PortPowerRole;
@@ -639,6 +643,60 @@
     ALOGI("UsbAidlTest nonCompliantChargerValues end");
 }
 
+/*
+ * Test PlugOrientation Values are within range in PortStatus
+ */
+TEST_P(UsbAidlTest, plugOrientationValues) {
+    ALOGI("UsbAidlTest plugOrientationValues start");
+    int64_t transactionId = rand() % 10000;
+    const auto& ret = usb->queryPortStatus(transactionId);
+    ASSERT_TRUE(ret.isOk());
+    EXPECT_EQ(std::cv_status::no_timeout, wait());
+    EXPECT_EQ(2, usb_last_cookie);
+    EXPECT_EQ(transactionId, last_transactionId);
+
+    EXPECT_TRUE((int)usb_last_port_status.plugOrientation >= (int)PlugOrientation::UNKNOWN);
+    EXPECT_TRUE((int)usb_last_port_status.plugOrientation <= (int)PlugOrientation::PLUGGED_FLIPPED);
+}
+
+/*
+ * Test DisplayPortAltMode Values when DisplayPort Alt Mode
+ * is active.
+ */
+TEST_P(UsbAidlTest, dpAltModeValues) {
+    ALOGI("UsbAidlTest dpAltModeValues start");
+    int64_t transactionId = rand() % 10000;
+    const auto& ret = usb->queryPortStatus(transactionId);
+    ASSERT_TRUE(ret.isOk());
+    EXPECT_EQ(std::cv_status::no_timeout, wait());
+    EXPECT_EQ(2, usb_last_cookie);
+    EXPECT_EQ(transactionId, last_transactionId);
+
+    // Discover DisplayPort Alt Mode
+    for (AltModeData altMode : usb_last_port_status.supportedAltModes) {
+      if (altMode.getTag() == AltModeData::displayPortAltModeData) {
+        AltModeData::DisplayPortAltModeData displayPortAltModeData =
+                altMode.get<AltModeData::displayPortAltModeData>();
+        EXPECT_TRUE((int)displayPortAltModeData.partnerSinkStatus >=
+                    (int)DisplayPortAltModeStatus::UNKNOWN);
+        EXPECT_TRUE((int)displayPortAltModeData.partnerSinkStatus <=
+                    (int)DisplayPortAltModeStatus::ENABLED);
+
+        EXPECT_TRUE((int)displayPortAltModeData.cableStatus >=
+                    (int)DisplayPortAltModeStatus::UNKNOWN);
+        EXPECT_TRUE((int)displayPortAltModeData.cableStatus <=
+                    (int)DisplayPortAltModeStatus::ENABLED);
+
+        EXPECT_TRUE((int)displayPortAltModeData.pinAssignment >=
+                    (int)DisplayPortAltModePinAssignment::NONE);
+        EXPECT_TRUE((int)displayPortAltModeData.pinAssignment <=
+                    (int)DisplayPortAltModePinAssignment::F);
+      }
+    }
+
+    ALOGI("UsbAidlTest dpAltModeValues end");
+}
+
 GTEST_ALLOW_UNINSTANTIATED_PARAMETERIZED_TEST(UsbAidlTest);
 INSTANTIATE_TEST_SUITE_P(
         PerInstance, UsbAidlTest,