commit 271a14d19dd66c5463810ed82be73ca2b6118af1
author Shunkai Yao <yaoshunkai@google.com> Fri Apr 05 16:26:47 2024 +0000
committer Shunkai Yao <yaoshunkai@google.com> Tue Apr 09 18:45:23 2024 +0000
tree cf831e02238cebd007503110b6fe9912a00011f4
parent b0cad00218f5cbb2baa279326438f59f28367ac5

Spatializer: allow vendor extension parameters

Bug: 332249644
Test: build
Change-Id: Ic4b27e5866d85f5eb96ef7100315f43e169f7de6

media/libaudiohal/impl/effectsAidlConversion/AidlConversionSpatializer.cpp

diff --git a/media/libaudiohal/impl/effectsAidlConversion/AidlConversionSpatializer.cpp b/media/libaudiohal/impl/effectsAidlConversion/AidlConversionSpatializer.cpp
index da28204..a88ef83 100644
--- a/media/libaudiohal/impl/effectsAidlConversion/AidlConversionSpatializer.cpp
+++ b/media/libaudiohal/impl/effectsAidlConversion/AidlConversionSpatializer.cpp
@@ -147,8 +147,12 @@
                 return statusTFromBinderStatus(mEffect->setParameter(aidlParam));
             }
             default: {
-                ALOGE("%s %d invalid command %u", __func__, __LINE__, command);
-                return BAD_VALUE;
+                // for vendor extension, copy data area to the DefaultExtension, parameter ignored
+                VendorExtension ext = VALUE_OR_RETURN_STATUS(
+                        aidl::android::legacy2aidl_EffectParameterReader_VendorExtension(param));
+                aidlParam =
+                        MAKE_SPECIFIC_PARAMETER(Spatializer, spatializer, vendor, ext);
+                break;
             }
         }
     } else {
@@ -317,8 +321,7 @@
                 return OK;
             }
             default: {
-                ALOGE("%s %d invalid command %u", __func__, __LINE__, command);
-                return BAD_VALUE;
+                VENDOR_EXTENSION_GET_AND_RETURN(Spatializer, spatializer, param);
             }
         }
     } else {