From 0b5c62fd7944c674f4c55cf2f86b4d5580748441 Mon Sep 17 00:00:00 2001 From: Yifan Hong Date: Mon, 19 Oct 2020 11:07:58 -0700 Subject: [PATCH] Remove legacy/O/O-MR1 matrices. Devices with legacy/O/O-MR1 VINTF won't be able to update to Android S. Devices launching with pre-O, O, or O-MR1 will need to update its VINTF to P and above to work with Android S. Test: builds Bug: 163419744 Bug: 171260360 Bug: 171260613 Bug: 171260670 Bug: 171260715 Change-Id: I2a65f5a24c0731df9c4247bf9dd2616c2279042f --- compatibility_matrices/Android.bp | 40 -- compatibility_matrices/Android.mk | 3 - compatibility_matrices/CleanSpec.mk | 3 + .../compatibility_matrix.1.xml | 326 --------------- .../compatibility_matrix.2.xml | 374 ------------------ .../compatibility_matrix.legacy.xml | 326 --------------- .../exclude/fcm_exclude.cpp | 12 + 7 files changed, 15 insertions(+), 1069 deletions(-) delete mode 100644 compatibility_matrices/compatibility_matrix.1.xml delete mode 100644 compatibility_matrices/compatibility_matrix.2.xml delete mode 100644 compatibility_matrices/compatibility_matrix.legacy.xml diff --git a/compatibility_matrices/Android.bp b/compatibility_matrices/Android.bp index d22aa68cb0..b2a815f7ca 100644 --- a/compatibility_matrices/Android.bp +++ b/compatibility_matrices/Android.bp @@ -12,46 +12,6 @@ // See the License for the specific language governing permissions and // limitations under the License. -vintf_compatibility_matrix { - name: "framework_compatibility_matrix.legacy.xml", - stem: "compatibility_matrix.legacy.xml", - srcs: [ - "compatibility_matrix.legacy.xml", - ], - kernel_configs: [ - // legacy uses O kernel requirements - "kernel_config_o_3.18", - "kernel_config_o_4.4", - "kernel_config_o_4.9", - ], -} - -vintf_compatibility_matrix { - name: "framework_compatibility_matrix.1.xml", - stem: "compatibility_matrix.1.xml", - srcs: [ - "compatibility_matrix.1.xml", - ], - kernel_configs: [ - "kernel_config_o_3.18", - "kernel_config_o_4.4", - "kernel_config_o_4.9", - ], -} - -vintf_compatibility_matrix { - name: "framework_compatibility_matrix.2.xml", - stem: "compatibility_matrix.2.xml", - srcs: [ - "compatibility_matrix.2.xml", - ], - kernel_configs: [ - "kernel_config_o_mr1_3.18", - "kernel_config_o_mr1_4.4", - "kernel_config_o_mr1_4.9", - ], -} - vintf_compatibility_matrix { name: "framework_compatibility_matrix.3.xml", stem: "compatibility_matrix.3.xml", diff --git a/compatibility_matrices/Android.mk b/compatibility_matrices/Android.mk index b88d40b53c..3951666fbf 100644 --- a/compatibility_matrices/Android.mk +++ b/compatibility_matrices/Android.mk @@ -92,9 +92,6 @@ include $(BUILD_FRAMEWORK_COMPATIBILITY_MATRIX) endif # DEVICE_PRODUCT_COMPATIBILITY_MATRIX_FILE my_system_matrix_deps := \ - framework_compatibility_matrix.legacy.xml \ - framework_compatibility_matrix.1.xml \ - framework_compatibility_matrix.2.xml \ framework_compatibility_matrix.3.xml \ framework_compatibility_matrix.4.xml \ framework_compatibility_matrix.5.xml \ diff --git a/compatibility_matrices/CleanSpec.mk b/compatibility_matrices/CleanSpec.mk index 9b150ede48..5b351a5340 100644 --- a/compatibility_matrices/CleanSpec.mk +++ b/compatibility_matrices/CleanSpec.mk @@ -45,3 +45,6 @@ #$(call add-clean-step, rm -rf $(PRODUCT_OUT)/data/*) $(call add-clean-step, rm -rf $(PRODUCT_OUT)/system/etc/vintf/compatibility_matrix.device.xml) +$(call add-clean-step, rm -rf $(PRODUCT_OUT)/system/etc/vintf/compatibility_matrix.legacy.xml) +$(call add-clean-step, rm -rf $(PRODUCT_OUT)/system/etc/vintf/compatibility_matrix.1.xml) +$(call add-clean-step, rm -rf $(PRODUCT_OUT)/system/etc/vintf/compatibility_matrix.2.xml) diff --git a/compatibility_matrices/compatibility_matrix.1.xml b/compatibility_matrices/compatibility_matrix.1.xml deleted file mode 100644 index cccf24ff99..0000000000 --- a/compatibility_matrices/compatibility_matrix.1.xml +++ /dev/null @@ -1,326 +0,0 @@ - - - android.hardware.audio - 2.0 - - IDevicesFactory - default - - - - android.hardware.audio.effect - 2.0 - - IEffectsFactory - default - - - - android.hardware.automotive.evs - 1.0 - - IEvsEnumerator - default - - - - android.hardware.automotive.vehicle - 2.0 - - IVehicle - default - - - - android.hardware.biometrics.fingerprint - 2.1 - - IBiometricsFingerprint - default - - - - android.hardware.bluetooth - 1.0 - - IBluetoothHci - default - - - - android.hardware.boot - 1.0 - - IBootControl - default - - - - android.hardware.broadcastradio - 1.0 - - IBroadcastRadioFactory - default - - - - android.hardware.camera.provider - 2.4 - - ICameraProvider - legacy/0 - - - - android.hardware.configstore - 1.0 - - ISurfaceFlingerConfigs - default - - - - android.hardware.contexthub - 1.0 - - IContexthub - default - - - - android.hardware.drm - 1.0 - - ICryptoFactory - default - - - IDrmFactory - default - - - - android.hardware.dumpstate - 1.0 - - IDumpstateDevice - default - - - - android.hardware.gatekeeper - 1.0 - - IGatekeeper - default - - - - android.hardware.gnss - 1.0 - - IGnss - default - - - - android.hardware.graphics.allocator - 2.0 - - IAllocator - default - - - - android.hardware.graphics.composer - 2.1 - - IComposer - default - - - - android.hardware.graphics.mapper - 2.0 - - IMapper - default - - - - android.hardware.health - 1.0 - - IHealth - default - - - - android.hardware.ir - 1.0 - - IConsumerIr - default - - - - android.hardware.keymaster - 3.0 - - IKeymasterDevice - default - - - - android.hardware.light - 2.0 - - ILight - default - - - - android.hardware.media.omx - 1.0 - - IOmx - default - - - IOmxStore - default - - - - android.hardware.memtrack - 1.0 - - IMemtrack - default - - - - android.hardware.nfc - 1.0 - - INfc - default - - - - android.hardware.power - 1.0 - - IPower - default - - - - android.hardware.radio - 1.0 - - IRadio - slot1 - - - ISap - slot1 - - - - android.hardware.radio.deprecated - 1.0 - - IOemHook - slot1 - - - - android.hardware.renderscript - 1.0 - - IDevice - default - - - - android.hardware.sensors - 1.0 - - ISensors - default - - - - android.hardware.soundtrigger - 2.0 - - ISoundTriggerHw - default - - - - android.hardware.thermal - 1.0 - - IThermal - default - - - - android.hardware.tv.cec - 1.0 - - IHdmiCec - default - - - - android.hardware.tv.input - 1.0 - - ITvInput - default - - - - android.hardware.usb - 1.0 - - IUsb - default - - - - android.hardware.vibrator - 1.0 - - IVibrator - default - - - - android.hardware.vr - 1.0 - - IVr - default - - - - android.hardware.wifi - 1.0 - - IWifi - default - - - - android.hardware.wifi.supplicant - 1.0 - - ISupplicant - default - - - diff --git a/compatibility_matrices/compatibility_matrix.2.xml b/compatibility_matrices/compatibility_matrix.2.xml deleted file mode 100644 index d4f9809b7e..0000000000 --- a/compatibility_matrices/compatibility_matrix.2.xml +++ /dev/null @@ -1,374 +0,0 @@ - - - android.hardware.audio - 2.0 - - IDevicesFactory - default - - - - android.hardware.audio.effect - 2.0 - - IEffectsFactory - default - - - - android.hardware.automotive.evs - 1.0 - - IEvsEnumerator - default - - - - android.hardware.automotive.vehicle - 2.0 - - IVehicle - default - - - - android.hardware.biometrics.fingerprint - 2.1 - - IBiometricsFingerprint - default - - - - android.hardware.bluetooth - 1.0 - - IBluetoothHci - default - - - - android.hardware.boot - 1.0 - - IBootControl - default - - - - android.hardware.broadcastradio - 1.0-1 - - IBroadcastRadioFactory - default - - - - android.hardware.camera.provider - 2.4 - - ICameraProvider - legacy/0 - - - - android.hardware.cas - 1.0 - - IMediaCasService - default - - - - android.hardware.configstore - 1.0 - - ISurfaceFlingerConfigs - default - - - - android.hardware.contexthub - 1.0 - - IContexthub - default - - - - android.hardware.drm - 1.0 - - ICryptoFactory - default - - - IDrmFactory - default - - - - android.hardware.dumpstate - 1.0 - - IDumpstateDevice - default - - - - android.hardware.gatekeeper - 1.0 - - IGatekeeper - default - - - - android.hardware.gnss - 1.0 - - IGnss - default - - - - android.hardware.graphics.allocator - 2.0 - - IAllocator - default - - - - android.hardware.graphics.composer - 2.1 - - IComposer - default - - - - android.hardware.graphics.mapper - 2.0 - - IMapper - default - - - - android.hardware.health - 1.0 - - IHealth - default - - - - android.hardware.ir - 1.0 - - IConsumerIr - default - - - - android.hardware.keymaster - 3.0 - - IKeymasterDevice - default - - - - android.hardware.light - 2.0 - - ILight - default - - - - android.hardware.media.omx - 1.0 - - IOmx - default - - - IOmxStore - default - - - - android.hardware.memtrack - 1.0 - - IMemtrack - default - - - - android.hardware.nfc - 1.0 - - INfc - default - - - - android.hardware.oemlock - 1.0 - - IOemLock - default - - - - android.hardware.power - 1.0-1 - - IPower - default - - - - android.hardware.radio - 1.0-1 - - IRadio - slot1 - - - ISap - slot1 - - - - android.hardware.radio.deprecated - 1.0 - - IOemHook - slot1 - - - - android.hardware.renderscript - 1.0 - - IDevice - default - - - - android.hardware.sensors - 1.0 - - ISensors - default - - - - android.hardware.soundtrigger - 2.0 - - ISoundTriggerHw - default - - - - android.hardware.tetheroffload.config - 1.0 - - IOffloadConfig - default - - - - android.hardware.tetheroffload.control - 1.0 - - IOffloadControl - default - - - - android.hardware.thermal - 1.0-1 - - IThermal - default - - - - android.hardware.tv.cec - 1.0 - - IHdmiCec - default - - - - android.hardware.tv.input - 1.0 - - ITvInput - default - - - - android.hardware.usb - 1.0-1 - - IUsb - default - - - - android.hardware.vibrator - 1.0-1 - - IVibrator - default - - - - android.hardware.vr - 1.0 - - IVr - default - - - - android.hardware.weaver - 1.0 - - IWeaver - default - - - - android.hardware.wifi - 1.0-1 - - IWifi - default - - - - android.hardware.wifi.offload - 1.0 - - IOffload - default - - - - android.hardware.wifi.supplicant - 1.0 - - ISupplicant - default - - - diff --git a/compatibility_matrices/compatibility_matrix.legacy.xml b/compatibility_matrices/compatibility_matrix.legacy.xml deleted file mode 100644 index 8a4d2ee60d..0000000000 --- a/compatibility_matrices/compatibility_matrix.legacy.xml +++ /dev/null @@ -1,326 +0,0 @@ - - - android.hardware.audio - 2.0 - - IDevicesFactory - default - - - - android.hardware.audio.effect - 2.0 - - IEffectsFactory - default - - - - android.hardware.automotive.evs - 1.0 - - IEvsEnumerator - default - - - - android.hardware.automotive.vehicle - 2.0 - - IVehicle - default - - - - android.hardware.biometrics.fingerprint - 2.1 - - IBiometricsFingerprint - default - - - - android.hardware.bluetooth - 1.0 - - IBluetoothHci - default - - - - android.hardware.boot - 1.0 - - IBootControl - default - - - - android.hardware.broadcastradio - 1.0 - - IBroadcastRadioFactory - default - - - - android.hardware.camera.provider - 2.4 - - ICameraProvider - legacy/0 - - - - android.hardware.configstore - 1.0 - - ISurfaceFlingerConfigs - default - - - - android.hardware.contexthub - 1.0 - - IContexthub - default - - - - android.hardware.drm - 1.0 - - ICryptoFactory - default - - - IDrmFactory - default - - - - android.hardware.dumpstate - 1.0 - - IDumpstateDevice - default - - - - android.hardware.gatekeeper - 1.0 - - IGatekeeper - default - - - - android.hardware.gnss - 1.0 - - IGnss - default - - - - android.hardware.graphics.allocator - 2.0 - - IAllocator - default - - - - android.hardware.graphics.composer - 2.1 - - IComposer - default - - - - android.hardware.graphics.mapper - 2.0 - - IMapper - default - - - - android.hardware.health - 1.0 - - IHealth - default - - - - android.hardware.ir - 1.0 - - IConsumerIr - default - - - - android.hardware.keymaster - 3.0 - - IKeymasterDevice - default - - - - android.hardware.light - 2.0 - - ILight - default - - - - android.hardware.media.omx - 1.0 - - IOmx - default - - - IOmxStore - default - - - - android.hardware.memtrack - 1.0 - - IMemtrack - default - - - - android.hardware.nfc - 1.0 - - INfc - default - - - - android.hardware.power - 1.0 - - IPower - default - - - - android.hardware.radio - 1.0 - - IRadio - slot1 - - - ISap - slot1 - - - - android.hardware.radio.deprecated - 1.0 - - IOemHook - slot1 - - - - android.hardware.renderscript - 1.0 - - IDevice - default - - - - android.hardware.sensors - 1.0 - - ISensors - default - - - - android.hardware.soundtrigger - 2.0 - - ISoundTriggerHw - default - - - - android.hardware.thermal - 1.0 - - IThermal - default - - - - android.hardware.tv.cec - 1.0 - - IHdmiCec - default - - - - android.hardware.tv.input - 1.0 - - ITvInput - default - - - - android.hardware.usb - 1.0 - - IUsb - default - - - - android.hardware.vibrator - 1.0 - - IVibrator - default - - - - android.hardware.vr - 1.0 - - IVr - default - - - - android.hardware.wifi - 1.0 - - IWifi - default - - - - android.hardware.wifi.supplicant - 1.0 - - ISupplicant - default - - - diff --git a/compatibility_matrices/exclude/fcm_exclude.cpp b/compatibility_matrices/exclude/fcm_exclude.cpp index 0eff450348..849df9490b 100644 --- a/compatibility_matrices/exclude/fcm_exclude.cpp +++ b/compatibility_matrices/exclude/fcm_exclude.cpp @@ -60,6 +60,18 @@ bool ShouldCheckMissingHalsInFcm(const std::string& package) { // does not depend on this HAL, hence it is not declared in any manifests or matrices. "android.hardware.fastboot@1.0", "android.hardware.fastboot@1.1", + + // Deprecated HALs. + // HIDL + // TODO(b/171260360) Remove when HAL definition is removed + "android.hardware.audio.effect@2.0", + "android.hardware.audio@2.0", + // TODO(b/171260613) Remove when HAL definition is removed + "android.hardware.health@1.0", + // TODO(b/171260670) Remove when HAL definition is removed + "android.hardware.nfc@1.0", + // TODO(b/171260715) Remove when HAL definition is removed + "android.hardware.radio.deprecated@1.0", }; auto package_has_prefix = [&](const std::string& prefix) {