Gitiles
Code Review Sign In
gerrit-public.fairphone.software / platform / system / tools / hidl / 1d565629a5e4acf48e5e175140078c3a337b3064
commit1d565629a5e4acf48e5e175140078c3a337b3064[log] [tgz]
authorNirav Atre <natre@google.com>Fri Jul 13 15:41:21 2018 -0700
committerNirav Atre <natre@google.com>Mon Jul 16 12:39:38 2018 -0700
tree041aeb55e5e75629f90684fca490ec65ea442810
parent2e57f01d9721a260b5c745957d25cb9320bb3f3a [diff]
Fix parcelling code for structs containing interfaces

This change implements the proper parcel serialization/deserialization
functionality for structs containing interfaces; previously, structs
containing interfaces+{arrays,strings,vectors} would not be handled
correctly. The compile-time error was of the form:

// Here, the member variable ("array") of a struct ("swi") is referenced
// using the member access operator. However, this results in an invalid
// identifier name.
error: expected ';' at end of declaration
    size_t _hidl_swi.array_parent;
                    ^

Also updated the C++ and Java hidl_test suites.

Bug: 111019943
Test: Ran the hidl_test suite, including the new test-case
Change-Id: Ibcd6517737ed9038832884df11f4274e0dd8970c
7 files changed
tree: 041aeb55e5e75629f90684fca490ec65ea442810
  1. build/
  2. c2hal/
  3. docs/
  4. include_hash/
  5. test/
  6. utils/
  7. .clang-format
  8. Android.bp
  9. Annotation.cpp
  10. Annotation.h
  11. ArrayType.cpp
  12. ArrayType.h
  13. AST.cpp
  14. AST.h
  15. CleanSpec.mk
  16. CompoundType.cpp
  17. CompoundType.h
  18. ConstantExpression.cpp
  19. ConstantExpression.h
  20. Coordinator.cpp
  21. Coordinator.h
  22. DeathRecipientType.cpp
  23. DeathRecipientType.h
  24. DocComment.cpp
  25. DocComment.h
  26. EnumType.cpp
  27. EnumType.h
  28. FmqType.cpp
  29. FmqType.h
  30. generateCpp.cpp
  31. generateCppAdapter.cpp
  32. generateCppImpl.cpp
  33. generateJava.cpp
  34. generateVts.cpp
  35. HandleType.cpp
  36. HandleType.h
  37. Hash.cpp
  38. hidl-gen_l.h
  39. hidl-gen_l.ll
  40. hidl-gen_y.yy
  41. HidlTypeAssertion.cpp
  42. HidlTypeAssertion.h
  43. Interface.cpp
  44. Interface.h
  45. Location.cpp
  46. Location.h
  47. main.cpp
  48. MemoryType.cpp
  49. MemoryType.h
  50. Method.cpp
  51. Method.h
  52. MODULE_LICENSE_APACHE2
  53. NamedType.cpp
  54. NamedType.h
  55. NOTICE
  56. OWNERS
  57. PointerType.cpp
  58. PointerType.h
  59. PREUPLOAD.cfg
  60. README.md
  61. Reference.h
  62. RefType.cpp
  63. RefType.h
  64. ScalarType.cpp
  65. ScalarType.h
  66. Scope.cpp
  67. Scope.h
  68. StringType.cpp
  69. StringType.h
  70. Type.cpp
  71. Type.h
  72. TypeDef.cpp
  73. TypeDef.h
  74. update-all-google-makefiles.sh
  75. update-makefiles-helper.sh
  76. VectorType.cpp
  77. VectorType.h
README.md

hidl-gen

Full documentation can be found here: https://source.android.com/devices/architecture/hidl/

hidl-gen is a compiler for the HIDL (HAL Interface Design Language) which generates C++ and Java endpoints for RPC mechanisms. The main userspace libraries which this compiler uses can be found at system/libhidl.

1. Build

m hidl-gen

2. Run

Note that options for hidl-gen expected to be invoked by the build system are marked with 'internal' in the help menu.

hidl-gen -h

hidl-gen -o output -L c++-impl -r android.hardware:hardware/interfaces -r android.hidl:system/libhidl/transport android.hardware.nfc@1.0

Some defaults for package roots are also provided

hidl-gen -o output -L c++-impl android.hardware.nfc@1.0
hidl-gen -o output -L vts android.hardware.nfc@1.0
hidl-gen -L hash android.hardware.nfc@1.0

Example command for vendor project

hidl-gen -L c++-impl -r vendor.foo:vendor/foo/interfaces vendor.foo.nfc@1.0

See update-makefiles-helper.sh and update-all-google-makefiles.sh for examples of how to generate HIDL makefiles (using the -Landroidbp option).

c2hal

This is a helper tool to convert C headers to valid .hal files.

m c2hal && c2hal -h

docs

This tool generates html documentation for hal interfaces.

m hidl-doc && hidl-doc -h