diff options
author | Roman Kovalivskyi <roman.kovalivskyi@globallogic.com> | 2019-11-21 17:15:02 +0200 |
---|---|---|
committer | John Stultz <john.stultz@linaro.org> | 2020-01-16 18:52:08 +0000 |
commit | 309570872cb989d5530d7a7a60ac0d9d5581f056 (patch) | |
tree | 920008d63bfb357d393c7d658e7d362cf507cd70 | |
parent | 521a4c85fc4e26fd1b439cbbbf3d421534e79485 (diff) | |
download | drm_hwcomposer-309570872cb989d5530d7a7a60ac0d9d5581f056.tar.gz |
drm_hwcomposer: Remove redundant hotplug call
Initial hotplug state is reported back during callback registration,
but during VTS tests we've noticed error that is caused by calling
hotplug with same display twice.
HandleInitialHotplugState calls hotplug function for each connected
display, so it covers HWC_DISPLAY_PRIMARY too, therefore first call
should be removed, as it is redundant and causes minor error.
Signed-off-by: Roman Kovalivskyi <roman.kovalivskyi@globallogic.com>
-rw-r--r-- | drmhwctwo.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/drmhwctwo.cpp b/drmhwctwo.cpp index 33ad0fb..2b7f684 100644 --- a/drmhwctwo.cpp +++ b/drmhwctwo.cpp @@ -203,9 +203,6 @@ HWC2::Error DrmHwcTwo::RegisterCallback(int32_t descriptor, switch (callback) { case HWC2::Callback::Hotplug: { - auto hotplug = reinterpret_cast<HWC2_PFN_HOTPLUG>(function); - hotplug(data, HWC_DISPLAY_PRIMARY, - static_cast<int32_t>(HWC2::Connection::Connected)); auto &drmDevices = resource_manager_.getDrmDevices(); for (auto &device : drmDevices) HandleInitialHotplugState(device.get()); |