Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-02 10:05:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge changes from topic "fcm_exclude_aidl_version" into main am: 76d87305d5 am: 74e0c01e15 am: fa24ef60cb

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2688568

Change-Id: I67ec14dbbfd45216a2a46021cc68a9003f73be02
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Yifan Hong
2023-08-09 22:18:19 +00:00
committed by Automerger Merge Worker
parent 273b775841 fa24ef60cb
commit d9a5e4104f
2 changed files with 30 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
compatibility_matrices/exclude/fcm_exclude.cpp
View File

@@ -104,7 +104,7 @@ bool ShouldCheckMissingHidlHalsInFcm(const std::string& packageAndVersion) {
} }
// The predicate to VintfObject::checkMissingHalsInMatrices. // The predicate to VintfObject::checkMissingHalsInMatrices.
bool ShouldCheckMissingAidlHalsInFcm(const std::string& package) { bool ShouldCheckMissingAidlHalsInFcm(const std::string& packageAndVersion) {
static std::vector<std::string> included_prefixes{ static std::vector<std::string> included_prefixes{
// Other AOSP HALs (e.g. android.frameworks.*) are not added because only framework // Other AOSP HALs (e.g. android.frameworks.*) are not added because only framework
// matrix is checked. // matrix is checked.
@@ -112,36 +112,43 @@ bool ShouldCheckMissingAidlHalsInFcm(const std::string& package) {
}; };
static std::vector<std::string> excluded_prefixes{ static std::vector<std::string> excluded_prefixes{
// Packages without top level interfaces (including types-only packages) are exempted.
"android.hardware.audio.common@",
"android.hardware.biometrics.common@",
"android.hardware.camera.metadata@",
"android.hardware.camera.device@",
"android.hardware.camera.common@",
"android.hardware.common@",
"android.hardware.common.fmq@",
"android.hardware.gnss.measurement_corrections@",
"android.hardware.gnss.visibility_control@",
"android.hardware.graphics.common@",
"android.hardware.input.common@",
"android.hardware.keymaster@",
"android.hardware.media.bufferpool2@",
"android.hardware.radio@",
"android.hardware.uwb.fira_android@",
// Test packages are exempted. // Test packages are exempted.
"android.hardware.tests.", "android.hardware.tests.",
// Fastboot HAL is only used by recovery. Recovery is owned by OEM. Framework
// does not depend on this HAL, hence it is not declared in any manifests or matrices.
"android.hardware.fastboot@",
}; };
static std::vector<std::string> excluded_exact{ static std::vector<std::string> excluded_exact{
// Packages without top level interfaces (including types-only packages) are exempted. // Packages without top level interfaces (including types-only packages) are exempted.
// AIDL // AIDL
"android.hardware.audio.common", "android.hardware.audio.core.sounddose@1",
"android.hardware.audio.core.sounddose",
"android.hardware.biometrics.common",
"android.hardware.camera.metadata",
"android.hardware.camera.device",
"android.hardware.camera.common",
"android.hardware.common",
"android.hardware.common.fmq",
"android.hardware.graphics.common",
"android.hardware.input.common",
"android.hardware.keymaster",
"android.hardware.media.bufferpool2",
"android.hardware.radio",
"android.hardware.uwb.fira_android",
// Fastboot HAL is only used by recovery. Recovery is owned by OEM. Framework // Deprecated HALs.
// does not depend on this HAL, hence it is not declared in any manifests or matrices. "android.hardware.bluetooth.audio@1",
"android.hardware.fastboot",
}; };
auto package_has_prefix = [&](const std::string& prefix) { auto package_has_prefix = [&](const std::string& prefix) {
return android::base::StartsWith(package, prefix); return android::base::StartsWith(packageAndVersion, prefix);
}; };
// Only check packageAndVersions that are in the include list and not in the exclude list. // Only check packageAndVersions that are in the include list and not in the exclude list.
@@ -149,7 +156,8 @@ bool ShouldCheckMissingAidlHalsInFcm(const std::string& package) {
return false; return false;
} }
if (std::find(excluded_exact.begin(), excluded_exact.end(), package) != excluded_exact.end()) { if (std::find(excluded_exact.begin(), excluded_exact.end(), packageAndVersion) !=
excluded_exact.end()) {
return false; return false;
} }

4
compatibility_matrices/exclude/include/vintf/fcm_exclude.h
View File

@@ -24,8 +24,8 @@ namespace android::vintf::details {
// Determine whether VINTF checks |package| is missing from FCMs. // Determine whether VINTF checks |package| is missing from FCMs.
// |package| can be a HIDL package and version like // |package| can be a HIDL package and version like
// "android.hardware.foo@1.0", or an AIDL package name like // "android.hardware.foo@1.0", or an AIDL package name like
// "android.hardware.foo". // "android.hardware.foo@1".
bool ShouldCheckMissingHidlHalsInFcm(const std::string& packageAndVersion); bool ShouldCheckMissingHidlHalsInFcm(const std::string& packageAndVersion);
bool ShouldCheckMissingAidlHalsInFcm(const std::string& package); bool ShouldCheckMissingAidlHalsInFcm(const std::string& packageAndVersion);
} // namespace android::vintf::details } // namespace android::vintf::details