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

Commit c8fc2c39 authored by Wonsik Kim's avatar Wonsik Kim
Browse files

RELAND "codec2 hal: Codec2ConfigurableClient to take HIDL or AIDL base" 

This reverts commit 43804173.

Bug: 251850069
Bug: 262059797
Test: builds
Merged-In: I8c83c6b74a5042406f203b80b564e2768def1a9a
Change-Id: I8c83c6b74a5042406f203b80b564e2768def1a9a
parent 912902c1

media/codec2/hidl/client/Android.bp

+2 −0
Original line number Diff line number Diff line
@@ -43,8 +43,10 @@ cc_library { 
        "android.hardware.media.c2@1.0",

        "android.hardware.media.c2@1.1",

        "android.hardware.media.c2@1.2",

        "android.hardware.media.c2-V1-ndk",

        "libbase",

        "libbinder",

        "libbinder_ndk",

        "libcodec2",

        "libcodec2_hidl_client@1.0",

        "libcodec2_hidl_client@1.1",

media/codec2/hidl/client/client.cpp

+289 −129

File changed.

Preview size limit exceeded, changes collapsed.

media/codec2/hidl/client/include/codec2/hidl/client.h

+41 −9
Original line number Diff line number Diff line
@@ -84,6 +84,13 @@ struct IComponent; 
struct IComponentStore;

}  // namespace android::hardware::media::c2::V1_2



namespace aidl::android::hardware::media::c2 {

class IComponent;

class IComponentInterface;

class IComponentStore;

class IConfigurable;

}  // namespace aidl::android::hardware::media::c2



namespace android::hardware::media::bufferpool::V2_0 {

struct IClientManager;

}  // namespace android::hardware::media::bufferpool::V2_0
@@ -106,7 +113,36 @@ namespace android { 
// declaration of an inner class is not possible.

struct Codec2ConfigurableClient {



    typedef ::android::hardware::media::c2::V1_0::IConfigurable Base;

    typedef ::android::hardware::media::c2::V1_0::IConfigurable HidlBase;

    typedef ::aidl::android::hardware::media::c2::IConfigurable AidlBase;



    struct ImplBase {

        virtual ~ImplBase() = default;



        virtual const C2String& getName() const = 0;



        virtual c2_status_t query(

                const std::vector<C2Param*>& stackParams,

                const std::vector<C2Param::Index> &heapParamIndices,

                c2_blocking_t mayBlock,

                std::vector<std::unique_ptr<C2Param>>* const heapParams) const = 0;



        virtual c2_status_t config(

                const std::vector<C2Param*> &params,

                c2_blocking_t mayBlock,

                std::vector<std::unique_ptr<C2SettingResult>>* const failures) = 0;



        virtual c2_status_t querySupportedParams(

                std::vector<std::shared_ptr<C2ParamDescriptor>>* const params

                ) const = 0;



        virtual c2_status_t querySupportedValues(

                std::vector<C2FieldSupportedValuesQuery>& fields,

                c2_blocking_t mayBlock) const = 0;

    };



    explicit Codec2ConfigurableClient(const sp<HidlBase> &hidlBase);

    explicit Codec2ConfigurableClient(const std::shared_ptr<AidlBase> &aidlBase);



    const C2String& getName() const;
@@ -128,15 +164,11 @@ struct Codec2ConfigurableClient { 
    c2_status_t querySupportedValues(

            std::vector<C2FieldSupportedValuesQuery>& fields,

            c2_blocking_t mayBlock) const;

private:

    struct HidlImpl;

    struct AidlImpl;



    // base cannot be null.

    Codec2ConfigurableClient(const sp<Base>& base);



protected:

    sp<Base> mBase;

    C2String mName;



    friend struct Codec2Client;

    const std::unique_ptr<ImplBase> mImpl;

};



struct Codec2Client : public Codec2ConfigurableClient {