Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 189cd6a5 authored by Mikhail Naganov's avatar Mikhail Naganov
Browse files

audio: Clarify that MMap h/w position must be continuous

The VTS test already verifies that the MMap hardware position
never resets or goes back, but the official wording in the AIDL
interface was lacking.

Bug: 350998390
Bug: 363139283
Test: none, doc change
Change-Id: I07b9842f53c89549fe648ee839d875d31a7a849d
parent bee59e3b
Loading
Loading
Loading
Loading
+12 −3
Original line number Diff line number Diff line
@@ -386,14 +386,23 @@ parcelable StreamDescriptor {
         * For input streams: the moment when data at the specified stream position
         *   was acquired (i.e. capture position).
         *
         * The observable position must never be reset by the HAL module.
         * The data type of the frame counter is large enough to support
         * continuous counting for years of operation.
         * The observable position must never be reset by the HAL module,
         * providing an abstraction of continuous audio data flow. The data
         * type of the frame counter is large enough to support continuous
         * counting for years of operation.
         */
        Position observable;
        /**
         * Used only for MMap streams to provide the hardware read / write
         * position for audio data in the shared memory buffer 'audio.mmap'.
         * Similar to the observable position, the 'Position::UNKNOWN' value
         * can be returned when the HAL module is unable to retrieve the current
         * position.
         *
         * The hardware position must never be reset by the HAL module,
         * providing an abstraction of continuous audio data flow. The data
         * type of the frame counter is large enough to support continuous
         * counting for years of operation.
         */
        Position hardware;
        /**