commit 85793d8c864cfa5b2b03348412366c7c7312702d
author Sham Rathod <sham.rathod@ittiam.com> Thu Dec 22 19:09:10 2022 +0530
committer Sham Rathod <sham.rathod@ittiam.com> Wed Dec 28 20:46:51 2022 +0530
tree 3aea372f9ca41d7dd36e412dc71ea8613e113de4
parent c6d1c3840a7976682b75002c7d30780187afd268

Volume : Add minimum and maximum capabilities for params.

Bug: 263416041
Test: atest VtsHalVolumeTargetTest

Change-Id: Iea1253d9bfb049916f8f60e675eacf8e9040d087

audio/aidl/default/volume/VolumeSw.h

diff --git a/audio/aidl/default/volume/VolumeSw.h b/audio/aidl/default/volume/VolumeSw.h
index a101c59..b6f6077 100644
--- a/audio/aidl/default/volume/VolumeSw.h
+++ b/audio/aidl/default/volume/VolumeSw.h
@@ -33,23 +33,11 @@
         LOG(DEBUG) << __func__;
     }
 
-    RetCode setVolLevel(int level) {
-        if (level < Volume::MIN_LEVEL_DB || level > Volume::MAX_LEVEL_DB) {
-            LOG(ERROR) << __func__ << " invalid level " << level;
-            return RetCode::ERROR_ILLEGAL_PARAMETER;
-        }
-        // TODO : Add implementation to apply new level
-        mLevel = level;
-        return RetCode::SUCCESS;
-    }
+    RetCode setVolLevel(int level);
 
     int getVolLevel() const { return mLevel; }
 
-    RetCode setVolMute(bool mute) {
-        // TODO : Add implementation to modify mute
-        mMute = mute;
-        return RetCode::SUCCESS;
-    }
+    RetCode setVolMute(bool mute);
 
     bool getVolMute() const { return mMute; }