Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 301bb309 authored by Courtney Goeltzenleuchter's avatar Courtney Goeltzenleuchter
Browse files

Do not send HdrMetadata on older interface 

Need to make sure composer@2.2 is supported before sending
HDR data across composer interface.
Test: boot
Bug: 74523164

Change-Id: I551c8568098990193309939483f10a30cc47350c
parent 990cc3e3

services/surfaceflinger/BufferLayer.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -631,7 +631,7 @@ void BufferLayer::setPerFrameData(const sp<const DisplayDevice>& displayDevice) 




    const HdrMetadata& metadata = mConsumer->getCurrentHdrMetadata();

    const HdrMetadata& metadata = mConsumer->getCurrentHdrMetadata();

    error = hwcLayer->setHdrMetadata(metadata);

    error = hwcLayer->setHdrMetadata(metadata);

    if (error != HWC2::Error::None) {

    if (error != HWC2::Error::None && error != HWC2::Error::Unsupported) {

        ALOGE("[%s] Failed to set hdrMetadata: %s (%d)", mName.string(),

        ALOGE("[%s] Failed to set hdrMetadata: %s (%d)", mName.string(),

              to_string(error).c_str(), static_cast<int32_t>(error));

              to_string(error).c_str(), static_cast<int32_t>(error));

    }

    }

services/surfaceflinger/DisplayHardware/ComposerHal.cpp

+3 −0
Original line number Original line Diff line number Diff line
@@ -706,6 +706,9 @@ Error Composer::setLayerDataspace(Display display, Layer layer, 
}

}





Error Composer::setLayerHdrMetadata(Display display, Layer layer, const HdrMetadata& metadata) {

Error Composer::setLayerHdrMetadata(Display display, Layer layer, const HdrMetadata& metadata) {

    if (!mClient_2_2) {

        return Error::UNSUPPORTED;

    }





    mWriter.selectDisplay(display);

    mWriter.selectDisplay(display);

    mWriter.selectLayer(layer);

    mWriter.selectLayer(layer);