Skip to contentSkip to conversation

drm_hwcomposer: fix return values for ValidateDisplay

PATCH New Rob Herring 1 participant 1 message Oct 9, 2017

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 drm_hwcomposer: fix return values for ValidateDisplay

According to the API documentation, if we've set num_types, then the return value should be HWC2::Error::HasChanges.

---
 drmhwctwo.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drmhwctwo.cpp +1 -1
index 89399bf1ba33..0f258770a8bb 100644
@@ -701,7 +701,7 @@ HWC2::Error DrmHwcTwo::HwcDisplay::ValidateDisplay(uint32_t *num_types,
         break;
     }
   }
-  return HWC2::Error::None;
+  return *num_types ? HWC2::Error::HasChanges : HWC2::Error::None;
 }
 
 HWC2::Error DrmHwcTwo::HwcLayer::SetCursorPosition(int32_t x, int32_t y) {
Signature · 6 lines
2.11.0

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel