Merge "Removed TODOs from radio hal." am: c5447fce20 am: ca1388166a am: b2cdacaa6e
am: 1ef858b769

Change-Id: I1d7da242ff8297b01241e6de1a0c549924f56f08

Merge "Removed TODOs from radio hal." am: c5447fce20 am: ca1388166a
am: b2cdacaa6e

Change-Id: Ie26aded569367bb5046011d4b4393dddc501bd08

Merge "Removed TODOs from radio hal." am: c5447fce20
am: ca1388166a

Change-Id: Ifd295cd847a1540075ff493d32cf39ea9a5bb1b4

Merge "Removed TODOs from radio hal."
am: c5447fce20

Change-Id: Ib97124740651fbbe7c8dfb205129bbcc7f6ef04f

Merge "Removed TODOs from radio hal."

Merge "vibrator HAL uses "default" service name"

vibrator HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: service registers correctly as default

Bug: 33844934
Change-Id: Ic0e9ff98b16bfb66e67222054653b7347442bacb

Merge "remove unused include subdir line from Android.mk file at leaf directory"

Merge "Use HIDL's memory for shared memory and add wrapper interfaces for IGraphicBufferProducer."

Merge "Add logic to actually read OBD2 sensor properties to VehicleHidlTest"

remove unused include subdir line from Android.mk file at leaf directory

Bug: 34695422
Test: make vts
Change-Id: I1f7bcacdd8790f3bb13bc6aa7d80e485db4a4dc4

Add logic to actually read OBD2 sensor properties to VehicleHidlTest

Test: make vts BUILD_GOOGLE_VTS=true -j32 && vts-tradefed run commandAndExit vts --skip-all-system-status-check --primary-abi-only --skip-preconditions --module VehicleHidlTest -l INFO

Change-Id: I0817823ea00354b27c90bdd412012320868b9e8f

Merge "Introduce ComposerBase interface to allow custom Hwc implementations"

Merge "Rename the VMDS property to VMS which is how it is called now. Test: Fixed HAL tests with the new property name Bug:34682582"

Introduce ComposerBase interface to allow custom Hwc implementations

Decouples the ComposerClient code which deals with parsing the command buffer
messages sent by SurfaceFlinger from the Hwc code that handles the
commands.

Bug: 33297270
Test: Compiled and verified on device that hwcomposer-2-1 service can
start correctly. SurfaceFlinger can connect to the service and the
system boots up. Also verified SurfaceFlinger runs correctly without
hwcomposer-2-1.
Change-Id: I43357306caea57d262735f4756c379ba9d0138cd

Merge "Fix HalGatekeeperHidlTargetBasicProfilingTest."

Merge "clean up default fingerprint HIDL@2.1 implementation"

Fix HalGatekeeperHidlTargetBasicProfilingTest.

* Similar fix for HalMemtrackHidlTargetProfilingTest

Test: make vts.
Change-Id: I999d18c53e321921f4bc1a4babe34520cc496321

Merge "Increase the timeout for NfcHidlTargetProfilingTest." am: 6d88fc9a6a am: 62cc546eec am: 21d5b70212
am: b6492bc464

Change-Id: I10e9cee844367c63f74754ee451b316738f56906

Merge "Increase the timeout for NfcHidlTargetProfilingTest." am: 6d88fc9a6a am: 62cc546eec
am: 21d5b70212

Change-Id: I96fa7e8a6816a13409b049248db18731ae9caf27

Merge "Increase the timeout for NfcHidlTargetProfilingTest." am: 6d88fc9a6a
am: 62cc546eec

Change-Id: I6cae2e1ddedcd2424739a309a6c27b08d1277a93

Merge "Increase the timeout for NfcHidlTargetProfilingTest."
am: 6d88fc9a6a

Change-Id: Ib6c2cd2bce720180ff2365b936e07bd217a265dd

Merge "Increase the timeout for NfcHidlTargetProfilingTest."

Merge "fix errata in comment"

Merge "EVS HAL with multi-buffer support via BufferDesc"

EVS HAL with multi-buffer support via BufferDesc

HAL changes required in order to enable required functionality.
Specifically, support for use of supplied buffers as OpenGL textures.
Addition of getDisplayState is needed in order to meet original API
spec (allow secondary clients to monitor display state).  Expected to be
needed specifically by Car Service.  Note that this HAL is not used on
the phone and only relevant to the Android Auto at this point.

Test:  compile & run evs_test.  Still issues with buffer locking.

Added getDisplayState() to IEvsEnumerator to allow secondary clients to
query for display state without having to acquire the display.

Introduced the use of weak pointers in the service to detect when
clients holding an object disappear unexpectedly.

Plus squashed commit of the following:

commit 26c685b430028384157f573b31264ff03ea4c37b
Author: Scott Randolph <randolphs@google.com>
Date:   Thu Jan 19 17:29:54 2017 -0800

    Convert EVS HAL to use BufferDesc (vs raw handles)

    Use a structure that includes the buffer properties in addition to the
    memory handle itself to make it easier/safer for the HAL client code to
    reconstruct ANativeWindowBuffer objects (as required when using the
    buffer as a GL texture, for example)

    Change-Id: Ifebf0c03b9ad167d61152b85f8229d5970cc41af

commit 0286385694282e017c12f238f5aaa86d8d1bd600
Author: Scott Randolph <randolphs@google.com>
Date:   Tue Jan 17 12:01:09 2017 -0800

    Incremental cleanup of EVS driver

    Change-Id: Ibba2b7d9a1c55629d76ee6faf2a9f36c715e7bd7

commit 71e7b85f3919a60c5a194028a586ce81846c6c0d
Author: Scott Randolph <randolphs@google.com>
Date:   Thu Jan 12 10:40:45 2017 -0800

    Fix error check bug that cause stream start to fail

    Also fixes a minor warning related to log message string formatting.

    Change-Id: I74edf9a02db87c3632b4883ca40207fe861fd75d

commit 164e7ef048eba9bc184319ea928d231a26b695bc
Author: Scott Randolph <randolphs@google.com>
Date:   Wed Jan 11 17:27:46 2017 -0800

    Fix HAL update related compile bugs

    Minor changes to accomodate recent HIDL/HAL changes.

    Change-Id: I466d8979e8e56424442840aff8d17762eac0a590

commit 67391db6a04b2cbbda1a0b5c8b717fe79994f5b9
Author: Scott Randolph <randolphs@google.com>
Date:   Wed Jan 11 13:56:23 2017 -0800

    Avoid mutex reentrancy when setting frame count

    At initialization time, we need to be able to set the number of frames
    available for capture.  This change makes the code to do this a private
    function called from within a mutex in order to avoid having one mutex
    protected call trying to make another mutex protected call as would
    happen if it called directly into the public API.

    Change-Id: I05c383976dd8db1acf5874bd47b2a3c3521d7032

commit 993f6e7e1e43f2bc4a48f6aae32d9ba2c3125d23
Author: Scott Randolph <randolphs@google.com>
Date:   Mon Dec 19 18:39:38 2016 -0800

    Avoid forced exit if frame delivery fails

    If we don't read the error code from the frame delivery call, the
    underlying code forces an application exit if an error was returned.
    Even when we don't care about the result, we have to read it to avoid
    this.

    Change-Id: Ie3695faa10d1c5e340f66c369bfc2595ac516434

commit 08298db0f2d5d0ae7cd0781fde834fc7e63375d0
Author: Scott Randolph <randolphs@google.com>
Date:   Mon Dec 19 18:21:16 2016 -0800

    Support adjustable number of frames in flight

    Add enhanced support for SetMaxFramesInFlight which is required to
    support concurrent camera clients at the EVS Manager level.

    Change-Id: I23b606a017529514370a03cd2eca3ee286af8fec

commit 198050b2cff70d1a232745ce8e7f35e87581562e
Author: Scott Randolph <randolphs@google.com>
Date:   Fri Dec 16 16:27:19 2016 -0800

    Add frame contents validation to EvsDisplay

    Update the default "mock" implementation of EvsDisplay hardware layer to
    provide a frame contents validation based on knowledge of what the
    "mock" EvsCamera implementation will emit.

    Change-Id: If0624f855a440f52a47af7751ed3b09e8b21ff74

commit c830b7ca466155049d6eee047f6c323521451e8b
Author: Scott Randolph <randolphs@google.com>
Date:   Fri Dec 16 15:25:48 2016 -0800

    Add multi-buffer support to EvsCamera

    Extend the EvsCamera implmentation to support more than one buffer in
    flight at a time.  This is necessary in order to support concurrent
    client access through the EvsManager proxy layer.

    Change-Id: I0c32336be40c7bedd797140569650906458d27c1

commit 1bdbb9d641f95d494ec2a398b6d1f5202276d17b
Author: Scott Randolph <randolphs@google.com>
Date:   Fri Dec 16 15:24:28 2016 -0800

    Add additional validation on round trip pointers

    Now that HIDL support returning the same pointer to an interface object
    after a round trip through the transport, we can add a bit more
    validation.

    Change-Id: Ia9a3a6ec7552f5e5523c0f9ad66ceb02b46b2db4

Change-Id: I47468932d6a7b2b5eab47819917cdb20154d8f18

Merge "use the device's default binder/passthrough mode in vts tests"

Use HIDL's memory for shared memory and add wrapper interfaces for
IGraphicBufferProducer.

Test: Compiles
Bug: 31399200
Change-Id: I17a3b1c8f20e9c11474761d120edd2e75353cc6d

Merge "Don't deref a NULL shared buffer base"

fix errata in comment

bug: 32371653
Test: build
Change-Id: Ia7889199a53c461df5f1367c07117aab2a775fd6

Merge "Fix syntax error in Vehicle VTS"

Merge "Update fingerprint makefile."

Increase the timeout for NfcHidlTargetProfilingTest.

* Given there's still test failue due to timeout:
  https://android-vts-internal.googleplex.com/show_table?testName=NfcHidlTargetProfilingTest,
  increased the limit to 25m.

Test: run vts -m NfcHidlTargetProfilingTest
Change-Id: Ie5fee6a399281ec538e6724207916908bb8da4d0

clean up default fingerprint HIDL@2.1 implementation

- remove Binder dependency by passing keystore token to onAuthenticated()
- move notify() to cpp file

Test: recovers from killing keystore, fingerprint unlocks device,
FingerprintDialog works with crypto objects.

Fixes bug 34264028

Change-Id: Ic0de31603f4bc4147d6faf014af89e787b1ef244

use the device's default binder/passthrough mode in vts tests

Test: mma
Change-Id: I3f2775873f5b6c091f70413adbdf7b7ae298ff44

use the profiler test's name and a precondition to decide when to run.

Bug: 34715432
Test: make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module
HalSoundTriggerHidlTargetBasicProfilingTest -l INFO
Change-Id: I116ed144076f07f752c756d9ddb6e425442d39cc

Merge "supplicant(interface): Add the new radio work methods" am: 4aceb63bc7 am: c637017b66 am: 538cbc9a29
am: af58817b1d

Change-Id: I8437ef232d576fa13132d4fa8e8f0d7a19a156e1

Merge "supplicant(interface): Add the new radio work methods" am: 4aceb63bc7 am: c637017b66
am: 538cbc9a29

Change-Id: Ifbf8f30310120ba00af0ea2383d9db9cccabe4db

Merge "supplicant(interface): Add the new radio work methods" am: 4aceb63bc7
am: c637017b66

Change-Id: I9f0d9b906930d646516462ba4f4b4c42d6619a7f

Merge "supplicant(interface): Add the new radio work methods"
am: 4aceb63bc7

Change-Id: If222b06393c5ea32b39cb8737a9cfd00a6e1cc3c

Merge "supplicant(interface): Add the new radio work methods"

memtrack HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds; verify HAL still works

Bug: 33844934
Change-Id: Ide36b5470a393b5ed0f60fd37661dd50d0db817b

Merge "NFC: Update VTS test" am: 3bce41d5ec am: 4d3c1ea989 am: d4cffbf1ec
am: 7b51c28e24

Change-Id: I4123a9bfa33c0a65ddee0d20ab3a316d5171c4cf

Merge "NFC: Update VTS test" am: 3bce41d5ec am: 4d3c1ea989
am: d4cffbf1ec

Change-Id: Ia1ee033655a51a8a7734c0626e33ae3ec42bc51a

Fix syntax error in Vehicle VTS

Test: make vts BUILD_GOOGLE_VTS=true -j32 && vts-tradefed run
commandAndExit vts --skip-all-system-status-check --primary-abi-only
--skip-preconditions --module VehicleHidlTest -l INFO

Bug: b/32021794

Change-Id: I855aa4792405e54824e85c3982912866fcb10968

Merge "NFC: Update VTS test" am: 3bce41d5ec
am: 4d3c1ea989

Change-Id: I01ab41ebd15c0237b41e3777ccc6fffd5868c3b7

Merge "NFC: Update VTS test"
am: 3bce41d5ec

Change-Id: Id275ca5ed4dbcc0f0644e632cd31231f47998309

Merge "Vehicle HAL: add VTS case for DRIVING_STATUS"

Merge "NFC: Update VTS test"

Don't deref a NULL shared buffer base

bug:34624618
Change-Id: I5caee20a944d0c7b5d30f8abdbd8734507914a3a

Merge "Add LOG_TAGs to drm hal default modules"

Merge "hidl_memory no longer owns passed-in handles."

Vehicle HAL: add VTS case for DRIVING_STATUS

Test: ENABLE_TREBLE=true make vts BUILD_GOOGLE_VTS=true -j32 && vts-tradefed run commandAndExit vts --skip-all-system-status-check --primary-abi-only --skip-preconditions --module VehicleHidlTest -l INFO
Change-Id: I66a4c7105412da9efd05eb0ef1caed275676488c

Merge "power HAL uses "default" service name"

Merge "contexthub: Add VTS target tests"

Merge "Add device id attestation"

use the default binder/passthrough mode in vts tests am: 3079b3c68e am: 9af3e139e2 am: 5594ca35af
am: 00f03ce59e

Change-Id: I4ebf2873c8ecf9bc0d9f19f44b314ba053ca984f

use the default binder/passthrough mode in vts tests am: 3079b3c68e am: 9af3e139e2
am: 5594ca35af

Change-Id: Ie004e69235ab61dd86b1b2af4117c51f280039da

use the default binder/passthrough mode in vts tests am: 3079b3c68e
am: 9af3e139e2

Change-Id: Icc0e947ed456e3aaa049726fe07d16620f226e9b

use the default binder/passthrough mode in vts tests
am: 3079b3c68e

Change-Id: Ib5a739feafa151329737b830d3fae977c2b60238

power HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds; verify HAL still works

Bug: 33844934
Change-Id: I7a5d9e848efabf624299ee81ee42aa046b61c660

Update fingerprint makefile.

Update fingerprint makefile with VTS support in HIDL.

Test: pass
Bug: 34518442
Change-Id: I4556b027f4217ef9748e07555705911c03e1cbbd

contexthub: Add VTS target tests

Introduce target-side VTS tests covering the Context Hub HAL.

Bug: 32021659
Test: make vts -j42 && \
      vts-tradefed run commandAndExit vts --primary-abi-only \
        --skip-all-system-status-check \
--module ContexthubHidlTargetTest
Change-Id: I387ad0cccbfa824d292e2364f175dcec21134be1

use the default binder/passthrough mode in vts tests

Test: mma
Change-Id: I9146d599c623aae1196fcf7e37b7b2715796c748

Merge "regenerate types.vts file for sensors and update its test."

Merge "use a precondition var in soundtrigger."

Merge changes from topic 'usb_hal'

* changes:
  USB: HIDL: default implentation for the type-c interface
  USB: HIDL: Type-c usb interface V1_0

Merge "update the used precondition var name for IR" am: 17f3c1c445 am: 3bfb0f1cfc am: cf21f92a2a
am: 846d643af6

Change-Id: I5caf0cc1c57be74d0556a20bd9e2db4992946a88

Merge "update the used precondition var name for IR" am: 17f3c1c445 am: 3bfb0f1cfc
am: cf21f92a2a

Change-Id: I89a5aa24666b05c3f57b17a28743c1893a0cfa42

Merge "update the used precondition var name for IR" am: 17f3c1c445
am: 3bfb0f1cfc

Change-Id: Idd042bf443bafb878462daa80afeccffa8b99ff8

Merge "update the used precondition var name for IR"
am: 17f3c1c445

Change-Id: I0075cdc824f527ed124cd06f96429743fe968e54

Merge "update the used precondition var name for IR"

supplicant(interface): Add the new radio work methods

These radio work methods can be used by external programs (wificond in
our case) to request wpa_supplicant to stop doing any offchannel
operations.

Also,
1. Add the setUpdateIdentifier method in ISupplicantStaNetwork.
2. Add the setProactiveKeyCaching method in ISupplicantStaNetwork.
3. Add sendNetworkEapSimGsmAuthFailure &
sendNetworkEapSimUmtsAuthResponse in ISupplicantStaNetwork.
4. Add the setWpsDeviceType() method in ISupplicantIface.

Bug: 34454675
Bug: 32581374
Test: /hardware/interfaces/update_makefiles.sh
Change-Id: I83fe577f6bd39d2daef2cb203ad9a8a93657c988

Removed TODOs from radio hal.

+ Opened new bug b/34683858 for addressing one of the TODO in P.

Bug: 34109705
Test: Removed only comments.
Change-Id: Ie78eacbc80def2df3ec448f14a89fbac185bf2af

use a precondition var in soundtrigger.

Test: make vts BUILD_GOOGLE_VTS=true -j32 && vts-tradefed run
commandAndExit vts --skip-all-system-status-check --primary-abi-only
--skip-preconditions --module HalSoundTriggerHidlTargetBasicTest -l INFO
Fixes: 34256132

Change-Id: Iab82492503d57a589fd3bed3db609a67064ec98b

update the used precondition var name for IR

Test: mma
Change-Id: Ibec34c82e41d1f93568c29e390a1fc031fa3d3bc

Add device id attestation

This adds device id attestation to the Keymaster 3.0 HAL. Device
id attestation must only be offered if the device can permanently
destroy device ids on request. The default implementation cannot
do this because it lacks storage that would survive device wipes.
Hence, the implementation refuses all device id attestation requests.

Bug: 34597337
Test: CTS CtsKeystoreTestCases and GTS DeviceIdAttestationHostTest

Change-Id: I6ff6146fad4656b8e1367650de922124b3d7f7b2

Rename the VMDS property to VMS which is how it is called now.
Test: Fixed HAL tests with the new property name
Bug:34682582

Change-Id: Ibed6a7d82de070087409da04e5a42e30f5cf3252

Merge "Fix failure to load memtrack HAL implementation"

Merge "Use the feature-based precondition checking for VR Test."

USB: HIDL: default implentation for the type-c interface

The default implementation should take care of legacy devices that
lauched before O.

BUG: 31015010
Test: Manually tested on Angler and sailfish for type-c role switches.
Change-Id: I8762598f4f82da5b2fe8a87aacc30728eb7921f5

Fix failure to load memtrack HAL implementation

Since some implementations of the libhardware memtrack HAL do not
implement an open() method, the HIDL implementation should not rely
on that method being available.

Test: Memtrack VTS tests pass
Bug: 31180823
Bug: 34103653
Change-Id: Ia0dda5e027894009bdcf12cd5c2e9eb635aca87e
Signed-off-by: Connor O'Brien <connoro@google.com>

NFC: Update VTS test

- Order of data and CORE_CONN_CREDITS_NTF is not defined, check for the
order in which they were received before checking for contents
- Configuration status is added to WriteCoreReset test
- Add WriteCoreResetConfigReset test

Test: VTS test passes
Change-Id: Ifb730bbb51c366807028fe7e6757c0f8eae943c3
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

regenerate types.vts file for sensors and update its test.

Test: make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module SensorsHidlTest -l INFO
Bug: 34378195

Change-Id: Ib4e83728d6725304c52b0c4f61def413eb7962f0

Merge "Add annotations to create gnss-base.h"

Merge "Fixed typos and incorrect types"

Fix some ordered comparisons with zero for pointers.

Bug: http://b/31532493

The latest clang will trigger an error diagnostic "ordered comparison
between pointer and zero ..." when you do something like "p > 0". This
change replaces the ordered comparisons with proper checks instead.

Test: Validated with latest toolchain.
Change-Id: I4a127fe2551688c8176bdd31210cf7f2f40baeba

Use the feature-based precondition checking for VR Test.

This is to run the test on both passthrough and binderized
devices.

Test: make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module
HalNfcHidlTargetBasicTest -l INFO
Bug: 32022964

Change-Id: Ib1288b7288431da6ec2d8a93aa0bc6f2c86fc4da

Merge "light HAL uses "default" service name"

Add LOG_TAGs to drm hal default modules

Change-Id: I1267205999b7cc660a6c908f0d183e43277a9c71

light HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds

Bug: 33844934
Change-Id: I042320d62f88ba9585ee05f33fe7592b5ffa1884

Fixed typos and incorrect types

Fixed typos, incorrect types for bitmap, and comment
alignment.

Test: Build
Change-Id: I51a99fa71877cfe168ee2cf31de702f36f57d013

Merge "Remove "false" parameter from getService"

Merge "Add host-side code for wifi HAL test" am: 688a9e1485 am: a48d0c425b am: 2c8160aeb0
am: 15df064e84

Change-Id: Ica434f314d5eb2c95fb81b11f66ed38151906f8d

Merge "Add host-side code for wifi HAL test" am: 688a9e1485 am: a48d0c425b
am: 2c8160aeb0

Change-Id: Ic0933c0820d0a623a6219ab3551a91ba12e9df3b

Merge "Add host-side code for wifi HAL test" am: 688a9e1485
am: a48d0c425b

Change-Id: I8468f30bb3a1166d0e6ff5868a265f9ce006a57a

Merge "Add host-side code for wifi HAL test"
am: 688a9e1485

Change-Id: Icf6e0ac782006fff547babff02aa727219ebd44d

Merge "Add host-side code for wifi HAL test"

Merge "Rename CommandWriter to CommandWriterBase"

Merge "bootctrl HAL uses "default" service name" am: 9877cf296f am: c68beb8712 am: f6835e18ef
am: f8fd472a50  -s ours

Change-Id: I320558eb3b6dabebb3c5b05a2d6633e9355f952c