release-request-c8b8af6b-4899-402a-9ff7-72255ba5300b-for-git_oc-mr1-release-42... (37412003) · Commits · e / os / android_hardware_interfaces

Android.bp

+6 −0

Original line number	Diff line number	Diff line
		@@ -10,15 +10,21 @@ cc_defaults {
		],
		}

		// VTS tests must link to HAL definition libraries statically.
		cc_defaults {
		name: "VtsHalTargetTestDefaults",
		defaults: ["hidl_defaults"],

		// Lists all dependencies that can not be expected on the device.
		static_libs: [
		"VtsHalHidlTargetTestBase",
		],
		group_static_libs: true,

		// Lists all system dependencies that can be expected on the device.
		shared_libs: [
		"libbase",
		// All the following are dependencies of any HAL definition library.
		"libcutils",
		"liblog",
		"libhidlbase",

broadcastradio/1.1/types.hal

+15 −9

Original line number	Diff line number	Diff line
		@@ -128,11 +128,14 @@ enum Modulation : uint32_t {
		/**
		* Type of a radio technology.
		*
		* VENDOR program types must be opaque to the framework.
		*
		* There are multiple VENDOR program types just to make vendor implementation
		* easier with multiple properitary radio technologies. They are treated the
		* same by the framework.
		*
		* All other values are reserved for future use.
		* Values not matching any enumerated constant must be ignored.
		*/
		enum ProgramType : uint32_t {
		AM = 1, // analogue AM radio (with or without RDS)
		@@ -142,10 +145,10 @@ enum ProgramType : uint32_t {
		DAB, // Digital audio broadcasting
		DRMO, // Digital Radio Mondiale
		SXM, // SiriusXM Satellite Radio
		VENDOR1, // Vendor-specific, not synced across devices.
		VENDOR2, // Vendor-specific, not synced across devices.
		VENDOR3, // Vendor-specific, not synced across devices.
		VENDOR4, // Vendor-specific, not synced across devices.

		// Vendor-specific, not synced across devices.
		VENDOR_START = 1000,
		VENDOR_END = 1999,
		};

		/**
		@@ -155,10 +158,13 @@ enum ProgramType : uint32_t {
		* it for secondary IDs. For example, a satellite program may set AM/FM fallback
		* frequency, if a station broadcasts both via satellite and AM/FM.
		*
		* VENDOR identifier types must be opaque to the framework.
		*
		* The value format for each (but VENDOR_PRIMARY) identifier is strictly defined
		* to maintain interoperability between devices made by different vendors.
		*
		* All other values are reserved for future use.
		* Values not matching any enumerated constant must be ignored.
		*/
		enum IdentifierType : uint32_t {
		AMFM_FREQUENCY = 1, // kHz
		@@ -208,12 +214,12 @@ enum IdentifierType : uint32_t {
		* Primary identifier for vendor-specific radio technology.
		* The value format is determined by a vendor.
		*
		* It must not be used in any other programType than VENDORx.
		* It must not be used in any other programType than corresponding VENDOR
		* type between VENDOR_START and VENDOR_END (eg. identifier type 1015 must
		* not be used in any program type other than 1015).
		*/
		VENDOR1_PRIMARY,
		VENDOR2_PRIMARY,
		VENDOR3_PRIMARY,
		VENDOR4_PRIMARY,
		VENDOR_PRIMARY_START = ProgramType:VENDOR_START,
		VENDOR_PRIMARY_END = ProgramType:VENDOR_END,
		};

		/**

broadcastradio/1.1/utils/Utils.cpp

+1 −5

Original line number	Diff line number	Diff line
		@@ -93,11 +93,7 @@ bool tunesTo(const ProgramSelector& a, const ProgramSelector& b) {
		return haveEqualIds(a, b, IdentifierType::SXM_SERVICE_ID);
		}
		return haveEqualIds(a, b, IdentifierType::SXM_CHANNEL);
		case ProgramType::VENDOR1:
		case ProgramType::VENDOR2:
		case ProgramType::VENDOR3:
		case ProgramType::VENDOR4:
		default:
		default: // includes all vendor types
		ALOGW("Unsupported program type: %s", toString(type).c_str());
		return false;
		}

broadcastradio/1.1/vts/functional/VtsHalBroadcastradioV1_1TargetTest.cpp

+6 −13

Original line number	Diff line number	Diff line
		@@ -69,10 +69,6 @@ static constexpr ProgramType kStandardProgramTypes[] = {
		ProgramType::AM, ProgramType::FM, ProgramType::AM_HD, ProgramType::FM_HD,
		ProgramType::DAB, ProgramType::DRMO, ProgramType::SXM};

		static constexpr IdentifierType kVendorPrimartIds[] = {
		IdentifierType::VENDOR1_PRIMARY, IdentifierType::VENDOR2_PRIMARY,
		IdentifierType::VENDOR3_PRIMARY, IdentifierType::VENDOR4_PRIMARY};

		static void printSkipped(std::string msg) {
		std::cout << "[ SKIPPED ] " << msg << std::endl;
		}
		@@ -382,17 +378,14 @@ TEST_P(BroadcastRadioHalTest, TuneFailsForPrimaryVendor) {

		for (auto ptype : kStandardProgramTypes) {
		ALOGD("Checking %s...", toString(ptype).c_str());
		for (auto idtype : kVendorPrimartIds) {
		ALOGD("...with %s", toString(idtype).c_str());
		ProgramSelector sel = {};
		sel.programType = static_cast<uint32_t>(ptype);
		sel.primaryId.type = static_cast<uint32_t>(idtype);
		sel.primaryId.type = static_cast<uint32_t>(IdentifierType::VENDOR_PRIMARY_START);

		auto tuneResult = mTuner->tuneByProgramSelector(sel);
		ASSERT_NE(Result::OK, tuneResult);
		}
		}
		}

		/**
		* Test that tune with unknown program type fails.

current.txt

+1 −0

Original line number	Diff line number	Diff line
		@@ -193,6 +193,7 @@ fe3c3c2f572b72f15f8594c538b0577bd5c28722c31879cfe6231330cddb6747 android.hardwar
		dc7e6d4f537b9943e27edc4f86c5a03bb643b18f18f866f8c3c71c0ac4ea8cbc android.hardware.broadcastradio@1.0::types
		760485232f6cce07f8bb05e3475509956996b702f77415ee5bff05e2ec5a5bcc android.hardware.dumpstate@1.0::IDumpstateDevice
		e822cb7f4a1bdd45689c5e92ccd19a2201c20b771bd4b2ec1ae627e324591f9d android.hardware.radio@1.0::IRadioResponse
		6e69adb24d7c0b0ca3a54a38c49a5625b161b3f5d5f7d6fda0befdbbfc8e9e06 android.hardware.radio@1.0::IRadioResponse
		28e929b453df3d9f5060af2764e6cdb123ddb893e3e86923c877f6ff7e5f02c9 android.hardware.wifi@1.0::types

		# HALs released in Android O MR1