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

Commit 01b7e025 authored by Vignesh Venkatasubramanian's avatar Vignesh Venkatasubramanian Committed by Android (Google) Code Review
Browse files

Merge "av1: Allow odd width/height for YUV420"

parents a65405fa db0859ec

media/codec2/components/base/SimpleC2Component.cpp

+7 −7
Original line number Diff line number Diff line
@@ -45,23 +45,23 @@ void convertYUV420Planar8ToYV12(uint8_t *dstY, uint8_t *dstU, uint8_t *dstV, con 


    if (isMonochrome) {

        // Fill with neutral U/V values.

        for (size_t i = 0; i < height / 2; ++i) {

            memset(dstV, kNeutralUVBitDepth8, width / 2);

            memset(dstU, kNeutralUVBitDepth8, width / 2);

        for (size_t i = 0; i < (height + 1) / 2; ++i) {

            memset(dstV, kNeutralUVBitDepth8, (width + 1) / 2);

            memset(dstU, kNeutralUVBitDepth8, (width + 1) / 2);

            dstV += dstUVStride;

            dstU += dstUVStride;

        }

        return;

    }



    for (size_t i = 0; i < height / 2; ++i) {

        memcpy(dstV, srcV, width / 2);

    for (size_t i = 0; i < (height + 1) / 2; ++i) {

        memcpy(dstV, srcV, (width + 1) / 2);

        srcV += srcVStride;

        dstV += dstUVStride;

    }



    for (size_t i = 0; i < height / 2; ++i) {

        memcpy(dstU, srcU, width / 2);

    for (size_t i = 0; i < (height + 1) / 2; ++i) {

        memcpy(dstU, srcU, (width + 1) / 2);

        srcU += srcUStride;

        dstU += dstUVStride;

    }

media/codec2/components/gav1/C2SoftGav1Dec.cpp

+8 −4
Original line number Diff line number Diff line
@@ -55,8 +55,8 @@ class C2SoftGav1Dec::IntfImpl : public SimpleInterface<void>::BaseParams { 
        DefineParam(mSize, C2_PARAMKEY_PICTURE_SIZE)

            .withDefault(new C2StreamPictureSizeInfo::output(0u, 320, 240))

            .withFields({

                C2F(mSize, width).inRange(2, 4096, 2),

                C2F(mSize, height).inRange(2, 4096, 2),

                C2F(mSize, width).inRange(2, 4096),

                C2F(mSize, height).inRange(2, 4096),

            })

            .withSetter(SizeSetter)

            .build());
@@ -650,8 +650,12 @@ bool C2SoftGav1Dec::outputBuffer(const std::shared_ptr<C2BlockPool> &pool, 
  }

  C2MemoryUsage usage = {C2MemoryUsage::CPU_READ, C2MemoryUsage::CPU_WRITE};



  c2_status_t err = pool->fetchGraphicBlock(align(mWidth, 16), mHeight, format,

                                            usage, &block);

  // We always create a graphic block that is width aligned to 16 and height

  // aligned to 2. We set the correct "crop" value of the image in the call to

  // createGraphicBuffer() by setting the correct image dimensions.

  c2_status_t err = pool->fetchGraphicBlock(align(mWidth, 16),

                                            align(mHeight, 2), format, usage,

                                            &block);



  if (err != C2_OK) {

    ALOGE("fetchGraphicBlock for Output failed with status %d", err);