commit cc5ff9944a3567e377dee00455a1d974cb6e5d0d
author Roman Kiryanov <rkir@google.com> Wed Aug 03 18:59:55 2022 -0700
committer Cherrypicker Worker <android-build-cherrypicker-worker@google.com> Thu Aug 04 17:36:54 2022 +0000
tree 8dbe42dc66c75c9eff06a5e1c746816eee25adbb
parent 96036441c83c81cf897ce546ef8ed3f043520196

Reduce logcat noise in the audio HAL

Bug: 241305429
Test: presubmit
Signed-off-by: Roman Kiryanov <rkir@google.com>
Change-Id: I53edd1e85351c9c54f185d625fb2b1963190c3c3
(cherry picked from commit d0878960747a626133478d43f2c507c939d39bae)
Merged-In: I53edd1e85351c9c54f185d625fb2b1963190c3c3

audio/stream_in.cpp

diff --git a/audio/stream_in.cpp b/audio/stream_in.cpp
index 7148e52..50c89e8 100644
--- a/audio/stream_in.cpp
+++ b/audio/stream_in.cpp
@@ -294,7 +294,7 @@
                                      const hidl_vec<hidl_string>& keys,
                                      getParameters_cb _hidl_cb) {
     (void)context;
-    _hidl_cb((keys.size() > 0) ? FAILURE(Result::NOT_SUPPORTED) : Result::OK, {});
+    _hidl_cb((keys.size() > 0) ? Result::NOT_SUPPORTED : Result::OK, {});
     return Void();
 }
 

audio/stream_out.cpp

diff --git a/audio/stream_out.cpp b/audio/stream_out.cpp
index 1eabed3..ecf5f2e 100644
--- a/audio/stream_out.cpp
+++ b/audio/stream_out.cpp
@@ -310,7 +310,7 @@
                                       const hidl_vec<hidl_string>& keys,
                                       getParameters_cb _hidl_cb) {
     (void)context;
-    _hidl_cb((keys.size() > 0) ? FAILURE(Result::NOT_SUPPORTED) : Result::OK, {});
+    _hidl_cb((keys.size() > 0) ? Result::NOT_SUPPORTED : Result::OK, {});
     return Void();
 }
 
@@ -386,7 +386,7 @@
 
 Return<Result> StreamOut::updateSourceMetadata(const SourceMetadata& sourceMetadata) {
     (void)sourceMetadata;
-    return FAILURE(Result::NOT_SUPPORTED);
+    return Result::NOT_SUPPORTED;
 }
 
 Return<void> StreamOut::prepareForWriting(uint32_t frameSize,
@@ -440,7 +440,7 @@
 
 Return<Result> StreamOut::setEventCallback(const sp<IStreamOutEventCallback>& callback) {
     (void)callback;
-    return FAILURE(Result::NOT_SUPPORTED);
+    return Result::NOT_SUPPORTED;
 }
 
 Return<void> StreamOut::supportsPauseAndResume(supportsPauseAndResume_cb _hidl_cb) {