commit 5891f1197fdaba27a34d8345eae7bec8659e3c31
author Jason Macnak <natsu@google.com> Mon Oct 24 13:11:14 2022 -0700
committer Jason Macnak <natsu@google.com> Mon Oct 24 13:11:14 2022 -0700
tree e13830c71b0d5b89158ae29eed2825295961b33a
parent d183844aa237d309cd30df7b68069d432cec98e9

Fix invalid preferred detailed timing descriptor check

This was previously getting lucky based on the values of clock
but could be considered invalid with the correct refresh rate.

Bug: b/254906981
Test: auto tests
Change-Id: If53b7f270cb6858cbf9a8101a88cc5c72dfc4917

system/hwc3/DrmConnector.cpp

diff --git a/system/hwc3/DrmConnector.cpp b/system/hwc3/DrmConnector.cpp
index a6a8b29..0924af5 100644
--- a/system/hwc3/DrmConnector.cpp
+++ b/system/hwc3/DrmConnector.cpp
@@ -109,7 +109,7 @@
     edid.remove_prefix(kEdidDescriptorOffset);
 
     byte_view descriptor(edid.data(), kEdidDescriptorLength);
-    if (descriptor[0] == 0 || descriptor[1] == 0) {
+    if (descriptor[0] == 0 && descriptor[1] == 0) {
         ALOGE("%s: display:%" PRIu32 " is missing preferred detailed timing descriptor.",
               __FUNCTION__, mId);
         return -1;