commit 90cef9e6c8eaa528ce278157d2447a23c75e5190
author John Stultz <john.stultz@linaro.org> Mon Apr 15 15:30:33 2019 -0700
committer Alistair Strachan <astrachan@google.com> Tue Apr 16 15:07:03 2019 +0000
tree f336cae3596350a1d7566303442e6e4bd0618654
parent 2f49f81c06000a6e514860d7f94b37ea49c20a3b
parent a9ed46acc12d7e7bb0cdaf5b53ff175156589dc9

Merge remote-tracking branch 'aosp/upstream-master' into HEAD

Resync w/ upstream master branch

* aosp/upstream-master:
  drm_hwcomposer: platformhisi: extend conditional of AFBC support
  drm_hwcomposer: Move hwcomposer.drm_hikey and hwcomposer.drm_hikey960
  drm_hwcomposer: Convert external/drm_hwcomposer to Android.bp
  drm_hwcomposer: Remove unnecessary parentheses in the conditional
  drm_hwcomposer: Tweak mode selection to pick the first DRM_MODE_TYPE_PREFERRED mode

Change-Id: I12fcd32cde036074a79ef56fbaae85fb9af2b43b
Signed-off-by: John Stultz <john.stultz@linaro.org>

drmconnector.cpp

diff --git a/drmconnector.cpp b/drmconnector.cpp
index f272024..ead9312 100644
--- a/drmconnector.cpp
+++ b/drmconnector.cpp
@@ -138,13 +138,15 @@
       m.set_id(drm_->next_mode_id());
       new_modes.push_back(m);
     }
-    if (new_modes.back().type() & DRM_MODE_TYPE_PREFERRED) {
+    // Use only the first DRM_MODE_TYPE_PREFERRED mode found
+    if (!preferred_mode_found &&
+        (new_modes.back().type() & DRM_MODE_TYPE_PREFERRED)) {
       preferred_mode_id_ = new_modes.back().id();
       preferred_mode_found = true;
     }
   }
   modes_.swap(new_modes);
-  if ((!preferred_mode_found) && (modes_.size() != 0)) {
+  if (!preferred_mode_found && modes_.size() != 0) {
     preferred_mode_id_ = modes_[0].id();
   }
   return 0;

platformhisi.cpp

diff --git a/platformhisi.cpp b/platformhisi.cpp
index d4002f1..e022010 100644
--- a/platformhisi.cpp
+++ b/platformhisi.cpp
@@ -70,7 +70,8 @@
   return 0;
 }
 
-#ifdef MALI_GRALLOC_INTFMT_AFBC_BASIC
+#if defined(MALI_GRALLOC_INTFMT_AFBC_BASIC) && \
+    defined(AFBC_FORMAT_MOD_BLOCK_SIZE_16x16)
 uint64_t HisiImporter::ConvertGrallocFormatToDrmModifiers(uint64_t flags,
                                                           bool is_rgb) {
   uint64_t features = 0UL;