Merge "Clean up channel count and channel mask"

Clean up channel count and channel mask

Channel count is uint32_t.
Remove redundant mask parameter to AudioTrack::createTrack_l()
    and AudioRecord::openRecord_l().

Change-Id: I5dc2b18eb609b2c0dc3091994cbaa4628062c17f

delete -> free

Strings duplicated with strdup() should be free()d, not deleted.

Change-Id: I42bb3df9625bb8d35f80b02d15364b94c36496f8

Merge ""if" statements use curly braces per media style"

Merge "Fix time vs. bytes units bug in getRenderPosition"

Merge "Don't use control block frame count after create"

Merge "Don't explicitly log tid"

Don't use control block frame count after create

This is part of a series to clean up the control block.

Change-Id: I7f4cb05aef63053f8e2ab05b286d302260ef4758

am abae71d3: am d983364b: Static AudioTrack plays twice initially

* commit 'abae71d37d4860e297de7ee06f49efa5254b90ee':
  Static AudioTrack plays twice initially

am d983364b: Static AudioTrack plays twice initially

* commit 'd983364b3655a547b55bb11dbe148103198c011d':
  Static AudioTrack plays twice initially

Fix a crash when the stop might be called due to some error before start in RTSPSource

o related-to-bug: 7507224

Change-Id: Ic8bfec13097b824ba337a01c9b00c98af2a33f43

"if" statements use curly braces per media style

Change-Id: I130e7849fd1da7a0b7fe56c3c53919d26e3843b8

Don't explicitly log tid

If needed, it can be obtained with adb logcat -v threadtime

Change-Id: I91b3911d20f7bcfc3361db4052db21ff9181f1cf

Fix time vs. bytes units bug in getRenderPosition

Rename correctLatency since it requires thread to be locked.
Use size_t for byte and frame counts.

Change-Id: I178fdd18bdb823813b9563927bdff8c0d28ca5a5

Static AudioTrack plays twice initially

Bug: 7528721
Change-Id: I10bc16a26f33dba6572b730a170cb3bf00e68e30

Merge "Only pass the surface to the video decoder."

resolved conflicts for merge of 205d7249 to master

Change-Id: I3df408b6e30e0c0b2a19a3336134ce49fb73a8bb

Only pass the surface to the video decoder.

Change-Id: Ice0cfc0021fdd9fe053be6ee324cbc64226ed122

am da33d66e: Merge "Add .mpga to acceptable file name extensions list."

* commit 'da33d66e68791d0bfeccebc8253a59467b5ef670':
  Add .mpga to acceptable file name extensions list.

Merge "Add .mpga to acceptable file name extensions list."

Merge "Add GSM 6.10 decoder"

Merge "Use size_t for frame counts"

Add .mpga to acceptable file name extensions list.

The *.mpga files are playable, but are not correctly scanned.
This is because they are prevented from being scanned in
StagefrightMediaScanner.cpp.
What this fix does is to add the extension to the list of valid
file extensions so that the scanner handles the filetype properly.
We have previously added the .mpga extension to the framework to
make it playable, but not added it so that the scanner scans it.

Change-Id: I02a44d770adb80d38e8bed77d0d76efa1b28a861

Add GSM 6.10 decoder

Supports Microsoft frame packing only, since that's what the sample
file used.
b/6620569

Change-Id: Ia89d95bcbf0f8dcbaad42148a7401728f60e079d

Use size_t for frame counts

Also fix typo: bufferCount should be frameCount.

Change-Id: Ibed539504db75ef99dc21c8ff1bf2987122063a5

Merge "Static AudioTrack plays twice initially"

Static AudioTrack plays twice initially

Bug: 7528721
Change-Id: I10bc16a26f33dba6572b730a170cb3bf00e68e30

Merge "wfd sink update."

wfd sink update.

Change-Id: Ib4e41ec1524d045699543536acdddc9a243db741

The length information of the chunks making up vorbis codec specific info

are "Xiph-style-lacing encoded" instead of individual bytes.

Change-Id: Ic1274a5bd8f082197bae6831da04002762a920c5
related-to-bug: 7401329

Merge "Stagefright command line tool: input file name last"

Clean up frame size in AudioTrack and AudioFlinger

TrackBase::mFrameSize, mChannelMask, and mChannelCount are now const.
Use TrackBase::mFrameSize instead of re-calculating frame size.
AudioFlinger only sees 16-bit PCM format, conversion from 8-bit is
  now entirely on the client side.  Previously a small part of the
  responsibility was on server side also.
size_t is unsigned, so use %u in logs.
Fix theoretical bug where TrackBase constructor was over-allocating space
  for non-linear AudioTrack or 8-bit PCM AudioRecord (probably benign).

Change-Id: I7cbbba0bf4dba29ea751d8af341ab8e5cbbdc206

Use uint32_t for sample rate

Change-Id: Ie240b48fb54b08359f69ecd4e5f8bda3d15cbe80

Merge "Update audio comments"

Merge "Remove deprecated AudioSystem methods"

Merge "Fix build warnings"

Merge changes I2ce3479d,Ibb56664d

* changes:
  more optimizations...
  refactor code to improve neon code

Update audio comments

Change-Id: I85d7d2f6381b251db5695202fec75128883a8662

am dbb74f4e: am f0937247: Merge "Properly signal an error if codec configuration goes wrong." into jb-mr1.1-dev

* commit 'dbb74f4ee1a971da71f26739d870fc9334100499':
  Properly signal an error if codec configuration goes wrong.

am 911d5f93: am 0224bf17: Various improvements of wifi display code

* commit '911d5f937adbd177c69bd6959603b8a3b776097e':
  Various improvements of wifi display code

am f0937247: Merge "Properly signal an error if codec configuration goes wrong." into jb-mr1.1-dev

* commit 'f0937247b6d92b7d9457c64e36fe4c10927685ac':
  Properly signal an error if codec configuration goes wrong.

am 0224bf17: Various improvements of wifi display code

* commit '0224bf170a3904576bba81593eaab113c5d3a4e7':
  Various improvements of wifi display code

Merge "Properly signal an error if codec configuration goes wrong." into jb-mr1.1-dev

Fix build warnings

Change-Id: Ic43bcca166a529a6431711b05a7fa21849b6a38b

Properly signal an error if codec configuration goes wrong.

previously any error signaled by setupXXX inside ACodec::configureCodec
would be overwritten with the result of setMinBufferSize at the end
of the function.

Change-Id: Id4beb571ca52ea4646239d0af006e09ce1130268
related-to-bug: 7542181

Remove deprecated AudioSystem methods

Change-Id: I952d504e03af9a1d3e1e0aa379c82dfb00197d9f

Various improvements of wifi display code

- manually prepend SPS/PPS if encoder doesn't support it
- latency improvements
- support for "our" method of optional RTP retransmission
- improvements to the wfd commandline tool for testing
- make it easier to turn on/off suspension of the video pipeline on idle
- fixes an issue where an error during encryption would cause a SEGV
- add HDCP descriptor if necessary

Squashed commit of the following:

commit 1115be0ebb3b885b4f1b7dba56761ca013d0ec4a
Author: Andreas Huber <andih@google.com>
Date:   Fri Nov 9 11:32:23 2012 -0800

    Better shutdown of wfd -l sessions.

    Change-Id: Id898a14ae21efd3b065b00a729830063d39195a7

commit 0e7d106dfe4eb6e2640b0b66c65deaba265f7ff0
Author: Andreas Huber <andih@google.com>
Date:   Thu Nov 8 16:38:55 2012 -0800

    No more sending delay, create rtp packets upfront.

    Change-Id: I809a225f664fdb485c7d9a49a27886601a6a26b2

commit d399e8571b77353d59afb57508dfd2a82c1ef93a
Author: Andreas Huber <andih@google.com>
Date:   Thu Nov 8 14:19:43 2012 -0800

    Restore AudioSource buffer size, factor out TimeSeries, make

    suspending video optional.

    Change-Id: Ifdfe4d447b901e714abf52856b4641d1d55a5d41

commit f8b649f0b8f917d59f4b8a2e8e6d7db61a684a78
Author: Andreas Huber <andih@google.com>
Date:   Thu Nov 8 09:34:06 2012 -0800

    Pull 480 frames at a time from AudioSource/AudioRecord

    Change-Id: I1e215abd329faec3da026631122c0f4c800c0ac4

commit 1bc13452eb35eebbba00f5da93fa86535be5db59
Author: Andreas Huber <andih@google.com>
Date:   Thu Nov 8 08:50:30 2012 -0800

    fixed bitrate traffic simulation

    Change-Id: Ic5efb7cbb0b5d3b4917bc77b8ba73d447249e695

commit 016cdff18e74bdc631a5679e97192645ed095aa2
Author: Andreas Huber <andih@google.com>
Date:   Wed Nov 7 14:00:03 2012 -0800

    resurrected "our" style of retransmission.

    Change-Id: I34d757aba67428437cb39b8293a9651750ad20d9

commit 384cf1a3c8fb4ec410bdf8fa5722c298e6028f3e
Author: Andreas Huber <andih@google.com>
Date:   Tue Nov 6 09:38:55 2012 -0800

    Changes to make wfd work on manta.

    Change-Id: I7a4e00cf16581fe2146edd1b359af195774090e4

commit 9628f24b22b35f28630d99dda3614babf51bc07e
Author: Andreas Huber <andih@google.com>
Date:   Wed Nov 7 09:15:44 2012 -0800

    Patch up rtp timestamps to more accurately measure network jitter.

    Change-Id: I9502a4615575f97f98a215a13131a89a6ae93c6d

commit 7c891a1a24f08bbd50f55be13f7d05f43e807eb8
Author: Andreas Huber <andih@google.com>
Date:   Tue Nov 6 09:37:24 2012 -0800

    Additions to the "wfd" tool to create a local wfd source.

    Change-Id: I99558653a70fdc703f9d13990b3ce1c4d3ae227a

Change-Id: Ia94c63fc390f597014531073485f0cfc53b3418a

Merge "Rename TrackBase::mFrameCount to mStepCount"

Rename TrackBase::mFrameCount to mStepCount

This prepares for adding a new field TrackBase::mFrameCount
with a different meaning.

Change-Id: I6bbe2c59f2a882be57caeec2e2e06f439a0e9e83

Simplify AudioRecord::restoreTrack_l()

Finish removing CBLK_RESTORING and CBLK_RESTORED from control block flags,
and remove constant RESTORE_TIMEOUT_MS.

Also minor cleanup:
 - Cache mCblk in local variable cblk and make cblk allocatable in a register.
 - Use "iMem" for sp<IMemory>.
 - Add missing error log to AudioRecord; it was already in AudioTrack.

This is part of a series to clean up the control block.

Change-Id: Ia5f5ab4763c392bc06a45851b167ddaee29e3455

Merge "Move frame size out of the control block"

Scan .awb files too

b/6122599

Change-Id: Ied3e0392939231447f1fc5685ca1fade1e55ce08

Move frame size out of the control block

This is part of a series to clean up the control block.

Change-Id: Ifab1c42ac0f8be704e571b292713cd2250d12a3f

Stagefright command line tool: input file name last

Show in usage that the source file name comes last, and is preceded
 by the options.

Change-Id: I8407fc36c8d19785cb2e6e1f7b7a352a8d86f889

Fix regression for AudioTrack::write() 8-bit PCM

Bug: 7526532
Change-Id: I8ddd1f0e9d035b54401788dcc422591281dcd97a

more optimizations...

calculate the offsets from the phase differently, this happens
to reduce the register pressure in the main loop, which in turns
allows the compiler to generate much better code (doesn't need
to spill a lot of stuff on the stack).

this gives another 15% performance increase

Change-Id: I2ce3479dd48b9e6941adb80e6d443d6e14d64d96

refactor code to improve neon code

we want to make sure we don't transfer data from the
neon unit to the arm register file, as this can be quite
slow. instead we do all the calculation on the neon side
and write the result directly to main memory.

Change-Id: Ibb56664d3ab03098ae2798b75e2b6927ac900187

Merge "Move buffers pointer out of the control block"

NEON optimized SINC resampler

this currently gives us a 60% to 80% boost depending
on the quality level selected.

Change-Id: I7db385007e811ed7bffe5fd3403b44e300894f5b

minor cleanups

Change-Id: Ia12ee4fb59e90221761bec85e6450db29197591f

Move buffers pointer out of the control block

This is part of a series to clean up the control block.

Change-Id: Ie474557db7cb360f2d9a0f11600a68f5a3d46f07

Simplify AudioTrack::restoreTrack_l()

Remove CBLK_RESTORING and CBLK_RESTORED from control block flags,
for AudioTrack only.  They are still used by AudioRecord.

This is part of a series to clean up the control block.

Change-Id: Iae4798f5b527c492bdaf789987ff3a1dadd0cb37

Merge "Add all-channel AudioTrack::setVolume() API"

Merge "Fix call to restoreTrack_l() without lock held"

Fix call to restoreTrack_l() without lock held

Also document lock order

Change-Id: I2c1f273a0a51fa79ee3dd766de8d23083e270051

Remove CBLK_DIRECTION from control block flags

This is part of a series to clean up the control block.

Change-Id: I0265fece3247356b585d4d48fbda6f37aea8a851

Merge "Remove CBLK_FAST from control block flags"

Merge "Remove unused fields in AudioTrack::Buffer"

Merge "Removed unused fields in AudioRecord::Buffer"

Remove CBLK_FAST from control block flags

This is part of a series to clean up the control block.

Change-Id: Ic881a3560d9547cb63fcc0cefec87aa3da480e0d

Merge changes I5d62a6c2,Iebf76958,I8068f0f7

* changes:
  improve resample test
  change how we store the FIR coefficients
  improve SINC resampler performance

Remove unused fields in AudioTrack::Buffer

Change-Id: Iab75f6e2348d8b6d1f3cec95aeb3fcd5135dfb50

Removed unused fields in AudioRecord::Buffer

Change-Id: I89fc6d8f695b48516d956b0a9a4a43d408f369f9

am 33cf49b1: (-s ours) am 78b012f5: Reconcile with jb-mr1-factory-release jb-mr1-release - do not merge

* commit '33cf49b168a88a88167c46b01787a57b49cd875f':

am 78b012f5: Reconcile with jb-mr1-factory-release jb-mr1-release - do not merge

* commit '78b012f5bc05706d1ce412811ccfaa9385dae0ab':

Reconcile with jb-mr1-factory-release jb-mr1-release - do not merge

Change-Id: Idf4c25115d89397ba668fc290344b2e7c1ea1993

Merge "Cache mCblk in local variable cblk"

am 3b31d805: am 1c6833af: Ignore SIGPIPE during write() to broken pipe

* commit '3b31d805517c0bde8a0ebed219cbd4b8409c71c2':
  Ignore SIGPIPE during write() to broken pipe

am 1c6833af: Ignore SIGPIPE during write() to broken pipe

* commit '1c6833afd6cfdbc6c696ed0719ceedc37ec0b534':
  Ignore SIGPIPE during write() to broken pipe

Cache mCblk in local variable cblk

Use "iMem" for sp<IMemory>

Change-Id: I2f1fbbc517fbd77cfc92f6c3b1f253c26bae93b0

Merge "AudioRecord comments"

Ignore SIGPIPE during write() to broken pipe

This happens occasionally when taking a bugreport.

Bug: 6447319
Change-Id: Ia6531a4a3658461f8fd3f7106e7996da7cc5933a

Merge duplicate code in TrackBase::TrackBase()

Change-Id: Id8a7db4d94888796a9dcbed4cac99941705cd174

Merge "Simplify control block flag names"

Add all-channel AudioTrack::setVolume() API

Add combined channel APIs setVolume to AudioTrack,
and remove obsolete getVolume.

Change-Id: I0c87bfdbff4f4292259fa33e65f67badbafd270b

Merge "Remove deprecated AudioTrack APIs"

Remove deprecated AudioTrack APIs

Change-Id: I88be6525f3e33df529c0c3cb701d12a484809477

Simplify control block flag names

Use only one symbol per flag

Change-Id: Ia3582e2134abd60c896d11337face65383e79c7c

improve resample test

- handle stereo input
- input file can now be ommited, in this case
  a linear chirp will be used automatically
- better usage information

Change-Id: I5d62a6c26a9054a1c1a517a065b4df5a2cdcda22

change how we store the FIR coefficients

The coefficient table is now transposed and shows
much better its polyphase nature: we now have a FIR
per line, each line corresponding to a phase.

This doesn't change at all the results produced by
the filter, but allows us to make slightly better
use of the data cache and improves performance a bit
(although not as much as I thought it would).

The main benefit is that it is the first step
before we can make much larger optimizations
(like using NEON).

Change-Id: Iebf7695825dcbd41f25861efcaefbaa3365ecb43

improve SINC resampler performance

The improvement is about 60% by just tweaking a few
things to help the compiler generate better code.
It turns out that inlining too much stuff manually was hurting us.

Change-Id: I8068f0f75051f95ac600e50ce552572dd1e8c304

Turn off executable bit on ordinary files

Change-Id: I0abea25b58fb1d03975bed9cca40f826fcd4c5e4

Merge "new coefficients for the vhq resampler"

Fix spurious wakeup waiting for new IAudioTrack

If there was a spurious wakeup while waiting for another thread to create
a new IAudioTrack, we assumed that the track has been created when it
might not have been.

Change-Id: I5f3999b4f7a06a00aabd65a746cc7222fff396ab

AudioRecord comments

Change-Id: Ibec910608948d778dc655d900255a80384e9b06f

Merge "Add NBAIO support for more sample rates"

Merge "AudioTrack comments"

AudioTrack comments

Change-Id: I08988f635c5bc6f4c35e9c4188f3d6fa557bc4c2

Merge "Remove unused default parameter for copyBuffer()"

am 8b6ce0a2: (-s ours) Reconcile with jb-mr1-release - do not merge

* commit '8b6ce0a269ba67995766980f5b83937302143e64':