On 12/12/2017 03:34 AM, Smitha T Murthy wrote: >> s/Lay/Layer here and below >> > Ok I will change it.
While it's fine to make such change for controls up to V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L6_QP... >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L1_QP: return "HEVC >>> Hierarchical Lay 1 QP"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L2_QP: return "HEVC >>> Hierarchical Lay 2 QP"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L3_QP: return "HEVC >>> Hierarchical Lay 3 QP"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L4_QP: return "HEVC >>> Hierarchical Lay 4 QP"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L5_QP: return "HEVC >>> Hierarchical Lay 5 QP"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L6_QP: return "HEVC >>> Hierarchical Lay 6 QP"; ...for the controls below we may need to replace "Lay" with "L." to make sure the length of the string don't exceed 31 characters (32 with terminating NULL). The names below seem to be 1 character too long and will be truncated when running VIDIOC_QUERY_CTRL ioctl. >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L0_BR: return "HEVC >>> Hierarchical Lay 0 Bit Rate"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L1_BR: return "HEVC >>> Hierarchical Lay 1 Bit Rate"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L2_BR: return "HEVC >>> Hierarchical Lay 2 Bit Rate"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L3_BR: return "HEVC >>> Hierarchical Lay 3 Bit Rate"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L4_BR: return "HEVC >>> Hierarchical Lay 4 Bit Rate"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L5_BR: return "HEVC >>> Hierarchical Lay 5 Bit Rate"; >>> + case V4L2_CID_MPEG_VIDEO_HEVC_HIER_CODING_L6_BR: return "HEVC >>> Hierarchical Lay 6 Bit Rate"; -- Regards, Sylwester
