summaryrefslogtreecommitdiffstats
path: root/automotive
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Add DISABLED_OPTIONAL_FEATURES property"Keun-young Park2019-11-251-0/+20
|\
| * Add DISABLED_OPTIONAL_FEATURES propertyKeun young Park2019-11-201-0/+20
| | | | | | | | | | | | | | | | | | | | - This allows disabling specific optional features in car service from vehicle hal layer. Bug: 144504820 Test: run with car service Change-Id: I419ad86b75151aba129730bdc75c9bc1e47333cf
* | Merge "Add GRPC Server as a Library to VHal Build File"Hao Chen2019-11-223-1/+108
|\ \
| * | Add GRPC Server as a Library to VHal Build FileHao Chen2019-11-213-1/+108
| | | | | | | | | | | | | | | | | | | | | | | | Bug: b/141493212 Test: Tested with ag/9775633, successfully compiled, also tested with ag/9775563 Change-Id: Icf046aafe3d5794a8b79183baa370bdbbcdb796a
* | | Merge changes Iab77a0ae,Ibf525ffeHao Chen2019-11-226-167/+395
|\ \ \ | |_|/ |/| | | | | | | | | | | * changes: Applying the vehicle connector to the VHAL Implement the connector for emulated vehicles
| * | Applying the vehicle connector to the VHALHao Chen2019-11-214-167/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make VHAL to use the connector/server interfaces instead of talking to the (faked) vehicle directly. Use passthrough connector for now since we have not moved to the virtualized scenario yet (but soon :) Bug: b/141493212 Test: On both Osprey and Hawk. Build and flash the image. If on Osprey, see go/enable-google-vhal-on-osprey; if on hawk, `aae app vhal apply google` to enable Google VHAL ``` # no VHAL crash $ adb logcat $ vts-tradefed > run vts-hal-auto -m VtsHalAutomotiveVehicleV2_0Host # 30 passed, 2 failed, the same as the result before this patch # See value changed in Vehicle HAL tab, KitchenSink app: $ python packages/services/Car/tools/emulator/prop_event_simulator.py --property VEHICLEPROPERTY_HVAC_AC_ON --area 0 --value 1 # unit tests $ atest packages/services/Car/tests/vehiclehal_test/src/com/android/car/vehiclehal/test/CarPropertyTest.java ``` Change-Id: Iab77a0ae32db2c55b4c65aa8f3e4f73ec9ef2644
| * | Implement the connector for emulated vehiclesHao Chen2019-11-212-0/+364
| |/ | | | | | | | | | | | | | | | | | | | | | | | | Some functions are copyed from the original VHAL implementation. The ones left in the old implementation will be removed after enabling the Vehicle Connector. Keeping them right now because otherwise it will break the build. Bug: b/141493212 Test: since nothing changed in the "active" code, just compiled by `mm` Change-Id: Ibf525ffe056d6a4c789a7a407eb0b54f61858041
* | Merge "VmsUtils: Add new methods to parse availability state messages" into ↵Prachi Hande2019-11-193-27/+247
|\ \ | | | | | | | | | | | | | | | | | | | | | qt-qpr1-dev am: 0d1d5a0697 am: 6533853937 Change-Id: I1380935021936bb530ddeeaad3b76e00664d0a1f
| * \ Merge "VmsUtils: Add new methods to parse availability state messages" into ↵Prachi Hande2019-11-193-27/+247
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | qt-qpr1-dev am: 0d1d5a0697 Change-Id: Id96237e0638dad9fa3d6edcba768c8a5606e5ee4
| | * | VmsUtils: Add new methods to parse availability state messagesPrachi Hande2019-11-193-27/+247
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These methods are needed in the HAL client to parse sequence number and associated layers when the availablity of the VMS layers change. HAL client relies on these messages by either explicitly requesting the availability or through availability change message sent by the Car service. Bug: 144434783 Fixes: 144434783 Test: Added new tests for the new methods. Ran the tests on Hawk. Change-Id: I09497640367a894e1dfb6143ac3bbdb63d64b53e
| | * | DO NOT MERGE Add DISTANCE_DISPLAY_UNIT to google VHALJordan Jozwiak2019-10-291-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 138816759 Test: aae app vhal apply google & dump properties Change-Id: I6a4d571975d5fe7ba6419cd4c37b734f21162f41 (cherry picked from commit 2619443e0a8b5bb8ca19c6c5761c8f6a834e4936) Merged-In: I6a4d571975d5fe7ba6419cd4c37b734f21162f41
* | | | Make HAL default service run on crosshatch_car (VINTF registration)Zhaoming Yin2019-11-192-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 144734708 Test: manually, once compiled and flashed, the service will be in the VINTF service list. Change-Id: I92d05557ddc7c95f697d2f5a88ab8f481c4d0e5b
* | | | Merge "Support chose permissions for vendor property"Kai Wang2019-11-192-0/+108
|\ \ \ \ | |_|_|/ |/| | |
| * | | Support chose permissions for vendor propertyKai2019-11-132-0/+108
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add property SUPPORT_CUSTOMIZE_VENDOR_PERMISSION. Add enum VehicleVendorPermission. The configArray of new property indicate vendor property Id and the permission selected by vendor. Update google VHAL as an example how to use it. Bug: 109950289 Test: build and flash with google HAL check permission via property test in KitchenSink Change-Id: Ic8dcb661ce69bfb82ebc00781426dfbb98fb393d
* | | | Merge "Interface Definition of Vehicle HAL Connector/Vehicle Server Pair"TreeHugger Robot2019-11-151-0/+141
|\ \ \ \
| * | | | Interface Definition of Vehicle HAL Connector/Vehicle Server PairHao Chen2019-11-141-0/+141
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test: compiled by `mm` Change-Id: I0616e9abfca28656242a82e6ca6addad9b9b812c
* | | | | Merge "Log and return error if failed to listen to a socket"TreeHugger Robot2019-11-131-1/+4
|\ \ \ \ \
| * | | | | Log and return error if failed to listen to a socketHao Chen2019-11-111-1/+4
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Test: build by `mm` Change-Id: I5cf38c4011274ff70a02ac7472a10670b779aa85
* | | | | Add support for SLCANchrisweir2019-11-1316-52/+275
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adding support for SLCAN type interfaces * Made some of the formatting more consistent Bug: 142656299 Test: Manually with canhalctrl, canhaldump, and canhalsend Change-Id: Ifa4c234beb75f8a0ea93bfd75555c5ed8d68dca4
* | | | | Merge "Remove static vndk dep from vendor modules" am: 437baabfde am: 5d5fee2ef2Inseob Kim2019-11-101-2/+2
|\ \ \ \ \ | |/ / / / |/| | / / | | |/ / | |/| | | | | | am: 936efb8177 Change-Id: Ifb7b89378fd22a8c8cf8a89b41845339c3e0c60f
| * | | Merge "Remove static vndk dep from vendor modules" am: 437baabfdeInseob Kim2019-11-101-2/+2
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | am: 5d5fee2ef2 Change-Id: I42aac9039542e01e230aef0d306ea30ae0221fc5
| | * \ \ Merge "Remove static vndk dep from vendor modules"Inseob Kim2019-11-101-2/+2
| | |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 437baabfde Change-Id: I75a47428508efbaa060d9c72a51b3e5f47472ec5
| | | * \ \ Merge "Remove static vndk dep from vendor modules"Treehugger Robot2019-11-111-2/+2
| | | |\ \ \
| | | | * | | Remove static vndk dep from vendor modulesInseob Kim2019-09-101-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead, vendor modules should link against shared vndk. Bug: 137178339 Test: mma Change-Id: Idfce608a3d62de7c4c0e6b02b7cbd80de0a5207f
| | * | | | | DO NOT MERGE - qt-qpr1-dev-plus-aosp-without-vendor@5915889 into ↵Xin Li2019-10-303-57/+115
| | |\ \ \ \ \ | | | |/ / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | stage-aosp-master Bug: 142003500 Change-Id: I8933ff57906fffacf1d48d606451533ec5b79fa2
| * | | | | | DO NOT MERGE Add DISTANCE_DISPLAY_UNIT to google VHALJordan Jozwiak2019-10-311-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 138816759 Test: aae app vhal apply google & dump properties Change-Id: I6a4d571975d5fe7ba6419cd4c37b734f21162f41 (cherry picked from commit 2619443e0a8b5bb8ca19c6c5761c8f6a834e4936)
| * | | | | | Merge "[hardware][interfaces][automotive] fix -Wreorder-init-list" am: ↵Nick Desaulniers2019-10-214-676/+717
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0d5782e99e am: 62a2c3a00e am: b1402d78ed Change-Id: Ica43b1fb52ef8286f49c5344eb48b1ee0d5dc95f
| | * | | | | [hardware][interfaces][automotive] fix -Wreorder-init-listNick Desaulniers2019-10-184-676/+717
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | C++20 will require members in a designated initializer to be in order unlike C99. Bug: 139945549 Test: mm Merged-In: I17d1d262d71eefa0aa5b89da7acc659bc2f97bbd Change-Id: I4e18c0f68606c3e8d965aab194f2a5fd5000c59a Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
* | | | | | | Merge "Improve error handling with separate ICanErrorListener"Tomasz Wasilczyk2019-11-0715-59/+209
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | |
| * | | | | | Improve error handling with separate ICanErrorListenerTomasz Wasilczyk2019-11-0615-59/+209
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Error handling highlights: - moved onError from ICanMessageListener to ICanErrorListener - added isFatal callback argument to request client disconnect - don't down interface that's already down Also: - don't crash if it's not possible to unregister ICanBus - don't crash while trying to down interface that failed - make hidl-utils available to vendor libraries Bug: 143779011 Test: implemented a VHAL service prototype that communicates with this HAL Change-Id: I98d054da9da0ead5ef952aebc086e052ac996212
* | | | | | | Merge "[AAE Power Mgt] Allow Suspend to RAM without triggering Garage Mode"Jim Kaye2019-10-211-1/+5
|\ \ \ \ \ \ \
| * | | | | | | [AAE Power Mgt] Allow Suspend to RAM without triggering Garage ModeJim Kaye2019-10-161-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Vehicle HAL can tell us to shut down immediately, but it cannot tell us to suspend immediately. Add that capability. This capability will be needed when the VHAL initiates a Silent Boot and then asks for suspension. In this case, we do not want to postpone the suspension by entering Garage Mode. Bug: 134521909 Test: Added CarPowerManagementTest.testSleepImmediateEntry() Test: Added CarPowerManagementServiceTest.testSleepImmediately() Change-Id: I8190d16cb6faa15672ec3d61f126f6d283d1189e
* | | | | | | | [hardware][interfaces][automotive] fix -Wreorder-init-listNick Desaulniers2019-10-174-47/+63
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | C++20 will require members in a designated initializer to be in order unlike C99. Bug: 139945549 Test: mm Change-Id: I17d1d262d71eefa0aa5b89da7acc659bc2f97bbd Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
* | | | | | | Merge "[hardware][interfaces][vehicle] fix -Wdangling-gsl in test" am: ↵Nick Desaulniers2019-10-151-3/+3
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 880ee3950a am: e126b55e4c am: d80d1d4fcb am: 3bdf845739 Change-Id: I64c6b866aba67f03711923efbaf52cb9b23cdc57
| * | | | | | Merge "[hardware][interfaces][vehicle] fix -Wdangling-gsl in test" am: ↵Nick Desaulniers2019-10-151-3/+3
| |\ \ \ \ \ \ | | | |/ / / / | | |/| / / / | | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | 880ee3950a am: e126b55e4c am: d80d1d4fcb Change-Id: I55e923c863608c795db1ac357548fe194b9fd582
| | * | | | [hardware][interfaces][vehicle] fix -Wdangling-gsl in testNick Desaulniers2019-10-141-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Chaining the method calls in such a way otherwise returns a dangling pointer to a temporary object. Bug: 139945549 Test: mm Change-Id: I0783fccbb6f11e7e37bd059445265227359649cf Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
| * | | | | Merge "Change AP_POWER_STATE_REPORT's access property." am: 86331df20a am: ↵Jim Kaye2019-10-022-2/+2
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a53b19fb08 am: 7a473c66f7 Change-Id: I48dad550404620168c74cc2b3ef2f321020ee014
| | * | | | Change AP_POWER_STATE_REPORT's access property.Min Yun2019-09-272-2/+2
| | | |/ / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change AP_POWER_STATE_REPORT's access property from WRITE to READ_WRITE. In Vehicle environment some hals communicate with external ecu, so they must know the boot reason. (Not only on, but DEEP_SLEEPY_EXIT, ON, CANCELD) To handle this, AOSP provide CarPowerManager, but it use AIDL. But, in treble, vendor process must communicate with system process via hidl. So, vendor processes(almost hal), can't use CarPowerManager. (Beside, CarPowerManager use /dev/binder but vendor process use /dev/vndbinder). If hals subscribe AP_POWER_STATE_REPORT via vehicle hal, they can handle the power state. bug: 140370318 Merged-In: Ic4c7f1d66a4fdee1cfb8b9da42cf76bc7aed5948 Change-Id: Ic4c7f1d66a4fdee1cfb8b9da42cf76bc7aed5948
* | | | | Explictly cast size_t into a long data typeChangyeon Jo2019-10-121-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change explictly casts size_t type variables to long data type in ALOG*, to avoid any build error due to inconsistent size_t type declarations between different build targets. Bug: 142545867 Change-Id: I587c7006a6ab69970b3a507a6160b3decd4bfd5a Signed-off-by: Changyeon Jo <changyeon@google.com>
* | | | | Revert "Revert "Revert "Revert "Extend EVS interfaces and data types""""Changyeon Jo2019-10-1220-409/+2225
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 468cc1d476ae4a78787e0730cac8cf53e1806ed1. Change-Id: If7e80111213ba02bf2c49f17f9880cd9cd14429e
* | | | | Revert "Revert "Revert "Extend EVS interfaces and data types"""Changyeon Jo2019-10-1220-2225/+409
| | | | | | | | | | | | | | | | | | | | This reverts commit 47b45af32cdea1788a1fd3e316f97d89aeddac2b.
* | | | | Revert "Revert "Extend EVS interfaces and data types""Changyeon Jo2019-10-1120-409/+2225
| | | | | | | | | | | | | | | | | | | | This reverts commit 1b73d28c04d250b581860dde082cf92c366371dc.
* | | | | Revert "Extend EVS interfaces and data types"Changyeon Jo2019-10-1120-2225/+409
| | | | | | | | | | | | | | | | | | | | This reverts commit e6976fce80a9a0951c6bc6f668b2de032ccc4d34.
* | | | | Extend EVS interfaces and data typesChangyeon Jo2019-10-1120-409/+2225
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change extends the definition of CameraDesc with the additional camera metadata field that various camera module informaiton will be stored. IEvsEnumerator is extended with three new methods: - getCameraList_1_1() returns a list of new camera descriptors - openCamera_1_1() tries to open a camera device with a given stream configuration. IEvsCameraStream has below changes: - deliverFrame_1_1() method is added to handle a frame in new BufferDesc. - Rename notifyEvent() as notify(). IEvsCamera also has below new methods: - getCameraInfo_1_1() returns a new camera descriptor. - getParameterList() returns a list of camera control parameters supported by the camera device. - getIntParameterRange() returns a valid range of parameter values. - Parameter setter and getter methods are renamed. VTS test cases are updated to use new methods and below test cases are added to validate their behaviors: - CameraUseStreamConfigToDisplay verifies end-to-end video stream with a stream configuration it finds from CameraDesc's metadata. - MultiCameraStreamUseConfig verifies two clients can start and stop video streams on the same underlying camera with same stream configuration. Bug: 128851019 Test: VtsHalEvsV1_1Target Change-Id: Ia6b0b94aff869129cb400d0a4c4df91e72682784 Signed-off-by: Changyeon Jo <changyeon@google.com>
* | | | | Edit a comment to reflect an actual enum type nameEnrico Granata2019-10-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is nothing named enum_vehicle_ap_power_state_shutdown_param_type in the Vehicle HAL source tree. The right greppable type name is VehicleApPowerStateShutdownParam. Test: git diff Change-Id: Ib6c8be491df9aa6cb3143b28a12b70bd4b8c29cc
* | | | | Remove libhwbinder/libhidltransport depsSteven Moreland2019-10-023-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since these were combined into libhidlbase. Bug: 135686713 Test: build only (libhwbinder/libhidltransport are empty) Change-Id: I878a4d04e65ef13f3a0ebaf64177f673958d213c
* | | | | Merge "Update OWNERS"TreeHugger Robot2019-10-011-1/+2
|\ \ \ \ \
| * | | | | Update OWNERSKeun young Park2019-09-301-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug: 141876695 Test: none Change-Id: I653304479634e7cc71b22705ff3384d04a3577dc
* | | | | | Change AP_POWER_STATE_REPORT's access property.Jim Kaye2019-09-272-2/+2
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change AP_POWER_STATE_REPORT's access property from WRITE to READ_WRITE. In Vehicle environment some hals communicate with external ecu, so they must know the boot reason. (Not only on, but DEEP_SLEEP_EXIT, ON, CANCELED) To handle this, AOSP provide CarPowerManager, but it use AIDL. But, in treble, vendor process must communicate with system process via hidl. So, vendor processes(almost hal), can't use CarPowerManager. (Beside, CarPowerManager use /dev/binder but vendor process use /dev/vndbinder). If hals subscribe AP_POWER_STATE_REPORT via vehicle hal, they can handle the power state. Bug: 140370318 Test: Verified on Hawk Change-Id: Ic4c7f1d66a4fdee1cfb8b9da42cf76bc7aed5948
* | | | | Merge "Add three vendor properties for e2e test"Kai Wang2019-09-242-4/+94
|\ \ \ \ \ | |_|_|/ / |/| | | |