summaryrefslogtreecommitdiff
path: root/modules
Commit message (Collapse)AuthorAgeFilesLines
* Merge tag 'android-10.0.0_r27' into q10.0HEADq10.0doc HD2020-02-051-0/+3
|\ | | | | | | | | | | Android 10.0.0 Release 27 (QQ1B.200205.002) Change-Id: Idaf2878e5364c443c0857112b7d94adc62234bc5
| * Fix accidental implicit fallthroughsColin Cross2019-12-181-0/+3
| | | | | | | | | | | | | | | | | | | | | | -Wimplicit-fallthrough was printing warnings for these, they look like real errors. Fix them and re-enable the error. Bug: 144040966 Test: treehugger Merged-In: I818e18a995760f17b85b4c72ee577e531fa2a601 Change-Id: I818e18a995760f17b85b4c72ee577e531fa2a601 (cherry picked from commit bfcf9140fe61eb3d3830daa396ed1992fe613303)
* | audio: Add audio amplifier HALEthan Chen2019-12-112-0/+173
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a squash of the following commits. commit dabc25c1d073a0f32b706e59b87ac89f67389107 Author: Ethan Chen <intervigil@gmail.com> Date: Fri, 03 Jul 2015 21:35:30 -0700 audio: Notify amplifier HAL of device enable/disable Change-Id: Ice808c9b55a9e3bc8bafe5ca3ff555377d38dd8f commit 5b2337c434acff34d85ae14b9cc2d3507fdffe52 Author: Scott Mertz <scott@cyngn.com> Date: Fri, 11 Sep 2015 12:09:06 -0700 amplifier: add set_parameters method - audio hal should pass the audio_device parameters to the amplifier to allow the amplifier to make decisions based on the additional parameters. For example, we may want to change settings for DTS TrueMedia use case. Change-Id: Iccf6ef7ced2abd2e12e857eea8e580cda15eec04 commit 6a6c815e74bc9c8da0cead508dfa722229ea878f Author: Scott Mertz <scott@cyngn.com> Date: Wed, 21 Oct 2015 10:11:49 -0700 audio_amplifier: fix compilation stdlib.h is needed explicitely with the new toolchain/toolchain options. Change-Id: I876041828310ec8f9fb1e0d636a38cce1977d9b9 commit dcc6b5fd693b129492a45ec75038ca582689bd5d Author: Ethan Chen <intervigil@gmail.com> Date: Fri, 15 Jan 2016 16:12:39 -0800 amplifier: Fix compilation warning * Forward declare str_parms Change-Id: Iafb0ad8e4b0696807cfe568d0f89ce5dad6bcade commit 978ac098d6cde55bc5090ea83a9bda2ee176577b Author: codeworkx <daniel.hillenbrand@codeworkx.de> Date: Sat Mar 24 21:30:34 2018 +0100 audio_amplifier: add hooks for stream parameter manipulation This allows to modify audio parameters before the first device selection happens which is necessary when setting ANC parameter since it must happen before audio hal hits voice_start_call() and selects devices. If setting ANC after stream has been started it will trigger a re-selection of devices and cause a short downtime. Change-Id: Icfddab5ce27235c1567a0557e63b3a7421778187 Change-Id: I0473143057d542c8fa0a3ec4a67ce277900badbe
* Strip sensors HAL module.Vineeta Srivastava2019-06-121-2/+0
| | | | | | Bug: 134685292 Test: verified on a device that the library size reduces. Change-Id: Ic22948a050f59b6ec0ae38dd89184f531d8b5c01
* Remove libBionicGtestMainChangyeon Jo2019-06-041-1/+0
| | | | | | | | | This change removes libBionicGtestMain from the dependency of camera.v4l2_test target. Bug: 134488555 Change-Id: Ia719f770b03fa1679bb9467f58806e64b05eb469 Signed-off-by: Changyeon Jo <changyeon@google.com>
* Fix remote submix MIX_RECORD stopFrançois Gaffie2019-04-151-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | Since rework of presentationTimestamp in AudioFlinger, it is expecting to drain correctly before stopping an output. When a MIX RECORDER is set, 1/ Dynamic Policy Mix is registered 2/ Extraction Sink Port is made available 3/ Player is started on this sink 4/ Extraction source port is connected upon start Output 4/ Capture may be launched on the source to be rerouted 5/ All Capture are stopped, player is stopped 6/ Extraction source port shall be disconnected on stopOutput Step 6 does not happen as the track is never set as drained. Upon capture stop, rxSink MonoPipe is put in shutdown state. Any further write will discard the buffer. However, frames written shall be incremented of these frame discarded to be able to set the track as drained. This CL fixes this issue. Test: AudioPolicyEmulatorTests --gtest_filter=RecordInjectionSourceMatch/DynamicPolicyMixRecordInjectionTest.RecordingInjection/* Bug: 130356101 Change-Id: I861c4867991da8bcc1090be56d64e69646d58dc2 Signed-off-by: François Gaffie <francois.gaffie@renault.com>
* remote submix: force pipe shutdown on input closeEric Laurent2019-03-041-0/+15
| | | | | | | | | Force mono pipe shutdown when all input stream clients are closed so that a pending write is not stuck waiting for available buffer. Bug: 119057550 Test: abort search on BLE remote Change-Id: I4391e73874e502da5000b1746436fb61bb2fe869
* Fixes incorrect array initialization.Changyeon Jo2019-02-261-1/+1
| | | | | | | | | This change fixes an array initialization failure due to recent resizing of camera_module structure's reserved variable. Bug: 126418509 Change-Id: Ifda61b5427f317192976aee788d0e6849f7d189f Signed-off-by: Changyeon Jo <changyeon@google.com>
* Merge "Explicitly include log/log.h instead of cutils/log.h" am: 68222be667 ↵Sasha Smundak2019-01-311-1/+1
|\ | | | | | | | | | | | | | | am: 4b0a646750 am: 99e3058602 Change-Id: I9ccccabeeadaf742ebb4e00264c9dfbd8e769e3c
| * Explicitly include log/log.h instead of cutils/log.hSasha Smundak2019-01-311-1/+1
| | | | | | | | | | | | | | Eliminates the warning. Test: treehugger Change-Id: I567d8fea169e688165ede1fb35d1338c565e3645
* | Merge "camera: add info about current status into README" am: 3d7f02b0ee am: ↵Sergii Piatakov2019-01-281-0/+8
|\| | | | | | | | | | | | | | | 11f257623b am: af36e97a4b Change-Id: I95f2896055e353060c1a3395520aeb719e986832
| * Merge "camera: add info about current status into README"Treehugger Robot2019-01-281-0/+8
| |\
| | * camera: add info about current status into READMESergii Piatakov2018-10-171-0/+8
| | | | | | | | | | | | | | | | | | Test: n/a. Change-Id: I9e62a8a6d8c04dac2c36bc483f011cb75ce20b2e Signed-off-by: Sergii Piatakov <sergii.piatakov@globallogic.com>
* | | Merge "Adding Audio HAL V5: Direction API"Paul Mclean2019-01-171-0/+28
|\ \ \
| * | | Adding Audio HAL V5: Direction APIPaul McLean2018-12-191-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 119137468 Test: Tesbed App Change-Id: I0dfc773344654f61f2ad9f5e8ed22a96e6b26ad5
* | | | Merge "Revert "Disable XOM in camera.v4l2."" am: 06ea58579b am: 36de427e99Ivan Lozano2019-01-151-7/+0
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | am: ac77fa8c78 Change-Id: Ia803f91a5487bb0d42ecc73163c34c151795d1c3
| * | | Merge "Revert "Disable XOM in camera.v4l2.""Treehugger Robot2019-01-161-7/+0
| |\ \ \
| | * | | Revert "Disable XOM in camera.v4l2."Ivan Lozano2019-01-151-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 7c7c62db251ba3deade6ba8bb1536b7ef0001d24. Reason for revert: libjpeg-turbo has been updated with XOM compatibility. Change-Id: If1daae895f9cfe08c03169ed2546c81c51a94117
* | | | | Merge "camera: add a lock for the request tracker" am: 8c3812f639 am: 421cd310cfAri Hausman-Cohen2019-01-102-8/+11
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 7bac9b0e96 Change-Id: Ie4877dd1169e99fe8e2dcfa64cc9a77de44492ae
| * | | | Merge "camera: add a lock for the request tracker"Ari Hausman-Cohen2019-01-102-8/+11
| |\ \ \ \
| | * | | | camera: add a lock for the request trackerSergii Piatakov2018-10-172-8/+11
| | | |_|/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The request tracker manages "in flights" buffers and frames. These two containers are modified by calling the tracker's member functions. Such functions are called from at least three threads: (see `v4l2_camera.cpp` for more details): - "main" thread; - separate buffer enqueue thread ("Enqueue buffers"); - separate buffer dequeue thread ("Dequeue buffers"). It is proposed to introduce additional mutex for the request tracker to prevent a race condition. An alternative approach is introducing an additional mutex for the shared containers inside the `RequresTracker` class. But such approach requires more changes and may lead to problems with design in the future. This commit contains a few additional changes which are not directly related to the topic: - as far as more than one local lock is used all of them are renamed; - the `mFlushLock` member is removed because it is completely covered by the new mutex. Test: try to use a camera with more than one buffer and get SIGSEGV crash of the `cameraserver` in the `RequestTracker` class. Change-Id: I2c1901722289cee9f0cb05a11a4dabe70d18f2ba Signed-off-by: Sergii Piatakov <sergii.piatakov@globallogic.com>
* | | | | Merge "camera: remove redundant and unnecessary includes" am: 9dae907ea0 am: ↵Sergii Piatakov2019-01-0963-144/+69
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 547abc4ae4 am: eb83d293ff Change-Id: I38c53a926e1cabd67c194714ca52e4f5f0b417e3
| * | | | Merge "camera: remove redundant and unnecessary includes"Treehugger Robot2019-01-1063-144/+69
| |\ \ \ \
| | * | | | camera: remove redundant and unnecessary includesSergii Piatakov2018-10-1863-144/+69
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove unused or already included headers. Sort and format includes accordind to the "Google C++ Style Guide". Replace C-headers by C++ analog when it's possible. Test: build Change-Id: I19fa6abb5bb68e0257c9c933e68f42f92b394ce7 Signed-off-by: Sergii Piatakov <sergii.piatakov@globallogic.com>
* | | | / Merge "camera: allow to get size of JPEG after conversion" am: 3664a25aa7 ↵Sergii Piatakov2019-01-081-0/+5
|\| | | | | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | am: 18033cb5d5 am: ddcd9c949f Change-Id: I00e6cba64605a7669b7fc127bb70cc2b83dda4bc
| * | | Merge "camera: allow to get size of JPEG after conversion"Treehugger Robot2019-01-091-0/+5
| |\ \ \
| | * | | camera: allow to get size of JPEG after conversionSergii Piatakov2018-10-221-0/+5
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The member function `ImageProcessor::GetConvertedSize` is called by the `ImageProcessor::ConvertFormat` every time when conversion is performed. But for JPEG destination format it leads to error. It is proposed to add the handling of the JPEG format with postponed size calculation. Test: try to take a picture via camera with YUYV output format. Change-Id: I1ec564e2434361c82b5d93348511116494249f06 Signed-off-by: Sergii Piatakov <sergii.piatakov@globallogic.com>
* | | | Merge "Fix cert-dcl16-c clang-tidy warnings" am: 48d500c9e3 am: 71062f3f98Chih-Hung Hsieh2018-12-131-1/+1
|\| | | | | | | | | | | | | | | | | | | | | | | am: d93a0ebf49 Change-Id: Id249d36d6f6c3be45c9d426365ce69e85a05269b
| * | | Merge "Fix cert-dcl16-c clang-tidy warnings"Treehugger Robot2018-12-131-1/+1
| |\ \ \
| | * | | Fix cert-dcl16-c clang-tidy warningsChih-Hung Hsieh2018-12-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 120614316 Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,cert-dcl16-c Change-Id: I744040edc476bf2bb58413a2a14965c27113a5bd
* | | | | Merge "Fix performance-for-range-copy warnings" am: a6a462a015 am: 71e5bab1dbChih-Hung Hsieh2018-12-131-3/+3
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 922e84c2fd Change-Id: Idee25a2fa2009e203c1ecd378236fdfb92edd747
| * | | | Merge "Fix performance-for-range-copy warnings"Treehugger Robot2018-12-131-3/+3
| |\ \ \ \
| | * | | | Fix performance-for-range-copy warningsChih-Hung Hsieh2018-12-121-3/+3
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 30413223 Test: make with WITH_TIDY=1 DEFAULT_GLOBAL_TIDY_CHECKS=-*,performance* Change-Id: I800b659c039c54fd52a927a88904d642604722f7
* | | | | Merge "Disable XOM in camera.v4l2." am: 1908ba0732 am: 6ca60bc346Ivan Lozano2018-12-131-0/+7
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: c334599dfe Change-Id: I730f62bfb2f28cf919614af3a5f7e0b8445079b0
| * | | | Merge "Disable XOM in camera.v4l2."Treehugger Robot2018-12-121-0/+7
| |\ \ \ \ | | |/ / / | |/| | / | | | |/ | | |/|
| | * | Disable XOM in camera.v4l2.Ivan Lozano2018-11-291-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disable execute-only memory layouts in camera.v4l2. This relies on libjpeg_static_ndk, which contains some data mixed into code and thus requires a readable .text. Bug: 77958880 Test: Built the library, checked the program headers with readelf. Change-Id: I5646b8c5e9a0ad151663df29f019b5a84db4b15c
* | | | Merge "Add bstack to sensors OWNERS file" am: 0b2aabd6e6 am: 40283a933dBrian Duddie2018-11-301-0/+1
|\| | | | | | | | | | | | | | | | | | | | | | | am: 11c3ef4db8 Change-Id: I7d5ed4e11c48deabd10db640cd7059dc8f3a491f
| * | | Add bstack to sensors OWNERS fileBrian Duddie2018-11-301-0/+1
| |/ / | | | | | | | | | | | | Test: n/a Change-Id: I25ae89abe1dfe1552e92cccf0354a269959c0c13
* | | Converting sound trigger v2.2 getModelState to be asynchronousmike dooley2018-11-081-17/+19
| | | | | | | | | | | | | | | | | | | | | Test: built android with checkbuild flag Change-Id: I8e927125dcedbc4b98ceb2886806225b2851c082 Bug-Id: 70206501
* | | Merge "camera: initialize smart pointer before usage" am: 8f89886dad am: ↵Ari Hausman-Cohen2018-10-311-1/+2
|\| | | | | | | | | | | | | | | | | | | | | | | b893b0cedb am: c8083cac38 Change-Id: I11bdbcdb80fa772d17c9088f43c01581d00e7851
| * | Merge "camera: initialize smart pointer before usage"Ari Hausman-Cohen2018-10-311-1/+2
| |\ \
| | * | camera: initialize smart pointer before usageSergii Piatakov2018-10-221-1/+2
| | |/ | | | | | | | | | | | | | | | | | | Test: try to scale image and get SIGSEGV crash of the `cameraserver` in the `CachedFrame` class. Change-Id: I17ab13f7c9e7fc84cd38f89dce30e6e5e0535cc7 Signed-off-by: Sergii Piatakov <sergii.piatakov@globallogic.com>
* | | Merge "Revert "Revert "Adding getModelState API to sound trigger"""Michael Dooley2018-10-241-1/+34
|\ \ \
| * | | Revert "Revert "Adding getModelState API to sound trigger""Michael Dooley2018-10-161-1/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 5ac2672ce38540b55fc8ba2aa53e1e2936faf711. Reason for revert: rolling forward with fix Change-Id: Ib7d321f06c94f06a22e962542c9a4c8744702134
* | | | Merge "Allow implicit-fallthrough warnings locally." am: e45fc974db am: ↵Chih-hung Hsieh2018-10-201-0/+2
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | 7da8404d40 am: f68d1864d9 Change-Id: Id3dfe43b6d433e7cc627d294e7397622fa680897
| * | | Allow implicit-fallthrough warnings locally.Chih-Hung Hsieh2018-10-191-0/+2
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | It will be a global error by default. Test: make checkbuild Bug: 112564944 Change-Id: Ifdcaca758036cf9624f79befb65a1cbbd2f9b010 Exempt-From-Owner-Approval: do not block on new warnings
* | | Merge "camera: avoid narrowing of the ioctl request" am: eef57caed9 am: ↵Ari Hausman-Cohen2018-10-112-2/+2
|\| | | | | | | | | | | | | | | | | | | | | | | 3d22b204f2 am: 00d9bf50ec Change-Id: Idfca5de8347a38c3ab2cf0da002e3a2e4657d5e2
| * | camera: avoid narrowing of the ioctl requestSergii Piatakov2018-10-102-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `ioctl` take request of the `unsinged long` type but current implementation narrows it up to `int`. Test: if add something like this into `IoctlLocked`: ``` switch (request) { case VIDIOC_QBUF: break; default:; } ``` the following error is appears: "error: case value evaluates to 3227014671, which cannot be narrowed to type 'int' [-Wc++11-narrowing]". Change-Id: Icb1c99e082feb0c019393205a4b0b717ddc09c05 Signed-off-by: Sergii Piatakov <sergii.piatakov@globallogic.com>
* | | Revert "Revert "Revert "Adding getModelState API to sound trigger"""Michael Dooley2018-09-281-34/+1
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 052d69aedebf6136195726cbfab9486b8ededb05. Reason for revert: broke build Change-Id: Ic76b67ecf918ee9ea04ada608f2ecc3e75ce04fa
* | | Revert "Revert "Adding getModelState API to sound trigger""Michael Dooley2018-09-211-1/+34
| | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 5ac2672ce38540b55fc8ba2aa53e1e2936faf711. Reason for revert: restoring the original cl Change-Id: Ie72548add220fdd3466f5e4ac70cf917bd6920f2