Public API changes for audio offload support.

NOTE: this does _not_ include all private member variables added
to classes as part of offload support. Only public/protected functions
and stubs functions/variables needed to make the changes buildable.

- isOffloadSupported() added to audio policy service
    A stub implementation is required to build, this always returns false
- setParameters() added to IAudioTrack
    A stub implementation is required to build, this always returns
    INVALID_OPERATION

- CBlk flag for stream end

- Change AudioSystem::getRenderPosition() to take an audio_output_t
    so caller can specify which output to query

- Add AudioSystem::isOffloadSupported()
    This is fully implemented down to the AudioFlinger function
    AudioPolicyServer::isOffloadSupported() which is just a stub
    that always returns false.

- Add EVENT_STREAM_END to AudioTrack interface.
    STREAM_END is used to signal when the hardware has actually finished
    playing all the data it was sent.

- Add event type enumeration to media player interface AudioSink callbacks
    so that the same callback can be used to handle multiple types of
    event. For offloaded tracks we also have to handle STREAM_END and
    TEAR_DOWN events

- Pass audio_offload_info_t to various functions used for opening outputs,
    tracks and audio players. This passes additional information about the
    compressed stream down to the HAL when using offload.
    For publicly-available APIs this is an optional parameter (for some of
    the internal and low-level APIs around the HAL interface it is mandatory)

- Add getParameters() and setParameters() API to AudioTrack
    Currently dummy implementations.

- Change AudioPlayer contructor so that it takes a set of bitflags defining what
    options are required. This replaces the original bool which only specified
    whether to use deep buffering.

- Changes to StageFright class definition related to handling tearing-down of
    an offloaded track when we need to switch back to software decode

- Define new StageFright utility functions used for offloaded tracks
    Currently dummy implementations.

- AudioFlinger changes to use extended audio_config_t.
    Fills in audio_offload_info_t member if this info is passed in when
    opening an output.

- libvideoeditor changes required to add the new event type parameter
    to AudioSink callback functions

- libmediaplayerservice changes required to add the new event type parameter
    to AudioSink callback functions

Change-Id: I3ab41138aa1083d81fe83b886a9b1021ec7320f1
Signed-off-by: Richard Fitzgerald <rf@opensource.wolfsonmicro.com>
Signed-off-by: Eric Laurent <elaurent@google.com>

1 files changed, 18 insertions, 0 deletions

diff --git a/media/libmedia/IAudioTrack.cpp b/media/libmedia/IAudioTrack.cpp
index e92f8aa..a2b49a3 100644
--- a/media/libmedia/IAudioTrack.cpp
+++ b/media/libmedia/IAudioTrack.cpp
@@ -39,6 +39,7 @@ enum {
     ALLOCATE_TIMED_BUFFER,
     QUEUE_TIMED_BUFFER,
     SET_MEDIA_TIME_TRANSFORM,
+    SET_PARAMETERS
 };
 
 class BpAudioTrack : public BpInterface<IAudioTrack>
@@ -154,6 +155,17 @@ public:
         }
         return status;
     }
+
+    virtual status_t setParameters(const String8& keyValuePairs) {
+        Parcel data, reply;
+        data.writeInterfaceToken(IAudioTrack::getInterfaceDescriptor());
+        data.writeString8(keyValuePairs);
+        status_t status = remote()->transact(SET_PARAMETERS, data, &reply);
+        if (status == NO_ERROR) {
+            status = reply.readInt32();
+        }
+        return status;
+    }
 };
 
 IMPLEMENT_META_INTERFACE(AudioTrack, "android.media.IAudioTrack");
@@ -223,6 +235,12 @@ status_t BnAudioTrack::onTransact(
             reply->writeInt32(setMediaTimeTransform(xform, target));
             return NO_ERROR;
         } break;
+        case SET_PARAMETERS: {
+            CHECK_INTERFACE(IAudioTrack, data, reply);
+            String8 keyValuePairs(data.readString8());
+            reply->writeInt32(setParameters(keyValuePairs));
+            return NO_ERROR;
+        } break;
         default:
             return BBinder::onTransact(code, data, reply, flags);
     }