Update Display-P3 transfer function

Decided to use same transfer function that Apple is
using for their Display-P3 devices. Difference between
sRGB transfer function and Display-P3 is very small.
We will treat them an synonymous.

Test: make

Change-Id: Id4e52058f2b810b70c46821c5fe6830e623c5491

configstore: Add display configuration items

This adds to configstore@1.0 HAL two new configuration
items that indicate if the display support WideColor
or HDR.
Configuration is controlled by TARGET_HAS_WIDE_COLOR_DISPLAY
and TARGET_HAS_HDR_DISPLAY in the board config file.

Test: make tests in libs/gui/tests/
Test: adb sync
Test: adb shell /data/nativetest/libgui_test/libgui_test

Change-Id: I442febc602851577c470c038a7fbf056c8ed25a7

Merge "Add VTS tests for audio effects"

Merge "wifi(implementation): Make a boolean flag atomic" am: 5bdf7a0ef7 am: bdbc4162e3
am: 6a37a04653

Change-Id: Iee147c099291a004930a41691b71ad0aedd42241

Merge "wifi(implementation): Make a boolean flag atomic" am: 5bdf7a0ef7
am: bdbc4162e3

Change-Id: Ifafb945391cf9cec56e9ed5ed884dd74db6f6f84

Merge "wifi(implementation): Make a boolean flag atomic"
am: 5bdf7a0ef7

Change-Id: Ibe597faf04c91cb437cb95faf96cae8713802ea8

Merge "wifi(implementation): Make a boolean flag atomic"

Add VTS tests for audio effects

Added tests covering IEffect, IEqualizerEffect, and
ILoudnessEnhancer interfaces.

Minor corrections in the interface definitions and implementations:

 - fixed descriptions and @callflow annotations in IEffect;

 - fixed type used for band levels in IEqualizerEffect;

 - fixed specification of frequencies in IEqualizerEffect;

 - fixed some bugs in previously non-execrices Effects code;

 - warning messages changed to error messages.

Test: this is a test
Bug: 32022706
Change-Id: I0e0bc111b07d944ad8a0321e8b1ec703f8d1a73e

Merge "Create configStore SF useContextPriority"

Merge "Move camera hal imp to vendor partition"

Merge "Move FMQ benchmark/test services to hardware/interfaces" am: 178250d254 am: c35f1e7bc1
am: add0d3c8b8

Change-Id: I002bb4119971877061921bb260238c6dc2568fca

Merge "Move FMQ benchmark/test services to hardware/interfaces" am: 178250d254
am: c35f1e7bc1

Change-Id: I7bb5aa7967d439ad3021b4bf4f79f5b4a4f1acf9

Merge "Move FMQ benchmark/test services to hardware/interfaces"
am: 178250d254

Change-Id: I29e411c3c53fd608ea21a0fc136eb0eb1ec8e1a4

Merge "Move FMQ benchmark/test services to hardware/interfaces"

Merge "wifi: Add support for multiple radio stats" am: c0fbd8fe63 am: eaf6ed41dd
am: 0f93903c7c

Change-Id: If535abd69fa2e81f96943a526975d5462bdaa41a

Merge changes Id207ae31,Ib18c6d2e,I4af19080,I03686e55,Iea1d7d22, ...

* changes:
  Audio HAL VTS: Add tests for IStreamIn
  Audio HAL VTS: Add at least negative tests for IStream
  Audio HAL VTS: Test IStream::debugDump
  Audio HAL VTS: Split big test for better traceability
  Audio hal VTS: pretty print AudioDevice and ChannelMask
  Audio Hal VTS: Refactor ASSERT_RESULT helper
  Audio hal VTS: generate names for parametrized tests
  Audio hal VTS: factorize master volume test
  Fix potentiel deadlock in audio VTS

Merge "wifi: Add support for multiple radio stats" am: c0fbd8fe63
am: eaf6ed41dd

Change-Id: I17c637a274c26a28cf4331a7722c8a7c5bf26022

Merge "wifi: Add support for multiple radio stats"
am: c0fbd8fe63

Change-Id: Icca33f2745c8297d1f413e651fbf7506fc0521f6

Merge "wifi: Add support for multiple radio stats"

Create configStore SF useContextPriority

Change-Id: I154604428488a5824f6aadd4a6b43a72b327e280

Audio HAL VTS: Add tests for IStreamIn

For now most tests are negative.

Test: run test on board

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

Audio HAL VTS: Add at least negative tests for IStream

Add negative test for the remaining methods of IStream
and positive one for most.

Test: run test on board

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

Audio HAL VTS: Test IStream::debugDump

Test: run it on target
Bug: 34170075
Change-Id: I4af190806f79e0c03b59e1cd60b6f58a08b05322
Signed-off-by: Kevin Rocard <krocard@google.com>

Audio HAL VTS: Split big test for better traceability

As some part of this big test were failing, it was difficult to actually
track on the dashboard what was wrong.

Split it in small independent equivalent tests.

Test: mm & run on target

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

Audio hal VTS: pretty print AudioDevice and ChannelMask

This eases the debuging of failing tests.
Also print hex value of the enums as well of their literal values.

Test: run the test on target
Bug: 34170075
Change-Id: Iea1d7d2269e0e5b413240aaa905daae755846e74
Signed-off-by: Kevin Rocard <krocard@google.com>

Audio Hal VTS: Refactor ASSERT_RESULT helper

ASSERT_INVALID_ARGUMENTS was a macro that asserted that a given Result
or Return contained INVALID_ARGUMENT. The problem was that a result can
have lots of other values like INVALID_STATE or NOT_SUPPORTED.
Additionally not all test expect only one possible result.

Introduce two overload of ASSERT_RESULT()
The first one takes an expected Result value and compare it to the
obtained one.
The second take a list and expect the obtained one to be in this list.

Test: run the test on target
Bug: 34170075
Change-Id: I798729f27f723c98292610bfb43dbdb2724ec2ca
Signed-off-by: Kevin Rocard <krocard@google.com>

wifi(implementation): Make a boolean flag atomic

The flag |awaiting_event_loop_termination_| is accessed by multiple
threads, so make it atomic.

Bug: 33552618
Test: Compiles
Change-Id: I5ebe27fe88672565aae0daf34a89616d36f35f90

Merge changes from topic 'configstore' am: e3a0c607cd am: fb64badb3e
am: e0ca61a6be  -s ours

Change-Id: I12c920d8b2e1a1734c822d3badc4195dee82bcfc

configstore: add new configuration item am: cb71410f16 am: 35ad8cf50d
am: 4b7759c251  -s ours

Change-Id: Ieb5e5416a3eecd1fde02cc3caaabeb95c1d8a7c8

configstore: add utility library functions for configstore am: ad54c30863 am: 71e1808de8
am: 20a8fde005  -s ours

Change-Id: I072c59bd3c543ea6fb3cb9cc26d6c24e6ed47914

configstore: add configstore@1.0 HAL am: c5d7c8a27a am: 06be01518b
am: c5a9bd19f6  -s ours

Change-Id: I41c0b04abe4ffbf84ea7cd5581104543f3896190

Merge changes from topic 'configstore' am: e3a0c607cd
am: fb64badb3e

Change-Id: Ia4da72dc31ec235dd3aa0710846fc1504a0fd884

configstore: add new configuration item am: cb71410f16
am: 35ad8cf50d

Change-Id: I66849f126efe18b86cd6010027d59909e67799c5

configstore: add utility library functions for configstore am: ad54c30863
am: 71e1808de8

Change-Id: I9acd3b567d8daa137f60eeb456e31be05e20a396

configstore: add configstore@1.0 HAL am: c5d7c8a27a
am: 06be01518b

Change-Id: Ifa830fa1d3128c9319e52152e20451b0886f6db8

Merge changes from topic 'configstore'
am: e3a0c607cd

Change-Id: I3b50189813bde7b84e9e545e37717a4e8bda5801

configstore: add new configuration item
am: cb71410f16

Change-Id: I9dcf29ba10593d3bca6d0db4269c072687d7ab51

configstore: add utility library functions for configstore
am: ad54c30863

Change-Id: I4d2fed4d6c46b4ab22c955f020f590f1f66f5927

configstore: add configstore@1.0 HAL
am: c5d7c8a27a

Change-Id: Id8da8ce348021dd6d8976071370c5d75118c2fa2

Merge changes from topic 'configstore'

* changes:
  configstore: build configstore-utils as header library
  configstore: add new configuration item
  configstore: add utility library functions for configstore
  configstore: add configstore@1.0 HAL

wifi: Add support for multiple radio stats

Previously, we've always assumed that we have only one instance of
radio stats fetched in link layer stats. This is no longer true for the
newer devices which support multiple radios.

Also,
Changed the timestamp member from uint32_t to uint64_t.

Bug: 36148086
Test: The error logs no longer seen on newer devices.
Change-Id: I048a1db7cc0bfb0dc0dacafff2156f42a8ae1e63

Merge "Bluetooth: No power off before power on" am: 2e429d6b64 am: 3805d65284
am: 0dba862f10

Change-Id: I2277fc9cae7055f3b915d9556e2a17ed657fecb3

Merge "Bluetooth: No power off before power on" am: 2e429d6b64
am: 3805d65284

Change-Id: I03d5b9fc13750baa4a771b645d032ce2331b1523

Merge "Bluetooth: No power off before power on"
am: 2e429d6b64

Change-Id: If65b972e390b50add2412c1d91c259210405aa5c

Merge "Bluetooth: No power off before power on"

Move FMQ benchmark/test services to hardware/interfaces

Bug: 32284445
Test: Built and ran FMQ benchmarks

Change-Id: Ia6bacf21d276b55e4e590dc96f348464f2098992

wifi: Link the new libcld80211 library am: 48df739a4d am: 02354cb6b6
am: 50ad0c5171

Change-Id: I6d7ccaa1c960b3e650ee0cbea8cddd4bbb145ce5

wifi: Link the new libcld80211 library am: 48df739a4d
am: 02354cb6b6

Change-Id: Ie1a5036050c5edc42597a1b70467b9da08c15183

wifi: Link the new libcld80211 library
am: 48df739a4d

Change-Id: Ia0676ab9802360aaa67dcb0f67f478fa820e06d4

wifi: Link the new libcld80211 library

Bug: 32775496
Change-Id: I700dec21bfc3a36b8a3b09251b3004449fd49de7

Bluetooth: No power off before power on

Test: Switch users (toggles Bluetooth under load)
Bug: 36026072
Change-Id: I7eef35e19fbb4cf825c2583ec0af573733b24f9d

Audio hal VTS: generate names for parametrized tests

The gtest dashboard only shows test names, not their parameters.
This mean that parametrized tests are names 0,1,2...
Thus when one test fails, the dashboard info is not very useful.

This patch appends the audio config to the test name.

Test: run the test on target
Bug: 34170075
Change-Id: I77090a3a17e4a3e97ecb72a8a89ad2d88626d96f
Signed-off-by: Kevin Rocard <krocard@google.com>

Audio hal VTS: factorize master volume test

Master volume was the first pair of getter/setter tested
and did not use the generic test*Accessor function.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Fix potentiel deadlock in audio VTS

The IDevice::debugDump method dumps debug info in a file descriptor.
Such file descriptor was previously the writing end of a pipe.

As the test is not multithreaded, if the dump was bigger than the pipe
buffer, a deadlock would occur.

With this patch, the test uses a file instead of a pipe. Thus write
will never block infinitely.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

configstore: build configstore-utils as header library

This change makes android.hardware.configstore-utils library built as
a header-only library instead of a static library.

Design doc: go/design-confighal
Bug: 35397068
Test: build, run

Change-Id: I59c7c8802aeb1fb687af24cfdb2bbb38723e6221
Merged-In: Idee48816a1a271bdb9ba8cf907d98c3823925da4

Merge "[WIFI][HIDL] Add device-based feature flag support" am: 48b4ce2e99 am: d15376f05c
am: 733015b56e

Change-Id: Ia12e5aef8c2260a2a3b35da1318ee1a73f6a6728

Merge "[WIFI][HIDL] Add device-based feature flag support" am: 48b4ce2e99
am: d15376f05c

Change-Id: If13d2a7e8993d93f23082fcd4ccf2732aaacba6f

Merge "[WIFI][HIDL] Add device-based feature flag support"
am: 48b4ce2e99

Change-Id: I74d285740511b9e8604a9bac5bcc8ab54688c0d8

Merge "[WIFI][HIDL] Add device-based feature flag support"

Merge "Revert "Revert "Add return status to IGraphicBufferSource methods"""

Merge "EvsCamera: %zd -> %zu for size_t print"

wifi(implementation): Add null checks for RTT result am: 78cfef9f0c am: 841ba1b367
am: 53c158606b

Change-Id: Ia36f323770af2f72bf416f686ac6e521adcf5ff4

wifi(implementation): Add null checks for RTT result am: 78cfef9f0c
am: 841ba1b367

Change-Id: Id2ce5421a1d4ae08fbedb6983477c80542e90001

wifi(implementation): Add null checks for RTT result
am: 78cfef9f0c

Change-Id: I9a58ca156d3e2f61fda8346d39b8de355302690f

wifi(implementation): Add null checks for RTT result

According to the legacy HAL header file, the LCI & LCR fields may not
always be populated. So, add null checks for them.

Bug: 36141536
Test: Compiles
Change-Id: Id2a5f5fd0eaf21b3a4d5cfcd26124c9fb6c90869

EvsCamera: %zd -> %zu for size_t print

%zd doesn't throw a warning, but %zu is technically correct.

Change-Id: I3762d01cc424ecf84e73c21afe5a5a3d295fe782

Revert "Revert "Add return status to IGraphicBufferSource methods""

This reverts commit b729637648a319616654160785cb088d10ddf95b.

Bug: 36110748
Change-Id: Ia12c00d57b9e8d4b66265018c3970416c265091a

Merge "memtrack: eliminate dependency on surfaceflinger in VTS test"

Merge "Revert "Add return status to IGraphicBufferSource methods""

Merge "audio: use vts target test base class"

Merge "Do not crash fatally when recovering from system server restart."

audio: use vts target test base class

Test: make vts succeed
Change-Id: I8008392e016e0604f8aa9a292f1ec6c6e7c3b384

Revert "Add return status to IGraphicBufferSource methods"

Bug: 36110748

This reverts commit ed32ff5150ce5bf9897c4dd00cf3b8cad385fcab.

Change-Id: Ibb5e72fc11f705a571491103fc5c68005cf54a30

Change vts target base test class name

Test: make vts succeed, vts kernel tests and nfc hal tests passed
Change-Id: I95f10ad7a66f261c9030357edd411fe6d94150e8

Merge "Change vts target base test class name" am: 4f4fdaf664 am: a8239094f4
am: 649a2c13d7

Change-Id: I376e83c09515ba50a7e033e3cb75e0c5f8a24ccf

Merge "Change vts target base test class name" am: 4f4fdaf664
am: a8239094f4

Change-Id: If6245264c41f92dc6f89a0bfd6001db07e211de4

Merge "Change vts target base test class name"
am: 4f4fdaf664

Change-Id: Id82295a163c4abbd0b9fc1ac8fde12ebd9a43030

Merge "Change vts target base test class name"

Do not crash fatally when recovering from system server restart.

Log(FATAL) is creating lots of nuisance bugs.

Test: Sensor service continues to recover from a system server
      restart and sensors are available.
      No crash log is created.
Bug: 36094522
Bug: 36115683

Change-Id: Idc2d086709dc2132c98aab806aa44728fbdaa189

Merge "graphics: fix a potential use after free"

Merge "graphics: really reset HWC2 on SF crashes"

Merge "Add SurfaceFlinger phase offset to configStore"

Bluetooth: Power off after closing the UART am: 66a4ca3c92 am: a62b176c5c
am: 0eb1c4a753

Change-Id: I09afc5f2c4b086749a093fe4114ece747e498c2d

Bluetooth: Power off after closing the UART am: 66a4ca3c92
am: a62b176c5c

Change-Id: I18c58eb31b35b8f589134541babff80194232bbe

Bluetooth: Power off after closing the UART
am: 66a4ca3c92

Change-Id: Iccea3216281ddb5e6ccd608d9743c49dca21293a

graphics: fix a potential use after free

We cannot lookup _and_ update buffer cache entry in lookupBuffer.
The old buffer is still in use by hwcomposer2.  Add updateBuffer to
do the update after the new buffer has replaced the old buffer in
hwcomposer2.

While at it, s/BufferClone/BufferCacheEntry/g.

Bug: 36064845
Test: manual
Change-Id: I59b61c0198ad528c40020fdebbe27a6cc359226f

Merge changes Ic451761c,I0652f1bf,I928cae71

* changes:
  Add VTS for the audio primary device
  Fix segfault in Primary audio hal hidl wrapper getDevice
  Do not skip failing Audio vts tests

[WIFI][HIDL] Add device-based feature flag support

Add mechanism to define feature support flags to
configure set of Wi-Fi features supported on a particular
device.

Bug: 32997844
Test: integration tests
Change-Id: Iffbf3ba237013f1030b24c0173641234a9c27e5c

Bluetooth: Power off after closing the UART

Bug: 36084266
Test: angler & marlin switch users (Toggle Bluetooth under load)
Change-Id: I1e0c34e334d11b9dcf9df62cbde3d5556c537bd7

Merge "Add generic flags to ProgramInfo struct."

Change vts target base test class name

Test: make vts succeed, vts kernel tests and nfc hal tests passed
Change-Id: I38ae6cc940d0f17bb3352f360efdf9c70697c76b

configstore: add new configuration item

This change adds to configstore@1.0 HAL a new configuration item
for surfaceflinger; useTripleFramebuffer()

In the default implementation, useTripleFramebuffer() will return true
only if the build flag NUM_FRAMEBUFFER_SURFACE_BUFFERS is defined as 3.

Bug: 34723491
Test: build & run. log message "SurfaceFlinger FrameBuffer: ?"
      should be seen.

Change-Id: I7ab9f17f43d68ee7b37b5db8c24b542db4d1c216
Merged-In: Iac97e25cd69717c73a135f6b2a44538124544987

configstore: add utility library functions for configstore

this change adds a set of library functions to facilitate
accessing(retrieving) configuration items.

Specifically, added template functions for accessing configuration
items for following types:

OptionalBool, OptionalInt32, OptionalUInt32, OptionalInt64,
OptionalUInt64, OptionalString

Bug: 34724435
Test: build, run

Change-Id: I28d565dcfc04bb71bf891d1b48ce2ec6192eb21f
Merged-in: Icca56d0d9e086b9d94c9b1168df041bf7d418698

configstore: add configstore@1.0 HAL

This change adds configstore@1.0 HAL which provides static configuration values
to the framework. As a first attempt, a configuration item in the
surfaceflinger (VSYNC_EVENT_PHASE_OFFSET_NS) is choosen and converted to
a HAL function named vsyncEventPhaseoffsetNs().

Bug: 34314793
Test: build & run. make sure that configstore is up and running.
$ ps -A | grep configstore

Change-Id: I0692190180cadc8e3f64ee840c737e05b519a135
Merged-in: Ibc70b14176d2e0144a5745d71ceba49ebc3ca706

Add VTS for the audio primary device

Its base class was tested but not the primary
device getters and setters.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Fix segfault in Primary audio hal hidl wrapper getDevice

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Do not skip failing Audio vts tests

Previously failing tests were commented because I thought
our implementation had to pass all tests.

As a mater of facts, it is not the case and tests are allowed to fail.
Bugs will be raised for those failure.
This is much better way to track the fixes that FIXMEs.

Test: run the corresponding vts test
Test: vts-tradefed r vts-hal-hidl -m VtsHalAudioV2_0Target

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

Merge changes If83c1cbe,I12ad2665 am: cdca05557d am: 3b815eee2a
am: b005b3ae2d

Change-Id: Ib9f77fbe6c43c4706cce89a6bcd99d9b840915e1

Merge changes If83c1cbe,I12ad2665 am: cdca05557d
am: 3b815eee2a

Change-Id: I3fe19d41a610b80fcb28e0db4f1264770b1659f3

Merge changes If83c1cbe,I12ad2665
am: cdca05557d

Change-Id: Ifc3dbe11f88ceb83d77f0334894f739bbc64e1c9