Merge "Make inlines static" into stage-aosp-master

Merge "Make inlines static"
am: e338b979f9  -s ours

Change-Id: I2a94c355f424a516069077cb3aa8d4dd1d5cd716

Make inlines static

Compiling the non-static inlines with -Oz results in missing symbols,
because clang chooses not to inline the functions but also doesn't
emit a definition because they are not static or extern.  They are
only used in this .c file, so make them static.

Test: builds with -Oz
Bug: 62549703
Change-Id: Ibb78ad982c2911e151f0dd6b01eb6164fe6e8e63
Merged-In: Ibb78ad982c2911e151f0dd6b01eb6164fe6e8e63

Merge "Make inlines static"

Make inlines static

Compiling the non-static inlines with -Oz results in missing symbols,
because clang chooses not to inline the functions but also doesn't
emit a definition because they are not static or extern.  They are
only used in this .c file, so make them static.

Test: builds with -Oz
Bug: 62549703
Change-Id: Ibb78ad982c2911e151f0dd6b01eb6164fe6e8e63
Merged-In: Ibb78ad982c2911e151f0dd6b01eb6164fe6e8e63
(cherry picked from commit c7eae5d84a55de0450b9947a26127ef05cb089a5)

Merge "fix missing headers"
am: 10be5075f2

Change-Id: I8791011831e4979b4ef583307d8ed54e92a66279

Merge "fix missing headers"

Merge "Rename audio_utils/conversion.h to mono_blend.h"
am: 5ce9898e08  -s ours

Change-Id: I79fd6d39c7eb6fb445285fbf48ddc0c524375bee

fix missing headers

Bug: 33241851
Test: cd system/media && mma -j64 -k BOARD_VNDK_VERSION=current
Change-Id: I7c6e4b271b365f50c376af2925d0f8cbe8e58551

Merge "Rename audio_utils/conversion.h to mono_blend.h"

Rename audio_utils/conversion.h to mono_blend.h

There is only one function declared in that header file, and it is
mono_blend, so a more specific filename may be easier to understand.

Test: compiles OK
Merged-In: Ie2aec6c706b1822581b34d3702818b8200a1db8a
Change-Id: Ie2aec6c706b1822581b34d3702818b8200a1db8a
(cherry picked from commit 66c7057547bbbae28704a014a64068a5a6625a48)

Merge "Add OWNERS in system/media/camera" am: c134db6eaa
am: a73060cb62

Change-Id: Id50556eb466d29a6cc8865fa123999343f250657

Merge "Add OWNERS in system/media/camera"
am: c134db6eaa

Change-Id: Ic1bfebb355acfe77e8cca67b58db2431da0c71fb

Merge "Add OWNERS in system/media/camera"

Add OWNERS in system/media/camera

* This is copied from frameworks/av/camera/OWNERS.
* Owners will be suggested to review/approve future CLs.
* OWNERS files are recognized by the new find-owners plugin, see .md files in
  https://gerrit.googlesource.com/plugins/find-owners/+/master/src/main/resources/Documentation/

Test: build/make/tools/checkowners.py -c -v OWNERS
Change-Id: I5452c77ed8e9c476c2257718f239fbd56a56ee2e

Merge "make libaudioutils_fixedfft vendor_available:true" am: b50f705f5c
am: 026fd65638  -s ours

Change-Id: Ibdc5441ed8ea8de1c06aa4caf1d2d0b7446fd58b

Merge "make libaudioutils_fixedfft vendor_available:true"
am: b50f705f5c

Change-Id: Icd573a4fa9b08a6ea0e5587c60d1623901e6e8f4

Merge "make libaudioutils_fixedfft vendor_available:true"

make libaudioutils_fixedfft vendor_available:true

This, when used with BOARD_VNDK_VERSION flag, enables the
module to be built and installed twice, one for /system,
other for /vendor.

libaudioutils_fixedfft is used by libaudioutils which is in turn used by libs in
/vendor and thus marked as vendor_available. Therefore, libaudioutils_fixedfft
is also marked as vendor_available.

Details: https://android-review.googlesource.com/368372

Bug: 33241851
Test: BOARD_VNDK_VERSION=current m libaudioutils_fixedfft.vendor successful
Merged-In: Ic594eeb153b90ee45e7b5d9b682af9c3ff5471e4
Change-Id: Ic594eeb153b90ee45e7b5d9b682af9c3ff5471e4
(cherry picked from commit 09d178f4f3bc33b347b2f8f6b68e8165f7b4edfc)

Merge "make libaudioutils_fixedfft vendor_available:true" into oc-dev
am: b8ca85c9ad

Change-Id: I31e13f7517d8fbe708acd7110eb616be7104b95b

Merge "make libaudioutils_fixedfft vendor_available:true" into oc-dev

Moved libalsautils to vendor. Marked libaudioutils and libaudioroute as vendor_available
am: 43338e77ca  -s ours

Change-Id: If93a55a9fba325332ad20150c2600feaeaf740ab

Moved libalsautils to vendor. Marked libaudioutils and libaudioroute as
vendor_available

Bug: 33241851
Bug: 37527576
Test: Build sailfish and pass CTS audio
Test: With BOARD_VNDK_VERSION:=current, no link type check
errors related to these libs

(cherry picked from commit a25465b1c46c7ac172371b523e816988dd4aa018)

Merged-In: Ifacd2305a79a50f6bf3182344bc678082047022f
Change-Id: Icb17b4266a2c6443e482efd62769dbc430e7501c

make libaudioutils_fixedfft vendor_available:true

This, when used with BOARD_VNDK_VERSION flag, enables the
module to be built and installed twice, one for /system,
other for /vendor.

libaudioutils_fixedfft is used by libaudioutils which is in turn used by libs in
/vendor and thus marked as vendor_available. Therefore, libaudioutils_fixedfft
is also marked as vendor_available.

Details: https://android-review.googlesource.com/368372

Bug: 33241851
Test: BOARD_VNDK_VERSION=current m libaudioutils_fixedfft.vendor successful
Merged-In: Ic594eeb153b90ee45e7b5d9b682af9c3ff5471e4
Change-Id: Ic594eeb153b90ee45e7b5d9b682af9c3ff5471e4
(cherry picked from commit 09d178f4f3bc33b347b2f8f6b68e8165f7b4edfc)

Merge "Moved libalsautils to vendor. Marked libaudioutils and libaudioroute as vendor_available" am: 74b2fff8f4 am: 4d15f047d6
am: fcc32970cf

Change-Id: I658cf987c99c45bd10f72778aef18db9d66ce437

Merge "Moved libalsautils to vendor. Marked libaudioutils and libaudioroute as vendor_available" am: 74b2fff8f4
am: 4d15f047d6

Change-Id: I840a1f4f73b952187db8840d6d42980f15d10c90

Merge "Moved libalsautils to vendor. Marked libaudioutils and libaudioroute as vendor_available"
am: 74b2fff8f4

Change-Id: Idfe556963a27260bfff3233dd3caf4aebdc1770e

Merge "Moved libalsautils to vendor. Marked libaudioutils and libaudioroute as vendor_available"

Merge "libalsautils: moved to vendor partition" am: f03c1c09ed am: 1662e1e5c9
am: 6901b39778

Change-Id: I46ba1a35bc7891a5f124319645602e8ce865bc02

Merge "libalsautils: moved to vendor partition" am: f03c1c09ed
am: 1662e1e5c9

Change-Id: If6ad072fcca1e7d7b0064c610f4a64eafc1fe360

Merge "libalsautils: moved to vendor partition"
am: f03c1c09ed

Change-Id: Icdbc804f07606664882aba5024e4b43e32f1c340

Merge "libalsautils: moved to vendor partition"

libalsautils: moved to vendor partition

Generic AOSP system image doesn't need libalsautils, so move
it to vendor partition in case vendors use it.

Bug: 37117855
Test: Build it with "vendor: true," and make sure it's actually
moved to vendor and also boot successfully on Sailfish.
Change-Id: Ib8c08276f0435b2b37ac6179100b7287b08f6f03

Moved libalsautils to vendor. Marked libaudioutils and libaudioroute as
vendor_available

Bug: 33241851
Bug: 37527576
Test: Build sailfish and pass CTS audio
Test: With BOARD_VNDK_VERSION:=current, no link type check
errors related to these libs

Change-Id: Ifacd2305a79a50f6bf3182344bc678082047022f

Merge "camera: Clarify ZSL supports" into oc-dev
am: 5da5ad056c

Change-Id: I1b4b1401f4e7ba61bfd1647f93203e591c987ef4

Merge "camera: Clarify ZSL supports" into oc-dev

Merge "libaudioeffects: import/export libhardware_headers" am: b899f5d93c am: 2804dbe772
am: bfab76413e

Change-Id: Ie7e1dcb916de7c58b3c41bd9e4e4ee5c14446418

Merge "libaudioeffects: import/export libhardware_headers" am: b899f5d93c
am: 2804dbe772

Change-Id: I542617b846358990ba980cf8f8fefaacff87a7db

Merge "libaudioeffects: import/export libhardware_headers"
am: b899f5d93c

Change-Id: I449fa1eff5055f2cbfd197ebe0774db5b957adc3

Merge "libaudioeffects: import/export libhardware_headers"

Merge "Update path to camera ndk headers" am: 9d7eff8334 am: 334bca910c
am: 344a7212f4  -s ours

Change-Id: Ie4188461c8159b7d478636be9bc5f20ad1510cf8

Camera metadata: Fix validation alignment check, add more unit tests
am: 0b47c6ef73

Change-Id: I103aefe3900017e8b1dfc4834beaba0f409f3cf9

Merge "Update path to camera ndk headers" am: 9d7eff8334
am: 334bca910c

Change-Id: I692efffbd9e25e197f0874d69ec8ce991e2e4156

Merge "Update path to camera ndk headers"
am: 9d7eff8334

Change-Id: I028e232e95e0aef8c559f0f099fb4b5ef5e25e9c

Merge "Update path to camera ndk headers"

Update path to camera ndk headers

Test: generate-metadata
Change-Id: Idcafb23a5483d77296003230ec75bcbdd393cf52
Merged-In: Idcafb23a5483d77296003230ec75bcbdd393cf52
(cherry picked from commit abc7c4963b25567852b1cdcfbd90a1626dddb3ff)

Update path to camera ndk headers

Test: generate-metadata
Change-Id: Idcafb23a5483d77296003230ec75bcbdd393cf52
(cherry picked from commit abc7c4963b25567852b1cdcfbd90a1626dddb3ff)

Camera metadata: Fix validation alignment check, add more unit tests

- Add unit test to ensure validate_camera_metadata_structure correctly
  reports VALIDATION_SHIFTED for all shifts
- Fix alignment validation bug found by unit test - need to offset
  pointer on both sides of align check.

Test: New unit test passes, no camera CTS regression
Bug: 37980415
Change-Id: I08379bf3792b57c4eae129acc7aa4f22ab960657

libaudioeffects: import/export libhardware_headers

Required for #include <hardware/audio_effect.h>.

Test: build with BOARD_VNDK_VERSION=current
Bug: 33241851
Change-Id: Ie5b69eddbc47f7de7f1def062e1da54f33a991c2

alsa proxy: fix sampling rate checks in proxy_prepare()
am: 49bc03cfdc

Change-Id: Ica03849c6735d56d11c137dcd23c31ca89b171f7

alsa proxy: fix sampling rate checks in proxy_prepare()

- proxy_prepare() should not alter the requested properties (format,
sampling rate, channels)  without notifying the caller.
- It should not force the sampling rate to the max sampling rate supported
by the profile but just limit it if the requested combination of
sampling rate, channel count and format is not possible.

Bug: 37949012
Test: verify capture over USB at various sampling rates
Change-Id: Id79ca4fda86db1f756e171d131494c3f244b5f2c

camera: Clarify ZSL supports

For application-operated ZSL, use CAMERA3_TEMPLATE_ZERO_SHUTTER_LAG
template.

Test: make
Bug: 34689167
Change-Id: I4e3fab6abb93f697589d5ce541529f8d51095d17

Merge "camera: Update for enableZsl values in template" into oc-dev
am: 6bbcd2a35d

Change-Id: I35a5dea7747bdc2ed554b1c8e71624f2134eb3f9

Merge "camera: Update for enableZsl values in template" into oc-dev

Merge "Add libaudioeffects header library" am: 8f8b3b9d38 am: aa9d32b4ce
am: 2de81923ba

Change-Id: I2c05e2487881be992cee7b3663a0c8776b7e650a

Merge "Add libaudioeffects header library" am: 8f8b3b9d38
am: aa9d32b4ce

Change-Id: Ide40e758c2c51eef20c966b43906e63975f731d5

Merge "Add libaudioeffects header library"
am: 8f8b3b9d38

Change-Id: I0bff41dde5f612e617638426d280c4df0512103a

Merge "Add libaudioeffects header library"

Add libaudioeffects header library

Add a header library for audio-effects instead of requiring modules
to use $(call include-path-for, audio-effects).

Test: m -j checkbuild
Change-Id: I21a191e7fe2911b80396ddbe84cd032592ac43b1

Merge changes from topic 'system-vendor-internal-pick' am: 9b88fb1019 am: c53ca010d7
am: 24408d52e4

Change-Id: I5b8a498a6410aa6d715ade604b39398402645e23

Add libaudio_system_headers am: d3d48f7e5e am: e43eb26510
am: a712780e26  -s ours

Change-Id: I23bcea1f807e7502d5b879bf2d80d5c9c6d6700d

Mark as vendor_available am: 752eeb00b8 am: 6f244c0643
am: 779d253206

Change-Id: I63b511509c09e3be980716a3f60e98cc3226772f

libradio_metadata: vendor_available am: 05ba91aba7 am: ecb36e0706
am: db9365dbd3

Change-Id: Ia84ef0ba6b2ca890564b6ca07f0aab124de94caa

libradio_metadata: work with BOARD_VNDK_VERSION am: 4473f2d570 am: 1733c56958
am: 9356e9d781

Change-Id: Ia80cdc46f350a668933fe852d1920b052f651b3d

Merge changes from topic 'system-vendor-internal-pick' am: 9b88fb1019
am: c53ca010d7

Change-Id: I3c2e924dc4481b9e6c7c2111fdafd51d03c56027

Add libaudio_system_headers am: d3d48f7e5e
am: e43eb26510

Change-Id: Iac5bfc414eaeb953b6f66617e45ea1599f76056c

Mark as vendor_available am: 752eeb00b8
am: 6f244c0643

Change-Id: I14c5ac188b34bcb57086bfac02947ba933921330

libradio_metadata: vendor_available am: 05ba91aba7
am: ecb36e0706

Change-Id: I8da377c07dccc4203a4c7b149729b68431b4c0a8

libradio_metadata: work with BOARD_VNDK_VERSION am: 4473f2d570
am: 1733c56958

Change-Id: I464b03f0e495eed365e4c6d9e2b65a565a68f509

Merge changes from topic 'system-vendor-internal-pick'
am: 9b88fb1019

Change-Id: I195eeb8de2f4eca09d414bbf20c49c8d53fe0c66

Add libaudio_system_headers
am: d3d48f7e5e

Change-Id: I5b2cecb7bd799f13b2e5b7ec4b30cc51e32be50d

Mark as vendor_available
am: 752eeb00b8

Change-Id: I63d3f7d9425a0cd197f700ab2734a7d1baee43da

libradio_metadata: vendor_available
am: 05ba91aba7

Change-Id: I1b37866352d178b81ec41c03ce6f0d12b82275c5

libradio_metadata: work with BOARD_VNDK_VERSION
am: 4473f2d570

Change-Id: I8bcbb929fe109bd2abda0cf28f7878d1f08577ec

Merge changes from topic 'system-vendor-internal-pick'

* changes:
  libaudio_system_headers: include libcutils_headers
  Add libaudio_system_headers
  Mark as vendor_available
  libradio_metadata: vendor_available
  libradio_metadata: work with BOARD_VNDK_VERSION

Camera: doc cleanup
am: 1c0a88599d

Change-Id: I5993023a1388092d15288ecb2d7e9482b5771451

Camera: doc cleanup

Test: compile
Bug: 37095012
Change-Id: Iaad8885420ad031fc743ede81d127d0783803646

libaudio_system_headers: include libcutils_headers

This is required since system/audio.h (and one more) uses
utils/BitOps.h.

Test: works with BOARD_VNDK_VERSION := current
Bug: 33241851
(cherry picked from commit 013d4a633f73e24f877887060accc0cb74055c89)
Merged-In: I13ef898a702316ccbf8b41ec500b5fd60dce155b
Change-Id: I13ef898a702316ccbf8b41ec500b5fd60dce155b

Add libaudio_system_headers

Test: modules using this compiled with BOARD_VNDK_VERSION := current
find dependencies
Bug: 33241851
(cherry picked from commit a834562a91b97f8b2ab3a5d6ed99d778c071f2cf)
Merged-In: If2d7ef0ca806b721c65e75774a754a8ee1dbb5fd
Change-Id: If2d7ef0ca806b721c65e75774a754a8ee1dbb5fd

Mark as vendor_available

By setting vendor_available, the following may become true:

* a prebuilt library from this release may be used at runtime by
  in a later releasse (by vendor code compiled against this release).
  so this library shouldn't depend on runtime state that may change
  in the future.
* this library may be loaded twice into a single process (potentially
  an old version and a newer version). The symbols will be isolated
  using linker namespaces, but this may break assumptions about 1
  library in 1 process (your singletons will run twice).

Background:

This means that these modules may be built and installed twice --
once for the system partition and once for the vendor partition. The
system version will build just like today, and will be used by the
framework components on /system. The vendor version will build
against a reduced set of exports and libraries -- similar to, but
separate from, the NDK. This means that all your dependencies must
also mark vendor_available.

At runtime, /system binaries will load libraries from /system/lib*,
while /vendor binaries will load libraries from /vendor/lib*. There
are some exceptions in both directions -- bionic(libc,etc) and liblog
are always loaded from /system. And SP-HALs (OpenGL, etc) may load
/vendor code into /system processes, but the dependencies of those
libraries will load from /vendor until it reaches a library that's
always on /system. In the SP-HAL case, if both framework and vendor
libraries depend on a library of the same name, both versions will be
loaded, but they will be isolated from each other.

It's possible to compile differently -- reducing your source files,
exporting different include directories, etc. For details see:

https://android-review.googlesource.com/368372

None of this is enabled unless the device opts into the system/vendor
split with BOARD_VNDK_VERSION := current.

Bug: 36426473
Bug: 36079834
Test: m -j libcamera_metadata
Test: attempt to compile with BOARD_VNDK_VERSION := current
(cherry picked from commit 4c5e9d1995908cf03aeb91fa5ca0233d1c757fb4)
Merged-In: Ic267fbff784ddb8090266f82d505bcfac52fd1e1
Change-Id: Ic267fbff784ddb8090266f82d505bcfac52fd1e1

libradio_metadata: vendor_available

By setting vendor_available, the following may become true:

* a prebuilt library from this release may be used at runtime by
  in a later releasse (by vendor code compiled against this release).
  so this library shouldn't depend on runtime state that may change
  in the future.
* this library may be loaded twice into a single process (potentially
  an old version and a newer version). The symbols will be isolated
  using linker namespaces, but this may break assumptions about 1
  library in 1 process (your singletons will run twice).

Background:

This means that these modules may be built and installed twice --
once for the system partition and once for the vendor partition. The
system version will build just like today, and will be used by the
framework components on /system. The vendor version will build
against a reduced set of exports and libraries -- similar to, but
separate from, the NDK. This means that all your dependencies must
also mark vendor_available.

At runtime, /system binaries will load libraries from /system/lib*,
while /vendor binaries will load libraries from /vendor/lib*. There
are some exceptions in both directions -- bionic(libc,etc) and liblog
are always loaded from /system. And SP-HALs (OpenGL, etc) may load
/vendor code into /system processes, but the dependencies of those
libraries will load from /vendor until it reaches a library that's
always on /system. In the SP-HAL case, if both framework and vendor
libraries depend on a library of the same name, both versions will be
loaded, but they will be isolated from each other.

It's possible to compile differently -- reducing your source files,
exporting different include directories, etc. For details see:

https://android-review.googlesource.com/368372

None of this is enabled unless the device opts into the system/vendor
split with BOARD_VNDK_VERSION := current.

Bug: 36426473
Bug: 37343404
Test: m -j libradio_metadata
Test: attempt to compile with BOARD_VNDK_VERSION := current
(cherry picked from commit c06e7d5ed7abcfae2fac39b7e8988afcc25b9034)
Merged-In: Ia80038019847b24118850adce2e4180065082e9b
Change-Id: Ia80038019847b24118850adce2e4180065082e9b

libradio_metadata: work with BOARD_VNDK_VERSION

Test: works with BOARD_VNDK_VERSION
Bug: 37342627
(cherry picked from commit 131eff918c241ee5b016a050f13614588f6fc75f)
Merged-In: I734f94348ce5818ee6720f2ad4f3ad3ad6e74633
Change-Id: I734f94348ce5818ee6720f2ad4f3ad3ad6e74633

Merge "Audio route: log fopen xml_patch failure errno" into oc-dev
am: 11c984b733

Change-Id: I280594875c0395ee80b14c63f0c952c5a70d99ce

Merge "Audio route: log fopen xml_patch failure errno" into oc-dev

Merge "camerametadata: add SHIFTED return value to validate_camera_metadata_structure" into oc-dev
am: df49dc6bee

Change-Id: Ie0bbb77c77a4e54ce96137bc2c781a82a67a978c

Merge "camerametadata: add SHIFTED return value to validate_camera_metadata_structure" into oc-dev

camera: Update for enableZsl values in template

For apps targeting SDK versions O and newer, the value of enableZsl
may be true in still capture template. the value of enableZsl is
always false for other templates.

For apps targeting SDK versions older than O, the value of enableZsl
is always flase.

Test: make
Bug: 34983527
Change-Id: I4819495d5ce74773cc410f0d1aa9bef4bb4bf2be

Audio route: log fopen xml_patch failure errno

Vts fail a log because of this open. It is likely a result of previous
leaks (EMFILE) but in order to be sure, log fopen errno value in case of
failure.

Test: play music
Bug: 36311550

Change-Id: I834b346d5387b142d7bd0bd9c1c6d9b9e2a85339
Signed-off-by: Kevin Rocard <krocard@google.com>

camerametadata: add SHIFTED return value to validate_camera_metadata_structure

This allows client to check for camera metadata buffer passed
by some transports that might not preserve alignment property.

Test: compile, GCA working
Bug: 37095012
Change-Id: I2caefe73d86723bc0bc31bb81ff6f692166000c5

Merge "Add option to flush the entire buffer on -EOVERFLOW" into oc-dev
am: 10c34c7578

Change-Id: Ica333cff3f4973450a5467138c37f7190033d1f8

Merge "Add option to flush the entire buffer on -EOVERFLOW" into oc-dev

Merge "libradio_metadata: work with BOARD_VNDK_VERSION" into oc-dev
am: 2f25151524

Change-Id: I82afc3a6fa700204184039b8c0b5490538f62e35

Merge "libradio_metadata: work with BOARD_VNDK_VERSION" into oc-dev

Merge "libaudio_system_headers: include libcutils_headers" into oc-dev
am: f8d332188b

Change-Id: I9970defff8c733cfa75c6a284847895ad3abf542

Merge "libaudio_system_headers: include libcutils_headers" into oc-dev

libradio_metadata: work with BOARD_VNDK_VERSION

Test: works with BOARD_VNDK_VERSION
Bug: 37342627
Change-Id: I734f94348ce5818ee6720f2ad4f3ad3ad6e74633

Merge "libradio_metadata: vendor_available" into oc-dev
am: 2bcd51a9a3

Change-Id: I4a8d6582af20f3b962869971616ede1db0620b8c

Merge "libradio_metadata: vendor_available" into oc-dev

libaudio_system_headers: include libcutils_headers

This is required since system/audio.h (and one more) uses
utils/BitOps.h.

Test: works with BOARD_VNDK_VERSION := current
Bug: 33241851
Change-Id: I13ef898a702316ccbf8b41ec500b5fd60dce155b

Camera: add static assertion for camera_metadata
am: 7591a98dba

Change-Id: I1bbb711021005ad2e6764a137d9e11985e69af39