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

Commit 43804173 authored by Kensuke Miyagi's avatar Kensuke Miyagi
Browse files

Revert "codec2 hal: Codec2ConfigurableClient to take HIDL or AID..." 

Revert submission 20667833

Reason for revert: Build broken (b/262039340)
Reverted Changes:
I4aacbfea2:codec2 hal: move directories around
I4239c654e:codec2 hal: Codec2ConfigurableClient to take HIDL ...

Change-Id: Ibe4c48cf49013ace615f891992eb126c788ff117
parent 9fb10280

media/codec2/hidl/client/Android.bp

+0 −2
Original line number Diff line number Diff line
@@ -43,10 +43,8 @@ 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

+129 −289

File changed.

Preview size limit exceeded, changes collapsed.

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

+9 −41
Original line number Diff line number Diff line
@@ -83,13 +83,6 @@ 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
@@ -112,36 +105,7 @@ namespace android { 
// declaration of an inner class is not possible.

struct Codec2ConfigurableClient {



    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);

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



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

            std::vector<C2FieldSupportedValuesQuery>& fields,

            c2_blocking_t mayBlock) const;

private:

    struct HidlImpl;

    struct AidlImpl;



    const std::unique_ptr<ImplBase> mImpl;

    // base cannot be null.

    Codec2ConfigurableClient(const sp<Base>& base);



protected:

    sp<Base> mBase;

    C2String mName;



    friend struct Codec2Client;

};



struct Codec2Client : public Codec2ConfigurableClient {