Hi Sylwester, >> + static const char * const vpx_num_partitions[] = { >> + "1 partition", >> + "2 partitions", >> + "4 partitions", >> + "8 partitions", >> + NULL, >> + }; >> + static const char * const vpx_num_ref_frames[] = { >> + "1 reference frame", >> + "2 reference frame", >> + NULL, >> + }; > > Have you considered using V4L2_CTRL_TYPE_INTEGER_MENU control type for this ? > One example is V4L2_CID_ISO_SENSITIVITY control. > If I understand correctly, V4L2_CTRL_TYPE_INTEGER_MENU is used for controls where the driver / IP can support different values depending on its capabilities. But here VP8 standard supports only 4 options for no. of partitions that is 1, 2, 4 and 8. Also for number of ref frames, the standard allows only the options 1, 2 and 3 which cannot be extended more. So is it correct to use INTEGER_MENU control here and let the driver define the values? Regards Arun -- 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