diff options
| author | Naseer Ahmed <naseer@codeaurora.org> | 2017-10-05 18:47:15 -0400 |
|---|---|---|
| committer | Marissa Wall <marissaw@google.com> | 2018-04-18 12:45:46 -0700 |
| commit | 1ff191fed3ad95868b7b1723d43bee86d28ef6ff (patch) | |
| tree | 14c22d6a949bf43b9bca6c1a9f7f67986e9d5037 /msm8998 | |
| parent | 530894e77a27c6a32dec305719571072f03ed13e (diff) | |
| download | display-1ff191fed3ad95868b7b1723d43bee86d28ef6ff.tar.gz | |
hwc2: Skip validation fixes
* Disable skip validation for solid fill during calibration.
* Require validation when the client changes the composition type
Bug: 77925367
CRs-Fixed: 2122072
Change-Id: I9ded200e5c15701f653c73d67c18c41de8278d52
Diffstat (limited to 'msm8998')
| -rw-r--r-- | msm8998/sdm/libs/hwc2/hwc_display.cpp | 4 | ||||
| -rw-r--r-- | msm8998/sdm/libs/hwc2/hwc_layers.cpp | 4 |
2 files changed, 8 insertions, 0 deletions
diff --git a/msm8998/sdm/libs/hwc2/hwc_display.cpp b/msm8998/sdm/libs/hwc2/hwc_display.cpp index 9fdaf444..6b2e39dc 100644 --- a/msm8998/sdm/libs/hwc2/hwc_display.cpp +++ b/msm8998/sdm/libs/hwc2/hwc_display.cpp @@ -1883,6 +1883,10 @@ std::string HWCDisplay::Dump() { } bool HWCDisplay::CanSkipValidate() { + if (solid_fill_enable_) { + return false; + } + // Layer Stack checks if (layer_stack_.flags.hdr_present && (tone_mapper_ && tone_mapper_->IsActive())) { return false; diff --git a/msm8998/sdm/libs/hwc2/hwc_layers.cpp b/msm8998/sdm/libs/hwc2/hwc_layers.cpp index 4ceaa632..06ec8af6 100644 --- a/msm8998/sdm/libs/hwc2/hwc_layers.cpp +++ b/msm8998/sdm/libs/hwc2/hwc_layers.cpp @@ -329,6 +329,10 @@ HWC2::Error HWCLayer::SetLayerColor(hwc_color_t color) { } HWC2::Error HWCLayer::SetLayerCompositionType(HWC2::Composition type) { + // Validation is required when the client changes the composition type + if (client_requested_ != type) { + needs_validate_ = true; + } client_requested_ = type; switch (type) { case HWC2::Composition::Client: |