commit 38585c253279bbadf21d49512d315a92cf4ff291
author Pawin Vongmasa <pawin@google.com> Tue Dec 06 03:03:14 2016 -0800
committer Pawin Vongmasa <pawin@google.com> Mon Dec 19 13:04:51 2016 +0700
tree fe10f877d5a74e7054b202621faae76165b8e0fe
parent cba23c66310331d9cae00ed4726642c73410a6fa

Remove the return Status from IGraphicBufferSource's functions.

Now, only the transport-level status is needed, so
::android::hardware::Status is sufficient.

Also simplify IOmxNode::sendCommand().

Test: Compiles

Bug: 31399200
Change-Id: Ia4cfc16681d19eb96f89d8db715fcc14516cb41a

media/omx/1.0/IGraphicBufferSource.hal

diff --git a/media/omx/1.0/IGraphicBufferSource.hal b/media/omx/1.0/IGraphicBufferSource.hal
index a5b5813..9b3ab0c 100644
--- a/media/omx/1.0/IGraphicBufferSource.hal
+++ b/media/omx/1.0/IGraphicBufferSource.hal
@@ -29,32 +29,23 @@
  */
 interface IGraphicBufferSource {
 
-    configure(IOmxNode omxNode, Dataspace dataspace)
-        generates (Status status);
+    configure(IOmxNode omxNode, Dataspace dataspace);
 
-    setSuspend(bool suspend)
-        generates (Status status);
+    setSuspend(bool suspend);
 
-    setRepeatPreviousFrameDelayUs(int64_t repeatAfterUs)
-        generates (Status status);
+    setRepeatPreviousFrameDelayUs(int64_t repeatAfterUs);
 
-    setMaxFps(float maxFps)
-        generates (Status status);
+    setMaxFps(float maxFps);
 
-    setTimeLapseConfig(int64_t timePerFrameUs, int64_t timePerCaptureUs)
-        generates (Status status);
+    setTimeLapseConfig(int64_t timePerFrameUs, int64_t timePerCaptureUs);
 
-    setStartTimeUs(int64_t startTimeUs)
-        generates (Status status);
+    setStartTimeUs(int64_t startTimeUs);
 
-    setColorAspects(ColorAspects aspects)
-        generates (Status status);
+    setColorAspects(ColorAspects aspects);
 
-    setTimeOffsetUs(int64_t timeOffsetUs)
-        generates (Status status);
+    setTimeOffsetUs(int64_t timeOffsetUs);
 
-    signalEndOfInputStream()
-        generates (Status status);
+    signalEndOfInputStream();
 
 };