Hi Hans, > Did you run the latest v4l2-compliance tool from the v4l-utils.git > repository over your driver? I'm sure you didn't since VIP is missing > support for control events and v4l2-compliance would certainly > complain about that. > > Always check with v4l2-compliance whenever you make changes! It's > continuously improved as well, so a periodic check wouldn't hurt. I applied all your suggestions, and some extra simplification; now I'm running v4l2-compliance but I have this error: Allow for multiple opens: test second video open: OK test VIDIOC_QUERYCAP: OK fail: v4l2-compliance.cpp(322): doioctl(node, VIDIOC_G_PRIORITY, &prio) test VIDIOC_G/S_PRIORITY: FAIL which I don't undestand. I don't have vidio_{g|s}_priority functions in my implementation. And I'm using the V4L2_FL_USE_FH_PRIO flag as suggested in the documentation: --------------- - flags: optional. Set to V4L2_FL_USE_FH_PRIO if you want to let the framework handle the VIDIOC_G/S_PRIORITY ioctls. This requires that you use struct v4l2_fh. Eventually this flag will disappear once all drivers use the core priority handling. But for now it has to be set explicitly. -------------- -- Federico Vaga -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html