From 5ea2d1799ce1d20f1d39b8538a8d9e5b82545005 Mon Sep 17 00:00:00 2001 From: Roshan Pius Date: Fri, 4 Feb 2022 04:50:21 +0000 Subject: [PATCH 1/2] uwb(hal): Add an android specifci aoa_result_req value This is needed for supporting antenna interleaving feature. Bug: 205730040 Test: Compiles Change-Id: I06696c1624cbe5af67124ae5f807ef0bcc983194 --- .../fira_android/UwbAndroidCapabilities.aidl | 2 +- .../UwbVendorCapabilityTlvTypes.aidl | 1 + .../UwbVendorSessionAppConfigTlvValues.aidl | 38 ++++++++++++++++++ .../fira_android/UwbAndroidCapabilities.aidl | 2 +- .../UwbVendorCapabilityTlvTypes.aidl | 12 ++++++ .../UwbVendorSessionAppConfigTlvTypes.aidl | 2 +- .../UwbVendorSessionAppConfigTlvValues.aidl | 39 +++++++++++++++++++ 7 files changed, 93 insertions(+), 3 deletions(-) create mode 100644 uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl create mode 100644 uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl diff --git a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl index 7e3be56d64..61a5548301 100644 --- a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl +++ b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl @@ -35,5 +35,5 @@ package android.hardware.uwb.fira_android; @Backing(type="long") @VintfStability enum UwbAndroidCapabilities { POWER_STATS_QUERY = 1, - ANTENNAE_INTERLEAVING = 2, + ANTENNA_INTERLEAVING = 2, } diff --git a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl index bc73b89ced..f5cf8a388f 100644 --- a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl +++ b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl @@ -43,4 +43,5 @@ enum UwbVendorCapabilityTlvTypes { CCC_SUPPORTED_CHANNELS = 166, CCC_SUPPORTED_HOPPING_SEQUENCES = 167, CCC_SUPPORTED_HOPPING_CONFIG_MODES = 168, + SUPPORTED_AOA_RESULT_REQ_ANTENNA_INTERLEAVING = 227, } diff --git a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl new file mode 100644 index 0000000000..a7f487bcd2 --- /dev/null +++ b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl @@ -0,0 +1,38 @@ +/* + * Copyright (C) 2021 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * You may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ +/////////////////////////////////////////////////////////////////////////////// +// THIS FILE IS IMMUTABLE. DO NOT EDIT IN ANY CASE. // +/////////////////////////////////////////////////////////////////////////////// + +// This file is a snapshot of an AIDL file. Do not edit it manually. There are +// two cases: +// 1). this is a frozen version file - do not edit this in any case. +// 2). this is a 'current' file. If you make a backwards compatible change to +// the interface (from the latest frozen version), the build system will +// prompt you to update this file with `m -update-api`. +// +// You must not make a backward incompatible change to any AIDL file built +// with the aidl_interface module type with versions property set. The module +// type is used to build AIDL files in a way that they can be used across +// independently updatable components of the system. If a device is shipped +// with such a backward incompatible change, it has a high risk of breaking +// later when a module using the interface is updated, e.g., Mainline modules. + +package android.hardware.uwb.fira_android; +@Backing(type="int") @VintfStability +enum UwbVendorSessionAppConfigTlvValues { + AOA_RESULT_REQ_ANTENNA_INTERLEAVING = 240, +} diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl index 0af99e0f29..1f9c9bbce7 100644 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl @@ -27,5 +27,5 @@ package android.hardware.uwb.fira_android; @Backing(type="long") enum UwbAndroidCapabilities { POWER_STATS_QUERY = 0x1, - ANTENNAE_INTERLEAVING = 0x2, + ANTENNA_INTERLEAVING = 0x2, } diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl index 968cd17884..1f502c3835 100644 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl @@ -91,4 +91,16 @@ enum UwbVendorCapabilityTlvTypes { * HOPPING_CONFIG_MODE_ADAPTIVE = 2 */ CCC_SUPPORTED_HOPPING_CONFIG_MODES = 0xA8, + + /********************************************* + * FIRA specific + ********************************************/ + /** + * 1 byte value to indicate support for antennare interleaving + * feature. + * Values: + * 1 - Feature supported. + * 0 - Feature not supported. + */ + SUPPORTED_AOA_RESULT_REQ_ANTENNA_INTERLEAVING = 0xE3, } diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl index b770ee0145..2b21f35f8b 100644 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl @@ -58,7 +58,7 @@ enum UwbVendorSessionAppConfigTlvTypes { * Added in vendor version 0. * Interleaving ratio if AOA_RESULT_REQ is set to 0xF0. * Supported only if the value returned by getSupportedAndroidCapabilities() - * has the bit of UwbAndroidCapabilities.ANTENNAE_INTERLEAVING set to 1. + * has the bit of UwbAndroidCapabilities.ANTENNA_INTERLEAVING set to 1. */ /** 2 byte data */ NB_OF_RANGE_MEASUREMENTS = 0xE3, diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl new file mode 100644 index 0000000000..33932bb120 --- /dev/null +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl @@ -0,0 +1,39 @@ +/* + * Copyright (C) 2021 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * You may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package android.hardware.uwb.fira_android; + +/** + * Android specific vendor app config values set/expected in UCI command: + * GID: 0001b (UWB Session config Group) + * OID: 000011b (SESSION_SET_APP_CONFIG_CMD) + * OID: 000100b (SESSION_GET_APP_CONFIG_CMD) + * + * Note: Refer to Table 34 of the UCI specification for the other values + * expected in this command. + */ +@VintfStability +@Backing(type="int") +enum UwbVendorSessionAppConfigTlvValues { + /** + * Added in vendor version 0. + * Supported only if the value returned by getSupportedAndroidCapabilities() + * has the bit of UwbAndroidCapabilities.ANTENNA_INTERLEAVING set to 1. + * Set AOA_RESULT_REQ (Config ID - 0x0D) to this value to turn on antenna + * interleaving feature. + */ + AOA_RESULT_REQ_ANTENNA_INTERLEAVING = 0xF0, +} From b44de0093bd66d358bd790b90e5fbc9ab7755574 Mon Sep 17 00:00:00 2001 From: Roshan Pius Date: Mon, 7 Feb 2022 08:02:56 -0800 Subject: [PATCH 2/2] uwb(hal): Remove getAndroidCapabilities HAL API This is unnecessary since we are anyway using GET_CAPS_INFO for querying non-fira capabilities. Bug: 208678993 Bug: 209053358 Test: Compiles Change-Id: Id9af03b313ceaf00fb6d5231c454dc7c74a6287a --- .../fira_android/UwbAndroidCapabilities.aidl | 39 ------------------- .../UwbVendorCapabilityTlvTypes.aidl | 1 + .../android/hardware/uwb/IUwbChip.aidl | 1 - uwb/aidl/android/hardware/uwb/IUwbChip.aidl | 10 ----- .../fira_android/UwbAndroidCapabilities.aidl | 31 --------------- .../UwbVendorCapabilityTlvTypes.aidl | 14 ++++++- .../fira_android/UwbVendorGidAndroidOids.aidl | 4 +- .../UwbVendorSessionAppConfigTlvTypes.aidl | 4 +- .../UwbVendorSessionAppConfigTlvValues.aidl | 4 +- uwb/aidl/default/uwb_chip.cpp | 6 --- uwb/aidl/default/uwb_chip.h | 1 - uwb/aidl/vts/VtsHalUwbTargetTest.cpp | 8 ---- 12 files changed, 20 insertions(+), 103 deletions(-) delete mode 100644 uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl delete mode 100644 uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl diff --git a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl deleted file mode 100644 index 61a5548301..0000000000 --- a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl +++ /dev/null @@ -1,39 +0,0 @@ -/* - * Copyright (C) 2021 The Android Open Source Project - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * You may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ -/////////////////////////////////////////////////////////////////////////////// -// THIS FILE IS IMMUTABLE. DO NOT EDIT IN ANY CASE. // -/////////////////////////////////////////////////////////////////////////////// - -// This file is a snapshot of an AIDL file. Do not edit it manually. There are -// two cases: -// 1). this is a frozen version file - do not edit this in any case. -// 2). this is a 'current' file. If you make a backwards compatible change to -// the interface (from the latest frozen version), the build system will -// prompt you to update this file with `m -update-api`. -// -// You must not make a backward incompatible change to any AIDL file built -// with the aidl_interface module type with versions property set. The module -// type is used to build AIDL files in a way that they can be used across -// independently updatable components of the system. If a device is shipped -// with such a backward incompatible change, it has a high risk of breaking -// later when a module using the interface is updated, e.g., Mainline modules. - -package android.hardware.uwb.fira_android; -@Backing(type="long") @VintfStability -enum UwbAndroidCapabilities { - POWER_STATS_QUERY = 1, - ANTENNA_INTERLEAVING = 2, -} diff --git a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl index f5cf8a388f..fec044eb6c 100644 --- a/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl +++ b/uwb/aidl/aidl_api/android.hardware.uwb.fira_android/current/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl @@ -34,6 +34,7 @@ package android.hardware.uwb.fira_android; @Backing(type="int") @VintfStability enum UwbVendorCapabilityTlvTypes { + SUPPORTED_POWER_STATS_QUERY = 192, CCC_SUPPORTED_VERSIONS = 160, CCC_SUPPORTED_UWB_CONFIGS = 161, CCC_SUPPORTED_PULSE_SHAPE_COMBOS = 162, diff --git a/uwb/aidl/aidl_api/android.hardware.uwb/current/android/hardware/uwb/IUwbChip.aidl b/uwb/aidl/aidl_api/android.hardware.uwb/current/android/hardware/uwb/IUwbChip.aidl index c7708f1e93..0e640be2dd 100644 --- a/uwb/aidl/aidl_api/android.hardware.uwb/current/android/hardware/uwb/IUwbChip.aidl +++ b/uwb/aidl/aidl_api/android.hardware.uwb/current/android/hardware/uwb/IUwbChip.aidl @@ -42,6 +42,5 @@ interface IUwbChip { void coreInit(); void sessionInit(int sessionId); int getSupportedAndroidUciVersion(); - long getSupportedAndroidCapabilities(); int sendUciMessage(in byte[] data); } diff --git a/uwb/aidl/android/hardware/uwb/IUwbChip.aidl b/uwb/aidl/android/hardware/uwb/IUwbChip.aidl index f2bb0f114f..00cb8e0772 100644 --- a/uwb/aidl/android/hardware/uwb/IUwbChip.aidl +++ b/uwb/aidl/android/hardware/uwb/IUwbChip.aidl @@ -66,16 +66,6 @@ interface IUwbChip { */ int getSupportedAndroidUciVersion(); - /** - * Mechanism to allow HAL implementation to optionally expose features that are defined - * in the "android.hardware.uwb.fira_android" types-only package. - * - * @return Returns the bitmask of capabilities - * (android.hardware.uwb.fira_android.UwbAndroidCapabilities) that is supported by the - * HAL implementation. - */ - long getSupportedAndroidCapabilities(); - /** * Write the UCI message to the UWB Subsystem. * The UCI message format is as per UCI protocol and it is diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl deleted file mode 100644 index 1f9c9bbce7..0000000000 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbAndroidCapabilities.aidl +++ /dev/null @@ -1,31 +0,0 @@ -/* - * Copyright (C) 2021 The Android Open Source Project - * - * Licensed under the Apache License, Version 2.0 (the "License"); - * You may not use this file except in compliance with the License. - * You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, software - * distributed under the License is distributed on an "AS IS" BASIS, - * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. - * See the License for the specific language governing permissions and - * limitations under the License. - */ - -package android.hardware.uwb.fira_android; - -/** - * Android specific capabilities should be defined here. - * - * For any features enabled via the FIRA vendor commands for Android, use this bitmask - * to allow devices to expose the features supported by the HAL implementation. - * - */ -@VintfStability -@Backing(type="long") -enum UwbAndroidCapabilities { - POWER_STATS_QUERY = 0x1, - ANTENNA_INTERLEAVING = 0x2, -} diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl index 1f502c3835..4591dda0ee 100644 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorCapabilityTlvTypes.aidl @@ -29,6 +29,18 @@ package android.hardware.uwb.fira_android; @VintfStability @Backing(type="int") enum UwbVendorCapabilityTlvTypes { + + /********************************************* + * Protocol agnostic + ********************************************/ + /** + * 1 byte value to indicate support for power stats query + * Values: + * 1 - Feature supported. + * 0 - Feature not supported. + */ + SUPPORTED_POWER_STATS_QUERY = 0xC0, + /********************************************* * CCC specific ********************************************/ @@ -96,7 +108,7 @@ enum UwbVendorCapabilityTlvTypes { * FIRA specific ********************************************/ /** - * 1 byte value to indicate support for antennare interleaving + * 1 byte value to indicate support for antenna interleaving * feature. * Values: * 1 - Feature supported. diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorGidAndroidOids.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorGidAndroidOids.aidl index 1dfcd6ff5d..c04bdcfcd9 100644 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorGidAndroidOids.aidl +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorGidAndroidOids.aidl @@ -24,8 +24,8 @@ package android.hardware.uwb.fira_android; @Backing(type="byte") enum UwbVendorGidAndroidOids { // Used by the command and response to get UWB power related stats. - // Supported only if the value returned by getSupportedAndroidCapabilities() - // has the bit of UwbAndroidCapabilities.POWER_STATS_QUERY set to 1. + // Supported only if the UwbVendorCapabilityTlvTypes.SUPPORTED_POWER_STATS_QUERY + // set to 1. ANDROID_GET_POWER_STATS = 0x0, // Used to set the current regulatory country code (determined usinag // SIM or hardcoded by OEM). diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl index 2b21f35f8b..8547567ddf 100644 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvTypes.aidl @@ -57,8 +57,8 @@ enum UwbVendorSessionAppConfigTlvTypes { /** * Added in vendor version 0. * Interleaving ratio if AOA_RESULT_REQ is set to 0xF0. - * Supported only if the value returned by getSupportedAndroidCapabilities() - * has the bit of UwbAndroidCapabilities.ANTENNA_INTERLEAVING set to 1. + * Supported only if the UwbVendorCapabilityTlvTypes + * .SUPPORTED_AOA_RESULT_REQ_ANTENNA_INTERLEAVING set to 1. */ /** 2 byte data */ NB_OF_RANGE_MEASUREMENTS = 0xE3, diff --git a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl index 33932bb120..bd4ad6e93f 100644 --- a/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl +++ b/uwb/aidl/android/hardware/uwb/fira_android/UwbVendorSessionAppConfigTlvValues.aidl @@ -30,8 +30,8 @@ package android.hardware.uwb.fira_android; enum UwbVendorSessionAppConfigTlvValues { /** * Added in vendor version 0. - * Supported only if the value returned by getSupportedAndroidCapabilities() - * has the bit of UwbAndroidCapabilities.ANTENNA_INTERLEAVING set to 1. + * Supported only if the UwbVendorCapabilityTlvTypes + * .SUPPORTED_AOA_RESULT_REQ_ANTENNA_INTERLEAVING set to 1. * Set AOA_RESULT_REQ (Config ID - 0x0D) to this value to turn on antenna * interleaving feature. */ diff --git a/uwb/aidl/default/uwb_chip.cpp b/uwb/aidl/default/uwb_chip.cpp index a5a3f4a123..41f14fd894 100644 --- a/uwb/aidl/default/uwb_chip.cpp +++ b/uwb/aidl/default/uwb_chip.cpp @@ -18,7 +18,6 @@ namespace { constexpr static int32_t kAndroidUciVersion = 1; -constexpr static int64_t kAndroidCapabilities = 0; } namespace android { @@ -60,11 +59,6 @@ UwbChip::~UwbChip() {} return ndk::ScopedAStatus::ok(); } -::ndk::ScopedAStatus UwbChip::getSupportedAndroidCapabilities(int64_t* capabilities) { - *capabilities = kAndroidCapabilities; - return ndk::ScopedAStatus::ok(); -} - ::ndk::ScopedAStatus UwbChip::sendUciMessage(const std::vector& /* data */, int32_t* /* bytes_written */) { // TODO(b/195992658): Need emulator support for UCI stack. diff --git a/uwb/aidl/default/uwb_chip.h b/uwb/aidl/default/uwb_chip.h index 46cecd4739..e900cbe123 100644 --- a/uwb/aidl/default/uwb_chip.h +++ b/uwb/aidl/default/uwb_chip.h @@ -39,7 +39,6 @@ class UwbChip : public BnUwbChip { ::ndk::ScopedAStatus coreInit() override; ::ndk::ScopedAStatus sessionInit(int sesionId) override; ::ndk::ScopedAStatus getSupportedAndroidUciVersion(int32_t* version) override; - ::ndk::ScopedAStatus getSupportedAndroidCapabilities(int64_t* capabilities) override; ::ndk::ScopedAStatus sendUciMessage(const std::vector& data, int32_t* bytes_written) override; diff --git a/uwb/aidl/vts/VtsHalUwbTargetTest.cpp b/uwb/aidl/vts/VtsHalUwbTargetTest.cpp index 1da4432e47..edd8dd6608 100644 --- a/uwb/aidl/vts/VtsHalUwbTargetTest.cpp +++ b/uwb/aidl/vts/VtsHalUwbTargetTest.cpp @@ -180,14 +180,6 @@ TEST_P(UwbAidl, ChipGetSupportedAndroidUciVersion) { EXPECT_GT(version, 0); } -TEST_P(UwbAidl, ChipGetSupportedAndroidCapabilities) { - const auto iuwb_chip = getAnyChipAndOpen(); - EXPECT_TRUE(iuwb_chip->coreInit().isOk()); - - int64_t capabilities; - EXPECT_TRUE(iuwb_chip->getSupportedAndroidCapabilities(&capabilities).isOk()); -} - TEST_P(UwbAidl, ChipGetName) { std::string chip_name = getAnyChipName(); std::shared_ptr iuwb_chip;