Add property compliance supportprops2

Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>

2 files changed, 102 insertions, 3 deletions

diff --git a/utils/v4l2-compliance/v4l2-compliance.cpp b/utils/v4l2-compliance/v4l2-compliance.cpp
index a53fcc50..5e68defb 100644
--- a/utils/v4l2-compliance/v4l2-compliance.cpp
+++ b/utils/v4l2-compliance/v4l2-compliance.cpp
@@ -816,11 +816,12 @@ void testNode(struct node &node, struct node &expbuf_node, media_type type,
 		printf("Media Controller ioctls:\n");
 		printf("\ttest MEDIA_IOC_G_TOPOLOGY: %s\n", ok(testMediaTopology(&node)));
 		if (node.topology)
-			printf("\tEntities: %u Interfaces: %u Pads: %u Links: %u\n",
+			printf("\tEntities: %u Interfaces: %u Pads: %u Links: %u Properties: %u\n",
 			       node.topology->num_entities,
 			       node.topology->num_interfaces,
 			       node.topology->num_pads,
-			       node.topology->num_links);
+			       node.topology->num_links,
+			       node.topology->num_props);
 		printf("\ttest MEDIA_IOC_ENUM_ENTITIES/LINKS: %s\n", ok(testMediaEnum(&node)));
 		printf("\ttest MEDIA_IOC_SETUP_LINK: %s\n", ok(testMediaSetupLink(&node)));
 		printf("\n");
diff --git a/utils/v4l2-compliance/v4l2-test-media.cpp b/utils/v4l2-compliance/v4l2-test-media.cpp
index ae5f0c8b..36fdb00c 100644
--- a/utils/v4l2-compliance/v4l2-test-media.cpp
+++ b/utils/v4l2-compliance/v4l2-test-media.cpp
@@ -79,12 +79,15 @@ static media_v2_pad *v2_pads;
 static id_set v2_pads_set;
 static media_v2_link *v2_links;
 static id_set v2_links_set;
+static media_v2_prop *v2_props;
+static id_set v2_props_set;
 static std::map<__u32, __u32> entity_num_pads;
 static std::map<__u32, media_v2_entity *> v2_entity_map;
 static std::set<std::string> v2_entity_names_set;
 static std::map<__u32, media_v2_interface *> v2_iface_map;
 static std::map<__u32, media_v2_pad *> v2_pad_map;
 static std::set<__u64> v2_entity_pad_idx_set;
+static std::map<__u32, media_v2_prop *> v2_prop_map;
 static unsigned num_data_links;
 
 static int checkFunction(__u32 function, bool v2_api)
@@ -111,13 +114,25 @@ int testMediaTopology(struct node *node)
 	topology.ptr_interfaces = 0;
 	topology.ptr_pads = 0;
 	topology.ptr_links = 0;
+	//fail_on_test(doioctl(node, MEDIA_IOC_G_TOPOLOGY_OLD, &topology));
+	//fail_on_test(topology.ptr_props != ~0ULL);
+
+	memset(&topology, 0xff, sizeof(topology));
+	topology.ptr_entities = 0;
+	topology.ptr_interfaces = 0;
+	topology.ptr_pads = 0;
+	topology.ptr_links = 0;
+	topology.ptr_props = 0;
 	fail_on_test(doioctl(node, MEDIA_IOC_G_TOPOLOGY, &topology));
+
 	fail_on_test(!topology.num_entities);
 	fail_on_test(topology.topology_version == ~0ULL);
 	fail_on_test(topology.num_entities == ~0U);
 	fail_on_test(topology.num_interfaces == ~0U);
 	fail_on_test(topology.num_pads == ~0U);
 	fail_on_test(topology.num_links == ~0U);
+	fail_on_test(topology.num_props == ~0U);
+	fail_on_test(topology.props_payload_size == ~0U);
 	fail_on_test(topology.reserved1);
 	fail_on_test(topology.reserved2);
 	fail_on_test(topology.reserved3);
@@ -137,6 +152,10 @@ int testMediaTopology(struct node *node)
 	fail_on_test(topology.num_links &&
 		     doioctl(node, MEDIA_IOC_G_TOPOLOGY, &topology) != EFAULT);
 	topology.ptr_links = 0;
+	topology.ptr_props = 4;
+	fail_on_test(topology.num_props &&
+		     doioctl(node, MEDIA_IOC_G_TOPOLOGY, &topology) != EFAULT);
+	topology.ptr_props = 0;
 	v2_ents = new media_v2_entity[topology.num_entities];
 	memset(v2_ents, 0xff, topology.num_entities * sizeof(*v2_ents));
 	topology.ptr_entities = (__u64)v2_ents;
@@ -149,11 +168,18 @@ int testMediaTopology(struct node *node)
 	v2_links = new media_v2_link[topology.num_links];
 	memset(v2_links, 0xff, topology.num_links * sizeof(*v2_links));
 	topology.ptr_links = (__u64)v2_links;
+	__u8 *props = new __u8[topology.num_props * sizeof(media_v2_prop) +
+		topology.props_payload_size];
+	memset(props, 0xff, topology.num_props * sizeof(media_v2_prop) +
+	       topology.props_payload_size);
+	v2_props = (media_v2_prop *)props;
+	topology.ptr_props = (__u64)v2_props;
 	fail_on_test(doioctl(node, MEDIA_IOC_G_TOPOLOGY, &topology));
 	fail_on_test(v2_ents != (media_v2_entity *)topology.ptr_entities);
 	fail_on_test(v2_ifaces != (media_v2_interface *)topology.ptr_interfaces);
 	fail_on_test(v2_pads != (media_v2_pad *)topology.ptr_pads);
 	fail_on_test(v2_links != (media_v2_link *)topology.ptr_links);
+	fail_on_test(v2_props != (media_v2_prop *)topology.ptr_props);
 
 	for (unsigned i = 0; i < topology.num_entities; i++) {
 		media_v2_entity &ent = v2_ents[i];
@@ -194,7 +220,6 @@ int testMediaTopology(struct node *node)
 		fail_on_test(!iface.intf_type);
 		fail_on_test(iface.intf_type < MEDIA_INTF_T_DVB_BASE);
 		fail_on_test(iface.intf_type > MEDIA_INTF_T_V4L_BASE + 0xff);
-		fail_on_test(iface.flags);
 		fail_on_test(v2_interfaces_set.find(iface.id) != v2_interfaces_set.end());
 		v2_interfaces_set.insert(iface.id);
 		v2_iface_map[iface.id] = &iface;
@@ -270,6 +295,79 @@ int testMediaTopology(struct node *node)
 		}
 	}
 
+	for (unsigned i = 0; i < topology.num_props; i++) {
+		media_v2_prop &prop = v2_props[i];
+
+		if (show_info) {
+			printf("\t\tProperty: 0x%08x '%s' (Owner: 0x%08x): ",
+			       prop.id, prop.name, prop.owner_id);
+			switch (prop.type) {
+			case MEDIA_PROP_TYPE_GROUP:
+				printf("group\n");
+				break;
+			case MEDIA_PROP_TYPE_U64:
+				printf("0x%016llx %llu\n", media_prop2u64(&prop),
+				       media_prop2u64(&prop));
+				break;
+			case MEDIA_PROP_TYPE_S64:
+				printf("0x%016llx %lld\n", media_prop2s64(&prop),
+				       media_prop2s64(&prop));
+				break;
+			case MEDIA_PROP_TYPE_STRING:
+				printf("'%s'\n", media_prop2string(&prop));
+				break;
+			}
+		}
+		fail_on_test(check_0(prop.reserved, sizeof(prop.reserved)));
+		fail_on_test(!prop.id);
+		fail_on_test(!prop.owner_id);
+		fail_on_test(v2_entities_set.find(prop.owner_id) == v2_entities_set.end() &&
+			     v2_pads_set.find(prop.owner_id) == v2_pads_set.end() &&
+			     v2_props_set.find(prop.owner_id) == v2_props_set.end());
+		if (prop.payload_size)
+			fail_on_test(!prop.payload_offset);
+		else
+			fail_on_test(prop.payload_offset);
+
+		switch (prop.owner_type) {
+		case MEDIA_OWNER_TYPE_ENTITY:
+			fail_on_test(v2_entities_set.find(prop.owner_id) == v2_entities_set.end());
+			break;
+		case MEDIA_OWNER_TYPE_PAD:
+			fail_on_test(v2_pads_set.find(prop.owner_id) == v2_pads_set.end());
+			break;
+		case MEDIA_OWNER_TYPE_LINK:
+			fail_on_test(v2_links_set.find(prop.owner_id) == v2_links_set.end());
+			break;
+		case MEDIA_OWNER_TYPE_INTF:
+			fail_on_test(v2_interfaces_set.find(prop.owner_id) == v2_interfaces_set.end());
+			break;
+		case MEDIA_OWNER_TYPE_PROP:
+			fail_on_test(v2_props_set.find(prop.owner_id) == v2_props_set.end());
+			break;
+		default:
+			return fail("unknown property owner");
+		}
+
+		switch (prop.type) {
+		case MEDIA_PROP_TYPE_GROUP:
+			fail_on_test(prop.payload_size);
+			break;
+		case MEDIA_PROP_TYPE_U64:
+		case MEDIA_PROP_TYPE_S64:
+			fail_on_test(prop.payload_size != 8);
+			break;
+		case MEDIA_PROP_TYPE_STRING:
+			fail_on_test(!prop.payload_size);
+			break;
+		default:
+			return fail("unknown property type\n");
+		}
+		fail_on_test(v2_props_set.find(prop.id) != v2_props_set.end());
+		v2_props_set.insert(prop.id);
+		v2_prop_map[prop.id] = &prop;
+	}
+
 	for (unsigned i = 0; i < topology.num_entities; i++) {
 		media_v2_entity &ent = v2_ents[i];