diff options
author | Rob Herring <robh@kernel.org> | 2017-06-09 15:15:55 -0500 |
---|---|---|
committer | Rob Herring <robh@kernel.org> | 2017-10-09 11:03:49 -0500 |
commit | ee8f45b183406324156220e5326d42e85ae048c8 (patch) | |
tree | 8d0a7ba105d5c4da5eeb08b471e9ab3dd60fa12b /drmhwctwo.cpp | |
parent | 0f7487f10a8ef2fc212e115627ed4744c9801d3a (diff) |
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.
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'drmhwctwo.cpp')
-rw-r--r-- | drmhwctwo.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drmhwctwo.cpp b/drmhwctwo.cpp index 89399bf..0f25877 100644 --- a/drmhwctwo.cpp +++ b/drmhwctwo.cpp @@ -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) { |