Revert "libaudiohal: implement supportsBluetoothVariableLatency for HIDL"

This reverts commit 8ed6d79.

Signed-off-by: Edwiin Kusuma Jaya <kutemeikito0905@gmail.com>

Author: kutemeikito

media/libaudiohal/impl/DeviceHalAidl.cpp

@@ -951,7 +951,7 @@ status_t DeviceHalAidl::dump(int fd, const Vector<String16>& args) {
     return mModule->dump(fd, Args(args).args(), args.size());
 }
 
-status_t DeviceHalAidl::supportsBluetoothVariableLatency(bool* supports) {
+int32_t DeviceHalAidl::supportsBluetoothVariableLatency(bool* supports) {
     TIME_CHECK();
     if (!mModule) return NO_INIT;
     if (supports == nullptr) {

media/libaudiohal/impl/DeviceHalAidl.h

@@ -162,7 +162,7 @@ class DeviceHalAidl : public DeviceHalInterface, public ConversionHelperAidl,
 
     error::Result<audio_hw_sync_t> getHwAvSync() override;
 
-    status_t supportsBluetoothVariableLatency(bool* supports __unused) override;
+    int32_t supportsBluetoothVariableLatency(bool* supports __unused) override;
 
     status_t getSoundDoseInterface(const std::string& module,
                                    ::ndk::SpAIBinder* soundDoseBinder) override;

media/libaudiohal/impl/DeviceHalHidl.cpp

@@ -681,26 +681,4 @@ status_t DeviceHalHidl::getSoundDoseInterface(const std::string& module,
 }
 #endif
 
-status_t DeviceHalHidl::supportsBluetoothVariableLatency(bool* supports) {
-    if (supports == nullptr) {
-        return BAD_VALUE;
-    }
-    *supports = false;
-
-    String8 reply;
-    status_t status = getParameters(
-            String8(AUDIO_PARAMETER_BT_VARIABLE_LATENCY_SUPPORTED), &reply);
-    if (status != NO_ERROR) {
-        return status;
-    }
-    AudioParameter replyParams(reply);
-    String8 trueOrFalse;
-    status = replyParams.get(
-            String8(AUDIO_PARAMETER_BT_VARIABLE_LATENCY_SUPPORTED), trueOrFalse);
-    if (status != NO_ERROR) {
-        return status;
-    }
-    *supports = trueOrFalse == AudioParameter::valueTrue;
-    return NO_ERROR;
-}
 } // namespace android

media/libaudiohal/impl/DeviceHalHidl.h

@@ -125,7 +125,10 @@ class DeviceHalHidl : public DeviceHalInterface, public CoreConversionHelperHidl
         return INVALID_OPERATION;
     }
 
-    status_t supportsBluetoothVariableLatency(bool* supports) override;
+    int32_t supportsBluetoothVariableLatency(bool* supports __unused) override {
+        // TODO: Implement the HAL query when moving to AIDL HAL.
+        return INVALID_OPERATION;
+    }
 
     status_t setConnectedState(const struct audio_port_v7 *port, bool connected) override;
 

media/libaudiohal/include/media/audiohal/DeviceHalInterface.h

@@ -142,7 +142,7 @@ class DeviceHalInterface : public virtual RefBase
     virtual int32_t getAAudioMixerBurstCount() = 0;
     virtual int32_t getAAudioHardwareBurstMinUsec() = 0;
 
-    virtual status_t supportsBluetoothVariableLatency(bool* supports) = 0;
+    virtual int32_t supportsBluetoothVariableLatency(bool* supports) = 0;
 
     // Update the connection status of an external device.
     virtual status_t setConnectedState(const struct audio_port_v7* port, bool connected) = 0;