commit ae4324ce4a61ca1fa553abedf35c0bb32fd9b823
author Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com> Wed Mar 21 12:06:20 2018 +0000
committer Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com> Fri Jul 13 15:38:12 2018 +0100
tree f11c1239e8f48fd9fd13c314ae6bc8acd0fa03d7
parent 364f957f3eefafd0963f140b68bec314cc05f07a

drm_hwcomposer: Add display field to Drmencoder

In the current implementation TryEncoderForDisplay just looks
at the crtc linked to the display, if that's not assigned to
a display it means the encoder could be used, otherwise iterate
to the list of possible_crtcs and find one which is not used.

This logic works fine when you have just one encoder connected to a
crtc but with two or more, like is the case when we attach a writeback
connector, we need to know if we already assigned the encoder to a
display.

Signed-off-by: Alexandru Gheorghe <alexandru-cosmin.gheorghe@arm.com>

drmencoder.cpp

diff --git a/drmencoder.cpp b/drmencoder.cpp
index f61c8fd..6ae5d9a 100644
--- a/drmencoder.cpp
+++ b/drmencoder.cpp
@@ -27,6 +27,7 @@
                        const std::vector<DrmCrtc *> &possible_crtcs)
     : id_(e->encoder_id),
       crtc_(current_crtc),
+      display_(-1),
       possible_crtcs_(possible_crtcs) {
 }
 
@@ -40,5 +41,14 @@
 
 void DrmEncoder::set_crtc(DrmCrtc *crtc) {
   crtc_ = crtc;
+  display_ = crtc->display();
+}
+
+int DrmEncoder::display() const {
+  return display_;
+}
+
+bool DrmEncoder::can_bind(int display) const {
+  return display_ == -1 || display_ == display;
 }
 }