Commit graph

49 commits

Author SHA1 Message Date
Michael Wright
bbd6adda5a Merge "Remove PowerHint::DISABLE_TOUCH" 2017-02-01 16:27:05 +00:00
Chris Phoenix
487c24c14a 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
2017-01-24 20:06:32 -08:00
Stephen Hines
b45857ebc5 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
2017-01-23 17:26:30 -08:00
Michael Wright
f54971abe4 Remove PowerHint::DISABLE_TOUCH
This isn't really a power hint and doesn't given the opportunity for the
input system to clean up any on going input streams or input fusion, so
shouldn't be part of the power HAL.

Test: run vts, play with device
Bug: 30143923
Change-Id: I7b63a2f07a7851739d90099af45773acead58548
2017-01-19 13:00:39 +00:00
Steven Moreland
66816de87f Remove useless Android.mk files.
Test: none
Change-Id: I99f4d3e5b943742009e366b13bee2c32e38a3fba
2017-01-18 11:49:08 -08:00
Zhuoyao Zhang
e7f2556241 Fix HalPowerHidlTargetProfilingTest
* Remove test-configure-path as no configure file is available.

Test: run vts -m HalPowerHidlTargetProfilingTest
Change-Id: I49f10b5c82a4e0cef3bb58d16741ca850a79e677
2017-01-16 20:38:56 +00:00
TreeHugger Robot
2cfe8fcb2b Merge "liblog: use log/log.h when utilizing ALOG macros" 2017-01-12 18:58:25 +00:00
TreeHugger Robot
7a9bd6d44c Merge "Remove coverage from test configurations." 2017-01-12 02:40:16 +00:00
TreeHugger Robot
dc8e458145 Merge "add checks to avoid potential buffer overflows and prevent exceptions" 2017-01-12 01:24:00 +00:00
Ryan Campbell
f276d1923e Remove coverage from test configurations.
Remove converage from test configs or entire config if not used
to define other test params. Tested each one individually to ensure
coverage still works.

Test: run vts -m (each test affected)
Change-Id: Ic1495a38ae9d1b32e851bc646f9c8e778eee2df1
2017-01-11 16:20:44 -08:00
Mark Salyzyn
3ff5260601 liblog: use log/log.h when utilizing ALOG macros
Test: compile
Bug: 30465923
Change-Id: I67046a75be057e7905570748a1ed6bfd03c0e1c8
2017-01-11 15:47:40 -08:00
TreeHugger Robot
a8d358fb63 Merge changes from topic 'bphwfoo'
* changes:
  Bp/nFoo -> Bp/nHwFoo.
  Bp/nFoo -> Bp/nHwFoo
2017-01-11 23:43:16 +00:00
Keun Soo Yim
c591f88fae add checks to avoid potential buffer overflows and prevent exceptions
number_platform_modes variable is from another module (e.g.,
eModule->get_number_of_platform_modes(mModule)) and thus can be
a big number. If a big number is used as the size of new operation,
it can cause an exception as is.

Test: mma
Change-Id: I6cf6027804be980ad39c80a1571b284efabce7e8
2017-01-11 15:31:48 -08:00
Yifan Hong
a18049ad13 Bp/nFoo -> Bp/nHwFoo.
Test: mma
Bug: 33554989
Change-Id: I4b3f852d20fdfc49f2da671dd3c12d618ffb4140
2017-01-11 13:14:59 -08:00
Tri Vo
72620bea30 Merge "Add profiling for sensors, light, and power HALs." 2017-01-11 17:11:22 +00:00
Tri Vo
fbd762204b Add profiling for sensors, light, and power HALs.
Test: make vts
Change-Id: I564f3066dd61eacb4e138133d8df62840e24e99d
2017-01-10 17:46:16 -08:00
Wei Wang
16f56e8c65 Merge "Fix powerhint for NULL parameter" 2017-01-11 01:16:26 +00:00
Wei Wang
e0c1bbbe6d Fix powerhint for NULL parameter
Bug: 32640289
Test: manual on marlin
Signed-off-by: Wei Wang <wvw@google.com>
Change-Id: I00c85082f317cb2d47b799a8fd7e892f7b54dc0e
2017-01-09 12:34:26 -08:00
Steven Moreland
b643842192 Return<*> getStatus().isOk() -> isOk()
Bug: 31348667
Test: compiles
Change-Id: If8d5a23f80dbc04127635b49f1ef77398fc9f8a1
2017-01-04 10:40:05 -08:00
Zhuoyao Zhang
c57e359d1a Update Android.bp and vts/Android.mk for Hidl HALs
* Using update-makefiles.sh to add make rule for profiler in Android.bp
* Remove make rule for profiler in vts/Android.mk.

Bug:31930023
Test: mma
Change-Id: Iea834d8610501c2fce63f65742db31bafb85c0b8
2016-12-22 17:09:12 -08:00
Connor O'Brien
4fb7ea5238 Merge "Fix Power VTS test setup code" 2016-12-20 23:20:32 +00:00
Connor O'Brien
a6e34f36c0 Fix Power VTS test setup code
Bug: 33757678
Test: make vts; vts-tradefed run vts -m HalPowerHidlTargetTest
Change-Id: Ia6d5073d7df858a5799270f364f989d7fc26690b
Signed-off-by: Connor O'Brien <connoro@google.com>
2016-12-20 13:44:53 -08:00
Connor O'Brien
fe08f754be Add code coverage for power VTS tests
Bug: 32022775
Test: Built and ran tests
Change-Id: If8962d3cac793617e009c429c1c1937ba5073808
Signed-off-by: Connor O'Brien <connoro@google.com>
2016-12-20 12:32:12 -08:00
Keun Soo Yim
b62dc3e6ee use the new oneshot vts driver generation build rule
Test: mma
Change-Id: Ib15399d6e7d5049b7a1912623ac2d95de9dfba73
2016-12-16 13:40:33 -08:00
Connor O'Brien
6d9ec22dba Merge "Fix incomplete Power HAL VTS tests" 2016-12-14 19:11:35 +00:00
Yifan Hong
f35ee3b192 Revert "Revert "Update makefiles for Java package path.""
This reverts commit a8789f27cb.

Change-Id: I69980b580a556188ba868bb096ae1590b01759f5
2016-12-13 22:20:16 +00:00
Yifan Hong
a8789f27cb Revert "Update makefiles for Java package path."
This reverts commit ab0792ccdc.

Change-Id: I47fc272d9fa8875e3d7ccb10cb748c738fbf8135
2016-12-13 22:18:34 +00:00
Yifan Hong
ab0792ccdc Update makefiles for Java package path.
use V1_0 instead of 1.0.

Bug: 33554192

Test: hidl_test_java
Change-Id: Ic2d95265df10914a60a5bb743d75e839948838c3
2016-12-13 11:32:56 -08:00
Connor O'Brien
f04688cb67 Fix incomplete Power HAL VTS tests
Add missing Android.mk files and the option to test either binderized
or pass-through implementation.

Bug: 32022775
Test: Built and ran tests.
Change-Id: Ie28a53c92b3512c946416e43e8410ca5bef1274f
Signed-off-by: Connor O'Brien <connoro@google.com>
2016-12-07 19:05:39 -08:00
Yifan Hong
501839495f Update makefiles for hidlizing IBase.
A new dependency android.hidl.base@1.0 is added for
all projects.

Test: mma

Change-Id: Ia70d0eb0d74de06475a339698386d383d491a43a
2016-12-03 01:01:14 -08:00
Yifan Hong
f9d303435d Update to use the correct logging library.
Test: mma compiles
Change-Id: Idb33dea5faaf1a5e54719c2a5a683d2d47e9b8e8
2016-12-01 17:47:02 -08:00
Connor O'Brien
49d87ae92a Merge "Power HAL target-side test" 2016-11-30 18:55:09 +00:00
Steven Moreland
d6e4f03acf Remove HIDL_GENERATED from things written by hand.
find . -name "*.h" -exec sed -i 's/HIDL_GENERATED_\(.*\)_H_/\U\1_H/g' {}
+

They now match what would be generated by -Lc++-impl. This prevents
confusion over whether or not the files are autogenerated.

Change-Id: I4aa89a39907024aa9525df71ff3715979f431357
Similar commit: https://android-review.googlesource.com/#/c/305996/
Test: pass
2016-11-29 15:05:31 -08:00
Connor O'Brien
7106c887a2 Power HAL target-side test
Test: make vts; vts-tradefed run -m HalPowerHidlTargetTest
Bug: 32022775
Change-Id: Id6a14ceb24e6a13e49a236357198686b4348809a
Signed-off-by: Connor O'Brien <connoro@google.com>
2016-11-28 17:36:26 -08:00
Steven Moreland
bbff14d4bb Merge "Update makefiles for logging." 2016-11-22 17:25:36 +00:00
Keun Soo Yim
8a36c0b01c move VTS HIDL HAL tests to the corresponding interface directories
Bug: 32783461
Test: vts-tradefed; run vts-hal-hidl
Change-Id: Iead44eaf65dcd4de2286267467c4a4a322a88205
2016-11-21 19:16:12 -08:00
Steven Moreland
f8d68ec855 Update makefiles for logging.
Bug: 32943424
Test: pass
Change-Id: Ic88ce6adf04e0e36db53e4cc06ac2ec5a90c011c
2016-11-21 16:28:54 -08:00
TreeHugger Robot
17452436fb Merge "Remove -DENABLE_TREBLE from vts test LOCAL_CFLAGS." 2016-11-18 16:24:29 +00:00
Yifan Hong
6b920e43d3 Split libhidl into base and transport.
Bug: 32756130

Test: mma
Change-Id: Id21cf23abe2910f26bb7895511862443b93dc112
2016-11-18 00:00:54 -08:00
Steven Moreland
2b2cef2864 Remove -DENABLE_TREBLE from vts test LOCAL_CFLAGS.
I went ahead and removed some no-ops from the mk's.

Bug: 32978887
Test: mma -j64
Change-Id: I6f8341332aa9f249912f6a8b3a3664b76d362e99
2016-11-17 14:57:37 -08:00
Ryan Campbell
6678b1b366 VTS driver files for Power HAL.
Test: manual
Change-Id: Iab0356aa988b2c274f7444bcdddee56aaeab14ea
2016-11-09 16:12:00 -08:00
Colin Cross
b785f5b82e Support genrules with multiple tools
To allow genrules with more than one tool, rename the tool property to
tools and make it an array, replace $tool with $(location <label>),
and use $() for other variables for consistency.

Bug: 31948427
Test: compare build.ninja
Change-Id: I3d714f70a2af0dc60faeee10e09b6ed166601f1d
2016-11-04 22:49:15 -07:00
Ruchi Kandoi
f3602751e6 power: Add init_rc for android.hardware.power@1.0-service
Test: boots and service starts
Bug: 31177288
Change-Id: I154c2dc41317cbf489452d88030cee0271de56ef
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>
2016-11-03 13:18:59 -07:00
Steven Moreland
fcd74ebcdc Running update makefiles.
https://android-review.googlesource.com/#/c/298160/

Test: pass
Change-Id: I14fcde2d3aacc8f8cd9221f2d69e2feae9ae1226
2016-11-02 10:24:33 -07:00
Steven Moreland
69727778a7 Stop all these merge conflicts.
Test: mma -j64
Change-Id: I319198af32adf1fef180c5c2c22ba4b360fa7609
2016-10-26 12:04:37 +02:00
Steven Moreland
22566b8913 Use defaultPassthroughServiceImplementation.
Bug: 32279541
Bug: 32282345

Test: mma
Change-Id: Ic0222963c99bea41de7637e5f20c19d7c541cb97
2016-10-21 19:33:06 +00:00
Ruchi Kandoi
49ceebe51a power: Add android.hardware.power@1.0-service
Bug: 31177288
Change-Id: I9bca18bd66b2a39fe8b4db7d91c2248655658887
Signed-off-by: Ruchi Kandoi<kandoiruchi@google.com>
2016-10-11 09:39:14 -07:00
Ruchi Kandoi
8214a64afb power: Add android.hardware.power@1.0-impl
Bug: 31177288
Change-Id: I410768ef422ec11c2c9d955e7d3aec9a50cb361e
Signed-off-by: Ruchi Kandoi<kandoiruchi@google.com>
2016-10-11 09:39:14 -07:00
Ruchi Kandoi
3becdbb84d power: Add Power interface 1.0
Bug: 31177288
Change-Id: I9c243166515b565ac73e5f47ff86cfacf59d46d3
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>
2016-10-11 09:39:09 -07:00