summaryrefslogtreecommitdiff
path: root/compatibility_matrices/exclude/fcm_exclude.cpp
diff options
context:
space:
mode:
authorYifan Hong <elsk@google.com>2020-09-16 19:16:02 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2020-09-16 19:16:02 +0000
commit8f7f56ff3c50cc804c644fa8aae8be437cbc5c7f (patch)
treee5c4679d8105daff0e46a36de67dde4cb17d0ba3 /compatibility_matrices/exclude/fcm_exclude.cpp
parent59236470dcffbda2e97bb76cd71338d4d9a09898 (diff)
parent5ea7bf3c6bc87ad131eb1f44ccc94dc1453b2757 (diff)
Merge "Reorder/cleanup FCM exempt list #1" am: ef85febe6a am: 7380d36ae3 am: 7a07e92303 am: 5ea7bf3c6b
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1427724 Change-Id: I8cc4b9aee221af981caf2a6f0b0e2be95b04cf23
Diffstat (limited to 'compatibility_matrices/exclude/fcm_exclude.cpp')
-rw-r--r--compatibility_matrices/exclude/fcm_exclude.cpp21
1 files changed, 11 insertions, 10 deletions
diff --git a/compatibility_matrices/exclude/fcm_exclude.cpp b/compatibility_matrices/exclude/fcm_exclude.cpp
index 7a6ea2b476..50653a15a2 100644
--- a/compatibility_matrices/exclude/fcm_exclude.cpp
+++ b/compatibility_matrices/exclude/fcm_exclude.cpp
@@ -33,12 +33,12 @@ bool ShouldCheckMissingHalsInFcm(const std::string& package) {
};
static std::vector<std::string> excluded_prefixes{
- // TODO(b/110261831): reduce items in this list
- "android.hardware.gnss.measurement_corrections@",
+ // Packages without top level interfaces (including types-only packages) are exempted.
+ "android.hardware.camera.device@",
+ "android.hardware.gnss.measurement_corrections@1.",
"android.hardware.graphics.bufferqueue@",
- // Exempted.
- "android.hardware.camera.device@",
+ // Test packages are exempted.
"android.hardware.tests.",
};
@@ -46,20 +46,21 @@ bool ShouldCheckMissingHalsInFcm(const std::string& package) {
// TODO(b/110261831): reduce items in this list
"android.hardware.audio@7.0",
"android.hardware.audio.effect@7.0",
- "android.hardware.biometrics.fingerprint@2.3",
- "android.hardware.cas.native@1.0",
"android.hardware.fastboot@1.0",
- "android.hardware.gnss.visibility_control@1.0",
"android.hardware.media.bufferpool@1.0",
"android.hardware.media.bufferpool@2.0",
- "android.hardware.radio.config@1.2",
"android.hardware.tv.cec@2.0",
"android.hardware.tv.tuner@1.0",
- "android.hardware.keymaster",
- // Exempted
+ // Packages without top level interfaces (including types-only packages) are exempted.
+ // HIDL
+ "android.hardware.cas.native@1.0",
+ "android.hardware.gnss.visibility_control@1.0",
+ "android.hardware.radio.config@1.2",
+ // AIDL
"android.hardware.common",
"android.hardware.graphics.common",
+ "android.hardware.keymaster",
};
auto package_has_prefix = [&](const std::string& prefix) {