Hi Hans, On Mon, Nov 11, 2013 at 7:03 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > On 11/11/2013 11:44 AM, Arun Kumar K wrote: >> Hi Hans, >> >> On Mon, Nov 11, 2013 at 3:01 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: >>> Hi Arun, >>> >>> On 11/11/2013 10:19 AM, Arun Kumar K wrote: >>>> Adds v4l2 controls to set MIN, MAX QP values and >>>> I, P frame QP for vp8 encoder. >>> >>> I assume these parameters and their ranges are all defined by the VP8 standard? >>> Or are they HW specific? >>> >> >> These ranges are not defined by VP8 standard. I can see that the >> standard does not >> give any range. The ranges mentioned are defined by Samsung MFC hardware. >> Do you think that for these controls, I shouldnt mention the range as >> the standard >> does not have it? > > Either leave it out or just say that the range is hardware dependent. > Ok will do that. Thanks & Regards Arun -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html