tequilaOS/platform_hardware_libhardware
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use LLNDK guard instead of builtin_available

Browse source 
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
This commit is contained in:
Jooyung Han 2024-02-01 12:44:37 +09:00
parent 16e1d03400
commit 4695d8a846
1 changed files with 12 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
hardware.c
View file

@ -179,9 +179,9 @@ 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, *)) {
#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",
@ -190,9 +190,9 @@ static int hw_module_exists(char *path, size_t path_len, const char *name,
if (access(path, R_OK) == 0)
return 0;
}
} else {
ALOGE("hw_module_exists: libapexsupport is not supported in %d.", __ANDROID_API__);
}
#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);