summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJooyung Han <jooyung@google.com>2024-02-01 12:44:37 +0900
committerJooyung Han <jooyung@google.com>2024-02-01 12:44:37 +0900
commit4695d8a846325ca0e6910c8beb2fc172f35779d3 (patch)
tree83951ad01a20bcbef7ce5da82dcdc7e06f353ceb
parent16e1d034000cba2d993de762ff9fb0c00fcab375 (diff)
downloadlibhardware-4695d8a846325ca0e6910c8beb2fc172f35779d3.tar.gz
Use LLNDK guard instead of builtin_available
APEXSUPPORT_API was a workaround when we had no support for __ANDROID_VENDOR_API__. Since we have it now, use it instead of builtin_available(android APEXSUPPORT_API). Bug: 302088370 Test: consumerir HAL APEX loading consumerir.default from the same APEX Test: ConsumerIrTest VtsHalIrTargetTest Change-Id: I6c9cf7e4555f7f242cc61defe7a67e3d11d79f6e
-rw-r--r--hardware.c24
1 files changed, 12 insertions, 12 deletions
diff --git a/hardware.c b/hardware.c
index 79073312..94b5d5d3 100644
--- a/hardware.c
+++ b/hardware.c
@@ -179,20 +179,20 @@ static int hw_module_exists(char *path, size_t path_len, const char *name,
const char *subname)
{
#ifdef __ANDROID_APEX__
- // When used in APEX, it should look only into the same APEX because
+ // When used in VAPEX, it should look only into the same APEX because
// libhardware modules don't provide ABI stability.
- if (__builtin_available(android AAPEXSUPPORT_API, *)) {
- AApexInfo *apex_info;
- if (AApexInfo_create(&apex_info) == AAPEXINFO_OK) {
- snprintf(path, path_len, "/apex/%s/%s/%s.%s.so",
- AApexInfo_getName(apex_info), HAL_LIBRARY_SUBDIR, name, subname);
- AApexInfo_destroy(apex_info);
- if (access(path, R_OK) == 0)
- return 0;
- }
- } else {
- ALOGE("hw_module_exists: libapexsupport is not supported in %d.", __ANDROID_API__);
+#if __ANDROID_VENDOR_API__ >= 202404
+ AApexInfo *apex_info;
+ if (AApexInfo_create(&apex_info) == AAPEXINFO_OK) {
+ snprintf(path, path_len, "/apex/%s/%s/%s.%s.so",
+ AApexInfo_getName(apex_info), HAL_LIBRARY_SUBDIR, name, subname);
+ AApexInfo_destroy(apex_info);
+ if (access(path, R_OK) == 0)
+ return 0;
}
+#else // __ANDROID_VENDOR_API__
+ ALOGE("hw_module_exists: libapexsupport is not supported in %d.", __ANDROID_VENDOR_API__);
+#endif // __ANDROID_VENDOR_API__
#else // __ANDROID_APEX__
snprintf(path, path_len, "%s/%s.%s.so",
HAL_LIBRARY_PATH3, name, subname);