commit 2b43a5debe88c7845d63488e7526990cac856fda
author Nicholas Ambur <nambur@google.com> Wed Feb 12 11:19:48 2020 -0800
committer Nicholas Ambur <nambur@google.com> Thu Feb 13 14:59:20 2020 -0800
tree 2be6275ba6eec5aac546543a50bcb183104cd273
parent 2797c113c8605104aa2cce8091b194f102aa78f7

update SoundTrigger queryParameter

Update the documentation and usage when HAL implementation wants to
convey the parameter ID is not supported.

Bug: 141929369
Test: build and boot smoke test
&& verify unsupported parameter use case with test app

Change-Id: I2124d8e5e9b136bd0797c16e71aa0b4049c9ed58

soundtrigger/2.3/default/SoundTriggerHw.cpp

diff --git a/soundtrigger/2.3/default/SoundTriggerHw.cpp b/soundtrigger/2.3/default/SoundTriggerHw.cpp
index d3136b9..8fe3108 100644
--- a/soundtrigger/2.3/default/SoundTriggerHw.cpp
+++ b/soundtrigger/2.3/default/SoundTriggerHw.cpp
@@ -889,7 +889,7 @@
     int32_t status = mHwDevice->query_parameter(
             mHwDevice, client->getHalHandle(), convertModelParameterToHal(modelParam), &paramRange);
 
-    if (status == 0) {
+    if (status == 0 && paramRange.is_supported) {
         optionalParamRange.range({.start = paramRange.start, .end = paramRange.end});
     }
     _hidl_cb(status, optionalParamRange);