Hi Philipp, Could you add a one sentence description for this patch? I know that it is really simple, but still the description is still necessary. Best wishes, -- Kamil Debski Samsung R&D Institute Poland > -----Original Message----- > From: Philipp Zabel [mailto:p.zabel@xxxxxxxxxxxxxx] > Sent: Friday, January 23, 2015 5:51 PM > To: Kamil Debski > Cc: linux-media@xxxxxxxxxxxxxxx; Philipp Zabel > Subject: [PATCH 02/21] [media] coda: bitrate can only be set in kbps > steps > > Signed-off-by: Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> > --- > drivers/media/platform/coda/coda-common.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/coda/coda-common.c > b/drivers/media/platform/coda/coda-common.c > index 39330a7..1cc4e90 100644 > --- a/drivers/media/platform/coda/coda-common.c > +++ b/drivers/media/platform/coda/coda-common.c > @@ -1407,7 +1407,7 @@ static const struct v4l2_ctrl_ops coda_ctrl_ops = > { static void coda_encode_ctrls(struct coda_ctx *ctx) { > v4l2_ctrl_new_std(&ctx->ctrls, &coda_ctrl_ops, > - V4L2_CID_MPEG_VIDEO_BITRATE, 0, 32767000, 1, 0); > + V4L2_CID_MPEG_VIDEO_BITRATE, 0, 32767000, 1000, 0); > v4l2_ctrl_new_std(&ctx->ctrls, &coda_ctrl_ops, > V4L2_CID_MPEG_VIDEO_GOP_SIZE, 1, 60, 1, 16); > v4l2_ctrl_new_std(&ctx->ctrls, &coda_ctrl_ops, > -- > 2.1.4 -- 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